cosmos 3.0.1 → 3.1.0

data/spec/packets/limits_response_spec.rb

@@ -1,25 +1,25 @@
-# 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/packets/limits_response'
-
-module Cosmos
-
-  describe LimitsResponse do
-
-    describe "call" do
-      it "should raise an exception" do
-        expect { LimitsResponse.new.call(nil,nil,nil) }.to raise_error(/defined by subclass/)
-      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/packets/limits_response'
+
+module Cosmos
+
+  describe LimitsResponse do
+
+    describe "call" do
+      it "should raise an exception" do
+        expect { LimitsResponse.new.call(nil,nil,nil) }.to raise_error(/defined by subclass/)
+      end
+    end
+  end
+end

data/spec/packets/limits_spec.rb

@@ -1,326 +1,326 @@
-# 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/packets/limits'
-require 'tempfile'
-
-module Cosmos
-
-  describe Limits do
-
-    before(:each) do
-      tf = Tempfile.new('unittest')
-      tf.puts '# This is a comment'
-      tf.puts '#'
-      tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "TGT1 PKT1 Description"'
-      tf.puts '  APPEND_ID_ITEM item1 8 UINT 1 "Item1"'
-      tf.puts '    LIMITS DEFAULT 1 ENABLED 1 2 4 5'
-      tf.puts '    LIMITS TVAC 1 ENABLED 6 7 12 13 9 10'
-      tf.puts '  APPEND_ITEM item2 8 UINT "Item2"'
-      tf.puts '    LIMITS DEFAULT 1 ENABLED 1 2 4 5'
-      tf.puts '    LIMITS TVAC 1 ENABLED 6 7 12 13 9 10'
-      tf.puts '  APPEND_ITEM item3 8 UINT "Item3"'
-      tf.puts '    LIMITS DEFAULT 1 ENABLED 1 2 4 5'
-      tf.puts '    LIMITS TVAC 1 ENABLED 6 7 12 13 9 10'
-      tf.puts '  APPEND_ITEM item4 8 UINT "Item4"'
-      tf.puts '    LIMITS DEFAULT 1 ENABLED 1 2 4 5'
-      tf.puts '    LIMITS TVAC 1 ENABLED 6 7 12 13 9 10'
-      tf.puts '  APPEND_ITEM item5 8 UINT "Item5"'
-      tf.puts 'TELEMETRY tgt1 pkt2 LITTLE_ENDIAN "TGT1 PKT2 Description"'
-      tf.puts '  APPEND_ID_ITEM item1 8 UINT 2 "Item1"'
-      tf.puts '    LIMITS DEFAULT 1 ENABLED 1 2 4 5'
-      tf.puts '  APPEND_ITEM item2 8 UINT "Item2"'
-      tf.puts 'TELEMETRY tgt2 pkt1 LITTLE_ENDIAN "TGT2 PKT1 Description"'
-      tf.puts '  APPEND_ID_ITEM item1 8 UINT 3 "Item1"'
-      tf.puts '  APPEND_ITEM item2 8 UINT "Item2"'
-      tf.puts 'LIMITS_GROUP GROUP1'
-      tf.puts '  LIMITS_GROUP_ITEM TGT1 PKT1 ITEM1'
-      tf.puts '  LIMITS_GROUP_ITEM TGT1 PKT1 ITEM2'
-      tf.puts 'LIMITS_GROUP GROUP2'
-      tf.puts '  LIMITS_GROUP_ITEM TGT1 PKT1 ITEM1'
-      tf.puts '  LIMITS_GROUP_ITEM TGT1 PKT1 ITEM2'
-      tf.close
-
-      # Verify initially that everything is empty
-      pc = PacketConfig.new
-      pc.process_file(tf.path, "SYSTEM")
-      @tlm = Telemetry.new(pc)
-      @limits = Limits.new(pc)
-      tf.unlink
-    end
-
-    describe "initialize" do
-      it "should have no warnings" do
-        Limits.new(PacketConfig.new).warnings.should be_empty
-      end
-    end
-
-    describe "sets" do
-      it "should return the defined limits set" do
-        @limits.sets.should eql [:DEFAULT, :TVAC]
-      end
-    end
-
-    describe "groups" do
-      it "should return the limits groups" do
-        @limits.groups.should_not be_empty
-      end
-    end
-
-    describe "config=" do
-      it "should set the underlying configuration" do
-        tf = Tempfile.new('unittest')
-        tf.puts ''
-        tf.close
-        pc = PacketConfig.new
-        pc.process_file(tf.path, "SYSTEM")
-        @limits.sets.should eql [:DEFAULT, :TVAC]
-        @limits.groups.should_not be_empty
-        @limits.config = pc
-        @limits.sets.should eql [:DEFAULT]
-        @limits.groups.should eql({})
-        tf.unlink
-      end
-    end
-
-    describe "out_of_limits" do
-      it "should return all out of limits telemetry items" do
-        @tlm.update!("TGT1","PKT1","\x00\x03\x03\x04\x05")
-        @tlm.packet("TGT1","PKT1").check_limits
-        items = @limits.out_of_limits
-        items[0][0].should eql "TGT1"
-        items[0][1].should eql "PKT1"
-        items[0][2].should eql "ITEM1"
-        items[0][3].should eql :RED_LOW
-      end
-    end
-
-    describe "overall_limits_state" do
-      it "should return overall limits state of the system" do
-        # Cause packet 2 to be green
-        @tlm.update!("TGT1","PKT2","\x03\x03")
-        @tlm.packet("TGT1","PKT2").check_limits
-
-        @tlm.packet("TGT1","PKT1").set_stale
-        expect(@limits.overall_limits_state).to eq :STALE
-
-        # Cause packet 1 to be all BLUE values
-        @tlm.update!("TGT1","PKT1","\x0a\x0a\x0a\x0a\x00")
-        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
-        expect(@limits.overall_limits_state).to eq :GREEN
-
-        # Cause packet 1 to have a GREEN value
-        @tlm.update!("TGT1","PKT1","\x0a\x0a\x0a\x08\x00")
-        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
-        expect(@limits.overall_limits_state).to eq :GREEN
-
-        # Cause packet 1 to have a YELLOW value
-        @tlm.update!("TGT1","PKT1","\x0a\x0a\x0a\x07\x00")
-        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
-        expect(@limits.overall_limits_state).to eq :YELLOW
-
-        # Cause packet 1 to have a YELLOW and a RED value
-        @tlm.update!("TGT1","PKT1","\x0a\x0a\x07\x06\x00")
-        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
-        expect(@limits.overall_limits_state).to eq :RED
-      end
-
-      it "should mark a stale packet green if all items are ignored" do
-        # Cause packet 2 to be green
-        @tlm.update!("TGT1","PKT2","\x03\x03")
-        @tlm.packet("TGT1","PKT2").check_limits
-
-        # Ignore everything in pkt1
-        @tlm.packet("TGT1","PKT1").set_stale
-        expect(@limits.overall_limits_state).to eq :STALE
-        expect(@limits.overall_limits_state([%w(TGT1 PKT1 ITEM1),%w(TGT1 PKT1 ITEM2),%w(TGT1 PKT1 ITEM3),%w(TGT1 PKT1 ITEM4)])).to eq :GREEN
-      end
-
-      it "should handle non-existant ignored telemetry items" do
-        # Cause packet 2 to be green
-        @tlm.update!("TGT1","PKT2","\x03\x03")
-        @tlm.packet("TGT1","PKT2").check_limits
-
-        # Ignore a non-existant value
-        @tlm.update!("TGT1","PKT1","\x0a\x0a\x0a\x07\x00")
-        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
-        expect(@limits.overall_limits_state([%w(TGT1 PKT1 BLAH)])).to eq :YELLOW
-      end
-
-      it "should ignore specified telemetry items" do
-        # Cause packet 2 to be green
-        @tlm.update!("TGT1","PKT2","\x03\x03")
-        @tlm.packet("TGT1","PKT2").check_limits
-
-        # Cause packet 1 to have a YELLOW value but ignore it
-        @tlm.update!("TGT1","PKT1","\x0a\x0a\x0a\x07\x00")
-        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
-        expect(@limits.overall_limits_state([%w(TGT1 PKT1 ITEM4)])).to eq :GREEN
-
-        # Cause packet 1 to have a YELLOW and a RED value but ignore them
-        @tlm.update!("TGT1","PKT1","\x0a\x0a\x07\x06\x00")
-        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
-        expect(@limits.overall_limits_state([%w(TGT1 PKT1 ITEM4),%w(TGT1 PKT1 ITEM3)])).to eq :GREEN
-      end
-    end
-
-    describe "enable_group" do
-      it "should complain about undefined limits groups" do
-        expect { @limits.enable_group("MINE") }.to raise_error(RuntimeError, "LIMITS_GROUP MINE undefined. Ensure your telemetry definition contains the line: LIMITS_GROUP MINE")
-      end
-
-      it "should complain about undefined items" do
-        tf = Tempfile.new('unittest')
-        tf.puts 'LIMITS_GROUP GROUP1'
-        tf.puts '  LIMITS_GROUP_ITEM TGT1 PKT1 ITEM1'
-        tf.close
-        pc = PacketConfig.new
-        pc.process_file(tf.path, "SYSTEM")
-        limits = Limits.new(pc)
-        expect { limits.enable_group("group1") }.to raise_error(RuntimeError, "Telemetry target 'TGT1' does not exist")
-        tf.unlink
-      end
-
-      it "should enable limits for all items in the group" do
-        @limits.enable_group("group1")
-        pkt = @tlm.packet("TGT1","PKT1")
-        pkt.get_item("ITEM1").limits.enabled.should be_truthy
-        pkt.get_item("ITEM2").limits.enabled.should be_truthy
-        pkt.get_item("ITEM3").limits.enabled.should be_truthy
-        pkt.get_item("ITEM4").limits.enabled.should be_truthy
-        pkt.get_item("ITEM5").limits.enabled.should be_falsey
-      end
-    end
-
-    describe "disable_group" do
-      it "should complain about undefined limits groups" do
-        expect { @limits.disable_group("MINE") }.to raise_error(RuntimeError, "LIMITS_GROUP MINE undefined. Ensure your telemetry definition contains the line: LIMITS_GROUP MINE")
-      end
-
-      it "should disable limits for all items in the group" do
-        pkt = @tlm.packet("TGT1","PKT1")
-        pkt.enable_limits("ITEM1")
-        pkt.enable_limits("ITEM2")
-        pkt.get_item("ITEM1").limits.enabled.should be_truthy
-        pkt.get_item("ITEM2").limits.enabled.should be_truthy
-
-        @limits.disable_group("group1")
-        pkt.get_item("ITEM1").limits.enabled.should be_falsey
-        pkt.get_item("ITEM2").limits.enabled.should be_falsey
-      end
-    end
-
-    describe "enabled?" do
-      it "should complain about non-existant targets" do
-        expect { @limits.enabled?("TGTX","PKT1","ITEM1") }.to raise_error(RuntimeError, "Telemetry target 'TGTX' does not exist")
-      end
-
-      it "should complain about non-existant packets" do
-        expect { @limits.enabled?("TGT1","PKTX","ITEM1") }.to raise_error(RuntimeError, "Telemetry packet 'TGT1 PKTX' does not exist")
-      end
-
-      it "should complain about non-existant items" do
-        expect { @limits.enabled?("TGT1","PKT1","ITEMX") }.to raise_error(RuntimeError, "Packet item 'TGT1 PKT1 ITEMX' does not exist")
-      end
-
-      it "should return whether limits are enable for an item" do
-        pkt = @tlm.packet("TGT1","PKT1")
-        @limits.enabled?("TGT1","PKT1","ITEM5").should be_falsey
-        pkt.enable_limits("ITEM5")
-        @limits.enabled?("TGT1","PKT1","ITEM5").should be_truthy
-      end
-    end
-
-    describe "enable" do
-      it "should complain about non-existant targets" do
-        expect { @limits.enable("TGTX","PKT1","ITEM1") }.to raise_error(RuntimeError, "Telemetry target 'TGTX' does not exist")
-      end
-
-      it "should complain about non-existant packets" do
-        expect { @limits.enable("TGT1","PKTX","ITEM1") }.to raise_error(RuntimeError, "Telemetry packet 'TGT1 PKTX' does not exist")
-      end
-
-      it "should complain about non-existant items" do
-        expect { @limits.enable("TGT1","PKT1","ITEMX") }.to raise_error(RuntimeError, "Packet item 'TGT1 PKT1 ITEMX' does not exist")
-      end
-
-      it "should enable limits for an item" do
-        pkt = @tlm.packet("TGT1","PKT1")
-        @limits.enabled?("TGT1","PKT1","ITEM5").should be_falsey
-        @limits.enable("TGT1","PKT1","ITEM5")
-        @limits.enabled?("TGT1","PKT1","ITEM5").should be_truthy
-      end
-    end
-
-    describe "disable" do
-      it "should complain about non-existant targets" do
-        expect { @limits.disable("TGTX","PKT1","ITEM1") }.to raise_error(RuntimeError, "Telemetry target 'TGTX' does not exist")
-      end
-
-      it "should complain about non-existant packets" do
-        expect { @limits.disable("TGT1","PKTX","ITEM1") }.to raise_error(RuntimeError, "Telemetry packet 'TGT1 PKTX' does not exist")
-      end
-
-      it "should complain about non-existant items" do
-        expect { @limits.disable("TGT1","PKT1","ITEMX") }.to raise_error(RuntimeError, "Packet item 'TGT1 PKT1 ITEMX' does not exist")
-      end
-
-      it "should disable limits for an item" do
-        pkt = @tlm.packet("TGT1","PKT1")
-        @limits.enable("TGT1","PKT1","ITEM1")
-        @limits.enabled?("TGT1","PKT1","ITEM1").should be_truthy
-        @limits.disable("TGT1","PKT1","ITEM1")
-        @limits.enabled?("TGT1","PKT1","ITEM1").should be_falsey
-      end
-    end
-
-    describe "get" do
-      it "should get the limits for an item with limits" do
-        @limits.get("TGT1", "PKT1", "ITEM1").should eql [:DEFAULT, 1, true, 1.0, 2.0, 4.0, 5.0, nil, nil]
-      end
-
-      it "should handle an item without limits" do
-        @limits.get("TGT1", "PKT1", "ITEM5").should eql [nil, nil, nil, nil, nil, nil, nil, nil, nil]
-      end
-
-      it "should support a specified limits set" do
-        @limits.get("TGT1", "PKT1", "ITEM1", :TVAC).should eql [:TVAC, 1, true, 6.0, 7.0, 12.0, 13.0, 9.0, 10.0]
-      end
-
-      it "should handle an item without limits for the given limits set" do
-        @limits.get("TGT1", "PKT2", "ITEM1", :TVAC).should eql [nil, nil, nil, nil, nil, nil, nil, nil, nil]
-      end
-    end
-
-    describe "set" do
-      it "should set limits for an item" do
-        @limits.set("TGT1", "PKT1", "ITEM5", 1, 2, 3, 4, nil, nil, :DEFAULT).should eql [:DEFAULT, 1, true, 1.0, 2.0, 3.0, 4.0, nil, nil]
-      end
-
-      it "should enforce setting DEFAULT limits first" do
-        expect { @limits.set("TGT1", "PKT1", "ITEM5", 1, 2, 3, 4) }.to raise_error(RuntimeError, "DEFAULT limits must be defined for TGT1 PKT1 ITEM5 before setting limits set CUSTOM")
-        @limits.set("TGT1", "PKT1", "ITEM5", 5, 6, 7, 8, nil, nil, :DEFAULT).should eql [:DEFAULT, 1, true, 5.0, 6.0, 7.0, 8.0, nil, nil]
-        @limits.set("TGT1", "PKT1", "ITEM5", 1, 2, 3, 4).should eql [:CUSTOM, 1, true, 1.0, 2.0, 3.0, 4.0, nil, nil]
-      end
-
-      it "should allow setting other limits sets" do
-        @limits.set("TGT1", "PKT1", "ITEM1", 1, 2, 3, 4, nil, nil, :TVAC).should eql [:TVAC, 1, true, 1.0, 2.0, 3.0, 4.0, nil, nil]
-      end
-
-      it "should handle green limits" do
-        @limits.set("TGT1", "PKT1", "ITEM1", 1, 2, 5, 6, 3, 4, nil).should eql [:DEFAULT, 1, true, 1.0, 2.0, 5.0, 6.0, 3.0, 4.0]
-      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/packets/limits'
+require 'tempfile'
+
+module Cosmos
+
+  describe Limits do
+
+    before(:each) do
+      tf = Tempfile.new('unittest')
+      tf.puts '# This is a comment'
+      tf.puts '#'
+      tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "TGT1 PKT1 Description"'
+      tf.puts '  APPEND_ID_ITEM item1 8 UINT 1 "Item1"'
+      tf.puts '    LIMITS DEFAULT 1 ENABLED 1 2 4 5'
+      tf.puts '    LIMITS TVAC 1 ENABLED 6 7 12 13 9 10'
+      tf.puts '  APPEND_ITEM item2 8 UINT "Item2"'
+      tf.puts '    LIMITS DEFAULT 1 ENABLED 1 2 4 5'
+      tf.puts '    LIMITS TVAC 1 ENABLED 6 7 12 13 9 10'
+      tf.puts '  APPEND_ITEM item3 8 UINT "Item3"'
+      tf.puts '    LIMITS DEFAULT 1 ENABLED 1 2 4 5'
+      tf.puts '    LIMITS TVAC 1 ENABLED 6 7 12 13 9 10'
+      tf.puts '  APPEND_ITEM item4 8 UINT "Item4"'
+      tf.puts '    LIMITS DEFAULT 1 ENABLED 1 2 4 5'
+      tf.puts '    LIMITS TVAC 1 ENABLED 6 7 12 13 9 10'
+      tf.puts '  APPEND_ITEM item5 8 UINT "Item5"'
+      tf.puts 'TELEMETRY tgt1 pkt2 LITTLE_ENDIAN "TGT1 PKT2 Description"'
+      tf.puts '  APPEND_ID_ITEM item1 8 UINT 2 "Item1"'
+      tf.puts '    LIMITS DEFAULT 1 ENABLED 1 2 4 5'
+      tf.puts '  APPEND_ITEM item2 8 UINT "Item2"'
+      tf.puts 'TELEMETRY tgt2 pkt1 LITTLE_ENDIAN "TGT2 PKT1 Description"'
+      tf.puts '  APPEND_ID_ITEM item1 8 UINT 3 "Item1"'
+      tf.puts '  APPEND_ITEM item2 8 UINT "Item2"'
+      tf.puts 'LIMITS_GROUP GROUP1'
+      tf.puts '  LIMITS_GROUP_ITEM TGT1 PKT1 ITEM1'
+      tf.puts '  LIMITS_GROUP_ITEM TGT1 PKT1 ITEM2'
+      tf.puts 'LIMITS_GROUP GROUP2'
+      tf.puts '  LIMITS_GROUP_ITEM TGT1 PKT1 ITEM1'
+      tf.puts '  LIMITS_GROUP_ITEM TGT1 PKT1 ITEM2'
+      tf.close
+
+      # Verify initially that everything is empty
+      pc = PacketConfig.new
+      pc.process_file(tf.path, "SYSTEM")
+      @tlm = Telemetry.new(pc)
+      @limits = Limits.new(pc)
+      tf.unlink
+    end
+
+    describe "initialize" do
+      it "should have no warnings" do
+        Limits.new(PacketConfig.new).warnings.should be_empty
+      end
+    end
+
+    describe "sets" do
+      it "should return the defined limits set" do
+        @limits.sets.should eql [:DEFAULT, :TVAC]
+      end
+    end
+
+    describe "groups" do
+      it "should return the limits groups" do
+        @limits.groups.should_not be_empty
+      end
+    end
+
+    describe "config=" do
+      it "should set the underlying configuration" do
+        tf = Tempfile.new('unittest')
+        tf.puts ''
+        tf.close
+        pc = PacketConfig.new
+        pc.process_file(tf.path, "SYSTEM")
+        @limits.sets.should eql [:DEFAULT, :TVAC]
+        @limits.groups.should_not be_empty
+        @limits.config = pc
+        @limits.sets.should eql [:DEFAULT]
+        @limits.groups.should eql({})
+        tf.unlink
+      end
+    end
+
+    describe "out_of_limits" do
+      it "should return all out of limits telemetry items" do
+        @tlm.update!("TGT1","PKT1","\x00\x03\x03\x04\x05")
+        @tlm.packet("TGT1","PKT1").check_limits
+        items = @limits.out_of_limits
+        items[0][0].should eql "TGT1"
+        items[0][1].should eql "PKT1"
+        items[0][2].should eql "ITEM1"
+        items[0][3].should eql :RED_LOW
+      end
+    end
+
+    describe "overall_limits_state" do
+      it "should return overall limits state of the system" do
+        # Cause packet 2 to be green
+        @tlm.update!("TGT1","PKT2","\x03\x03")
+        @tlm.packet("TGT1","PKT2").check_limits
+
+        @tlm.packet("TGT1","PKT1").set_stale
+        expect(@limits.overall_limits_state).to eq :STALE
+
+        # Cause packet 1 to be all BLUE values
+        @tlm.update!("TGT1","PKT1","\x0a\x0a\x0a\x0a\x00")
+        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
+        expect(@limits.overall_limits_state).to eq :GREEN
+
+        # Cause packet 1 to have a GREEN value
+        @tlm.update!("TGT1","PKT1","\x0a\x0a\x0a\x08\x00")
+        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
+        expect(@limits.overall_limits_state).to eq :GREEN
+
+        # Cause packet 1 to have a YELLOW value
+        @tlm.update!("TGT1","PKT1","\x0a\x0a\x0a\x07\x00")
+        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
+        expect(@limits.overall_limits_state).to eq :YELLOW
+
+        # Cause packet 1 to have a YELLOW and a RED value
+        @tlm.update!("TGT1","PKT1","\x0a\x0a\x07\x06\x00")
+        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
+        expect(@limits.overall_limits_state).to eq :RED
+      end
+
+      it "should mark a stale packet green if all items are ignored" do
+        # Cause packet 2 to be green
+        @tlm.update!("TGT1","PKT2","\x03\x03")
+        @tlm.packet("TGT1","PKT2").check_limits
+
+        # Ignore everything in pkt1
+        @tlm.packet("TGT1","PKT1").set_stale
+        expect(@limits.overall_limits_state).to eq :STALE
+        expect(@limits.overall_limits_state([%w(TGT1 PKT1 ITEM1),%w(TGT1 PKT1 ITEM2),%w(TGT1 PKT1 ITEM3),%w(TGT1 PKT1 ITEM4)])).to eq :GREEN
+      end
+
+      it "should handle non-existant ignored telemetry items" do
+        # Cause packet 2 to be green
+        @tlm.update!("TGT1","PKT2","\x03\x03")
+        @tlm.packet("TGT1","PKT2").check_limits
+
+        # Ignore a non-existant value
+        @tlm.update!("TGT1","PKT1","\x0a\x0a\x0a\x07\x00")
+        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
+        expect(@limits.overall_limits_state([%w(TGT1 PKT1 BLAH)])).to eq :YELLOW
+      end
+
+      it "should ignore specified telemetry items" do
+        # Cause packet 2 to be green
+        @tlm.update!("TGT1","PKT2","\x03\x03")
+        @tlm.packet("TGT1","PKT2").check_limits
+
+        # Cause packet 1 to have a YELLOW value but ignore it
+        @tlm.update!("TGT1","PKT1","\x0a\x0a\x0a\x07\x00")
+        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
+        expect(@limits.overall_limits_state([%w(TGT1 PKT1 ITEM4)])).to eq :GREEN
+
+        # Cause packet 1 to have a YELLOW and a RED value but ignore them
+        @tlm.update!("TGT1","PKT1","\x0a\x0a\x07\x06\x00")
+        @tlm.packet("TGT1","PKT1").check_limits(:TVAC)
+        expect(@limits.overall_limits_state([%w(TGT1 PKT1 ITEM4),%w(TGT1 PKT1 ITEM3)])).to eq :GREEN
+      end
+    end
+
+    describe "enable_group" do
+      it "should complain about undefined limits groups" do
+        expect { @limits.enable_group("MINE") }.to raise_error(RuntimeError, "LIMITS_GROUP MINE undefined. Ensure your telemetry definition contains the line: LIMITS_GROUP MINE")
+      end
+
+      it "should complain about undefined items" do
+        tf = Tempfile.new('unittest')
+        tf.puts 'LIMITS_GROUP GROUP1'
+        tf.puts '  LIMITS_GROUP_ITEM TGT1 PKT1 ITEM1'
+        tf.close
+        pc = PacketConfig.new
+        pc.process_file(tf.path, "SYSTEM")
+        limits = Limits.new(pc)
+        expect { limits.enable_group("group1") }.to raise_error(RuntimeError, "Telemetry target 'TGT1' does not exist")
+        tf.unlink
+      end
+
+      it "should enable limits for all items in the group" do
+        @limits.enable_group("group1")
+        pkt = @tlm.packet("TGT1","PKT1")
+        pkt.get_item("ITEM1").limits.enabled.should be_truthy
+        pkt.get_item("ITEM2").limits.enabled.should be_truthy
+        pkt.get_item("ITEM3").limits.enabled.should be_truthy
+        pkt.get_item("ITEM4").limits.enabled.should be_truthy
+        pkt.get_item("ITEM5").limits.enabled.should be_falsey
+      end
+    end
+
+    describe "disable_group" do
+      it "should complain about undefined limits groups" do
+        expect { @limits.disable_group("MINE") }.to raise_error(RuntimeError, "LIMITS_GROUP MINE undefined. Ensure your telemetry definition contains the line: LIMITS_GROUP MINE")
+      end
+
+      it "should disable limits for all items in the group" do
+        pkt = @tlm.packet("TGT1","PKT1")
+        pkt.enable_limits("ITEM1")
+        pkt.enable_limits("ITEM2")
+        pkt.get_item("ITEM1").limits.enabled.should be_truthy
+        pkt.get_item("ITEM2").limits.enabled.should be_truthy
+
+        @limits.disable_group("group1")
+        pkt.get_item("ITEM1").limits.enabled.should be_falsey
+        pkt.get_item("ITEM2").limits.enabled.should be_falsey
+      end
+    end
+
+    describe "enabled?" do
+      it "should complain about non-existant targets" do
+        expect { @limits.enabled?("TGTX","PKT1","ITEM1") }.to raise_error(RuntimeError, "Telemetry target 'TGTX' does not exist")
+      end
+
+      it "should complain about non-existant packets" do
+        expect { @limits.enabled?("TGT1","PKTX","ITEM1") }.to raise_error(RuntimeError, "Telemetry packet 'TGT1 PKTX' does not exist")
+      end
+
+      it "should complain about non-existant items" do
+        expect { @limits.enabled?("TGT1","PKT1","ITEMX") }.to raise_error(RuntimeError, "Packet item 'TGT1 PKT1 ITEMX' does not exist")
+      end
+
+      it "should return whether limits are enable for an item" do
+        pkt = @tlm.packet("TGT1","PKT1")
+        @limits.enabled?("TGT1","PKT1","ITEM5").should be_falsey
+        pkt.enable_limits("ITEM5")
+        @limits.enabled?("TGT1","PKT1","ITEM5").should be_truthy
+      end
+    end
+
+    describe "enable" do
+      it "should complain about non-existant targets" do
+        expect { @limits.enable("TGTX","PKT1","ITEM1") }.to raise_error(RuntimeError, "Telemetry target 'TGTX' does not exist")
+      end
+
+      it "should complain about non-existant packets" do
+        expect { @limits.enable("TGT1","PKTX","ITEM1") }.to raise_error(RuntimeError, "Telemetry packet 'TGT1 PKTX' does not exist")
+      end
+
+      it "should complain about non-existant items" do
+        expect { @limits.enable("TGT1","PKT1","ITEMX") }.to raise_error(RuntimeError, "Packet item 'TGT1 PKT1 ITEMX' does not exist")
+      end
+
+      it "should enable limits for an item" do
+        pkt = @tlm.packet("TGT1","PKT1")
+        @limits.enabled?("TGT1","PKT1","ITEM5").should be_falsey
+        @limits.enable("TGT1","PKT1","ITEM5")
+        @limits.enabled?("TGT1","PKT1","ITEM5").should be_truthy
+      end
+    end
+
+    describe "disable" do
+      it "should complain about non-existant targets" do
+        expect { @limits.disable("TGTX","PKT1","ITEM1") }.to raise_error(RuntimeError, "Telemetry target 'TGTX' does not exist")
+      end
+
+      it "should complain about non-existant packets" do
+        expect { @limits.disable("TGT1","PKTX","ITEM1") }.to raise_error(RuntimeError, "Telemetry packet 'TGT1 PKTX' does not exist")
+      end
+
+      it "should complain about non-existant items" do
+        expect { @limits.disable("TGT1","PKT1","ITEMX") }.to raise_error(RuntimeError, "Packet item 'TGT1 PKT1 ITEMX' does not exist")
+      end
+
+      it "should disable limits for an item" do
+        pkt = @tlm.packet("TGT1","PKT1")
+        @limits.enable("TGT1","PKT1","ITEM1")
+        @limits.enabled?("TGT1","PKT1","ITEM1").should be_truthy
+        @limits.disable("TGT1","PKT1","ITEM1")
+        @limits.enabled?("TGT1","PKT1","ITEM1").should be_falsey
+      end
+    end
+
+    describe "get" do
+      it "should get the limits for an item with limits" do
+        @limits.get("TGT1", "PKT1", "ITEM1").should eql [:DEFAULT, 1, true, 1.0, 2.0, 4.0, 5.0, nil, nil]
+      end
+
+      it "should handle an item without limits" do
+        @limits.get("TGT1", "PKT1", "ITEM5").should eql [nil, nil, nil, nil, nil, nil, nil, nil, nil]
+      end
+
+      it "should support a specified limits set" do
+        @limits.get("TGT1", "PKT1", "ITEM1", :TVAC).should eql [:TVAC, 1, true, 6.0, 7.0, 12.0, 13.0, 9.0, 10.0]
+      end
+
+      it "should handle an item without limits for the given limits set" do
+        @limits.get("TGT1", "PKT2", "ITEM1", :TVAC).should eql [nil, nil, nil, nil, nil, nil, nil, nil, nil]
+      end
+    end
+
+    describe "set" do
+      it "should set limits for an item" do
+        @limits.set("TGT1", "PKT1", "ITEM5", 1, 2, 3, 4, nil, nil, :DEFAULT).should eql [:DEFAULT, 1, true, 1.0, 2.0, 3.0, 4.0, nil, nil]
+      end
+
+      it "should enforce setting DEFAULT limits first" do
+        expect { @limits.set("TGT1", "PKT1", "ITEM5", 1, 2, 3, 4) }.to raise_error(RuntimeError, "DEFAULT limits must be defined for TGT1 PKT1 ITEM5 before setting limits set CUSTOM")
+        @limits.set("TGT1", "PKT1", "ITEM5", 5, 6, 7, 8, nil, nil, :DEFAULT).should eql [:DEFAULT, 1, true, 5.0, 6.0, 7.0, 8.0, nil, nil]
+        @limits.set("TGT1", "PKT1", "ITEM5", 1, 2, 3, 4).should eql [:CUSTOM, 1, true, 1.0, 2.0, 3.0, 4.0, nil, nil]
+      end
+
+      it "should allow setting other limits sets" do
+        @limits.set("TGT1", "PKT1", "ITEM1", 1, 2, 3, 4, nil, nil, :TVAC).should eql [:TVAC, 1, true, 1.0, 2.0, 3.0, 4.0, nil, nil]
+      end
+
+      it "should handle green limits" do
+        @limits.set("TGT1", "PKT1", "ITEM1", 1, 2, 5, 6, 3, 4, nil).should eql [:DEFAULT, 1, true, 1.0, 2.0, 5.0, 6.0, 3.0, 4.0]
+      end
+    end
+
+  end
+end
+