cosmos 3.3.3 → 3.4.0

data/spec/packets/telemetry_spec.rb

@@ -366,17 +366,17 @@ module Cosmos
     describe "check_stale" do
       it "checks each packet for staleness" do
         @tlm.check_stale
-        expect(@tlm.packet("TGT1","PKT1").stale).to be_truthy
-        expect(@tlm.packet("TGT1","PKT2").stale).to be_truthy
-        expect(@tlm.packet("TGT2","PKT1").stale).to be_truthy
+        expect(@tlm.packet("TGT1","PKT1").stale).to be true
+        expect(@tlm.packet("TGT1","PKT2").stale).to be true
+        expect(@tlm.packet("TGT2","PKT1").stale).to be true
 
         @tlm.packet("TGT1","PKT1").check_limits
         @tlm.packet("TGT1","PKT2").check_limits
         @tlm.packet("TGT2","PKT1").check_limits
         @tlm.check_stale
-        expect(@tlm.packet("TGT1","PKT1").stale).to be_falsey
-        expect(@tlm.packet("TGT1","PKT2").stale).to be_falsey
-        expect(@tlm.packet("TGT2","PKT1").stale).to be_falsey
+        expect(@tlm.packet("TGT1","PKT1").stale).to be false
+        expect(@tlm.packet("TGT1","PKT2").stale).to be false
+        expect(@tlm.packet("TGT2","PKT1").stale).to be false
       end
     end
 

data/spec/script/cmd_tlm_server_spec.rb

@@ -0,0 +1,110 @@
+# encoding: ascii-8bit
+
+# Copyright 2014 Ball Aerospace & Technologies Corp.
+# All Rights Reserved.
+#
+# This program is free software; you can modify and/or redistribute it
+# under the terms of the GNU General Public License
+# as published by the Free Software Foundation; version 3 with
+# attribution addendums as found in the LICENSE.txt
+
+require 'spec_helper'
+require 'cosmos'
+require 'cosmos/script/script'
+require 'tempfile'
+
+module Cosmos
+
+  describe Script do
+
+    before(:all) do
+      cts = File.join(Cosmos::USERPATH,'config','tools','cmd_tlm_server','cmd_tlm_server.txt')
+      FileUtils.mkdir_p(File.dirname(cts))
+      File.open(cts,'w') do |file|
+        file.puts 'INTERFACE INST_INT interface.rb'
+        file.puts 'TARGET INST'
+      end
+      System.class_eval('@@instance = nil')
+
+      require 'cosmos/script'
+    end
+
+    after(:all) do
+      clean_config()
+      FileUtils.rm_rf File.join(Cosmos::USERPATH,'config','tools')
+    end
+
+    before(:each) do
+      allow_any_instance_of(Interface).to receive(:connected?).and_return(true)
+      allow_any_instance_of(Interface).to receive(:disconnect)
+
+      @server = CmdTlmServer.new
+      shutdown_cmd_tlm()
+      initialize_script_module()
+      sleep 0.1
+    end
+
+    after(:each) do
+      @server.stop
+      shutdown_cmd_tlm()
+      sleep(0.1)
+    end
+
+    describe "get_interface_names" do
+      it "returns all interfaces" do
+        expect(get_interface_names).to include("INST_INT")
+      end
+    end
+
+    describe "connect_interface, disconnect_interface, interface_state" do
+      it "connects, disconnect and return the state of the interface CTS-3" do
+        connect_interface("INST_INT")
+        expect(interface_state("INST_INT")).to eql "CONNECTED"
+        disconnect_interface("INST_INT")
+      end
+    end
+
+    describe "map_target_to_interface" do
+      it "maps a target name to an interface" do
+        map_target_to_interface("INST","INST_INT")
+      end
+    end
+
+    describe "connect_router, disconnect_router, get_router_names, router_state" do
+      it "returns connect, disconnect, and list the routers CTS-11" do
+        expect(get_router_names).to include("PREIDENTIFIED_ROUTER")
+        connect_router("PREIDENTIFIED_ROUTER")
+        expect(router_state("PREIDENTIFIED_ROUTER")).to eql "CONNECTED"
+        disconnect_router("PREIDENTIFIED_ROUTER")
+      end
+    end
+
+    describe "logging methods" do
+      it "starts and stop logging and get filenames CTS-14" do
+        start_logging
+        stop_logging
+        get_cmd_log_filename
+        get_tlm_log_filename
+        start_cmd_log
+        start_tlm_log
+        get_cmd_log_filename
+        get_tlm_log_filename
+        stop_cmd_log
+        stop_tlm_log
+        get_cmd_log_filename
+        get_tlm_log_filename
+        start_raw_logging_interface
+        start_raw_logging_router
+        stop_raw_logging_interface
+        stop_raw_logging_router
+
+        start_new_server_message_log
+        sleep 0.1
+        filename = get_server_message_log_filename
+        expect(filename).to match /server_messages.txt/
+      end
+    end
+
+  end
+end
+

data/spec/script/commands_disconnect_spec.rb

@@ -0,0 +1,270 @@
+# encoding: ascii-8bit
+
+# Copyright 2014 Ball Aerospace & Technologies Corp.
+# All Rights Reserved.
+#
+# This program is free software; you can modify and/or redistribute it
+# under the terms of the GNU General Public License
+# as published by the Free Software Foundation; version 3 with
+# attribution addendums as found in the LICENSE.txt
+
+require 'spec_helper'
+require 'cosmos'
+require 'cosmos/script/script'
+require 'tempfile'
+
+module Cosmos
+
+  describe Script do
+
+    before(:all) do
+      cts = File.join(Cosmos::USERPATH,'config','tools','cmd_tlm_server','cmd_tlm_server.txt')
+      FileUtils.mkdir_p(File.dirname(cts))
+      File.open(cts,'w') do |file|
+        file.puts 'INTERFACE INST_INT interface.rb'
+        file.puts 'TARGET INST'
+      end
+      System.class_eval('@@instance = nil')
+      require 'cosmos/script'
+      set_cmd_tlm_disconnect(true)
+    end
+
+    after(:all) do
+      clean_config()
+      FileUtils.rm_rf File.join(Cosmos::USERPATH,'config','tools')
+    end
+
+    describe "require cosmos/script.rb" do
+      it "should raise when inside CmdTlmServer" do
+        save = $0
+        $0 = "CmdTlmServer"
+        expect { load 'cosmos/script.rb' }.to raise_error(/must not be required/)
+        $0 = save
+      end
+
+      it "should raise when inside Replay" do
+        save = $0
+        $0 = "Replay"
+        expect { load 'cosmos/script.rb' }.to raise_error(/must not be required/)
+        $0 = save
+      end
+    end
+
+    describe "cmd" do
+      it "sends a command" do
+        capture_io do |stdout|
+          cmd("INST ABORT")
+          expect(stdout.string).to match /cmd\(\"INST ABORT\"\)/ #"
+          stdout.rewind
+          cmd("INST", "ABORT")
+          expect(stdout.string).to match /cmd\(\"INST ABORT\"\)/ #"
+        end
+      end
+
+      it "checks parameter ranges" do
+        expect { cmd("INST COLLECT with TYPE NORMAL, DURATION 20") }.to raise_error(/Command parameter 'INST COLLECT DURATION' = 20 not in valid range/)
+      end
+
+      it "prompts for a hazardous command" do
+        capture_io do |stdout|
+          expect(self).to receive(:gets) { 'y' } # Send hazardous command
+          cmd("INST COLLECT with TYPE SPECIAL")
+
+          expect(stdout.string).to match /cmd\(\"INST COLLECT/ # "
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring hazardous warnings"
+          stdout.rewind
+
+          expect(self).to receive(:gets) { 'n' } # Don't send hazardous
+          expect(self).to receive(:gets) { 'y' } # Stop running script
+          cmd("INST COLLECT with TYPE SPECIAL")
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+        end
+      end
+    end
+
+    describe "cmd_no_range_check" do
+      it "sends an out of range command" do
+        expect { cmd_no_range_check("INST COLLECT with TYPE NORMAL, DURATION 20") }.to_not raise_error
+      end
+
+      it "prompts for a hazardous command" do
+        capture_io do |stdout|
+          expect(self).to receive(:gets) { 'y' } # Send hazardous command
+          cmd_no_range_check("INST COLLECT with TYPE SPECIAL")
+
+          expect(stdout.string).to match /cmd\(\"INST COLLECT/ # "
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring hazardous warnings"
+          stdout.rewind
+
+          expect(self).to receive(:gets) { 'n' } # Don't send hazardous
+          expect(self).to receive(:gets) { 'y' } # Stop running script
+          cmd_no_range_check("INST COLLECT with TYPE SPECIAL")
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+        end
+      end
+    end
+
+    describe "cmd_no_hazardous_check" do
+      it "checks parameter ranges" do
+        expect { cmd_no_hazardous_check("INST COLLECT with TYPE SPECIAL, DURATION 20") }.to raise_error(/Command parameter 'INST COLLECT DURATION' = 20 not in valid range/)
+      end
+
+      it "sends a hazardous command without prompting" do
+        capture_io do |stdout|
+          cmd_no_hazardous_check("INST COLLECT with TYPE SPECIAL")
+
+          expect(stdout.string).to match /cmd\(\"INST COLLECT/ # "
+          expect(stdout.string).to_not match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring hazardous warnings"
+        end
+      end
+    end
+
+    describe "cmd_no_checks" do
+      it "sends an out of range hazardous command without prompting" do
+        capture_io do |stdout|
+          cmd_no_checks("INST COLLECT with TYPE SPECIAL, DURATION 20")
+
+          expect(stdout.string).to match /cmd\(\"INST COLLECT/ # "
+          expect(stdout.string).to_not match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring hazardous warnings"
+        end
+      end
+    end
+
+    describe "cmd_raw" do
+      it "sends a command" do
+        capture_io do |stdout|
+          cmd_raw("INST ABORT")
+          expect(stdout.string).to match /cmd_raw\(\"INST ABORT\"\)/ # "
+        end
+      end
+
+      it "checks parameter ranges" do
+        expect { cmd_raw("INST COLLECT with TYPE 0, DURATION 20") }.to raise_error(/Command parameter 'INST COLLECT DURATION' = 20 not in valid range/) # '
+      end
+
+      it "prompts for a hazardous command" do
+        capture_io do |stdout|
+          expect(self).to receive(:gets) { 'y' } # Send hazardous command
+          cmd_raw("INST COLLECT with TYPE 1")
+
+          expect(stdout.string).to match /cmd_raw\(\"INST COLLECT/ # "
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring hazardous warnings"
+          stdout.rewind
+
+          expect(self).to receive(:gets) { 'n' } # Don't send hazardous
+          expect(self).to receive(:gets) { 'y' } # Stop running script
+          cmd_raw("INST COLLECT with TYPE 1")
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+        end
+      end
+    end
+
+    describe "cmd_raw_no_range_check" do
+      it "sends an out of range command" do
+        expect { cmd_raw_no_range_check("INST COLLECT with TYPE 0, DURATION 20") }.to_not raise_error
+      end
+
+      it "prompts for a hazardous command" do
+        capture_io do |stdout|
+          expect(self).to receive(:gets) { 'y' } # Send hazardous command
+          cmd_raw_no_range_check("INST COLLECT with TYPE 1")
+
+          expect(stdout.string).to match /cmd_raw\(\"INST COLLECT/ # "
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring hazardous warnings"
+          stdout.rewind
+
+          expect(self).to receive(:gets) { 'n' } # Don't send hazardous
+          expect(self).to receive(:gets) { 'y' } # Stop running script
+          cmd_raw_no_range_check("INST COLLECT with TYPE 1")
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+        end
+      end
+    end
+
+    describe "cmd_raw_no_hazardous_check" do
+      it "checks parameter ranges" do
+        expect { cmd_raw_no_hazardous_check("INST COLLECT with TYPE 1, DURATION 20") }.to raise_error(/Command parameter 'INST COLLECT DURATION' = 20 not in valid range/)
+      end
+
+      it "sends a hazardous command without prompting" do
+        capture_io do |stdout|
+          cmd_raw_no_hazardous_check("INST COLLECT with TYPE 1")
+          expect(stdout.string).to match /cmd_raw\(\"INST COLLECT/ #"
+          expect(stdout.string).to_not match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring hazardous warnings"
+        end
+      end
+    end
+
+    describe "cmd_raw_no_checks" do
+      it "sends an out of range hazardous command without prompting" do
+        capture_io do |stdout|
+          cmd_raw_no_checks("INST COLLECT with TYPE 1, DURATION 20")
+          expect(stdout.string).to match /cmd_raw\(\"INST COLLECT/ #"
+          expect(stdout.string).to_not match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring hazardous warnings"
+        end
+      end
+    end
+
+    describe "send_raw" do
+      it "sends data to the write_raw interface method" do
+        expect_any_instance_of(Interface).to receive(:write_raw).with('\x00')
+        send_raw('INST_INT', '\x00')
+      end
+    end
+
+    describe "send_raw_file" do
+      it "sends file data to the write_raw interface method" do
+        file = File.open('raw_test_file.bin','wb')
+        file.write '\x00\x01\x02\x03'
+        file.close
+
+        expect_any_instance_of(Interface).to receive(:write_raw).with('\x00\x01\x02\x03')
+
+        send_raw_file('INST_INT', 'raw_test_file.bin')
+
+        File.delete('raw_test_file.bin')
+      end
+    end
+
+    describe "get_cmd_list" do
+      it "returns all the target commands" do
+        list = get_cmd_list("INST")
+        # Only check for the collect command to make this test list dependent
+        # on the demo INST command definition file
+        expect(list).to include(["COLLECT", "Starts a collect on the instrument"])
+      end
+    end
+
+    describe "get_cmd_param_list" do
+      it "returns all the parameters for a command" do
+        list = get_cmd_param_list("INST", "COLLECT")
+        expect(list).to include(["TYPE", 0, {"NORMAL"=>0, "SPECIAL"=>1}, "Collect type", nil, nil, true])
+      end
+    end
+
+    describe "get_cmd_hazardous" do
+      it "returns whether a command is hazardous" do
+        expect(get_cmd_hazardous("INST", "COLLECT", {"TYPE"=>"NORMAL"})).to be false
+        expect(get_cmd_hazardous("INST", "COLLECT", {"TYPE"=>"SPECIAL"})).to be true
+      end
+    end
+
+  end
+end
+

data/spec/script/commands_spec.rb

@@ -0,0 +1,288 @@
+# encoding: ascii-8bit
+
+# Copyright 2014 Ball Aerospace & Technologies Corp.
+# All Rights Reserved.
+#
+# This program is free software; you can modify and/or redistribute it
+# under the terms of the GNU General Public License
+# as published by the Free Software Foundation; version 3 with
+# attribution addendums as found in the LICENSE.txt
+
+require 'spec_helper'
+require 'cosmos'
+require 'cosmos/script/script'
+require 'tempfile'
+
+module Cosmos
+
+  describe Script do
+
+    before(:all) do
+      cts = File.join(Cosmos::USERPATH,'config','tools','cmd_tlm_server','cmd_tlm_server.txt')
+      FileUtils.mkdir_p(File.dirname(cts))
+      File.open(cts,'w') do |file|
+        file.puts 'INTERFACE INST_INT interface.rb'
+        file.puts 'TARGET INST'
+      end
+      System.class_eval('@@instance = nil')
+
+      require 'cosmos/script'
+    end
+
+    after(:all) do
+      clean_config()
+      FileUtils.rm_rf File.join(Cosmos::USERPATH,'config','tools')
+    end
+
+    before(:each) do
+      allow_any_instance_of(Interface).to receive(:connected?).and_return(true)
+      allow_any_instance_of(Interface).to receive(:disconnect)
+      allow_any_instance_of(Interface).to receive(:write)
+      allow_any_instance_of(Interface).to receive(:read)
+
+      @server = CmdTlmServer.new
+      shutdown_cmd_tlm()
+      initialize_script_module()
+      sleep 0.1
+    end
+
+    after(:each) do
+      @server.stop
+      shutdown_cmd_tlm()
+      sleep(0.1)
+    end
+
+    describe "require cosmos/script.rb" do
+      it "should raise when inside CmdTlmServer" do
+        save = $0
+        $0 = "CmdTlmServer"
+        expect { load 'cosmos/script.rb' }.to raise_error(/must not be required/)
+        $0 = save
+      end
+
+      it "should raise when inside Replay" do
+        save = $0
+        $0 = "Replay"
+        expect { load 'cosmos/script.rb' }.to raise_error(/must not be required/)
+        $0 = save
+      end
+    end
+
+    describe "cmd" do
+      it "sends a command" do
+        capture_io do |stdout|
+          cmd("INST ABORT")
+          expect(stdout.string).to match /cmd\(\"INST ABORT\"\)/ #"
+          stdout.rewind
+          cmd("INST", "ABORT")
+          expect(stdout.string).to match /cmd\(\"INST ABORT\"\)/ #"
+        end
+      end
+
+      it "checks parameter ranges" do
+        expect { cmd("INST COLLECT with TYPE NORMAL, DURATION 20") }.to raise_error(/Command parameter 'INST COLLECT DURATION' = 20 not in valid range/)
+      end
+
+      it "prompts for a hazardous command" do
+        capture_io do |stdout|
+          expect(self).to receive(:gets) { 'y' } # Send hazardous command
+          cmd("INST COLLECT with TYPE SPECIAL")
+
+          expect(stdout.string).to match /cmd\(\"INST COLLECT/ #"
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring hazardous warnings"
+          stdout.rewind
+
+          expect(self).to receive(:gets) { 'n' } # Don't send hazardous
+          expect(self).to receive(:gets) { 'y' } # Stop running script
+          cmd("INST COLLECT with TYPE SPECIAL")
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+        end
+      end
+    end
+
+    describe "cmd_no_range_check" do
+      it "sends an out of range command" do
+        expect { cmd_no_range_check("INST COLLECT with TYPE NORMAL, DURATION 20") }.to_not raise_error
+      end
+
+      it "prompts for a hazardous command" do
+        capture_io do |stdout|
+          expect(self).to receive(:gets) { 'y' } # Send hazardous command
+          cmd_no_range_check("INST COLLECT with TYPE SPECIAL")
+
+          expect(stdout.string).to match /cmd\(\"INST COLLECT/ #"
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring hazardous warnings"
+          stdout.rewind
+
+          expect(self).to receive(:gets) { 'n' } # Don't send hazardous
+          expect(self).to receive(:gets) { 'y' } # Stop running script
+          cmd_no_range_check("INST COLLECT with TYPE SPECIAL")
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+        end
+      end
+    end
+
+    describe "cmd_no_hazardous_check" do
+      it "checks parameter ranges" do
+        expect { cmd_no_hazardous_check("INST COLLECT with TYPE SPECIAL, DURATION 20") }.to raise_error(/Command parameter 'INST COLLECT DURATION' = 20 not in valid range/)
+      end
+
+      it "sends a hazardous command without prompting" do
+        capture_io do |stdout|
+          cmd_no_hazardous_check("INST COLLECT with TYPE SPECIAL")
+
+          expect(stdout.string).to match /cmd\(\"INST COLLECT/ #"
+          expect(stdout.string).to_not match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring hazardous warnings"
+        end
+      end
+    end
+
+    describe "cmd_no_checks" do
+      it "sends an out of range hazardous command without prompting" do
+        capture_io do |stdout|
+          cmd_no_checks("INST COLLECT with TYPE SPECIAL, DURATION 20")
+
+          expect(stdout.string).to match /cmd\(\"INST COLLECT/ #"
+          expect(stdout.string).to_not match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring hazardous warnings"
+        end
+      end
+    end
+
+    describe "cmd_raw" do
+      it "sends a command" do
+        capture_io do |stdout|
+          cmd_raw("INST ABORT")
+          expect(stdout.string).to match /cmd_raw\(\"INST ABORT\"\)/ #"
+        end
+      end
+
+      it "checks parameter ranges" do
+        expect { cmd_raw("INST COLLECT with TYPE 0, DURATION 20") }.to raise_error(/Command parameter 'INST COLLECT DURATION' = 20 not in valid range/)
+      end
+
+      it "prompts for a hazardous command" do
+        capture_io do |stdout|
+          expect(self).to receive(:gets) { 'y' } # Send hazardous command
+          cmd_raw("INST COLLECT with TYPE 1")
+
+          expect(stdout.string).to match /cmd_raw\(\"INST COLLECT/ #"
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring hazardous warnings"
+          stdout.rewind
+
+          expect(self).to receive(:gets) { 'n' } # Don't send hazardous
+          expect(self).to receive(:gets) { 'y' } # Stop running script
+          cmd_raw("INST COLLECT with TYPE 1")
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+        end
+      end
+    end
+
+    describe "cmd_raw_no_range_check" do
+      it "sends an out of range command" do
+        expect { cmd_raw_no_range_check("INST COLLECT with TYPE 0, DURATION 20") }.to_not raise_error
+      end
+
+      it "prompts for a hazardous command" do
+        capture_io do |stdout|
+          expect(self).to receive(:gets) { 'y' } # Send hazardous command
+          cmd_raw_no_range_check("INST COLLECT with TYPE 1")
+
+          expect(stdout.string).to match /cmd_raw\(\"INST COLLECT/ #"
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring hazardous warnings"
+          stdout.rewind
+
+          expect(self).to receive(:gets) { 'n' } # Don't send hazardous
+          expect(self).to receive(:gets) { 'y' } # Stop running script
+          cmd_raw_no_range_check("INST COLLECT with TYPE 1")
+          expect(stdout.string).to match "Warning: Command INST COLLECT is Hazardous"
+        end
+      end
+    end
+
+    describe "cmd_raw_no_hazardous_check" do
+      it "checks parameter ranges" do
+        expect { cmd_raw_no_hazardous_check("INST COLLECT with TYPE 1, DURATION 20") }.to raise_error(/Command parameter 'INST COLLECT DURATION' = 20 not in valid range/)
+      end
+
+      it "sends a hazardous command without prompting" do
+        capture_io do |stdout|
+          cmd_raw_no_hazardous_check("INST COLLECT with TYPE 1")
+          expect(stdout.string).to match /cmd_raw\(\"INST COLLECT/ #"
+          expect(stdout.string).to_not match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to_not match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring hazardous warnings"
+        end
+      end
+    end
+
+    describe "cmd_raw_no_checks" do
+      it "sends an out of range hazardous command without prompting" do
+        capture_io do |stdout|
+          cmd_raw_no_checks("INST COLLECT with TYPE 1, DURATION 20")
+          expect(stdout.string).to match /cmd_raw\(\"INST COLLECT/ #"
+          expect(stdout.string).to_not match "Warning: Command INST COLLECT is Hazardous"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring range checks"
+          expect(stdout.string).to match "Command INST COLLECT being sent ignoring hazardous warnings"
+        end
+      end
+    end
+
+    describe "send_raw" do
+      it "sends data to the write_raw interface method" do
+        expect_any_instance_of(Interface).to receive(:write_raw).with('\x00')
+        send_raw('INST_INT', '\x00')
+      end
+    end
+
+    describe "send_raw_file" do
+      it "sends file data to the write_raw interface method" do
+        file = File.open('raw_test_file.bin','wb')
+        file.write '\x00\x01\x02\x03'
+        file.close
+
+        expect_any_instance_of(Interface).to receive(:write_raw).with('\x00\x01\x02\x03')
+
+        send_raw_file('INST_INT', 'raw_test_file.bin')
+
+        File.delete('raw_test_file.bin')
+      end
+    end
+
+    describe "get_cmd_list" do
+      it "returns all the target commands" do
+        list = get_cmd_list("INST")
+        # Only check for the collect command to make this test list dependent
+        # on the demo INST command definition file
+        expect(list).to include(["COLLECT", "Starts a collect on the instrument"])
+      end
+    end
+
+    describe "get_cmd_param_list" do
+      it "returns all the parameters for a command" do
+        list = get_cmd_param_list("INST", "COLLECT")
+        expect(list).to include(["TYPE", 0, {"NORMAL"=>0, "SPECIAL"=>1}, "Collect type", nil, nil, true])
+      end
+    end
+
+    describe "get_cmd_hazardous" do
+      it "returns whether a command is hazardous" do
+        expect(get_cmd_hazardous("INST", "COLLECT", {"TYPE"=>"NORMAL"})).to be false
+        expect(get_cmd_hazardous("INST", "COLLECT", {"TYPE"=>"SPECIAL"})).to be true
+      end
+    end
+
+  end
+end
+