cosmos 3.5.1 → 3.5.2

data/spec/tools/cmd_tlm_server/api_spec.rb

@@ -1,1117 +1,1117 @@
-# 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("COSMOS", "STARTLOGGING")
-        packet2.received_time = time2
-        expect(@api.get_cmd_time()).to eql ['COSMOS', '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('COSMOS')).to eql ['COSMOS', '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('COSMOS', 'STARTLOGGING')).to eql ['COSMOS', '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(COSMOS INST META 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_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
-
+# 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("COSMOS", "STARTLOGGING")
+        packet2.received_time = time2
+        expect(@api.get_cmd_time()).to eql ['COSMOS', '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('COSMOS')).to eql ['COSMOS', '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('COSMOS', 'STARTLOGGING')).to eql ['COSMOS', '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(COSMOS INST META 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_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
+