cosmos 3.5.1 → 3.5.2

data/spec/packets/parsers/packet_parser_spec.rb

@@ -1,99 +1,99 @@
-# 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/packet_config'
-require 'cosmos/packets/parsers/packet_parser'
-require 'tempfile'
-
-module Cosmos
-
-  describe PacketParser do
-
-    describe "process_file" do
-      before(:each) do
-        @pc = PacketConfig.new
-      end
-
-      it "complains if there are not enough parameters" do
-        %w(COMMAND TELEMETRY).each do |keyword|
-          tf = Tempfile.new('unittest')
-          tf.puts(keyword)
-          tf.close
-          expect { @pc.process_file(tf.path, "SYSTEM") }.to raise_error(ConfigParser::Error, /Not enough parameters for #{keyword}/)
-          tf.unlink
-        end
-      end
-
-      it "complains if there are too many parameters" do
-        %w(COMMAND TELEMETRY).each do |keyword|
-          tf = Tempfile.new('unittest')
-          tf.puts "#{keyword} tgt1 pkt1 LITTLE_ENDIAN 'Packet' extra"
-          tf.close
-          expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Too many parameters for #{keyword}/)
-          tf.unlink
-        end
-      end
-
-      it "complains about invalid endianness" do
-        %w(COMMAND TELEMETRY).each do |keyword|
-          tf = Tempfile.new('unittest')
-          tf.puts keyword + ' tgt1 pkt1 MIDDLE_ENDIAN "Packet"'
-          tf.close
-          expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, "Invalid endianness MIDDLE_ENDIAN. Must be BIG_ENDIAN or LITTLE_ENDIAN.")
-          tf.unlink
-        end
-      end
-
-      it "processes target, packet, endianness, description" do
-        %w(COMMAND TELEMETRY).each do |keyword|
-          tf = Tempfile.new('unittest')
-          tf.puts keyword + ' tgt1 pkt1 LITTLE_ENDIAN "Packet"'
-          tf.close
-          @pc.process_file(tf.path, "TGT1")
-          pkt = @pc.commands["TGT1"]["PKT1"] if keyword == 'COMMAND'
-          pkt = @pc.telemetry["TGT1"]["PKT1"] if keyword == 'TELEMETRY'
-          expect(pkt.target_name).to eql "TGT1"
-          expect(pkt.packet_name).to eql "PKT1"
-          expect(pkt.default_endianness).to eql :LITTLE_ENDIAN
-          expect(pkt.description).to eql "Packet"
-          tf.unlink
-        end
-      end
-
-      it "substitutes the target name" do
-        %w(COMMAND TELEMETRY).each do |keyword|
-          tf = Tempfile.new('unittest')
-          tf.puts keyword + ' tgt1 pkt1 LITTLE_ENDIAN "Packet"'
-          tf.close
-          @pc.process_file(tf.path, "NEW")
-          pkt = @pc.commands["NEW"]["PKT1"] if keyword == 'COMMAND'
-          pkt = @pc.telemetry["NEW"]["PKT1"] if keyword == 'TELEMETRY'
-          expect(pkt.target_name).to eql "NEW"
-          tf.unlink
-        end
-      end
-
-      it "complains if a packet is redefined" do
-        %w(COMMAND TELEMETRY).each do |keyword|
-          tf = Tempfile.new('unittest')
-          tf.puts keyword + ' tgt1 pkt1 LITTLE_ENDIAN "Packet 1"'
-          tf.puts keyword + ' tgt1 pkt1 LITTLE_ENDIAN "Packet 2"'
-          tf.close
-          @pc.process_file(tf.path, "SYSTEM")
-          expect(@pc.warnings).to include("#{keyword.capitalize} Packet TGT1 PKT1 redefined.")
-          tf.unlink
-        end
-      end
-
-    end # describe "process_file"
-  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/packet_config'
+require 'cosmos/packets/parsers/packet_parser'
+require 'tempfile'
+
+module Cosmos
+
+  describe PacketParser do
+
+    describe "process_file" do
+      before(:each) do
+        @pc = PacketConfig.new
+      end
+
+      it "complains if there are not enough parameters" do
+        %w(COMMAND TELEMETRY).each do |keyword|
+          tf = Tempfile.new('unittest')
+          tf.puts(keyword)
+          tf.close
+          expect { @pc.process_file(tf.path, "SYSTEM") }.to raise_error(ConfigParser::Error, /Not enough parameters for #{keyword}/)
+          tf.unlink
+        end
+      end
+
+      it "complains if there are too many parameters" do
+        %w(COMMAND TELEMETRY).each do |keyword|
+          tf = Tempfile.new('unittest')
+          tf.puts "#{keyword} tgt1 pkt1 LITTLE_ENDIAN 'Packet' extra"
+          tf.close
+          expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Too many parameters for #{keyword}/)
+          tf.unlink
+        end
+      end
+
+      it "complains about invalid endianness" do
+        %w(COMMAND TELEMETRY).each do |keyword|
+          tf = Tempfile.new('unittest')
+          tf.puts keyword + ' tgt1 pkt1 MIDDLE_ENDIAN "Packet"'
+          tf.close
+          expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, "Invalid endianness MIDDLE_ENDIAN. Must be BIG_ENDIAN or LITTLE_ENDIAN.")
+          tf.unlink
+        end
+      end
+
+      it "processes target, packet, endianness, description" do
+        %w(COMMAND TELEMETRY).each do |keyword|
+          tf = Tempfile.new('unittest')
+          tf.puts keyword + ' tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+          tf.close
+          @pc.process_file(tf.path, "TGT1")
+          pkt = @pc.commands["TGT1"]["PKT1"] if keyword == 'COMMAND'
+          pkt = @pc.telemetry["TGT1"]["PKT1"] if keyword == 'TELEMETRY'
+          expect(pkt.target_name).to eql "TGT1"
+          expect(pkt.packet_name).to eql "PKT1"
+          expect(pkt.default_endianness).to eql :LITTLE_ENDIAN
+          expect(pkt.description).to eql "Packet"
+          tf.unlink
+        end
+      end
+
+      it "substitutes the target name" do
+        %w(COMMAND TELEMETRY).each do |keyword|
+          tf = Tempfile.new('unittest')
+          tf.puts keyword + ' tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+          tf.close
+          @pc.process_file(tf.path, "NEW")
+          pkt = @pc.commands["NEW"]["PKT1"] if keyword == 'COMMAND'
+          pkt = @pc.telemetry["NEW"]["PKT1"] if keyword == 'TELEMETRY'
+          expect(pkt.target_name).to eql "NEW"
+          tf.unlink
+        end
+      end
+
+      it "complains if a packet is redefined" do
+        %w(COMMAND TELEMETRY).each do |keyword|
+          tf = Tempfile.new('unittest')
+          tf.puts keyword + ' tgt1 pkt1 LITTLE_ENDIAN "Packet 1"'
+          tf.puts keyword + ' tgt1 pkt1 LITTLE_ENDIAN "Packet 2"'
+          tf.close
+          @pc.process_file(tf.path, "SYSTEM")
+          expect(@pc.warnings).to include("#{keyword.capitalize} Packet TGT1 PKT1 redefined.")
+          tf.unlink
+        end
+      end
+
+    end # describe "process_file"
+  end
+end

data/spec/packets/parsers/processor_parser_spec.rb

@@ -1,114 +1,114 @@
-# 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/packet_config'
-require 'cosmos/packets/parsers/processor_parser'
-require 'tempfile'
-
-module Cosmos
-
-  describe ProcessorParser do
-
-    describe "process_file" do
-      before(:each) do
-        @pc = PacketConfig.new
-      end
-
-      it "complains if a current packet is not defined" do
-        # Check for missing TELEMETRY line
-        tf = Tempfile.new('unittest')
-        tf.puts('PROCESSOR')
-        tf.close
-        expect { @pc.process_file(tf.path, "SYSTEM") }.to raise_error(ConfigParser::Error, "No current packet for PROCESSOR")
-        tf.unlink
-      end
-
-      it "complains if there are not enough parameters" do
-        tf = Tempfile.new('unittest')
-        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
-        tf.puts '  PROCESSOR'
-        tf.close
-        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Not enough parameters for PROCESSOR/)
-        tf.unlink
-      end
-
-      it "complains about missing processor file" do
-        filename = File.join(File.dirname(__FILE__), "../../test_only.rb")
-        File.delete(filename) if File.exist?(filename)
-        @pc = PacketConfig.new
-
-        tf = Tempfile.new('unittest')
-        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
-        tf.puts '  PROCESSOR TEST test_only.rb'
-        tf.close
-        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /TestOnly class not found/)
-        tf.unlink
-      end
-
-      it "complains about a non Cosmos::Processor class" do
-        filename = File.join(File.dirname(__FILE__), "../../processor1.rb")
-        File.open(filename, 'w') do |file|
-          file.puts "class Processor1"
-          file.puts "  def call(packet,buffer)"
-          file.puts "  end"
-          file.puts "end"
-        end
-        load 'processor1.rb'
-        File.delete(filename) if File.exist?(filename)
-
-        tf = Tempfile.new('unittest')
-        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
-        tf.puts '  PROCESSOR P1 processor1.rb'
-        tf.close
-        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /processor must be a Cosmos::Processor but is a Processor1/)
-        tf.unlink
-      end
-
-      it "parses the processor" do
-        filename = File.join(File.dirname(__FILE__), "../../processor2.rb")
-        File.open(filename, 'w') do |file|
-          file.puts "require 'cosmos/processors/processor'"
-          file.puts "class Processor2 < Cosmos::Processor"
-          file.puts "  def call(packet,buffer)"
-          file.puts "    @results[:TEST] = 5"
-          file.puts "  end"
-          file.puts "end"
-        end
-        load 'processor2.rb'
-
-        tf = Tempfile.new('unittest')
-        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
-        tf.puts '  ITEM item1 0 16 INT "Integer Item"'
-        tf.puts '    READ_CONVERSION processor_conversion.rb P2 TEST'
-        tf.puts '  PROCESSOR P2 processor2.rb'
-        tf.puts '  PROCESSOR P3 processor2.rb RAW'
-        tf.close
-        @pc.process_file(tf.path, "TGT1")
-        @pc.telemetry["TGT1"]["PKT1"].buffer = "\x01\x01"
-        expect(@pc.telemetry["TGT1"]["PKT1"].read("ITEM1")).to eql 5
-        tf.unlink
-
-        File.delete(filename) if File.exist?(filename)
-      end
-
-      it "complains if applied to a command packet" do
-        tf = Tempfile.new('unittest')
-        tf.puts 'COMMAND tgt1 pkt1 LITTLE_ENDIAN "Packet"'
-        tf.puts '  PROCESSOR P1 processor1.rb'
-        tf.close
-        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, "PROCESSOR only applies to telemetry packets")
-        tf.unlink
-      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/packet_config'
+require 'cosmos/packets/parsers/processor_parser'
+require 'tempfile'
+
+module Cosmos
+
+  describe ProcessorParser do
+
+    describe "process_file" do
+      before(:each) do
+        @pc = PacketConfig.new
+      end
+
+      it "complains if a current packet is not defined" do
+        # Check for missing TELEMETRY line
+        tf = Tempfile.new('unittest')
+        tf.puts('PROCESSOR')
+        tf.close
+        expect { @pc.process_file(tf.path, "SYSTEM") }.to raise_error(ConfigParser::Error, "No current packet for PROCESSOR")
+        tf.unlink
+      end
+
+      it "complains if there are not enough parameters" do
+        tf = Tempfile.new('unittest')
+        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+        tf.puts '  PROCESSOR'
+        tf.close
+        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Not enough parameters for PROCESSOR/)
+        tf.unlink
+      end
+
+      it "complains about missing processor file" do
+        filename = File.join(File.dirname(__FILE__), "../../test_only.rb")
+        File.delete(filename) if File.exist?(filename)
+        @pc = PacketConfig.new
+
+        tf = Tempfile.new('unittest')
+        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+        tf.puts '  PROCESSOR TEST test_only.rb'
+        tf.close
+        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /TestOnly class not found/)
+        tf.unlink
+      end
+
+      it "complains about a non Cosmos::Processor class" do
+        filename = File.join(File.dirname(__FILE__), "../../processor1.rb")
+        File.open(filename, 'w') do |file|
+          file.puts "class Processor1"
+          file.puts "  def call(packet,buffer)"
+          file.puts "  end"
+          file.puts "end"
+        end
+        load 'processor1.rb'
+        File.delete(filename) if File.exist?(filename)
+
+        tf = Tempfile.new('unittest')
+        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+        tf.puts '  PROCESSOR P1 processor1.rb'
+        tf.close
+        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /processor must be a Cosmos::Processor but is a Processor1/)
+        tf.unlink
+      end
+
+      it "parses the processor" do
+        filename = File.join(File.dirname(__FILE__), "../../processor2.rb")
+        File.open(filename, 'w') do |file|
+          file.puts "require 'cosmos/processors/processor'"
+          file.puts "class Processor2 < Cosmos::Processor"
+          file.puts "  def call(packet,buffer)"
+          file.puts "    @results[:TEST] = 5"
+          file.puts "  end"
+          file.puts "end"
+        end
+        load 'processor2.rb'
+
+        tf = Tempfile.new('unittest')
+        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+        tf.puts '  ITEM item1 0 16 INT "Integer Item"'
+        tf.puts '    READ_CONVERSION processor_conversion.rb P2 TEST'
+        tf.puts '  PROCESSOR P2 processor2.rb'
+        tf.puts '  PROCESSOR P3 processor2.rb RAW'
+        tf.close
+        @pc.process_file(tf.path, "TGT1")
+        @pc.telemetry["TGT1"]["PKT1"].buffer = "\x01\x01"
+        expect(@pc.telemetry["TGT1"]["PKT1"].read("ITEM1")).to eql 5
+        tf.unlink
+
+        File.delete(filename) if File.exist?(filename)
+      end
+
+      it "complains if applied to a command packet" do
+        tf = Tempfile.new('unittest')
+        tf.puts 'COMMAND tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+        tf.puts '  PROCESSOR P1 processor1.rb'
+        tf.close
+        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, "PROCESSOR only applies to telemetry packets")
+        tf.unlink
+      end
+    end
+
+  end
+end

data/spec/packets/parsers/state_parser_spec.rb

@@ -1,156 +1,156 @@
-# 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/packet_config'
-require 'cosmos/packets/parsers/state_parser'
-require 'tempfile'
-
-module Cosmos
-
-  describe StateParser do
-
-    describe "process_file" do
-      before(:each) do
-        @pc = PacketConfig.new
-      end
-
-      it "complains if a current item is not defined" do
-        # Check for missing ITEM definitions
-        tf = Tempfile.new('unittest')
-        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
-        tf.puts 'STATE'
-        tf.close
-        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, "No current item for STATE")
-        tf.unlink
-      end
-
-      it "complains if there are not enough parameters" do
-        tf = Tempfile.new('unittest')
-        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
-        tf.puts 'ITEM myitem 0 8 UINT "Test Item"'
-        tf.puts 'STATE'
-        tf.close
-        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Not enough parameters for STATE/)
-        tf.unlink
-      end
-
-      it "complains if there are too many parameters" do
-        tf = Tempfile.new('unittest')
-        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
-        tf.puts 'ITEM myitem 0 8 UINT "Test Item"'
-        tf.puts 'STATE mystate 0 RED extra'
-        tf.close
-        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Too many parameters for STATE/)
-        tf.unlink
-      end
-
-      it "supports STRING items" do
-        tf = Tempfile.new('unittest')
-        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Description"'
-        tf.puts '  APPEND_ITEM item1 128 STRING "state item"'
-        tf.puts '    STATE FALSE "FALSE STRING"'
-        tf.puts '    STATE TRUE "TRUE STRING"'
-        tf.close
-        @pc.process_file(tf.path, "TGT1")
-        @pc.telemetry["TGT1"]["PKT1"].write("ITEM1", "TRUE STRING")
-        expect(@pc.telemetry["TGT1"]["PKT1"].read("ITEM1")).to eql "TRUE"
-        @pc.telemetry["TGT1"]["PKT1"].write("ITEM1", "FALSE STRING")
-        expect(@pc.telemetry["TGT1"]["PKT1"].read("ITEM1")).to eql "FALSE"
-        tf.unlink
-      end
-
-      it "warns about duplicate states and replace the duplicate" do
-        tf = Tempfile.new('unittest')
-        tf.puts 'COMMAND tgt1 pkt1 LITTLE_ENDIAN "Description"'
-        tf.puts '  APPEND_PARAMETER item1 8 UINT 0 2 0 "state item"'
-        tf.puts '    STATE FALSE 0'
-        tf.puts '    STATE TRUE 1'
-        tf.puts '    STATE FALSE 2'
-        tf.close
-        @pc.process_file(tf.path, "TGT1")
-        expect(@pc.warnings).to include("Duplicate state defined on line 5: STATE FALSE 2")
-        @pc.commands["TGT1"]["PKT1"].buffer = "\x00"
-        expect(@pc.commands["TGT1"]["PKT1"].read("ITEM1")).to eql 0
-        @pc.commands["TGT1"]["PKT1"].buffer = "\x02"
-        expect(@pc.commands["TGT1"]["PKT1"].read("ITEM1")).to eql "FALSE"
-        tf.unlink
-      end
-
-      context "with telemetry" do
-        it "only allows GREEN YELLOW or RED" do
-          tf = Tempfile.new('unittest')
-          tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Description"'
-          tf.puts '  APPEND_ITEM item1 8 UINT "state item"'
-          tf.puts '    STATE WORST 1 ORANGE'
-          tf.close
-          expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Invalid state color ORANGE/)
-          tf.unlink
-        end
-
-        it "records the state values and colors" do
-          tf = Tempfile.new('unittest')
-          tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Description"'
-          tf.puts '  APPEND_ITEM item1 8 UINT "state item"'
-          tf.puts '    STATE STATE1 1 RED'
-          tf.puts '    STATE STATE2 2 YELLOW'
-          tf.puts '    STATE STATE3 3 GREEN'
-          tf.puts '    STATE STATE4 4'
-          tf.close
-          @pc.process_file(tf.path, "TGT1")
-          index = 1
-          colors = [:RED, :YELLOW, :GREEN]
-          @pc.telemetry["TGT1"]["PKT1"].items["ITEM1"].states.each do |name,val|
-            expect(name).to eql "STATE#{index}"
-            expect(val).to eql index
-            expect(@pc.telemetry["TGT1"]["PKT1"].items["ITEM1"].state_colors[name]).to eql colors[index - 1]
-
-            index += 1
-          end
-          expect(@pc.telemetry["TGT1"]["PKT1"].limits_items).to eql [@pc.telemetry["TGT1"]["PKT1"].items["ITEM1"]]
-          tf.unlink
-        end
-      end
-
-      context "with command" do
-        it "only allows HAZARDOUS as the third param" do
-          tf = Tempfile.new('unittest')
-          tf.puts 'COMMAND tgt1 pkt1 LITTLE_ENDIAN "Description"'
-          tf.puts '  APPEND_PARAMETER item1 8 UINT 0 0 0'
-          tf.puts '    STATE WORST 0 RED'
-          tf.close
-          expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /HAZARDOUS expected as third parameter/)
-          tf.unlink
-        end
-
-        it "takes HAZARDOUS and an optional description" do
-          tf = Tempfile.new('unittest')
-          tf.puts 'COMMAND tgt1 pkt1 LITTLE_ENDIAN "Description"'
-          tf.puts '  APPEND_PARAMETER item1 8 UINT 1 3 1'
-          tf.puts '    STATE GOOD 1'
-          tf.puts '    STATE BAD 2 HAZARDOUS'
-          tf.puts '    STATE WORST 3 HAZARDOUS "Hazardous description"'
-          tf.close
-          @pc.process_file(tf.path, "TGT1")
-          @pc.commands["TGT1"]["PKT1"].buffer = "\x01"
-          @pc.commands["TGT1"]["PKT1"].check_limits
-          expect(@pc.commands["TGT1"]["PKT1"].items["ITEM1"].hazardous["GOOD"]).to be_nil
-          expect(@pc.commands["TGT1"]["PKT1"].items["ITEM1"].hazardous["BAD"]).not_to be_nil
-          expect(@pc.commands["TGT1"]["PKT1"].items["ITEM1"].hazardous["WORST"]).not_to be_nil
-          expect(@pc.commands["TGT1"]["PKT1"].items["ITEM1"].hazardous["WORST"]).to eql "Hazardous description"
-          expect(@pc.commands["TGT1"]["PKT1"].limits_items).to be_empty
-          tf.unlink
-        end
-      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/packet_config'
+require 'cosmos/packets/parsers/state_parser'
+require 'tempfile'
+
+module Cosmos
+
+  describe StateParser do
+
+    describe "process_file" do
+      before(:each) do
+        @pc = PacketConfig.new
+      end
+
+      it "complains if a current item is not defined" do
+        # Check for missing ITEM definitions
+        tf = Tempfile.new('unittest')
+        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+        tf.puts 'STATE'
+        tf.close
+        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, "No current item for STATE")
+        tf.unlink
+      end
+
+      it "complains if there are not enough parameters" do
+        tf = Tempfile.new('unittest')
+        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+        tf.puts 'ITEM myitem 0 8 UINT "Test Item"'
+        tf.puts 'STATE'
+        tf.close
+        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Not enough parameters for STATE/)
+        tf.unlink
+      end
+
+      it "complains if there are too many parameters" do
+        tf = Tempfile.new('unittest')
+        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+        tf.puts 'ITEM myitem 0 8 UINT "Test Item"'
+        tf.puts 'STATE mystate 0 RED extra'
+        tf.close
+        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Too many parameters for STATE/)
+        tf.unlink
+      end
+
+      it "supports STRING items" do
+        tf = Tempfile.new('unittest')
+        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Description"'
+        tf.puts '  APPEND_ITEM item1 128 STRING "state item"'
+        tf.puts '    STATE FALSE "FALSE STRING"'
+        tf.puts '    STATE TRUE "TRUE STRING"'
+        tf.close
+        @pc.process_file(tf.path, "TGT1")
+        @pc.telemetry["TGT1"]["PKT1"].write("ITEM1", "TRUE STRING")
+        expect(@pc.telemetry["TGT1"]["PKT1"].read("ITEM1")).to eql "TRUE"
+        @pc.telemetry["TGT1"]["PKT1"].write("ITEM1", "FALSE STRING")
+        expect(@pc.telemetry["TGT1"]["PKT1"].read("ITEM1")).to eql "FALSE"
+        tf.unlink
+      end
+
+      it "warns about duplicate states and replace the duplicate" do
+        tf = Tempfile.new('unittest')
+        tf.puts 'COMMAND tgt1 pkt1 LITTLE_ENDIAN "Description"'
+        tf.puts '  APPEND_PARAMETER item1 8 UINT 0 2 0 "state item"'
+        tf.puts '    STATE FALSE 0'
+        tf.puts '    STATE TRUE 1'
+        tf.puts '    STATE FALSE 2'
+        tf.close
+        @pc.process_file(tf.path, "TGT1")
+        expect(@pc.warnings).to include("Duplicate state defined on line 5: STATE FALSE 2")
+        @pc.commands["TGT1"]["PKT1"].buffer = "\x00"
+        expect(@pc.commands["TGT1"]["PKT1"].read("ITEM1")).to eql 0
+        @pc.commands["TGT1"]["PKT1"].buffer = "\x02"
+        expect(@pc.commands["TGT1"]["PKT1"].read("ITEM1")).to eql "FALSE"
+        tf.unlink
+      end
+
+      context "with telemetry" do
+        it "only allows GREEN YELLOW or RED" do
+          tf = Tempfile.new('unittest')
+          tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Description"'
+          tf.puts '  APPEND_ITEM item1 8 UINT "state item"'
+          tf.puts '    STATE WORST 1 ORANGE'
+          tf.close
+          expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Invalid state color ORANGE/)
+          tf.unlink
+        end
+
+        it "records the state values and colors" do
+          tf = Tempfile.new('unittest')
+          tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Description"'
+          tf.puts '  APPEND_ITEM item1 8 UINT "state item"'
+          tf.puts '    STATE STATE1 1 RED'
+          tf.puts '    STATE STATE2 2 YELLOW'
+          tf.puts '    STATE STATE3 3 GREEN'
+          tf.puts '    STATE STATE4 4'
+          tf.close
+          @pc.process_file(tf.path, "TGT1")
+          index = 1
+          colors = [:RED, :YELLOW, :GREEN]
+          @pc.telemetry["TGT1"]["PKT1"].items["ITEM1"].states.each do |name,val|
+            expect(name).to eql "STATE#{index}"
+            expect(val).to eql index
+            expect(@pc.telemetry["TGT1"]["PKT1"].items["ITEM1"].state_colors[name]).to eql colors[index - 1]
+
+            index += 1
+          end
+          expect(@pc.telemetry["TGT1"]["PKT1"].limits_items).to eql [@pc.telemetry["TGT1"]["PKT1"].items["ITEM1"]]
+          tf.unlink
+        end
+      end
+
+      context "with command" do
+        it "only allows HAZARDOUS as the third param" do
+          tf = Tempfile.new('unittest')
+          tf.puts 'COMMAND tgt1 pkt1 LITTLE_ENDIAN "Description"'
+          tf.puts '  APPEND_PARAMETER item1 8 UINT 0 0 0'
+          tf.puts '    STATE WORST 0 RED'
+          tf.close
+          expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /HAZARDOUS expected as third parameter/)
+          tf.unlink
+        end
+
+        it "takes HAZARDOUS and an optional description" do
+          tf = Tempfile.new('unittest')
+          tf.puts 'COMMAND tgt1 pkt1 LITTLE_ENDIAN "Description"'
+          tf.puts '  APPEND_PARAMETER item1 8 UINT 1 3 1'
+          tf.puts '    STATE GOOD 1'
+          tf.puts '    STATE BAD 2 HAZARDOUS'
+          tf.puts '    STATE WORST 3 HAZARDOUS "Hazardous description"'
+          tf.close
+          @pc.process_file(tf.path, "TGT1")
+          @pc.commands["TGT1"]["PKT1"].buffer = "\x01"
+          @pc.commands["TGT1"]["PKT1"].check_limits
+          expect(@pc.commands["TGT1"]["PKT1"].items["ITEM1"].hazardous["GOOD"]).to be_nil
+          expect(@pc.commands["TGT1"]["PKT1"].items["ITEM1"].hazardous["BAD"]).not_to be_nil
+          expect(@pc.commands["TGT1"]["PKT1"].items["ITEM1"].hazardous["WORST"]).not_to be_nil
+          expect(@pc.commands["TGT1"]["PKT1"].items["ITEM1"].hazardous["WORST"]).to eql "Hazardous description"
+          expect(@pc.commands["TGT1"]["PKT1"].limits_items).to be_empty
+          tf.unlink
+        end
+      end
+
+    end
+  end
+end