cosmos 4.1.0 → 4.1.1

data/spec/install/config/targets/INST/screens/hs.txt

@@ -0,0 +1,42 @@
+SCREEN AUTO AUTO 0.5
+GLOBAL_SETTING LABELVALUELIMITSBAR COLORBLIND TRUE
+
+VERTICAL
+
+  TITLE "Instrument Health and Status"
+  SETTING BACKCOLOR 162 181 205
+  SETTING TEXTCOLOR black
+
+  VERTICALBOX
+    SECTIONHEADER "General Telemetry"
+    NAMED_WIDGET COLLECT_TYPE COMBOBOX NORMAL SPECIAL
+    BUTTON 'Start Collect' 'target_name = get_target_name("INST"); cmd("#{target_name} COLLECT with TYPE NORMAL, DURATION 5")'
+    FORMATVALUE INST HEALTH_STATUS COLLECTS "0x%08X"
+    LABELVALUE INST HEALTH_STATUS COLLECT_TYPE
+    LABELVALUE INST HEALTH_STATUS DURATION
+    LABELVALUE INST HEALTH_STATUS ASCIICMD WITH_UNITS 30
+  END
+  SETTING BACKCOLOR 163 185 163
+
+  VERTICALBOX
+    SECTIONHEADER "Temperatures"
+    LABELTRENDLIMITSBAR INST HEALTH_STATUS TEMP1 WITH_UNITS 5
+    LABELVALUELIMITSBAR INST HEALTH_STATUS TEMP2 CONVERTED 25
+    # LABELVALUELIMITSBAR INST HEALTH_STATUS TEMP2 RAW 20 # RAW is not allowed for LIMITSBAR widgets
+    LABELVALUELIMITSBAR INST HEALTH_STATUS TEMP2 FORMATTED
+    LABELVALUELIMITSBAR INST HEALTH_STATUS TEMP2 WITH_UNITS
+    LABELVALUELIMITSBAR INST HEALTH_STATUS TEMP3
+    LABELVALUELIMITSBAR INST HEALTH_STATUS TEMP4
+    SETTING GRAY_TOLERANCE 0.1
+  END
+  SETTING BACKCOLOR 203 173 158
+
+  VERTICALBOX
+    SECTIONHEADER "Ground Station"
+    LABELVALUE INST HEALTH_STATUS GROUND1STATUS
+    LABELVALUE INST HEALTH_STATUS GROUND2STATUS
+  END
+
+  SETTING BACKCOLOR 207 171 169
+END
+SETTING BACKCOLOR 162 181 205

data/spec/install/config/targets/INST/target.txt

@@ -8,3 +8,5 @@ IGNORE_PARAMETER CCSDSSEQFLAGS
 IGNORE_PARAMETER CCSDSSEQCNT
 IGNORE_PARAMETER CCSDSLENGTH
 IGNORE_PARAMETER PKTID
+
+AUTO_SCREEN_SUBSTITUTE

data/spec/interfaces/protocols/burst_protocol_spec.rb

@@ -197,6 +197,24 @@ module Cosmos
         pkt = @interface.read
         expect(pkt.length).to eql 3 # sync plus one byte
       end
+
+      it "handle auto allow_empty_data correctly" do
+        @interface.add_protocol(BurstProtocol, [0, nil, false, nil], :READ_WRITE)
+        expect(@interface.read_protocols[0].read_data("")).to eql :STOP
+        expect(@interface.read_protocols[0].read_data("A")).to eql "A"
+        @interface.add_protocol(BurstProtocol, [0, nil, false, nil], :READ_WRITE)
+        expect(@interface.read_protocols[0].read_data("")).to eql ""
+        expect(@interface.read_protocols[1].read_data("")).to eql :STOP
+        expect(@interface.read_protocols[0].read_data("A")).to eql "A"
+        expect(@interface.read_protocols[1].read_data("A")).to eql "A"
+        @interface.add_protocol(BurstProtocol, [0, nil, false, nil], :READ_WRITE)
+        expect(@interface.read_protocols[0].read_data("")).to eql ""
+        expect(@interface.read_protocols[1].read_data("")).to eql ""
+        expect(@interface.read_protocols[2].read_data("")).to eql :STOP
+        expect(@interface.read_protocols[0].read_data("A")).to eql "A"
+        expect(@interface.read_protocols[1].read_data("A")).to eql "A"
+        expect(@interface.read_protocols[2].read_data("A")).to eql "A"
+      end
     end
 
     describe "write" do

data/spec/interfaces/protocols/length_protocol_spec.rb

@@ -46,6 +46,55 @@ module Cosmos
     end
 
     describe "read" do
+      it "caches data for reads correctly" do
+        @interface.instance_variable_set(:@stream, LengthStream.new)
+        @interface.add_protocol(LengthProtocol, [
+          0, # bit offset
+          8, # bit size
+          0,  # length offset
+          1,  # bytes per count
+          'BIG_ENDIAN'], :READ_WRITE)
+        $buffer = "\x02\x03\x02\x05"
+        packet = @interface.read
+        expect(packet.buffer.length).to eql 2
+        expect(packet.buffer).to eql "\x02\x03"
+        packet = @interface.read
+        expect(packet.buffer.length).to eql 2
+        expect(packet.buffer).to eql "\x02\x05"
+        expect(@interface.read_protocols[0].read_data("\x03\x01\x02\x03\x04\x05")).to eql "\x03\x01\x02"
+        expect(@interface.read_protocols[0].read_data("")).to eql "\x03\x04\x05"
+        expect(@interface.read_protocols[0].read_data("")).to eql :STOP
+      end
+
+      # This test case uses two length protocols to verify that data flows correctly between the two protocols and that earlier data
+      # is removed correctly using discard leading bytes.  In general it is not typical to use two different length protocols, but it could
+      # be useful to pull out a packet inside of a packet.
+      it "caches data for reads correctly with multiple protocols" do
+        @interface.instance_variable_set(:@stream, LengthStream.new)
+        @interface.add_protocol(LengthProtocol, [
+          0, # bit offset
+          8, # bit size
+          0,  # length offset
+          1,  # bytes per count
+          'BIG_ENDIAN'], :READ_WRITE)
+        @interface.add_protocol(LengthProtocol, [
+          0, # bit offset
+          8, # bit size
+          0,  # length offset
+          1,  # bytes per count
+          'BIG_ENDIAN',
+          1], :READ_WRITE) # Discard leading bytes set to 1
+        # The second protocol above will receive the two byte packets from the first protocol and
+        # then drop the length field.
+        $buffer = "\x02\x03\x02\x05"
+        packet = @interface.read
+        expect(packet.buffer.length).to eql 1
+        expect(packet.buffer).to eql "\x03"
+        packet = @interface.read
+        expect(packet.buffer.length).to eql 1
+        expect(packet.buffer).to eql "\x05"
+      end
+
       it "reads LITTLE_ENDIAN length fields from the stream" do
         @interface.instance_variable_set(:@stream, LengthStream.new)
         @interface.add_protocol(LengthProtocol, [

data/spec/interfaces/udp_interface_spec.rb

@@ -100,15 +100,6 @@ module Cosmos
     end
 
     describe "read" do
-      it "stops the read thread if no read port given" do
-        i = UdpInterface.new('localhost','8888','nil')
-        i.connect
-        thread = Thread.new { i.read }
-        sleep 0.1
-        expect(thread.stop?).to be true
-        Cosmos.kill_thread(nil, thread)
-      end
-
       it "stops the read thread if there is an IOError" do
         read = double("read")
         allow(read).to receive(:read).and_raise(IOError)

data/spec/packets/packet_config_spec.rb

@@ -13,36 +13,6 @@ require 'cosmos'
 require 'cosmos/packets/packet_config'
 require 'tempfile'
 
-XTCE_START =<<END
-<?xml version="1.0" encoding="UTF-8"?>
-<xtce:SpaceSystem xmlns:xtce="http://www.omg.org/space/xtce" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" name="INST" xsi:schemaLocation="http://www.omg.org/space/xtce http://www.omg.org/spec/XTCE/20061101/06-11-06.xsd">
-  <xtce:TelemetryMetaData>
-    <xtce:ParameterTypeSet>
-      <xtce:IntegerParameterType name="A_Type" shortDescription="A" signed="false">
-        <xtce:UnitSet/>
-END
-XTCE_END =<<END
-      </xtce:IntegerParameterType>
-    </xtce:ParameterTypeSet>
-    <xtce:ParameterSet>
-      <xtce:Parameter name="A" parameterTypeRef="A_Type"/>
-    </xtce:ParameterSet>
-    <xtce:ContainerSet>
-      <xtce:SequenceContainer name="B_Base" abstract="true">
-        <xtce:EntryList>
-          <xtce:ParameterRefEntry parameterRef="A"/>
-        </xtce:EntryList>
-      </xtce:SequenceContainer>
-      <xtce:SequenceContainer name="B" shortDescription="B">
-        <xtce:EntryList/>
-        <xtce:BaseContainer containerRef="B_Base">
-        </xtce:BaseContainer>
-      </xtce:SequenceContainer>
-    </xtce:ContainerSet>
-  </xtce:TelemetryMetaData>
-</xtce:SpaceSystem>
-END
-
 module Cosmos
 
   describe PacketConfig do
@@ -52,120 +22,6 @@ module Cosmos
         @pc = PacketConfig.new
       end
 
-      describe "xtce support" do
-        it "processes xtce telemetry" do
-          tf = Tempfile.new(['unittest', '.xtce'])
-          tf.puts XTCE_START
-          tf.puts '<xtce:IntegerDataEncoding sizeInBits="32" encoding="unsigned"/>'
-          tf.puts XTCE_END
-          tf.close
-
-          @pc.process_file(tf.path, 'TEST')
-
-          packet = @pc.telemetry['TEST']['B']
-          expect(packet).to_not be_nil
-          expect(packet.get_item('A').endianness).to eql :BIG_ENDIAN
-
-          tf.unlink
-        end
-
-        it "processes explicit big endian xtce telemetry" do
-          tf = Tempfile.new(['unittest', '.xtce'])
-          tf.puts XTCE_START
-          tf.puts '<xtce:IntegerDataEncoding sizeInBits="32" encoding="unsigned">' + "\n"
-          tf.puts '  <xtce:ByteOrderList>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="3"/>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="2"/>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="1"/>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="0"/>' + "\n"
-          tf.puts '  </xtce:ByteOrderList>' + "\n"
-          tf.puts '</xtce:IntegerDataEncoding>' + "\n"
-          tf.puts XTCE_END
-          tf.close
-
-          @pc.process_file(tf.path, 'TEST')
-
-          packet = @pc.telemetry['TEST']['B']
-          expect(packet).to_not be_nil
-          expect(packet.get_item('A').endianness).to eql :BIG_ENDIAN
-          expect(@pc.warnings).to be_empty
-
-          tf.unlink
-        end
-
-        it "processes explicit little endian xtce telemetry" do
-          tf = Tempfile.new(['unittest', '.xtce'])
-          tf.puts XTCE_START
-          tf.puts '<xtce:IntegerDataEncoding sizeInBits="32" encoding="unsigned">' + "\n"
-          tf.puts '  <xtce:ByteOrderList>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="0"/>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="1"/>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="2"/>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="3"/>' + "\n"
-          tf.puts '  </xtce:ByteOrderList>' + "\n"
-          tf.puts '</xtce:IntegerDataEncoding>' + "\n"
-          tf.puts XTCE_END
-          tf.close
-
-          @pc.process_file(tf.path, 'TEST')
-
-          packet = @pc.telemetry['TEST']['B']
-          expect(packet).to_not be_nil
-          expect(packet.get_item('A').endianness).to eql :LITTLE_ENDIAN
-          expect(@pc.warnings).to be_empty
-
-          tf.unlink
-        end
-
-        it "warn of bad byteorderlist no zero xtce telemetry" do
-          tf = Tempfile.new(['unittest', '.xtce'])
-          tf.puts XTCE_START
-          tf.puts '<xtce:IntegerDataEncoding sizeInBits="32" encoding="unsigned">' + "\n"
-          tf.puts '  <xtce:ByteOrderList>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="1"/>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="2"/>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="3"/>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="4"/>' + "\n"
-          tf.puts '  </xtce:ByteOrderList>' + "\n"
-          tf.puts '</xtce:IntegerDataEncoding>' + "\n"
-          tf.puts XTCE_END
-          tf.close
-
-          @pc.process_file(tf.path, 'TEST')
-
-          packet = @pc.telemetry['TEST']['B']
-          expect(packet).to_not be_nil
-          expect(packet.get_item('A').endianness).to eql :BIG_ENDIAN
-          expect(@pc.warnings).to_not be_empty
-
-          tf.unlink
-        end
-
-        it "warn of bad byteorderlist scrambled xtce telemetry" do
-          tf = Tempfile.new(['unittest', '.xtce'])
-          tf.puts XTCE_START
-          tf.puts '<xtce:IntegerDataEncoding sizeInBits="32" encoding="unsigned">' + "\n"
-          tf.puts '  <xtce:ByteOrderList>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="0"/>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="2"/>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="1"/>' + "\n"
-          tf.puts '    <xtce:Byte byteSignificance="3"/>' + "\n"
-          tf.puts '  </xtce:ByteOrderList>' + "\n"
-          tf.puts '</xtce:IntegerDataEncoding>' + "\n"
-          tf.puts XTCE_END
-          tf.close
-
-          @pc.process_file(tf.path, 'TEST')
-
-          packet = @pc.telemetry['TEST']['B']
-          expect(packet).to_not be_nil
-          expect(packet.get_item('A').endianness).to eql :LITTLE_ENDIAN
-          expect(@pc.warnings).to_not be_empty
-
-          tf.unlink
-        end
-      end
-
       it "complains about unknown keywords" do
         tf = Tempfile.new('unittest')
         tf.puts("BLAH")
@@ -174,6 +30,27 @@ module Cosmos
         tf.unlink
       end
 
+      it "outputs parsed definitions back to a file" do
+        tf = Tempfile.new('unittest')
+        tlm = "TELEMETRY TGT1 PKT1 LITTLE_ENDIAN \"Telemetry\"\n"\
+              "  ITEM BYTE 0 8 UINT \"Item\"\n"
+        tf.write tlm
+        cmd = "COMMAND TGT1 PKT1 LITTLE_ENDIAN \"Command\"\n"\
+              "  PARAMETER PARAM 0 16 UINT 0 0 0 \"Param\"\n"
+        tf.write cmd
+        limits = "LIMITS_GROUP TVAC\n"\
+                 "  LIMITS_GROUP_ITEM TGT1 PKT1 ITEM1\n"
+        tf.write limits
+        tf.close
+        @pc.process_file(tf.path, "TGT1")
+        @pc.to_config(System.paths["LOGS"])
+        @pc.to_xtce(System.paths["LOGS"])
+        expect(cmd.strip).to eql File.read(File.join(System.paths["LOGS"], 'TGT1', 'cmd_tlm', 'tgt1_cmd.txt')).strip
+        expect(tlm.strip).to eql File.read(File.join(System.paths["LOGS"], 'TGT1', 'cmd_tlm', 'tgt1_tlm.txt')).strip
+        expect(limits.strip).to eql File.read(File.join(System.paths["LOGS"], 'SYSTEM', 'cmd_tlm', 'limits_groups.txt')).strip
+        tf.unlink
+      end
+
       context "with all telemetry keywords" do
         before(:all) do
           # top level keywords

data/spec/packets/packet_item_spec.rb

@@ -24,6 +24,7 @@ module Cosmos
       it "sets the format_string" do
         @pi.format_string = "%5.1f"
         expect(@pi.format_string).to eql "%5.1f"
+        expect(@pi.to_config(:TELEMETRY, :BIG_ENDIAN)).to match /FORMAT_STRING %5.1f/
       end
 
       it "sets the format_string to nil" do
@@ -47,6 +48,10 @@ module Cosmos
         c = GenericConversion.new("value / 2")
         @pi.read_conversion = c
         expect(@pi.read_conversion.to_s == c.to_s).to be true
+        config = @pi.to_config(:TELEMETRY, :BIG_ENDIAN)
+        expect(config).to match /GENERIC_READ_CONVERSION_START/
+        expect(config).to match /value \/ 2/
+        expect(config).to match /GENERIC_READ_CONVERSION_END/
       end
 
       it "sets the read_conversion to nil" do
@@ -64,6 +69,10 @@ module Cosmos
         c = GenericConversion.new("value / 2")
         @pi.write_conversion = c
         expect(@pi.write_conversion.to_s == c.to_s).to be true
+        config = @pi.to_config(:TELEMETRY, :BIG_ENDIAN)
+        expect(config).to match /GENERIC_WRITE_CONVERSION_START/
+        expect(config).to match /value \/ 2/
+        expect(config).to match /GENERIC_WRITE_CONVERSION_END/
       end
 
       it "sets the write_conversion to nil" do
@@ -78,14 +87,22 @@ module Cosmos
 
     describe "id_value=" do
       it "accepts id values according to data_type" do
+        @pi.range = (0..10)
         @pi.id_value = 10
         expect(@pi.id_value).to eql 10
         @pi.data_type = :FLOAT
         @pi.id_value = 10.0
         expect(@pi.id_value).to eql 10.0
+        expect(@pi.to_config(:COMMAND, :BIG_ENDIAN)).to match(/ID_PARAMETER TEST 0 32 FLOAT 0 10 10.0/)
+        expect(@pi.to_config(:TELEMETRY, :BIG_ENDIAN)).to match(/ID_ITEM TEST 0 32 FLOAT 10.0/)
         @pi.data_type = :STRING
         @pi.id_value = "HI"
         expect(@pi.id_value).to eql "HI"
+        expect(@pi.to_config(:COMMAND, :BIG_ENDIAN)).to match(/ID_PARAMETER TEST 0 32 STRING "HI"/)
+        expect(@pi.to_config(:TELEMETRY, :BIG_ENDIAN)).to match(/ID_ITEM TEST 0 32 STRING "HI"/)
+        @pi.id_value = "\xDE\xAD\xBE\xEF" # binary
+        expect(@pi.to_config(:COMMAND, :BIG_ENDIAN)).to match(/ID_PARAMETER TEST 0 32 STRING 0xDEADBEEF/)
+        expect(@pi.to_config(:TELEMETRY, :BIG_ENDIAN)).to match(/ID_ITEM TEST 0 32 STRING 0xDEADBEEF/)
       end
 
       it "sets the id_value to nil" do
@@ -105,6 +122,9 @@ module Cosmos
         states = {"TRUE"=>1, "FALSE"=>0}
         @pi.states = states
         expect(@pi.states).to eql states
+        config = @pi.to_config(:TELEMETRY, :BIG_ENDIAN)
+        expect(config).to match(/STATE TRUE 1/)
+        expect(config).to match(/STATE FALSE 0/)
       end
 
       it "sets the states to nil" do
@@ -122,6 +142,7 @@ module Cosmos
         description = "this is it"
         @pi.description = description
         expect(@pi.description).to eql description
+        expect(@pi.to_config(:TELEMETRY, :BIG_ENDIAN)).to match(/ITEM TEST 0 32 UINT "this is it"/)
       end
 
       it "sets the description to nil" do
@@ -156,6 +177,8 @@ module Cosmos
         units = "V"
         @pi.units = units
         expect(@pi.units).to eql units
+        @pi.units_full = "Volts"
+        expect(@pi.to_config(:TELEMETRY, :BIG_ENDIAN)).to match(/UNITS Volts V/)
       end
 
       it "sets the units to nil" do
@@ -173,15 +196,25 @@ module Cosmos
         pi = PacketItem.new("test", 0, 8, :INT, :BIG_ENDIAN, 16)
         pi.default = [1, -1]
         expect(pi.default).to eql [1, -1]
+        expect(pi.to_config(:COMMAND, :BIG_ENDIAN)).to match(/ARRAY_PARAMETER TEST 0 8 INT 16/)
+        expect(pi.to_config(:TELEMETRY, :BIG_ENDIAN)).to match(/ARRAY_ITEM TEST 0 8 INT 16/)
         pi = PacketItem.new("test", 0, 32, :UINT, :BIG_ENDIAN, nil)
+        pi.range = (0..10)
         pi.default = 0x01020304
         expect(pi.default).to eql 0x01020304
+        expect(pi.to_config(:COMMAND, :BIG_ENDIAN)).to match(/PARAMETER TEST 0 32 UINT 0 10 16909060/)
         pi = PacketItem.new("test", 0, 32, :FLOAT, :BIG_ENDIAN, nil)
+        pi.range = (-10..10)
         pi.default = 5.5
         expect(pi.default).to eql 5.5
+        expect(pi.to_config(:COMMAND, :BIG_ENDIAN)).to match(/PARAMETER TEST 0 32 FLOAT -10 10 5.5/)
         pi = PacketItem.new("test", 0, 32, :STRING, :BIG_ENDIAN, nil)
         pi.default = "HI"
         expect(pi.default).to eql "HI"
+        expect(pi.to_config(:COMMAND, :BIG_ENDIAN)).to match(/PARAMETER TEST 0 32 STRING "HI"/)
+        pi = PacketItem.new("test", 0, 32, :STRING, :BIG_ENDIAN, nil)
+        pi.default = "\xDE\xAD\xBE\xEF"
+        expect(pi.to_config(:COMMAND, :BIG_ENDIAN)).to match(/PARAMETER TEST 0 32 STRING 0xDEADBEEF/)
       end
 
       it "sets the default to nil" do
@@ -269,6 +302,12 @@ module Cosmos
         expect(@pi.hazardous).to eql hazardous
         expect(@pi.hazardous["TRUE"]).to eql hazardous["TRUE"]
         expect(@pi.hazardous["FALSE"]).to eql hazardous["FALSE"]
+
+        @pi.range = (0..1)
+        @pi.states = {"TRUE"=>1, "FALSE"=>0}
+        config = @pi.to_config(:COMMAND, :BIG_ENDIAN)
+        expect(config).to match(/STATE TRUE 1/)
+        expect(config).to match(/STATE FALSE 0 HAZARDOUS "NO FALSE ALLOWED"/)
       end
 
       it "sets hazardous to nil" do
@@ -286,6 +325,12 @@ module Cosmos
         state_colors = {"TRUE"=>:GREEN, "FALSE"=>:RED}
         @pi.state_colors = state_colors
         expect(@pi.state_colors).to eql state_colors
+
+        @pi.range = (0..1)
+        @pi.states = {"TRUE"=>1, "FALSE"=>0}
+        config = @pi.to_config(:TELEMETRY, :BIG_ENDIAN)
+        expect(config).to match(/STATE TRUE 1 GREEN/)
+        expect(config).to match(/STATE FALSE 0 RED/)
       end
 
       it "sets the state_colors to nil" do
@@ -301,7 +346,16 @@ module Cosmos
     describe "limits=" do
       it "accepts limits as a PacketItemLimits" do
         limits = PacketItemLimits.new
+        limits.values = {DEFAULT: [10, 20, 80, 90, 40, 50], TVAC: [100, 200, 800, 900]}
         @pi.limits = limits
+        config = @pi.to_config(:TELEMETRY, :BIG_ENDIAN)
+        expect(config).to match(/LIMITS DEFAULT 1 DISABLED 10 20 80 90 40 50/)
+        expect(config).to match(/LIMITS TVAC 1 DISABLED 100 200 800 900/)
+        @pi.limits.enabled = true
+        @pi.limits.persistence_setting = 3
+        config = @pi.to_config(:TELEMETRY, :BIG_ENDIAN)
+        expect(config).to match(/LIMITS DEFAULT 3 ENABLED 10 20 80 90 40 50/)
+        expect(config).to match(/LIMITS TVAC 3 ENABLED 100 200 800 900/)
       end
 
       it "sets the limits to nil" do
@@ -314,10 +368,20 @@ module Cosmos
       end
     end
 
-    describe "meta" do
-      it "allows adding items to the meta hash" do
-        @pi.meta['TYPE'] = 'float32'
-        expect(@pi.meta['TYPE']).to eql 'float32'
+    describe "meta=" do
+      it "only allows a hash" do
+        expect { @pi.meta = 1 }.to raise_error(ArgumentError, /must be a Hash/)
+      end
+
+      it "sets the meta hash" do
+        @pi.meta = { 'TYPE' => ['float32', 'uint8'], 'TEST' => ["test string"] }
+        expect(@pi.meta['TYPE']).to eql ['float32', 'uint8']
+        expect(@pi.meta['TEST']).to eql ["test string"]
+        config = @pi.to_config(:TELEMETRY, :BIG_ENDIAN)
+        expect(config).to match(/META TYPE float32 uint8/)
+        expect(config).to match(/META TEST "test string"/)
+        @pi.meta = nil # Clear the meta hash
+        expect(@pi.meta.empty?).to be true # Clearing it results in empty hash
       end
     end