cosmos 4.0.3-java

data/spec/system/target_spec.rb

@@ -0,0 +1,304 @@
+# 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/system/target'
+require 'tempfile'
+
+module Cosmos
+
+  describe Target do
+    after(:all) do
+      FileUtils.rm_rf File.join(Cosmos::USERPATH,'target_spec_temp')
+    end
+
+    describe "initialize" do
+      it "creates a target with the given name" do
+        expect(Target.new("TGT").name).to eql "TGT"
+      end
+
+      it "creates a target with the given substitute name" do
+        tgt = Target.new("TGT","TGT2")
+        expect(tgt.name).to eql "TGT2"
+        expect(tgt.original_name).to eql "TGT"
+      end
+
+      it "creates a target with the default dir" do
+        expect(Target.new("TGT").dir).to eql File.join(USERPATH,'config','targets','TGT')
+      end
+
+      it "creates a target with an override path" do
+        saved = File.join(USERPATH,'saved')
+        expect(Target.new("TGT",nil,saved).dir).to eql File.join(saved,'TGT')
+      end
+
+      it "records all the command and telemetry files in the target directory" do
+        tgt_path = File.join(Cosmos::USERPATH,'target_spec_temp')
+        tgt_name = "TEST"
+        cmd_tlm = File.join(tgt_path,tgt_name,'cmd_tlm')
+        FileUtils.mkdir_p(cmd_tlm)
+        File.open(File.join(cmd_tlm,'cmd1.txt'),'w') {}
+        File.open(File.join(cmd_tlm,'cmd2.txt'),'w') {}
+        File.open(File.join(cmd_tlm,'tlm1.txt'),'w') {}
+        File.open(File.join(cmd_tlm,'tlm2.txt'),'w') {}
+        File.open(File.join(cmd_tlm,'tlm2.txt~'),'w') {}
+        File.open(File.join(cmd_tlm,'tlm2.txt.mine'),'w') {}
+        File.open(File.join(cmd_tlm,'tlm3.xtce'),'w') {}
+        File.open(File.join(cmd_tlm,'tlm3.xtce~'),'w') {}
+        File.open(File.join(cmd_tlm,'tlm3.xtce.bak'),'w') {}
+
+        tgt = Target.new(tgt_name,nil,tgt_path)
+        expect(tgt.dir).to eql File.join(tgt_path,tgt_name)
+        files = Dir[File.join(cmd_tlm,'*.txt')]
+        files.concat(Dir[File.join(cmd_tlm,'*.xtce')])
+        expect(files).not_to be_empty
+        expect(tgt.cmd_tlm_files.length).to eql 5
+        expect(tgt.cmd_tlm_files.sort).to eql files.sort
+
+        FileUtils.rm_r(tgt_path)
+      end
+
+      it "processes a target.txt in the target directory" do
+        tgt_path = File.join(Cosmos::USERPATH,'target_spec_temp')
+        tgt_name = "TEST"
+        tgt_dir = File.join(tgt_path,tgt_name)
+        FileUtils.mkdir_p(tgt_dir)
+        File.open(File.join(tgt_dir,'target.txt'),'w') do |file|
+          file.puts("IGNORE_PARAMETER TEST")
+        end
+
+        tgt = Target.new(tgt_name,nil,tgt_path)
+        expect(tgt.dir).to eql tgt_dir
+        expect(tgt.ignored_parameters).to eql ["TEST"]
+
+        FileUtils.rm_r(tgt_path)
+      end
+
+      it "processes an alternative target.txt in the target directory" do
+        tgt_path = File.join(Cosmos::USERPATH,'target_spec_temp')
+        tgt_name = "TEST"
+        tgt_dir = File.join(tgt_path,tgt_name)
+        FileUtils.mkdir_p(tgt_dir)
+        File.open(File.join(tgt_dir,'target_other.txt'),'w') do |file|
+          file.puts("IGNORE_PARAMETER BOB")
+        end
+
+        tgt = Target.new(tgt_name,nil,tgt_path, 'target_other.txt')
+        expect(tgt.dir).to eql tgt_dir
+        expect(tgt.ignored_parameters).to eql ["BOB"]
+
+        FileUtils.rm_r(tgt_path)
+      end
+    end
+
+    describe "process_file" do
+      it "complains about unknown keywords" do
+        tf = Tempfile.new('unittest')
+        tf.puts("BLAH")
+        tf.close
+        expect { Target.new("TGT").process_file(tf.path) }.to raise_error(ConfigParser::Error, /Unknown keyword 'BLAH'/)
+        tf.unlink
+      end
+
+      context "with REQUIRE" do
+        it "takes 1 parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("REQUIRE")
+          tf.close
+          expect { Target.new("INST").process_file(tf.path) }.to raise_error(ConfigParser::Error, /Not enough parameters for REQUIRE./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("REQUIRE my_file.rb TRUE")
+          tf.close
+          expect { Target.new("INST").process_file(tf.path) }.to raise_error(ConfigParser::Error, /Too many parameters for REQUIRE./)
+          tf.unlink
+        end
+
+        it "complains if the file doesn't exist" do
+          tf = Tempfile.new('unittest')
+          tf.puts("REQUIRE my_file.rb")
+          tf.close
+          expect { Target.new("INST").process_file(tf.path) }.to raise_error(ConfigParser::Error, /Unable to require my_file.rb/)
+          tf.unlink
+        end
+
+        it "requires the file in the target lib directory over system lib" do
+          filename1 = File.join(Cosmos::USERPATH, 'config', 'targets', 'INST', 'lib', 'tgt_file.rb')
+          File.open(filename1, 'w') do |file|
+            file.puts "class TgtLibFile"
+            file.puts "end"
+          end
+          filename2 = File.join(Cosmos::USERPATH, 'lib', 'tgt_file.rb')
+          File.open(filename2, 'w') do |file|
+            file.puts "class SystemLibFile"
+            file.puts "end"
+          end
+
+          tf = Tempfile.new('unittest')
+          tf.puts("REQUIRE tgt_file.rb")
+          tf.close
+          Target.new("INST").process_file(tf.path)
+          expect { TgtLibFile.new }.to_not raise_error
+          expect(Object.const_defined?('TgtLibFile')).to be true
+          expect(Object.const_defined?('SystemLibFile')).to be false
+          File.delete filename1
+          File.delete filename2
+          tf.unlink
+        end
+
+        it "requires the file in the system lib directory" do
+          filename = File.join(Cosmos::USERPATH, 'lib', 'system_file.rb')
+          File.open(filename, 'w') do |file|
+            file.puts "class SystemFile"
+            file.puts "end"
+          end
+          tf = Tempfile.new('unittest')
+          tf.puts("REQUIRE system_file.rb")
+          tf.close
+          Target.new("INST").process_file(tf.path)
+          expect { SystemFile.new }.to_not raise_error
+          File.delete filename
+          tf.unlink
+        end
+      end
+
+      context "with IGNORE_PARAMETER" do
+        it "takes 1 parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("IGNORE_PARAMETER")
+          tf.close
+          expect { Target.new("TGT").process_file(tf.path) }.to raise_error(ConfigParser::Error, /Not enough parameters for IGNORE_PARAMETER./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("IGNORE_PARAMETER my_file.rb TRUE")
+          tf.close
+          expect { Target.new("TGT").process_file(tf.path) }.to raise_error(ConfigParser::Error, /Too many parameters for IGNORE_PARAMETER./)
+          tf.unlink
+        end
+
+        it "stores the parameter" do
+          tf = Tempfile.new('unittest')
+          tf.puts("IGNORE_PARAMETER TEST")
+          tf.close
+          tgt = Target.new("TGT")
+          tgt.process_file(tf.path)
+          expect(tgt.ignored_parameters).to eql ["TEST"]
+          tf.unlink
+        end
+      end
+
+      context "with COMMANDS and TELEMETRY" do
+        it "takes 1 parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("COMMANDS")
+          tf.close
+          expect { Target.new("TGT").process_file(tf.path) }.to raise_error(ConfigParser::Error, /Not enough parameters for COMMANDS./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("COMMANDS tgt_cmds.txt TRUE")
+          tf.close
+          expect { Target.new("TGT").process_file(tf.path) }.to raise_error(ConfigParser::Error, /Too many parameters for COMMANDS./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("TELEMETRY")
+          tf.close
+          expect { Target.new("TGT").process_file(tf.path) }.to raise_error(ConfigParser::Error, /Not enough parameters for TELEMETRY./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("TELEMETRY tgt_tlm.txt TRUE")
+          tf.close
+          expect { Target.new("TGT").process_file(tf.path) }.to raise_error(ConfigParser::Error, /Too many parameters for TELEMETRY./)
+          tf.unlink
+        end
+
+        it "stores the filename" do
+          tgt_path = File.join(Cosmos::USERPATH,'target_spec_temp')
+          tgt_name = "TEST"
+          tgt_dir = File.join(tgt_path,tgt_name)
+          FileUtils.mkdir_p(tgt_dir)
+          FileUtils.mkdir_p(tgt_dir + '/cmd_tlm')
+          File.open(tgt_dir + '/cmd_tlm/tgt_cmds.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_cmds2.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_cmds3.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_tlm.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_tlm2.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_tlm3.txt', 'w') {|file| file.puts "# comment"}
+          File.open(File.join(tgt_dir,'target.txt'),'w') do |file|
+            file.puts("COMMANDS tgt_cmds2.txt")
+            file.puts("TELEMETRY tgt_tlm3.txt")
+          end
+
+          tgt = Target.new(tgt_name,nil,tgt_path)
+          expect(tgt.dir).to eql tgt_dir
+          expect(tgt.cmd_tlm_files.length).to eql 2
+          expect(tgt.cmd_tlm_files).to eql [tgt_dir + '/cmd_tlm/tgt_cmds2.txt', tgt_dir + '/cmd_tlm/tgt_tlm3.txt']
+
+          FileUtils.rm_r(tgt_dir)
+        end
+
+        it "filenames must exist" do
+          tgt_path = File.join(Cosmos::USERPATH,'target_spec_temp')
+          tgt_name = "TEST"
+          tgt_dir = File.join(tgt_path,tgt_name)
+          FileUtils.mkdir_p(tgt_dir)
+          FileUtils.mkdir_p(tgt_dir + '/cmd_tlm')
+          File.open(tgt_dir + '/cmd_tlm/tgt_cmds.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_cmds2.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_cmds3.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_tlm.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_tlm2.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_tlm3.txt', 'w') {|file| file.puts "# comment"}
+          File.open(File.join(tgt_dir,'target.txt'),'w') do |file|
+            file.puts("COMMANDS tgt_cmds4.txt")
+            file.puts("TELEMETRY tgt_tlm4.txt")
+          end
+
+          expect { Target.new(tgt_name,nil,tgt_path) }.to raise_error(ConfigParser::Error, /#{tgt_dir + '/cmd_tlm/tgt_cmds4.txt'} not found/)
+
+          FileUtils.rm_r(tgt_dir)
+        end
+
+        it "filename order must be preserved" do
+          tgt_path = File.join(Cosmos::USERPATH,'target_spec_temp')
+          tgt_name = "TEST"
+          tgt_dir = File.join(tgt_path,tgt_name)
+          FileUtils.mkdir_p(tgt_dir)
+          FileUtils.mkdir_p(tgt_dir + '/cmd_tlm')
+          File.open(tgt_dir + '/cmd_tlm/tgt_cmds.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_cmds2.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_cmds3.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_tlm.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_tlm2.txt', 'w') {|file| file.puts "# comment"}
+          File.open(tgt_dir + '/cmd_tlm/tgt_tlm3.txt', 'w') {|file| file.puts "# comment"}
+          File.open(File.join(tgt_dir,'target.txt'),'w') do |file|
+            file.puts("COMMANDS tgt_cmds3.txt")
+            file.puts("COMMANDS tgt_cmds2.txt")
+            file.puts("TELEMETRY tgt_tlm3.txt")
+            file.puts("TELEMETRY tgt_tlm.txt")
+          end
+
+          tgt = Target.new(tgt_name,nil,tgt_path)
+          expect(tgt.dir).to eql tgt_dir
+          expect(tgt.cmd_tlm_files.length).to eql 4
+          expect(tgt.cmd_tlm_files).to eql [tgt_dir + '/cmd_tlm/tgt_cmds3.txt', tgt_dir + '/cmd_tlm/tgt_cmds2.txt', tgt_dir + '/cmd_tlm/tgt_tlm3.txt', tgt_dir + '/cmd_tlm/tgt_tlm.txt']
+
+          FileUtils.rm_rf(tgt_dir)
+        end
+      end
+    end
+
+  end
+end

data/spec/tools/cmd_tlm_server/api_spec.rb

@@ -0,0 +1,1123 @@
+# 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/tools/cmd_tlm_server/cmd_tlm_server'
+require 'cosmos/tools/cmd_tlm_server/api'
+
+module Cosmos
+
+  describe Api 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 INT interface.rb'
+        file.puts 'ROUTER ROUTE interface.rb'
+      end
+    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?)
+      allow_any_instance_of(Interface).to receive(:connect)
+      allow_any_instance_of(Interface).to receive(:disconnect)
+      allow_any_instance_of(Interface).to receive(:write_raw)
+      allow_any_instance_of(Interface).to receive(:read)
+      @api = CmdTlmServer.new
+      allow(@api.commanding).to receive(:send_command_to_target)
+    end
+
+    after(:each) do
+      @api.stop
+    end
+
+    def test_cmd_unknown(method)
+      expect { @api.send(method,"BLAH COLLECT with TYPE NORMAL") }.to raise_error(/does not exist/)
+      expect { @api.send(method,"INST UNKNOWN with TYPE NORMAL") }.to raise_error(/does not exist/)
+      expect { @api.send(method,"INST COLLECT with BLAH NORMAL") }.to raise_error(/does not exist/)
+      expect { @api.send(method,"BLAH","COLLECT","TYPE"=>"NORMAL") }.to raise_error(/does not exist/)
+      expect { @api.send(method,"INST","UNKNOWN","TYPE"=>"NORMAL") }.to raise_error(/does not exist/)
+      expect { @api.send(method,"INST","COLLECT","BLAH"=>"NORMAL") }.to raise_error(/does not exist/)
+    end
+
+    describe "cmd" do
+      it "complains about unknown targets, commands, and parameters" do
+        test_cmd_unknown(:cmd)
+        sleep(0.5)
+      end
+
+      it "processes a string" do
+        target_name, cmd_name, params = @api.cmd("INST COLLECT with TYPE NORMAL, DURATION 5")
+        expect(target_name).to eql 'INST'
+        expect(cmd_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>'NORMAL', 'DURATION'=>5)
+      end
+
+      it "complains if parameters are not separated by commas" do
+        expect { @api.cmd("INST COLLECT with TYPE NORMAL DURATION 5") }.to raise_error(/Missing comma/)
+      end
+
+      it "complains if parameters don't have values" do
+        expect { @api.cmd("INST COLLECT with TYPE") }.to raise_error(/Missing value/)
+      end
+
+      it "processes parameters" do
+        target_name, cmd_name, params = @api.cmd("INST","COLLECT","TYPE"=>"NORMAL","DURATION"=>5)
+        expect(target_name).to eql 'INST'
+        expect(cmd_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>'NORMAL', 'DURATION'=>5)
+      end
+
+      it "processes commands without parameters" do
+        target_name, cmd_name, params = @api.cmd("INST","ABORT")
+        expect(target_name).to eql 'INST'
+        expect(cmd_name).to eql 'ABORT'
+        expect(params).to be {}
+      end
+
+      it "complains about too many parameters" do
+        expect { @api.cmd("INST","COLLECT","TYPE","DURATION") }.to raise_error(/Invalid number of arguments/)
+      end
+
+      it "warns about required parameters" do
+        expect { @api.cmd("INST COLLECT with DURATION 5") }.to raise_error(/Required/)
+      end
+
+      it "warns about out of range parameters" do
+        expect { @api.cmd("INST COLLECT with TYPE NORMAL, DURATION 1000") }.to raise_error(/not in valid range/)
+      end
+
+      it "warns about hazardous parameters" do
+        expect { @api.cmd("INST COLLECT with TYPE SPECIAL") }.to raise_error(/Hazardous/)
+      end
+
+      it "warns about hazardous commands" do
+        expect { @api.cmd("INST CLEAR") }.to raise_error(/Hazardous/)
+      end
+    end
+
+    describe "cmd_no_range_check" do
+      it "complains about unknown targets, commands, and parameters" do
+        test_cmd_unknown(:cmd_no_range_check)
+      end
+
+      it "processes a string" do
+        target_name, cmd_no_range_check_name, params = @api.cmd_no_range_check("INST COLLECT with TYPE NORMAL, DURATION 5")
+        expect(target_name).to eql 'INST'
+        expect(cmd_no_range_check_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>'NORMAL', 'DURATION'=>5)
+      end
+
+      it "processes parameters" do
+        target_name, cmd_no_range_check_name, params = @api.cmd_no_range_check("INST","COLLECT","TYPE"=>"NORMAL","DURATION"=>5)
+        expect(target_name).to eql 'INST'
+        expect(cmd_no_range_check_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>'NORMAL', 'DURATION'=>5)
+      end
+
+      it "warns about required parameters" do
+        expect { @api.cmd_no_range_check("INST COLLECT with DURATION 5") }.to raise_error(/Required/)
+      end
+
+      it "does not warn about out of range parameters" do
+        expect { @api.cmd_no_range_check("INST COLLECT with TYPE NORMAL, DURATION 1000") }.to_not raise_error
+      end
+
+      it "warns about hazardous parameters" do
+        expect { @api.cmd_no_range_check("INST COLLECT with TYPE SPECIAL") }.to raise_error(/Hazardous/)
+      end
+
+      it "warns about hazardous commands" do
+        expect { @api.cmd_no_range_check("INST CLEAR") }.to raise_error(/Hazardous/)
+      end
+    end
+
+    describe "cmd_no_hazardous_check" do
+      it "complains about unknown targets, commands, and parameters" do
+        test_cmd_unknown(:cmd_no_hazardous_check)
+      end
+
+      it "processes a string" do
+        target_name, cmd_no_hazardous_check_name, params = @api.cmd_no_hazardous_check("INST COLLECT with TYPE NORMAL, DURATION 5")
+        expect(target_name).to eql 'INST'
+        expect(cmd_no_hazardous_check_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>'NORMAL', 'DURATION'=>5)
+      end
+
+      it "processes parameters" do
+        target_name, cmd_no_hazardous_check_name, params = @api.cmd_no_hazardous_check("INST","COLLECT","TYPE"=>"NORMAL","DURATION"=>5)
+        expect(target_name).to eql 'INST'
+        expect(cmd_no_hazardous_check_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>'NORMAL', 'DURATION'=>5)
+      end
+
+      it "processes parameters that are strings" do
+        target_name, cmd_name, params = @api.cmd_no_hazardous_check("INST ASCIICMD with STRING 'ARM LASER'")
+        expect(target_name).to eql 'INST'
+        expect(cmd_name).to eql 'ASCIICMD'
+        expect(params).to include('STRING'=>'ARM LASER')
+      end
+
+      it "warns about required parameters" do
+        expect { @api.cmd_no_hazardous_check("INST COLLECT with DURATION 5") }.to raise_error(/Required/)
+      end
+
+      it "warns about out of range parameters" do
+        expect { @api.cmd_no_hazardous_check("INST COLLECT with TYPE NORMAL, DURATION 1000") }.to raise_error(/not in valid range/)
+      end
+
+      it "does not warn about hazardous parameters" do
+        expect { @api.cmd_no_hazardous_check("INST COLLECT with TYPE SPECIAL") }.to_not raise_error
+      end
+
+      it "does not warn about hazardous commands" do
+        expect { @api.cmd_no_hazardous_check("INST CLEAR") }.to_not raise_error
+      end
+    end
+
+    describe "cmd_no_checks" do
+      it "complains about unknown targets, commands, and parameters" do
+        test_cmd_unknown(:cmd_no_checks)
+      end
+
+      it "processes a string" do
+        target_name, cmd_no_checks_name, params = @api.cmd_no_checks("INST COLLECT with TYPE NORMAL, DURATION 5")
+        expect(target_name).to eql 'INST'
+        expect(cmd_no_checks_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>'NORMAL', 'DURATION'=>5)
+      end
+
+      it "processes parameters" do
+        target_name, cmd_no_checks_name, params = @api.cmd_no_checks("INST","COLLECT","TYPE"=>"NORMAL","DURATION"=>5)
+        expect(target_name).to eql 'INST'
+        expect(cmd_no_checks_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>'NORMAL', 'DURATION'=>5)
+      end
+
+      it "warns about required parameters" do
+        expect { @api.cmd_no_checks("INST COLLECT with DURATION 5") }.to raise_error(/Required/)
+      end
+
+      it "does not warn about out of range parameters" do
+        expect { @api.cmd_no_checks("INST COLLECT with TYPE NORMAL, DURATION 1000") }.to_not raise_error
+      end
+
+      it "does not warn about hazardous parameters" do
+        expect { @api.cmd_no_checks("INST COLLECT with TYPE SPECIAL") }.to_not raise_error
+      end
+
+      it "does not warn about hazardous commands" do
+        expect { @api.cmd_no_checks("INST CLEAR") }.to_not raise_error
+      end
+    end
+
+    describe "cmd_raw" do
+      it "complains about unknown targets, commands, and parameters" do
+        test_cmd_unknown(:cmd_raw)
+      end
+
+      it "processes a string" do
+        target_name, cmd_name, params = @api.cmd_raw("INST COLLECT with TYPE 0, DURATION 5")
+        expect(target_name).to eql 'INST'
+        expect(cmd_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>0, 'DURATION'=>5)
+      end
+
+      it "complains if parameters are not separated by commas" do
+        expect { @api.cmd_raw("INST COLLECT with TYPE 0 DURATION 5") }.to raise_error(/Missing comma/)
+      end
+
+      it "complains if parameters don't have values" do
+        expect { @api.cmd_raw("INST COLLECT with TYPE") }.to raise_error(/Missing value/)
+      end
+
+      it "processes parameters" do
+        target_name, cmd_name, params = @api.cmd_raw("INST","COLLECT","TYPE"=>0,"DURATION"=>5)
+        expect(target_name).to eql 'INST'
+        expect(cmd_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>0, 'DURATION'=>5)
+      end
+
+      it "processes commands without parameters" do
+        target_name, cmd_name, params = @api.cmd_raw("INST","ABORT")
+        expect(target_name).to eql 'INST'
+        expect(cmd_name).to eql 'ABORT'
+        expect(params).to be {}
+      end
+
+      it "complains about too many parameters" do
+        expect { @api.cmd_raw("INST","COLLECT","TYPE","DURATION") }.to raise_error(/Invalid number of arguments/)
+      end
+
+      it "warns about required parameters" do
+        expect { @api.cmd_raw("INST COLLECT with DURATION 5") }.to raise_error(/Required/)
+      end
+
+      it "warns about out of range parameters" do
+        expect { @api.cmd_raw("INST COLLECT with TYPE 0, DURATION 1000") }.to raise_error(/not in valid range/)
+      end
+
+      it "warns about hazardous parameters" do
+        expect { @api.cmd_raw("INST COLLECT with TYPE 1") }.to raise_error(/Hazardous/)
+      end
+
+      it "warns about hazardous commands" do
+        expect { @api.cmd_raw("INST CLEAR") }.to raise_error(/Hazardous/)
+      end
+    end
+
+    describe "cmd_no_range_check" do
+      it "complains about unknown targets, commands, and parameters" do
+        test_cmd_unknown(:cmd_raw_no_range_check)
+      end
+
+      it "processes a string" do
+        target_name, cmd_no_range_check_name, params = @api.cmd_raw_no_range_check("INST COLLECT with TYPE 0, DURATION 5")
+        expect(target_name).to eql 'INST'
+        expect(cmd_no_range_check_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>0, 'DURATION'=>5)
+      end
+
+      it "processes parameters" do
+        target_name, cmd_no_range_check_name, params = @api.cmd_raw_no_range_check("INST","COLLECT","TYPE"=>0,"DURATION"=>5)
+        expect(target_name).to eql 'INST'
+        expect(cmd_no_range_check_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>0, 'DURATION'=>5)
+      end
+
+      it "warns about required parameters" do
+        expect { @api.cmd_raw_no_range_check("INST COLLECT with DURATION 5") }.to raise_error(/Required/)
+      end
+
+      it "does not warn about out of range parameters" do
+        expect { @api.cmd_raw_no_range_check("INST COLLECT with TYPE 0, DURATION 1000") }.to_not raise_error
+      end
+
+      it "warns about hazardous parameters" do
+        expect { @api.cmd_raw_no_range_check("INST COLLECT with TYPE 1") }.to raise_error(/Hazardous/)
+      end
+
+      it "warns about hazardous commands" do
+        expect { @api.cmd_raw_no_range_check("INST CLEAR") }.to raise_error(/Hazardous/)
+      end
+    end
+
+    describe "cmd_raw_no_hazardous_check" do
+      it "complains about unknown targets, commands, and parameters" do
+        test_cmd_unknown(:cmd_raw_no_hazardous_check)
+      end
+
+      it "processes a string" do
+        target_name, cmd_no_hazardous_check_name, params = @api.cmd_raw_no_hazardous_check("INST COLLECT with TYPE 0, DURATION 5")
+        expect(target_name).to eql 'INST'
+        expect(cmd_no_hazardous_check_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>0, 'DURATION'=>5)
+      end
+
+      it "processes parameters" do
+        target_name, cmd_no_hazardous_check_name, params = @api.cmd_raw_no_hazardous_check("INST","COLLECT","TYPE"=>0,"DURATION"=>5)
+        expect(target_name).to eql 'INST'
+        expect(cmd_no_hazardous_check_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>0, 'DURATION'=>5)
+      end
+
+      it "processes parameters that are strings" do
+        target_name, cmd_name, params = @api.cmd_raw_no_hazardous_check("INST ASCIICMD with STRING 'ARM LASER'")
+        expect(target_name).to eql 'INST'
+        expect(cmd_name).to eql 'ASCIICMD'
+        expect(params).to include('STRING'=>'ARM LASER')
+      end
+
+      it "warns about required parameters" do
+        expect { @api.cmd_raw_no_hazardous_check("INST COLLECT with DURATION 5") }.to raise_error(/Required/)
+      end
+
+      it "warns about out of range parameters" do
+        expect { @api.cmd_raw_no_hazardous_check("INST COLLECT with TYPE 0, DURATION 1000") }.to raise_error(/not in valid range/)
+      end
+
+      it "does not warn about hazardous parameters" do
+        expect { @api.cmd_raw_no_hazardous_check("INST COLLECT with TYPE 1") }.to_not raise_error
+      end
+
+      it "does not warn about hazardous commands" do
+        expect { @api.cmd_raw_no_hazardous_check("INST CLEAR") }.to_not raise_error
+      end
+    end
+
+    describe "cmd_raw_no_checks" do
+      it "complains about unknown targets, commands, and parameters" do
+        test_cmd_unknown(:cmd_raw_no_checks)
+      end
+
+      it "processes a string" do
+        target_name, cmd_no_checks_name, params = @api.cmd_raw_no_checks("INST COLLECT with TYPE 0, DURATION 5")
+        expect(target_name).to eql 'INST'
+        expect(cmd_no_checks_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>0, 'DURATION'=>5)
+      end
+
+      it "processes parameters" do
+        target_name, cmd_no_checks_name, params = @api.cmd_raw_no_checks("INST","COLLECT","TYPE"=>0,"DURATION"=>5)
+        expect(target_name).to eql 'INST'
+        expect(cmd_no_checks_name).to eql 'COLLECT'
+        expect(params).to include('TYPE'=>0, 'DURATION'=>5)
+      end
+
+      it "warns about required parameters" do
+        expect { @api.cmd_raw_no_checks("INST COLLECT with DURATION 5") }.to raise_error(/Required/)
+      end
+
+      it "does not warn about out of range parameters" do
+        expect { @api.cmd_raw_no_checks("INST COLLECT with TYPE 0, DURATION 1000") }.to_not raise_error
+      end
+
+      it "does not warn about hazardous parameters" do
+        expect { @api.cmd_raw_no_checks("INST COLLECT with TYPE 1") }.to_not raise_error
+      end
+
+      it "does not warn about hazardous commands" do
+        expect { @api.cmd_raw_no_checks("INST CLEAR") }.to_not raise_error
+      end
+    end
+
+    describe "get_cmd_list" do
+      it "returns command names sorted" do
+        result = @api.get_cmd_list("INST")
+        expect(result.sort).to eql result
+      end
+
+      it "complains with a unknown target" do
+        expect { @api.get_cmd_list("BLAH") }.to raise_error(/does not exist/)
+      end
+
+      it "returns command names and descriptions for a given target" do
+        result = @api.get_cmd_list("INST")
+        expect(result[0][0]).to eql "ABORT"
+        # The second parameter is the description ... only test one
+        expect(result[0][1]).to eql "Aborts a collect on the instrument"
+        expect(result[1][0]).to eql "ARYCMD"
+        expect(result[2][0]).to eql "ASCIICMD"
+        expect(result[3][0]).to eql "CLEAR"
+        expect(result[4][0]).to eql "COLLECT"
+        expect(result[5][0]).to eql "COSMOS_ERROR_HANDLE"
+        expect(result[6][0]).to eql "COSMOS_ERROR_IGNORE"
+        expect(result[7][0]).to eql "COSMOS_HANDSHAKE_DS"
+        expect(result[8][0]).to eql "COSMOS_HANDSHAKE_EN"
+        expect(result[9][0]).to eql "FLTCMD"
+        expect(result[10][0]).to eql "LINC_COMMAND"
+        expect(result[11][0]).to eql "SETPARAMS"
+        expect(result[12][0]).to eql "SLRPNLDEPLOY"
+        expect(result[13][0]).to eql "SLRPNLRESET"
+      end
+    end
+
+    describe "get_cmd_param_list" do
+      it "returns parameters for the command" do
+        result = @api.get_cmd_param_list("INST","COLLECT")
+        # Each element in the results array contains:
+        #   name, default, states, description, full units, units, required
+        expect(result).to include ['TYPE',0,{"NORMAL"=>0,"SPECIAL"=>1},'Collect type',nil,nil,true]
+        expect(result).to include ['TEMP',0.0,nil,'Collect temperature','Celcius','C',false]
+      end
+
+      it "returns array parameters for the command" do
+        result = @api.get_cmd_param_list("INST","ARYCMD")
+        # Each element in the results array contains:
+        #   name, default, states, description, full units, units, required
+        expect(result).to include ['ARRAY',[],nil,'Array parameter',nil,nil,false]
+      end
+    end
+
+    describe "get_cmd_hazardous" do
+      it "returns whether the command with parameters is hazardous" do
+        expect(@api.get_cmd_hazardous("INST","COLLECT",{"TYPE"=>"NORMAL"})).to be false
+        expect(@api.get_cmd_hazardous("INST","COLLECT",{"TYPE"=>"SPECIAL"})).to be true
+      end
+
+      it "returns whether the command is hazardous" do
+        expect(@api.get_cmd_hazardous("INST","CLEAR")).to be true
+      end
+    end
+
+    describe "get_cmd_value" do
+      it "returns command values" do
+        time = Time.now
+        packet = System.commands.packet("INST", "COLLECT")
+        packet.received_time = time
+        packet.restore_defaults
+        packet.received_count = 5
+        expect(@api.get_cmd_value("INST", "COLLECT", "TYPE")).to eql 'NORMAL'
+        expect(@api.get_cmd_value("INST", "COLLECT", "RECEIVED_TIMEFORMATTED")).to eql time.formatted
+        expect(@api.get_cmd_value("INST", "COLLECT", "RECEIVED_TIMESECONDS")).to eql time.to_f
+        expect(@api.get_cmd_value("INST", "COLLECT", "RECEIVED_COUNT")).to eql 5
+      end
+    end
+
+    describe "get_cmd_time" do
+      it "returns command times" do
+        time = Time.now
+        time2 = Time.now + 2
+        packet = System.commands.packet("INST", "COLLECT")
+        packet.received_time = time
+        packet2 = System.commands.packet("SYSTEM", "STARTLOGGING")
+        packet2.received_time = time2
+        expect(@api.get_cmd_time()).to eql ['SYSTEM', 'STARTLOGGING', time2.tv_sec, time2.tv_usec]
+        expect(@api.get_cmd_time('INST')).to eql ['INST', 'COLLECT', time.tv_sec, time.tv_usec]
+        expect(@api.get_cmd_time('SYSTEM')).to eql ['SYSTEM', 'STARTLOGGING', time2.tv_sec, time2.tv_usec]
+        expect(@api.get_cmd_time('INST', 'COLLECT')).to eql ['INST', 'COLLECT', time.tv_sec, time.tv_usec]
+        expect(@api.get_cmd_time('SYSTEM', 'STARTLOGGING')).to eql ['SYSTEM', 'STARTLOGGING', time2.tv_sec, time2.tv_usec]
+      end
+    end
+
+    def test_tlm_unknown(method)
+      expect { @api.send(method,"BLAH HEALTH_STATUS COLLECTS") }.to raise_error(/does not exist/)
+      expect { @api.send(method,"INST UNKNOWN COLLECTS") }.to raise_error(/does not exist/)
+      expect { @api.send(method,"INST HEALTH_STATUS BLAH") }.to raise_error(/does not exist/)
+      expect { @api.send(method,"BLAH","HEALTH_STATUS","COLLECTS") }.to raise_error(/does not exist/)
+      expect { @api.send(method,"INST","UNKNOWN","COLLECTS") }.to raise_error(/does not exist/)
+      expect { @api.send(method,"INST","HEALTH_STATUS","BLAH") }.to raise_error(/does not exist/)
+    end
+
+    describe "tlm" do
+      it "complains about unknown targets, commands, and parameters" do
+        test_tlm_unknown(:tlm)
+      end
+
+      it "processes a string" do
+        value = @api.tlm("INST HEALTH_STATUS TEMP1")
+        expect(value).to eql -100.0
+      end
+
+      it "processes parameters" do
+        value = @api.tlm("INST","HEALTH_STATUS","TEMP1")
+      end
+
+      it "complains if too many parameters" do
+        expect { @api.tlm("INST","HEALTH_STATUS","TEMP1","TEMP2") }.to raise_error(/Invalid number of arguments/)
+      end
+    end
+
+    describe "tlm_raw" do
+      it "complains about unknown targets, commands, and parameters" do
+        test_tlm_unknown(:tlm_raw)
+      end
+
+      it "processes a string" do
+        expect(@api.tlm_raw("INST HEALTH_STATUS TEMP1")).to eql 0
+      end
+
+      it "returns the value using LATEST" do
+        expect(@api.tlm_raw("INST LATEST TEMP1")).to eql 0
+      end
+
+      it "processes parameters" do
+        expect(@api.tlm_raw("INST","HEALTH_STATUS","TEMP1")).to eql 0
+      end
+    end
+
+    describe "tlm_formatted" do
+      it "complains about unknown targets, commands, and parameters" do
+        test_tlm_unknown(:tlm_formatted)
+      end
+
+      it "processes a string" do
+        expect(@api.tlm_formatted("INST HEALTH_STATUS TEMP1")).to eql "-100.000"
+      end
+
+      it "returns the value using LATEST" do
+        expect(@api.tlm_formatted("INST LATEST TEMP1")).to eql "-100.000"
+      end
+
+      it "processes parameters" do
+        expect(@api.tlm_formatted("INST","HEALTH_STATUS","TEMP1")).to eql "-100.000"
+      end
+    end
+
+    describe "tlm_with_units" do
+      it "complains about unknown targets, commands, and parameters" do
+        test_tlm_unknown(:tlm_with_units)
+      end
+
+      it "processes a string" do
+        expect(@api.tlm_with_units("INST HEALTH_STATUS TEMP1")).to eql "-100.000 C"
+      end
+
+      it "returns the value using LATEST" do
+        expect(@api.tlm_with_units("INST LATEST TEMP1")).to eql "-100.000 C"
+      end
+
+      it "processes parameters" do
+        expect(@api.tlm_with_units("INST","HEALTH_STATUS","TEMP1")).to eql "-100.000 C"
+      end
+    end
+
+    describe "tlm_variable" do
+      it "complains about unknown targets, commands, and parameters" do
+        expect { @api.tlm_variable("BLAH HEALTH_STATUS COLLECTS",:RAW) }.to raise_error(/does not exist/)
+        expect { @api.tlm_variable("INST UNKNOWN COLLECTS",:RAW) }.to raise_error(/does not exist/)
+        expect { @api.tlm_variable("INST HEALTH_STATUS BLAH",:RAW) }.to raise_error(/does not exist/)
+        expect { @api.tlm_variable("BLAH","HEALTH_STATUS","COLLECTS",:RAW) }.to raise_error(/does not exist/)
+        expect { @api.tlm_variable("INST","UNKNOWN","COLLECTS",:RAW) }.to raise_error(/does not exist/)
+        expect { @api.tlm_variable("INST","HEALTH_STATUS","BLAH",:RAW) }.to raise_error(/does not exist/)
+      end
+
+      it "processes a string" do
+        expect(@api.tlm_variable("INST HEALTH_STATUS TEMP1",:CONVERTED)).to eql -100.0
+        expect(@api.tlm_variable("INST HEALTH_STATUS TEMP1",:RAW)).to eql 0
+        expect(@api.tlm_variable("INST HEALTH_STATUS TEMP1",:FORMATTED)).to eql "-100.000"
+        expect(@api.tlm_variable("INST HEALTH_STATUS TEMP1",:WITH_UNITS)).to eql "-100.000 C"
+      end
+
+      it "returns the value using LATEST" do
+        expect(@api.tlm_variable("INST LATEST TEMP1",:CONVERTED)).to eql -100.0
+        expect(@api.tlm_variable("INST LATEST TEMP1",:RAW)).to eql 0
+        expect(@api.tlm_variable("INST LATEST TEMP1",:FORMATTED)).to eql "-100.000"
+        expect(@api.tlm_variable("INST LATEST TEMP1",:WITH_UNITS)).to eql "-100.000 C"
+      end
+
+      it "processes parameters" do
+        expect(@api.tlm_variable("INST","HEALTH_STATUS","TEMP1",:CONVERTED)).to eql -100.0
+        expect(@api.tlm_variable("INST","HEALTH_STATUS","TEMP1",:RAW)).to eql 0
+        expect(@api.tlm_variable("INST","HEALTH_STATUS","TEMP1",:FORMATTED)).to eql "-100.000"
+        expect(@api.tlm_variable("INST","HEALTH_STATUS","TEMP1",:WITH_UNITS)).to eql "-100.000 C"
+      end
+
+      it "complains with too many parameters" do
+        expect { @api.tlm_variable("INST","HEALTH_STATUS","TEMP1","TEMP2",:CONVERTED) }.to raise_error(/Invalid number of arguments/)
+      end
+    end
+
+    describe "set_tlm" do
+      it "complains about unknown targets, commands, and parameters" do
+        expect { @api.set_tlm("BLAH HEALTH_STATUS COLLECTS = 1") }.to raise_error(/does not exist/)
+        expect { @api.set_tlm("INST UNKNOWN COLLECTS = 1") }.to raise_error(/does not exist/)
+        expect { @api.set_tlm("INST HEALTH_STATUS BLAH = 1") }.to raise_error(/does not exist/)
+        expect { @api.set_tlm("BLAH","HEALTH_STATUS","COLLECTS",1) }.to raise_error(/does not exist/)
+        expect { @api.set_tlm("INST","UNKNOWN","COLLECTS",1) }.to raise_error(/does not exist/)
+        expect { @api.set_tlm("INST","HEALTH_STATUS","BLAH",1) }.to raise_error(/does not exist/)
+      end
+
+      it "processes a string" do
+        @api.set_tlm("INST HEALTH_STATUS TEMP1 = 0.0")
+        expect(@api.tlm("INST HEALTH_STATUS TEMP1")).to be_within(0.00001).of(-0.05759)
+      end
+
+      it "processes parameters" do
+        @api.set_tlm("INST","HEALTH_STATUS","TEMP1", 0.0)
+        expect(@api.tlm("INST HEALTH_STATUS TEMP1")).to be_within(0.00001).of(-0.05759)
+      end
+
+      it "complains with too many parameters" do
+        expect { @api.set_tlm("INST","HEALTH_STATUS","TEMP1","TEMP2",0.0) }.to raise_error(/Invalid number of arguments/)
+      end
+    end
+
+    describe "set_tlm_raw" do
+      it "complains about unknown targets, commands, and parameters" do
+        expect { @api.set_tlm_raw("BLAH HEALTH_STATUS COLLECTS = 1") }.to raise_error(/does not exist/)
+        expect { @api.set_tlm_raw("INST UNKNOWN COLLECTS = 1") }.to raise_error(/does not exist/)
+        expect { @api.set_tlm_raw("INST HEALTH_STATUS BLAH = 1") }.to raise_error(/does not exist/)
+        expect { @api.set_tlm_raw("BLAH","HEALTH_STATUS","COLLECTS",1) }.to raise_error(/does not exist/)
+        expect { @api.set_tlm_raw("INST","UNKNOWN","COLLECTS",1) }.to raise_error(/does not exist/)
+        expect { @api.set_tlm_raw("INST","HEALTH_STATUS","BLAH",1) }.to raise_error(/does not exist/)
+      end
+
+      it "processes a string" do
+        @api.set_tlm_raw("INST HEALTH_STATUS TEMP1 = 0.0")
+        expect(@api.tlm("INST HEALTH_STATUS TEMP1")).to eql -100.0
+      end
+
+      it "processes parameters" do
+        @api.set_tlm_raw("INST","HEALTH_STATUS","TEMP1", 0.0)
+        expect(@api.tlm("INST HEALTH_STATUS TEMP1")).to eql -100.0
+      end
+    end
+
+    describe "get_tlm_packet" do
+      it "complains about non-existant targets" do
+        expect { @api.get_tlm_packet("BLAH","HEALTH_STATUS") }.to raise_error(RuntimeError, "Telemetry target 'BLAH' does not exist")
+      end
+
+      it "complains about non-existant packets" do
+        expect { @api.get_tlm_packet("INST","BLAH") }.to raise_error(RuntimeError, "Telemetry packet 'INST BLAH' does not exist")
+      end
+
+      it "complains using LATEST" do
+        expect { @api.get_tlm_packet("INST","LATEST") }.to raise_error(RuntimeError, "Telemetry packet 'INST LATEST' does not exist")
+      end
+
+      it "complains about non-existant value_types" do
+        expect { @api.get_tlm_packet("INST","HEALTH_STATUS",:MINE) }.to raise_error(ArgumentError, "Unknown value type on read: MINE")
+      end
+
+      it "reads all telemetry items with their limits states" do
+        vals = @api.get_tlm_packet("INST","HEALTH_STATUS")
+        expect(vals[0][0]).to eql "RECEIVED_TIMESECONDS"
+        expect(vals[0][1]).to eql 0.0
+        expect(vals[0][2]).to be_nil
+        expect(vals[1][0]).to eql "RECEIVED_TIMEFORMATTED"
+        expect(vals[1][1]).to eql "No Packet Received Time"
+        expect(vals[1][2]).to be_nil
+        expect(vals[2][0]).to eql "RECEIVED_COUNT"
+        expect(vals[2][1]).to eql 0
+        expect(vals[2][2]).to be_nil
+        # Spot check a few more
+        expect(vals[22][0]).to eql "TEMP1"
+        expect(vals[22][1]).to eql -100.0
+        expect(vals[22][2]).to eql :RED_LOW
+        expect(vals[23][0]).to eql "TEMP2"
+        expect(vals[23][1]).to eql -100.0
+        expect(vals[23][2]).to eql :RED_LOW
+        expect(vals[24][0]).to eql "TEMP3"
+        expect(vals[24][1]).to eql -100.0
+        expect(vals[24][2]).to eql :RED_LOW
+        expect(vals[25][0]).to eql "TEMP4"
+        expect(vals[25][1]).to eql -100.0
+        expect(vals[25][2]).to eql :RED_LOW
+      end
+    end
+
+    describe "get_tlm_values" do
+      it "handles an empty request" do
+        expect(@api.get_tlm_values([])).to eql [[], [], [], :DEFAULT]
+      end
+
+      it "complains about non-existant targets" do
+        expect { @api.get_tlm_values([["BLAH","HEALTH_STATUS","TEMP1"]]) }.to raise_error(RuntimeError, "Telemetry target 'BLAH' does not exist")
+      end
+
+      it "complains about non-existant packets" do
+        expect { @api.get_tlm_values([["INST","BLAH","TEMP1"]]) }.to raise_error(RuntimeError, "Telemetry packet 'INST BLAH' does not exist")
+      end
+
+      it "complains about non-existant items" do
+        expect { @api.get_tlm_values([["INST","LATEST","BLAH"]]) }.to raise_error(RuntimeError, "Telemetry item 'INST LATEST BLAH' does not exist")
+      end
+
+      it "complains about non-existant value_types" do
+        expect { @api.get_tlm_values([["INST","HEALTH_STATUS","TEMP1"]],:MINE) }.to raise_error(ArgumentError, "Unknown value type on read: MINE")
+      end
+
+      it "complains about bad arguments" do
+        expect { @api.get_tlm_values("INST",:MINE) }.to raise_error(ArgumentError, /item_array must be nested array/)
+        expect { @api.get_tlm_values(["INST","HEALTH_STATUS","TEMP1"],:MINE) }.to raise_error(ArgumentError, /item_array must be nested array/)
+        expect { @api.get_tlm_values([["INST","HEALTH_STATUS","TEMP1"]],10) }.to raise_error(ArgumentError, /value_types must be a single symbol or array of symbols/)
+        expect { @api.get_tlm_values([["INST","HEALTH_STATUS","TEMP1"]],[10]) }.to raise_error(ArgumentError, /value_types must be a single symbol or array of symbols/)
+      end
+
+      it "reads all the specified items" do
+        items = []
+        items << %w(INST HEALTH_STATUS TEMP1)
+        items << %w(INST HEALTH_STATUS TEMP2)
+        items << %w(INST HEALTH_STATUS TEMP3)
+        items << %w(INST HEALTH_STATUS TEMP4)
+        vals = @api.get_tlm_values(items)
+        expect(vals[0][0]).to eql -100.0
+        expect(vals[0][1]).to eql -100.0
+        expect(vals[0][2]).to eql -100.0
+        expect(vals[0][3]).to eql -100.0
+        expect(vals[1][0]).to eql :RED_LOW
+        expect(vals[1][1]).to eql :RED_LOW
+        expect(vals[1][2]).to eql :RED_LOW
+        expect(vals[1][3]).to eql :RED_LOW
+        expect(vals[2][0]).to eql [-80.0, -70.0, 60.0, 80.0, -20.0, 20.0]
+        expect(vals[2][1]).to eql [-60.0, -55.0, 30.0, 35.0]
+        expect(vals[2][2]).to eql [-25.0, -10.0, 50.0, 55.0]
+        expect(vals[2][3]).to eql [-80.0, -70.0, 60.0, 80.0]
+        expect(vals[3]).to eql :DEFAULT
+      end
+
+      it "reads all the specified items with one conversion" do
+        items = []
+        items << %w(INST HEALTH_STATUS TEMP1)
+        items << %w(INST HEALTH_STATUS TEMP2)
+        items << %w(INST HEALTH_STATUS TEMP3)
+        items << %w(INST HEALTH_STATUS TEMP4)
+        vals = @api.get_tlm_values(items, :RAW)
+        expect(vals[0][0]).to eql 0
+        expect(vals[0][1]).to eql 0
+        expect(vals[0][2]).to eql 0
+        expect(vals[0][3]).to eql 0
+        expect(vals[1][0]).to eql :RED_LOW
+        expect(vals[1][1]).to eql :RED_LOW
+        expect(vals[1][2]).to eql :RED_LOW
+        expect(vals[1][3]).to eql :RED_LOW
+        expect(vals[2][0]).to eql [-80.0, -70.0, 60.0, 80.0, -20.0, 20.0]
+        expect(vals[2][1]).to eql [-60.0, -55.0, 30.0, 35.0]
+        expect(vals[2][2]).to eql [-25.0, -10.0, 50.0, 55.0]
+        expect(vals[2][3]).to eql [-80.0, -70.0, 60.0, 80.0]
+        expect(vals[3]).to eql :DEFAULT
+      end
+
+      it "reads all the specified items with different conversions" do
+        items = []
+        items << %w(INST HEALTH_STATUS TEMP1)
+        items << %w(INST HEALTH_STATUS TEMP2)
+        items << %w(INST HEALTH_STATUS TEMP3)
+        items << %w(INST HEALTH_STATUS TEMP4)
+        vals = @api.get_tlm_values(items, [:RAW, :CONVERTED, :FORMATTED, :WITH_UNITS])
+        expect(vals[0][0]).to eql 0
+        expect(vals[0][1]).to eql -100.0
+        expect(vals[0][2]).to eql "-100.000"
+        expect(vals[0][3]).to eql "-100.000 C"
+        expect(vals[1][0]).to eql :RED_LOW
+        expect(vals[1][1]).to eql :RED_LOW
+        expect(vals[1][2]).to eql :RED_LOW
+        expect(vals[1][3]).to eql :RED_LOW
+        expect(vals[2][0]).to eql [-80.0, -70.0, 60.0, 80.0, -20.0, 20.0]
+        expect(vals[2][1]).to eql [-60.0, -55.0, 30.0, 35.0]
+        expect(vals[2][2]).to eql [-25.0, -10.0, 50.0, 55.0]
+        expect(vals[2][3]).to eql [-80.0, -70.0, 60.0, 80.0]
+        expect(vals[3]).to eql :DEFAULT
+      end
+
+      it "complains if items length != conversions length" do
+        items = []
+        items << %w(INST HEALTH_STATUS TEMP1)
+        items << %w(INST HEALTH_STATUS TEMP2)
+        items << %w(INST HEALTH_STATUS TEMP3)
+        items << %w(INST HEALTH_STATUS TEMP4)
+        expect { @api.get_tlm_values(items, [:RAW, :CONVERTED]) }.to raise_error(ArgumentError, "Passed 4 items but only 2 value types")
+      end
+    end
+
+    describe "get_tlm_list" do
+      it "complains about non-existant targets" do
+        expect { @api.get_tlm_list("BLAH") }.to raise_error(RuntimeError, "Telemetry target 'BLAH' does not exist")
+      end
+
+      it "returns the sorted packet names for a target" do
+        pkts = @api.get_tlm_list("INST")
+        expect(pkts[0][0]).to eql "ADCS"
+        expect(pkts[1][0]).to eql "ERROR"
+        expect(pkts[2][0]).to eql "HANDSHAKE"
+        expect(pkts[3][0]).to eql "HEALTH_STATUS"
+        expect(pkts[4][0]).to eql "IMAGE"
+        expect(pkts[5][0]).to eql "MECH"
+        expect(pkts[6][0]).to eql "PARAMS"
+      end
+    end
+
+    describe "get_tlm_item_list" do
+      it "complains about non-existant targets" do
+        expect { @api.get_tlm_item_list("BLAH","HEALTH_STATUS") }.to raise_error(RuntimeError, "Telemetry target 'BLAH' does not exist")
+      end
+
+      it "complains about non-existant packets" do
+        expect { @api.get_tlm_item_list("INST","BLAH") }.to raise_error(RuntimeError, "Telemetry packet 'INST BLAH' does not exist")
+      end
+
+      it "returns all the items for a target/packet" do
+        items = @api.get_tlm_item_list("INST","HEALTH_STATUS")
+        expect(items[0][0]).to eql "RECEIVED_TIMESECONDS"
+        expect(items[1][0]).to eql "RECEIVED_TIMEFORMATTED"
+        expect(items[2][0]).to eql "RECEIVED_COUNT"
+        # Spot check a few more
+        expect(items[22][0]).to eql "TEMP1"
+        expect(items[22][1]).to be_nil
+        expect(items[22][2]).to eql "Temperature #1"
+        expect(items[28][0]).to eql "COLLECT_TYPE"
+        expect(items[28][1]).to include("NORMAL"=>0, "SPECIAL"=>1)
+        expect(items[28][2]).to eql "Most recent collect type"
+      end
+    end
+
+    describe "get_tlm_details" do
+      it "complains about non-existant targets" do
+        expect { @api.get_tlm_details([["BLAH","HEALTH_STATUS","TEMP1"]]) }.to raise_error(RuntimeError, "Telemetry target 'BLAH' does not exist")
+      end
+
+      it "complains about non-existant packets" do
+        expect { @api.get_tlm_details([["INST","BLAH","TEMP1"]]) }.to raise_error(RuntimeError, "Telemetry packet 'INST BLAH' does not exist")
+      end
+
+      it "complains about non-existant items" do
+        expect { @api.get_tlm_details([["INST","LATEST","BLAH"]]) }.to raise_error(RuntimeError, "Telemetry item 'INST LATEST BLAH' does not exist")
+      end
+
+      it "complains about bad parameters" do
+        expect { @api.get_tlm_details("INST") }.to raise_error(ArgumentError, /item_array must be nested array/)
+        expect { @api.get_tlm_details(["INST","LATEST","BLAH"]) }.to raise_error(ArgumentError, /item_array must be nested array/)
+      end
+
+      it "reads all the specified items" do
+        items = []
+        items << %w(INST HEALTH_STATUS TEMP1)
+        items << %w(INST HEALTH_STATUS TEMP2)
+        items << %w(INST HEALTH_STATUS TEMP3)
+        items << %w(INST HEALTH_STATUS TEMP4)
+        details = @api.get_tlm_details(items)
+        expect(details.length).to eql 4
+        expect(details[0]["name"]).to eql "TEMP1"
+        expect(details[1]["name"]).to eql "TEMP2"
+        expect(details[2]["name"]).to eql "TEMP3"
+        expect(details[3]["name"]).to eql "TEMP4"
+      end
+    end
+
+    describe "get_out_of_limits" do
+      it "returns all out of limits items" do
+        items = @api.get_out_of_limits
+        (0..3).each do |i|
+          expect(items[i][0]).to eql "INST"
+          expect(items[i][1]).to eql "HEALTH_STATUS"
+          expect(items[i][2]).to eql "TEMP#{i+1}"
+          expect(items[i][3]).to eql :RED_LOW
+        end
+      end
+    end
+
+    describe "limits_enabled?" do
+      it "complains about non-existant targets" do
+        expect { @api.limits_enabled?("BLAH","HEALTH_STATUS","TEMP1") }.to raise_error(RuntimeError, "Telemetry target 'BLAH' does not exist")
+      end
+
+      it "complains about non-existant packets" do
+        expect { @api.limits_enabled?("INST","BLAH","TEMP1") }.to raise_error(RuntimeError, "Telemetry packet 'INST BLAH' does not exist")
+      end
+
+      it "complains about non-existant items" do
+        expect { @api.limits_enabled?("INST","HEALTH_STATUS","BLAH") }.to raise_error(RuntimeError, "Packet item 'INST HEALTH_STATUS BLAH' does not exist")
+      end
+
+      it "returns whether limits are enable for an item" do
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be true
+      end
+    end
+
+    describe "enable_limits" do
+      it "complains about non-existant targets" do
+        expect { @api.enable_limits("BLAH","HEALTH_STATUS","TEMP1") }.to raise_error(RuntimeError, "Telemetry target 'BLAH' does not exist")
+      end
+
+      it "complains about non-existant packets" do
+        expect { @api.enable_limits("INST","BLAH","TEMP1") }.to raise_error(RuntimeError, "Telemetry packet 'INST BLAH' does not exist")
+      end
+
+      it "complains about non-existant items" do
+        expect { @api.enable_limits("INST","HEALTH_STATUS","BLAH") }.to raise_error(RuntimeError, "Packet item 'INST HEALTH_STATUS BLAH' does not exist")
+      end
+
+      it "enables limits for an item" do
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be true
+        @api.disable_limits("INST","HEALTH_STATUS","TEMP1")
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be false
+        @api.enable_limits("INST","HEALTH_STATUS","TEMP1")
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be true
+      end
+    end
+
+    describe "disable_limits" do
+      it "complains about non-existant targets" do
+        expect { @api.disable_limits("BLAH","HEALTH_STATUS","TEMP1") }.to raise_error(RuntimeError, "Telemetry target 'BLAH' does not exist")
+      end
+
+      it "complains about non-existant packets" do
+        expect { @api.disable_limits("INST","BLAH","TEMP1") }.to raise_error(RuntimeError, "Telemetry packet 'INST BLAH' does not exist")
+      end
+
+      it "complains about non-existant items" do
+        expect { @api.disable_limits("INST","HEALTH_STATUS","BLAH") }.to raise_error(RuntimeError, "Packet item 'INST HEALTH_STATUS BLAH' does not exist")
+      end
+
+      it "disables limits for an item" do
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be true
+        @api.disable_limits("INST","HEALTH_STATUS","TEMP1")
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be false
+        @api.enable_limits("INST","HEALTH_STATUS","TEMP1")
+      end
+    end
+
+    describe "get_limits" do
+      it "complains about non-existant targets" do
+        expect { @api.get_limits("BLAH","HEALTH_STATUS","TEMP1") }.to raise_error(RuntimeError, "Telemetry target 'BLAH' does not exist")
+      end
+
+      it "complains about non-existant packets" do
+        expect { @api.get_limits("INST","BLAH","TEMP1") }.to raise_error(RuntimeError, "Telemetry packet 'INST BLAH' does not exist")
+      end
+
+      it "complains about non-existant items" do
+        expect { @api.get_limits("INST","HEALTH_STATUS","BLAH") }.to raise_error(RuntimeError, "Packet item 'INST HEALTH_STATUS BLAH' does not exist")
+      end
+
+      it "gets limits for an item" do
+        expect(@api.get_limits("INST","HEALTH_STATUS","TEMP1")).to eql([:DEFAULT, 1, true, -80.0, -70.0, 60.0, 80.0, -20.0, 20.0])
+        expect(@api.get_limits("INST","HEALTH_STATUS","TEMP1",:TVAC)).to eql([:TVAC, 1, true, -80.0, -30.0, 30.0, 80.0, nil, nil])
+      end
+    end
+
+    describe "set_limits" do
+      it "complains about non-existant targets" do
+        expect { @api.set_limits("BLAH","HEALTH_STATUS","TEMP1",0.0,10.0,20.0,30.0) }.to raise_error(RuntimeError, "Telemetry target 'BLAH' does not exist")
+      end
+
+      it "complains about non-existant packets" do
+        expect { @api.set_limits("INST","BLAH","TEMP1",0.0,10.0,20.0,30.0) }.to raise_error(RuntimeError, "Telemetry packet 'INST BLAH' does not exist")
+      end
+
+      it "complains about non-existant items" do
+        expect { @api.set_limits("INST","HEALTH_STATUS","BLAH",0.0,10.0,20.0,30.0) }.to raise_error(RuntimeError, "Packet item 'INST HEALTH_STATUS BLAH' does not exist")
+      end
+
+      it "gets limits for an item" do
+        expect(@api.set_limits("INST","HEALTH_STATUS","TEMP1",0.0,10.0,20.0,30.0)).to eql([:CUSTOM, 1, true, 0.0, 10.0, 20.0, 30.0, nil, nil])
+        expect(@api.set_limits("INST","HEALTH_STATUS","TEMP1",0.0,10.0,20.0,30.0,12.0,15.0,:CUSTOM2,2,false)).to eql([:CUSTOM2, 2, false, 0.0, 10.0, 20.0, 30.0, 12.0, 15.0])
+        expect(@api.set_limits("INST","HEALTH_STATUS","TEMP1",0.0,10.0,20.0,30.0,12.0,15.0,:CUSTOM,1,true)).to eql([:CUSTOM, 1, true, 0.0, 10.0, 20.0, 30.0, 12.0, 15.0])
+      end
+    end
+
+    describe "get_limits_groups" do
+      it "returns all the limits groups" do
+        expect(@api.get_limits_groups).to eql %w(FIRST SECOND)
+      end
+    end
+
+    describe "enable_limits_group" do
+      it "complains about undefined limits groups" do
+        expect { @api.enable_limits_group("MINE") }.to raise_error(RuntimeError, "LIMITS_GROUP MINE undefined. Ensure your telemetry definition contains the line: LIMITS_GROUP MINE")
+      end
+
+      it "enables limits for all items in the group" do
+        @api.disable_limits("INST","HEALTH_STATUS","TEMP1")
+        @api.disable_limits("INST","HEALTH_STATUS","TEMP3")
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be false
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP3")).to be false
+        @api.enable_limits_group("FIRST")
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be true
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP3")).to be true
+      end
+    end
+
+    describe "disable_limits_group" do
+      it "complains about undefined limits groups" do
+        expect { @api.disable_limits_group("MINE") }.to raise_error(RuntimeError, "LIMITS_GROUP MINE undefined. Ensure your telemetry definition contains the line: LIMITS_GROUP MINE")
+      end
+
+      it "disables limits for all items in the group" do
+        @api.enable_limits("INST","HEALTH_STATUS","TEMP1")
+        @api.enable_limits("INST","HEALTH_STATUS","TEMP3")
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be true
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP3")).to be true
+        @api.disable_limits_group("FIRST")
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be false
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP3")).to be false
+      end
+    end
+
+    describe "get_limits_sets, get_limits_set, set_limits_set" do
+      it "gets and set the active limits set" do
+        if @api.get_limits_sets.include?(:CUSTOM)
+          expect(@api.get_limits_sets).to eql [:DEFAULT,:TVAC, :CUSTOM, :CUSTOM2]
+        else
+          expect(@api.get_limits_sets).to eql [:DEFAULT,:TVAC]
+        end
+        @api.set_limits_set("TVAC")
+        expect(@api.get_limits_set).to eql "TVAC"
+        @api.set_limits_set("DEFAULT")
+        expect(@api.get_limits_set).to eql "DEFAULT"
+      end
+    end
+
+    describe "get_target_list" do
+      it "returns all target names" do
+        expect(@api.get_target_list).to eql %w(INST SYSTEM)
+      end
+    end
+
+    describe "subscribe_limits_events" do
+      it "calls CmdTlmServer" do
+        stub_const("Cosmos::CmdTlmServer::DEFAULT_LIMITS_EVENT_QUEUE_SIZE", 100)
+        expect(CmdTlmServer).to receive(:subscribe_limits_events)
+        @api.subscribe_limits_events
+      end
+    end
+
+    describe "unsubscribe_limits_events" do
+      it "calls CmdTlmServer" do
+        expect(CmdTlmServer).to receive(:unsubscribe_limits_events)
+        @api.unsubscribe_limits_events(0)
+      end
+    end
+
+    describe "get_limits_event" do
+      it "gets a limits event" do
+        expect(CmdTlmServer).to receive(:get_limits_event)
+        @api.get_limits_event(0)
+      end
+    end
+
+    describe "subscribe_packet_data" do
+      it "calls CmdTlmServer" do
+        stub_const("Cosmos::CmdTlmServer::DEFAULT_PACKET_DATA_QUEUE_SIZE", 100)
+        expect(CmdTlmServer).to receive(:subscribe_packet_data)
+        @api.subscribe_packet_data([["TGT","PKT1"],["TGT","PKT2"]])
+      end
+    end
+
+    describe "unsubscribe_packet_datas" do
+      it "calls CmdTlmServer" do
+        expect(CmdTlmServer).to receive(:unsubscribe_packet_data)
+        @api.unsubscribe_packet_data(10)
+      end
+    end
+
+    describe "get_packet_data" do
+      it "calls CmdTlmServer" do
+        expect(CmdTlmServer).to receive(:get_packet_data)
+        @api.get_packet_data(10)
+      end
+    end
+
+    # All these methods simply pass through directly to CmdTlmServer without
+    # adding any functionality. Thus we just test that they are are received
+    # by the CmdTlmServer.
+    describe "CmdTlmServer pass-throughs" do
+      it "calls through to the CmdTlmServer" do
+        @api.get_interface_names
+        @api.connect_interface("INT")
+        @api.disconnect_interface("INT")
+        @api.interface_state("INT")
+        @api.map_target_to_interface("INST", "INT")
+        @api.get_interface_info("INT")
+        @api.get_target_info("INST")
+        @api.get_cmd_cnt("INST", "COLLECT")
+        @api.get_tlm_cnt("INST", "HEALTH_STATUS")
+        @api.get_router_info("ROUTE")
+        @api.get_packet_logger_info('DEFAULT')
+        @api.get_router_names
+        @api.connect_router("ROUTE")
+        @api.disconnect_router("ROUTE")
+        @api.router_state("ROUTE")
+        @api.send_raw("INT","\x00\x01")
+        @api.get_cmd_log_filename('DEFAULT')
+        @api.get_tlm_log_filename('DEFAULT')
+        @api.start_logging('ALL')
+        @api.stop_logging('ALL')
+        @api.start_cmd_log('ALL')
+        @api.start_tlm_log('ALL')
+        @api.stop_cmd_log('ALL')
+        @api.stop_tlm_log('ALL')
+        @api.get_server_message_log_filename
+        @api.start_new_server_message_log
+      end
+    end
+
+  end
+end
+