cosmos 4.0.3-java

data/spec/packets/parsers/packet_parser_spec.rb

@@ -0,0 +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

data/spec/packets/parsers/processor_parser_spec.rb

@@ -0,0 +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

data/spec/packets/parsers/state_parser_spec.rb

@@ -0,0 +1,189 @@
+# 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 LIMITS defined" do
+        tf = Tempfile.new('unittest')
+        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+        tf.puts '  ITEM myitem 0 8 UINT "Test Item"'
+        tf.puts '    LIMITS DEFAULT 3 ENABLED 1 2 6 7 3 5'
+        tf.puts '    STATE ONE 1'
+        tf.close
+        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Items with LIMITS can't define STATE/)
+        tf.unlink
+      end
+
+      it "complains if FORMAT_STRING defined" do
+        tf = Tempfile.new('unittest')
+        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+        tf.puts '  ITEM myitem 0 8 UINT "Test Item"'
+        tf.puts '    FORMAT_STRING "0x%x"'
+        tf.puts '    STATE ONE 1'
+        tf.close
+        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Items with FORMAT_STRING can't define STATE/)
+        tf.unlink
+      end
+
+      it "complains if UNITS defined" do
+        tf = Tempfile.new('unittest')
+        tf.puts 'TELEMETRY tgt1 pkt1 LITTLE_ENDIAN "Packet"'
+        tf.puts '  ITEM myitem 0 8 UINT "Test Item"'
+        tf.puts '    UNITS Kelvin K'
+        tf.puts '    STATE ONE 1'
+        tf.close
+        expect { @pc.process_file(tf.path, "TGT1") }.to raise_error(ConfigParser::Error, /Items with UNITS can't define 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

data/spec/packets/structure_item_spec.rb

@@ -0,0 +1,213 @@
+# 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/structure_item'
+
+module Cosmos
+
+  describe StructureItem do
+
+    describe "name=" do
+      it "creates new structure items" do
+        expect(StructureItem.new("test", 0, 8, :UINT, :BIG_ENDIAN, nil).name).to eql "TEST"
+      end
+
+      it "complains about non String names" do
+        expect { StructureItem.new(nil, 0, 8, :UINT, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "name must be a String but is a NilClass")
+        expect { StructureItem.new(5.1, 0, 8, :UINT, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "name must be a String but is a Float")
+      end
+
+      it "complains about blank names" do
+        expect { StructureItem.new("", 0, 8, :UINT, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "name must contain at least one character")
+      end
+    end
+
+    describe "endianness=" do
+      it "accepts BIG_ENDIAN and LITTLE_ENDIAN" do
+        expect(StructureItem.new("test", 0, 8, :UINT, :BIG_ENDIAN, nil).endianness).to eql :BIG_ENDIAN
+        expect(StructureItem.new("test", 0, 8, :UINT, :LITTLE_ENDIAN, nil).endianness).to eql :LITTLE_ENDIAN
+      end
+
+      it "complains about bad endianness" do
+        expect { StructureItem.new("test", 0, 8, :UINT, :BLAH, nil) }.to raise_error(ArgumentError, "TEST: unknown endianness: BLAH - Must be :BIG_ENDIAN or :LITTLE_ENDIAN")
+      end
+    end
+
+    describe "data_type=" do
+      it "accepts INT, UINT, FLOAT, STRING, BLOCK, and DERIVED data types" do
+        %w(INT UINT FLOAT STRING BLOCK).each do |type|
+          expect(StructureItem.new("test", 0, 32, type.to_sym, :BIG_ENDIAN, nil).data_type).to eql type.to_sym
+        end
+        expect(StructureItem.new("test", 0, 0, :DERIVED, :BIG_ENDIAN, nil).data_type).to eql :DERIVED
+      end
+
+      it "complains about bad data types" do
+        expect { StructureItem.new("test", 0, 0, :UNKNOWN, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "TEST: unknown data_type: UNKNOWN - Must be :INT, :UINT, :FLOAT, :STRING, :BLOCK, or :DERIVED")
+      end
+    end
+
+    describe "overflow=" do
+      it "accepts ERROR, ERROR_ALLOW_HEX, TRUNCATE and SATURATE overflow types" do
+        %w(ERROR ERROR_ALLOW_HEX TRUNCATE SATURATE).each do |type|
+          expect(StructureItem.new("test", 0, 32, :INT, :BIG_ENDIAN, nil, type.to_sym).overflow).to eql type.to_sym
+        end
+      end
+
+      it "complains about bad overflow types" do
+        expect { StructureItem.new("test", 0, 32, :INT, :BIG_ENDIAN, nil, :UNKNOWN) }.to raise_error(ArgumentError, "TEST: unknown overflow type: UNKNOWN - Must be :ERROR, :ERROR_ALLOW_HEX, :TRUNCATE, or :SATURATE")
+      end
+    end
+
+    describe "bit_offset=" do
+      it "compains about bad bit offsets types" do
+        if 0.class == Integer
+          # Ruby version >= 2.4.0
+          expect { StructureItem.new("test", nil, 8, :UINT, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "TEST: bit_offset must be an Integer")
+        else
+          # Ruby version < 2.4.0
+          expect { StructureItem.new("test", nil, 8, :UINT, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "TEST: bit_offset must be a Fixnum")
+        end
+      end
+
+      it "complains about unaligned bit offsets" do
+        %w(FLOAT STRING BLOCK).each do |type|
+          expect { StructureItem.new("test", 1, 32, type.to_sym, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "TEST: bit_offset for :FLOAT, :STRING, and :BLOCK items must be byte aligned")
+        end
+      end
+
+      it "complains about non zero DERIVED bit offsets" do
+        expect { StructureItem.new("test", 8, 0, :DERIVED, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "TEST: DERIVED items must have bit_offset of zero")
+      end
+    end
+
+    describe "bit_size=" do
+      it "complains about bad bit sizes types" do
+        if 0.class == Integer
+          # Ruby version >= 2.4.0
+          expect { StructureItem.new("test", 0, nil, :UINT, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "TEST: bit_size must be an Integer")
+          else
+          # Ruby version < 2.4.0
+          expect { StructureItem.new("test", 0, nil, :UINT, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "TEST: bit_size must be a Fixnum")
+        end
+      end
+
+      it "complains about 0 size INT, UINT, and FLOAT" do
+        %w(INT UINT FLOAT).each do |type|
+          expect { StructureItem.new("test", 0, 0, type.to_sym, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "TEST: bit_size cannot be negative or zero for :INT, :UINT, and :FLOAT items: 0")
+        end
+      end
+
+      it "complains about bad float bit sizes" do
+        expect { StructureItem.new("test", 0, 8, :FLOAT, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "TEST: bit_size for FLOAT items must be 32 or 64. Given: 8")
+      end
+
+      it "creates 32 and 64 bit floats" do
+        expect(StructureItem.new("test", 0, 32, :FLOAT, :BIG_ENDIAN, nil).bit_size).to eql 32
+        expect(StructureItem.new("test", 0, 64, :FLOAT, :BIG_ENDIAN, nil).bit_size).to eql 64
+      end
+
+      it "complains about non zero DERIVED bit sizes" do
+        expect { StructureItem.new("test", 0, 8, :DERIVED, :BIG_ENDIAN, nil) }.to raise_error(ArgumentError, "TEST: DERIVED items must have bit_size of zero")
+      end
+    end
+
+    describe "array_size=" do
+      it "complains about bad array size types" do
+        if 0.class == Integer
+          # Ruby version >= 2.4.0
+          expect { StructureItem.new("test", 0, 8, :UINT, :BIG_ENDIAN, "") }.to raise_error(ArgumentError, "TEST: array_size must be an Integer")
+        else
+          # Ruby version < 2.4.0
+          expect { StructureItem.new("test", 0, 8, :UINT, :BIG_ENDIAN, "") }.to raise_error(ArgumentError, "TEST: array_size must be a Fixnum")
+        end
+      end
+
+      it "complains about array size != multiple of bit size" do
+        expect { StructureItem.new("test", 0, 8, :UINT, :BIG_ENDIAN, 10) }.to raise_error(ArgumentError, "TEST: array_size must be a multiple of bit_size")
+      end
+
+      it "does not complain about array size != multiple of bit size with negative array size" do
+        expect { StructureItem.new("test", 0, 32, :UINT, :BIG_ENDIAN, -8) }.not_to raise_error
+      end
+    end
+
+    describe "<=>" do
+      it "sorts items according to positive bit offset" do
+        si1 = StructureItem.new("si1", 0, 8, :UINT, :BIG_ENDIAN, nil)
+        si2 = StructureItem.new("si2", 8, 8, :UINT, :BIG_ENDIAN, nil)
+        expect(si1 < si2).to be true
+        expect(si1 == si2).to be false
+        expect(si1 > si2).to be false
+
+        si2 = StructureItem.new("si2", 0, 8, :UINT, :BIG_ENDIAN, nil)
+        expect(si1 < si2).to be false
+        expect(si1 == si2).to be true
+        expect(si1 > si2).to be false
+      end
+
+      it "sorts items with 0 bit offset according to bit size" do
+        si1 = StructureItem.new("si1", 0, 8, :UINT, :BIG_ENDIAN, nil)
+        si2 = StructureItem.new("si2", 0, 0, :BLOCK, :BIG_ENDIAN, nil)
+        expect(si1 < si2).to be false
+        expect(si1 == si2).to be false
+        expect(si1 > si2).to be true
+      end
+
+      it "sorts items according to negative bit offset" do
+        si1 = StructureItem.new("si1", -8, 8, :UINT, :BIG_ENDIAN, nil)
+        si2 = StructureItem.new("si2", -16, 8, :UINT, :BIG_ENDIAN, nil)
+        expect(si1 < si2).to be false
+        expect(si1 == si2).to be false
+        expect(si1 > si2).to be true
+
+        si2 = StructureItem.new("si2", -8, 8, :UINT, :BIG_ENDIAN, nil)
+        expect(si1 < si2).to be false
+        # si1 == si2 even though they have different names and sizes
+        expect(si1 == si2).to be true
+        expect(si1 > si2).to be false
+      end
+
+      it "sorts items according to mixed bit offset" do
+        si1 = StructureItem.new("si1", 16, 8, :UINT, :BIG_ENDIAN, nil)
+        si2 = StructureItem.new("si2", -8, 8, :UINT, :BIG_ENDIAN, nil)
+        expect(si1 < si2).to be true
+        expect(si1 == si2).to be false
+        expect(si1 > si2).to be false
+      end
+    end
+
+    describe "clone" do
+      it "duplicates the entire structure item " do
+        si1 = StructureItem.new("si1", -8, 1, :UINT, :LITTLE_ENDIAN, nil)
+        si2 = si1.clone
+        expect(si1 == si2).to be true
+      end
+    end
+
+    describe "to_hash" do
+      it "creates a Hash" do
+        item = StructureItem.new("test", 0, 8, :UINT, :BIG_ENDIAN, 16)
+        hash = item.to_hash
+        expect(hash.keys.length).to eql 7
+        expect(hash.keys).to include('name','bit_offset','bit_size','data_type','endianness','array_size', 'overflow')
+        expect(hash["name"]).to eql "TEST"
+        expect(hash["bit_offset"]).to eql 0
+        expect(hash["bit_size"]).to eql 8
+        expect(hash["data_type"]).to eql :UINT
+        expect(hash["endianness"]).to eql :BIG_ENDIAN
+        expect(hash["array_size"]).to eql 16
+        expect(hash["overflow"]).to eql :ERROR
+      end
+    end
+
+  end
+end