cosmos 4.2.4-java → 4.3.0-java

data/make_gems.sh

@@ -0,0 +1,10 @@
+# export VERSION= before running this
+git checkout master
+git pull
+rbenv global 2.4.4
+rake release VERSION=$VERSION
+rake commit_release VERSION=$VERSION
+mv *.gem ~/share/.
+rbenv global jruby-9.1.12.0
+gem build cosmos.gemspec
+mv *.gem ~/share/.

data/spec/conversions/packet_time_formatted_conversion_spec.rb

@@ -0,0 +1,58 @@
+# encoding: ascii-8bit
+
+# Copyright 2018 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/conversions/packet_time_formatted_conversion'
+require 'cosmos/packets/packet'
+
+module Cosmos
+
+  describe PacketTimeFormattedConversion do
+
+    describe "initialize" do
+      it "initializes converted_type and converted_bit_size" do
+        gc = PacketTimeFormattedConversion.new()
+        expect(gc.converted_type).to eql :STRING
+        expect(gc.converted_bit_size).to eql 0
+      end
+    end
+
+    describe "call" do
+      it "returns the formatted packet time" do
+        gc = PacketTimeFormattedConversion.new()
+        packet = Packet.new("TGT","PKT")
+        packet.received_time = Time.new(2020,1,31,12,15,30)
+        expect(gc.call(nil,packet,nil)).to eql "2020/01/31 12:15:30.000"
+      end
+
+      it "returns the formatted packet time from a packet item" do
+        gc = PacketTimeFormattedConversion.new()
+        packet = Packet.new("TGT","PKT")
+        item = packet.define_item("PACKET_TIME", 0, 0, :DERIVED)
+        item.read_conversion = GenericConversion.new("Time.new(2020,1,31,12,15,30)")
+        packet.received_time = nil
+        expect(gc.call(nil,packet,nil)).to eql "2020/01/31 12:15:30.000"
+      end
+
+      it "returns a string if packet time isn't set" do
+        gc = PacketTimeFormattedConversion.new()
+        packet = Packet.new("TGT","PKT")
+        expect(gc.call(nil,packet,nil)).to eql "No Packet Time"
+      end
+    end
+
+    describe "to_s" do
+      it "returns the class" do
+        expect(PacketTimeFormattedConversion.new().to_s).to eql "PacketTimeFormattedConversion"
+      end
+    end
+  end
+end
+

data/spec/conversions/packet_time_seconds_conversion_spec.rb

@@ -0,0 +1,60 @@
+# encoding: ascii-8bit
+
+# Copyright 2018 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/conversions/packet_time_seconds_conversion'
+require 'cosmos/packets/packet'
+
+module Cosmos
+
+  describe PacketTimeSecondsConversion do
+
+    describe "initialize" do
+      it "initializes converted_type and converted_bit_size" do
+        gc = PacketTimeSecondsConversion.new()
+        expect(gc.converted_type).to eql :FLOAT
+        expect(gc.converted_bit_size).to eql 64
+      end
+    end
+
+    describe "call" do
+      it "returns the formatted packet time" do
+        gc = PacketTimeSecondsConversion.new()
+        packet = Packet.new("TGT","PKT")
+        time = Time.new(2020,1,31,12,15,30)
+        packet.received_time = time
+        expect(gc.call(nil,packet,nil)).to eql time.to_f
+      end
+
+      it "returns the formatted packet time from a packet item" do
+        gc = PacketTimeSecondsConversion.new()
+        packet = Packet.new("TGT","PKT")
+        item = packet.define_item("PACKET_TIME", 0, 0, :DERIVED)
+        time = Time.new(2020,1,31,12,15,30)
+        item.read_conversion = GenericConversion.new("Time.new(2020,1,31,12,15,30)")
+        packet.received_time = nil
+        expect(gc.call(nil,packet,nil)).to eql time.to_f
+      end
+
+      it "returns 0.0 if packet time isn't set" do
+        gc = PacketTimeSecondsConversion.new()
+        packet = Packet.new("TGT","PKT")
+        expect(gc.call(nil,packet,nil)).to eql 0.0
+      end
+    end
+
+    describe "to_s" do
+      it "returns the class" do
+        expect(PacketTimeSecondsConversion.new().to_s).to eql "PacketTimeSecondsConversion"
+      end
+    end
+  end
+end
+

data/spec/interfaces/protocols/ignore_packet_protocol_spec.rb

@@ -0,0 +1,243 @@
+# 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/interfaces/protocols/ignore_packet_protocol'
+require 'cosmos/interfaces/interface'
+require 'cosmos/streams/stream'
+
+module Cosmos
+  describe IgnorePacketProtocol do
+    before(:each) do
+      $buffer = nil
+      @interface = StreamInterface.new
+      @interface.target_names = ['SYSTEM', 'INST']
+      allow(@interface).to receive(:connected?) { true }
+    end
+
+    class PreStream < Stream
+      def connect; end
+      def connected?; true; end
+      def disconnect; end
+      def read; $buffer; end
+      def write(data); $buffer = data; end
+    end
+
+    describe "initialize" do
+      it "complains if target is not given" do
+        expect { @interface.add_protocol(IgnorePacketProtocol, [], :READ_WRITE) }.to raise_error(ArgumentError)
+      end
+
+      it "complains if packet is not given" do
+        expect { @interface.add_protocol(IgnorePacketProtocol, ['SYSTEM'], :READ_WRITE) }.to raise_error(ArgumentError)
+      end
+
+      it "complains if the target is not found" do
+        expect { @interface.add_protocol(IgnorePacketProtocol, ['BLAH', 'META'], :READ_WRITE) }.to raise_error(/target 'BLAH' does not exist/)
+      end
+
+      it "complains if the target is not found" do
+        expect { @interface.add_protocol(IgnorePacketProtocol, ['SYSTEM', 'BLAH'], :READ_WRITE) }.to raise_error(/packet 'SYSTEM BLAH' does not exist/)
+      end
+    end
+
+    describe "read" do
+      it "ignores the packet specified" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        pkt = System.telemetry.packet("SYSTEM","META")
+        # Ensure the ID items are set so this packet can be identified
+        pkt.id_items.each do |item|
+          pkt.write_item(item, item.id_value)
+        end
+        pkt.received_time = Time.now
+        $buffer = nil
+        @interface.write(pkt)
+        # Verify the write went out
+        expect(pkt.buffer).to eql $buffer
+        # Verify we read the packet back
+        packet = @interface.read
+        expect(packet.buffer).to eql $buffer
+
+        # Now add the protocol to ignore the packet
+        @interface.add_protocol(IgnorePacketProtocol, ['SYSTEM', 'META'], :READ)
+        $buffer = nil
+        @interface.write(pkt)
+        packet = nil
+        # Try to read the interface
+        # We put this in a thread because it calls it continuously
+        thread = Thread.new do
+          packet = @interface.read
+        end
+        sleep 0.1
+        thread.kill
+        sleep 0.2 # Allow thread to die
+        expect(packet).to be_nil
+      end
+
+      it "can be added multiple times to ignore different packets" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+
+        pkt = System.telemetry.packet("INST","HEALTH_STATUS")
+        # Ensure the ID items are set so this packet can be identified
+        pkt.id_items.each do |item|
+          pkt.write_item(item, item.id_value)
+        end
+        pkt.received_time = Time.now
+        $buffer = nil
+        @interface.write(pkt)
+        expect($buffer).to eql pkt.buffer
+
+        # Verify we read the packet back
+        packet = @interface.read
+        expect(packet.buffer).to eql $buffer
+
+        # Now add the protocol to ignore the packet
+        @interface.add_protocol(IgnorePacketProtocol, ['INST', 'HEALTH_STATUS'], :READ)
+        $buffer = nil
+        @interface.write(pkt)
+        expect($buffer).to eql pkt.buffer
+        packet = nil
+        # Try to read the interface
+        # We put this in a thread because it calls it continuously
+        thread = Thread.new do
+          packet = @interface.read
+        end
+        sleep 0.1
+        thread.kill
+        sleep 0.2 # Allow thread to die
+        expect(packet).to be_nil
+
+        # Add another protocol to ignore another packet
+        @interface.add_protocol(IgnorePacketProtocol, ['INST', 'ADCS'], :READ)
+
+        pkt = System.telemetry.packet("INST","ADCS")
+        # Ensure the ID items are set so this packet can be identified
+        pkt.id_items.each do |item|
+          pkt.write_item(item, item.id_value)
+        end
+        pkt.received_time = Time.now
+        $buffer = nil
+        @interface.write(pkt)
+        expect($buffer).to eql pkt.buffer
+
+        packet = nil
+        # Try to read the interface
+        # We put this in a thread because it calls it continuously
+        thread = Thread.new do
+          packet = @interface.read
+        end
+        sleep 0.1
+        thread.kill
+        sleep 0.2 # Allow thread to die
+        expect(packet).to be_nil
+
+        pkt = System.telemetry.packet("INST","PARAMS")
+        # Ensure the ID items are set so this packet can be identified
+        pkt.id_items.each do |item|
+          pkt.write_item(item, item.id_value)
+        end
+        pkt.received_time = Time.now
+        $buffer = nil
+        @interface.write(pkt)
+        # Verify the write went out
+        expect($buffer).to eql pkt.buffer
+
+        packet = @interface.read
+        expect(packet.buffer).to eql pkt.buffer
+      end
+    end
+
+    describe "write" do
+      it "ignores the packet specified" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        @interface.add_protocol(IgnorePacketProtocol, ['SYSTEM', 'META'], :WRITE)
+        pkt = System.telemetry.packet("SYSTEM","META")
+        pkt.write("COSMOS_VERSION", "TEST")
+        pkt.received_time = Time.now
+        $buffer = nil
+        @interface.write(pkt)
+        # Verify the write was ignored
+        expect($buffer).to be_nil
+
+        # Verify reading the interface works
+        $buffer = pkt.buffer
+        packet = @interface.read
+        expect(packet.buffer).to eql $buffer
+      end
+
+      it "can be added multiple times to ignore different packets" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        @interface.add_protocol(IgnorePacketProtocol, ['INST', 'HEALTH_STATUS'], :WRITE)
+        @interface.add_protocol(IgnorePacketProtocol, ['INST', 'ADCS'], :WRITE)
+
+        pkt = System.telemetry.packet("INST","HEALTH_STATUS")
+        pkt.received_time = Time.now
+        $buffer = nil
+        @interface.write(pkt)
+        # Verify the write was ignored
+        expect($buffer).to be_nil
+
+        pkt = System.telemetry.packet("INST","ADCS")
+        pkt.received_time = Time.now
+        $buffer = nil
+        @interface.write(pkt)
+        # Verify the write was ignored
+        expect($buffer).to be_nil
+
+        pkt = System.telemetry.packet("INST","PARAMS")
+        pkt.received_time = Time.now
+        $buffer = nil
+        @interface.write(pkt)
+        # Verify the write went out
+        expect($buffer).to eql pkt.buffer
+      end
+    end
+
+    describe "read/write" do
+      it "ignores the packet specified" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        @interface.add_protocol(IgnorePacketProtocol, ['SYSTEM', 'META'], :READ_WRITE)
+        pkt = System.telemetry.packet("SYSTEM","META")
+        pkt.write("COSMOS_VERSION", "TEST")
+        pkt.received_time = Time.now
+        $buffer = nil
+        @interface.write(pkt)
+        # Verify the write was ignored
+        expect($buffer).to be_nil
+
+        packet = nil
+        # Try to read the interface
+        thread = Thread.new do
+          packet = @interface.read
+        end
+        sleep 0.1
+        thread.kill
+        sleep 0.2 # Allow thread to die
+        expect(packet).to be_nil
+      end
+
+      it "reads and writes unknown packets" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        @interface.add_protocol(IgnorePacketProtocol, ['SYSTEM', 'META'], :READ_WRITE)
+        $buffer = nil
+        pkt = Packet.new("TGT","PTK")
+        pkt.append_item("ITEM", 8, :INT)
+        pkt.write("ITEM", 33, :RAW)
+        @interface.write(pkt)
+        # Verify the write went out
+        expect(pkt.buffer).to eql $buffer
+
+        # Verify the read works
+        packet = @interface.read
+        expect(packet.buffer).to eq $buffer
+      end
+    end
+  end
+end

data/spec/interfaces/protocols/preidentified_protocol_spec.rb

@@ -56,6 +56,229 @@ module Cosmos
         time = Time.new(2020,1,31,12,15,30.5)
         pkt.received_time = time
         @interface.write(pkt)
+        expect($buffer[0..0].unpack('C')[0]).to eql 0
+        expect($buffer[1..4].unpack('N')[0]).to eql time.to_f.to_i
+        expect($buffer[5..8].unpack('N')[0]).to eql 500000
+        offset = 9
+        tgt_name_length = $buffer[offset].unpack('C')[0]
+        offset += 1 # for the length field
+        expect($buffer[offset...(offset+tgt_name_length)]).to eql 'SYSTEM'
+        offset += tgt_name_length
+        pkt_name_length = $buffer[offset].unpack('C')[0]
+        offset += 1 # for the length field
+        expect($buffer[offset...(offset+pkt_name_length)]).to eql 'META'
+        offset += pkt_name_length
+        expect($buffer[offset..(offset+3)].unpack('N')[0]).to eql pkt.buffer.length
+        offset += 4
+        expect($buffer[offset..-1]).to eql pkt.buffer
+      end
+
+      it "creates a packet header with stored" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        @interface.add_protocol(PreidentifiedProtocol, [nil, 5], :READ_WRITE)
+        pkt = System.telemetry.packet("SYSTEM","META").clone
+        time = Time.new(2020,1,31,12,15,30.5)
+        pkt.received_time = time
+        pkt.stored = true
+        @interface.write(pkt)
+        expect($buffer[0..0].unpack('C')[0]).to eql 0x80
+        expect($buffer[1..4].unpack('N')[0]).to eql time.to_f.to_i
+        expect($buffer[5..8].unpack('N')[0]).to eql 500000
+        offset = 9
+        tgt_name_length = $buffer[offset].unpack('C')[0]
+        offset += 1 # for the length field
+        expect($buffer[offset...(offset+tgt_name_length)]).to eql 'SYSTEM'
+        offset += tgt_name_length
+        pkt_name_length = $buffer[offset].unpack('C')[0]
+        offset += 1 # for the length field
+        expect($buffer[offset...(offset+pkt_name_length)]).to eql 'META'
+        offset += pkt_name_length
+        expect($buffer[offset..(offset+3)].unpack('N')[0]).to eql pkt.buffer.length
+        offset += 4
+        expect($buffer[offset..-1]).to eql pkt.buffer
+      end
+
+      it "creates a packet header with extra" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        @interface.add_protocol(PreidentifiedProtocol, [nil, 5], :READ_WRITE)
+        pkt = System.telemetry.packet("SYSTEM","META").clone
+        time = Time.new(2020,1,31,12,15,30.5)
+        pkt.received_time = time
+        pkt.stored = false
+        extra_data = {"vcid" => 2}
+        pkt.extra = extra_data
+        @interface.write(pkt)
+        offset = 0
+        expect($buffer[0..0].unpack('C')[0]).to eql 0x40
+        json_extra = extra_data.to_json
+        offset += 1
+        expect($buffer[offset..(offset+3)].unpack('N')[0]).to eql json_extra.length
+        offset += 4
+        expect($buffer[offset..(offset + json_extra.length - 1)]).to eql json_extra
+        offset += json_extra.length
+        expect($buffer[offset..(offset + 3)].unpack('N')[0]).to eql time.to_f.to_i
+        expect($buffer[(offset + 4)..(offset + 7)].unpack('N')[0]).to eql 500000
+        offset = offset += 8 # time fields
+        tgt_name_length = $buffer[offset].unpack('C')[0]
+        offset += 1 # for the length field
+        expect($buffer[offset...(offset+tgt_name_length)]).to eql 'SYSTEM'
+        offset += tgt_name_length
+        pkt_name_length = $buffer[offset].unpack('C')[0]
+        offset += 1 # for the length field
+        expect($buffer[offset...(offset+pkt_name_length)]).to eql 'META'
+        offset += pkt_name_length
+        expect($buffer[offset..(offset+3)].unpack('N')[0]).to eql pkt.buffer.length
+        offset += 4
+        expect($buffer[offset..-1]).to eql pkt.buffer
+      end
+
+      it "creates a packet header with stored and extra" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        @interface.add_protocol(PreidentifiedProtocol, [nil, 5], :READ_WRITE)
+        pkt = System.telemetry.packet("SYSTEM","META").clone
+        time = Time.new(2020,1,31,12,15,30.5)
+        pkt.received_time = time
+        pkt.stored = true
+        extra_data = {"vcid" => 2}
+        pkt.extra = extra_data
+        @interface.write(pkt)
+        offset = 0
+        expect($buffer[0..0].unpack('C')[0]).to eql 0xC0
+        json_extra = extra_data.to_json
+        offset += 1
+        expect($buffer[offset..(offset+3)].unpack('N')[0]).to eql json_extra.length
+        offset += 4
+        expect($buffer[offset..(offset + json_extra.length - 1)]).to eql json_extra
+        offset += json_extra.length
+        expect($buffer[offset..(offset + 3)].unpack('N')[0]).to eql time.to_f.to_i
+        expect($buffer[(offset + 4)..(offset + 7)].unpack('N')[0]).to eql 500000
+        offset = offset += 8 # time fields
+        tgt_name_length = $buffer[offset].unpack('C')[0]
+        offset += 1 # for the length field
+        expect($buffer[offset...(offset+tgt_name_length)]).to eql 'SYSTEM'
+        offset += tgt_name_length
+        pkt_name_length = $buffer[offset].unpack('C')[0]
+        offset += 1 # for the length field
+        expect($buffer[offset...(offset+pkt_name_length)]).to eql 'META'
+        offset += pkt_name_length
+        expect($buffer[offset..(offset+3)].unpack('N')[0]).to eql pkt.buffer.length
+        offset += 4
+        expect($buffer[offset..-1]).to eql pkt.buffer
+      end
+
+      it "handles a sync pattern" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        @interface.add_protocol(PreidentifiedProtocol, ["DEAD"], :READ_WRITE)
+        pkt = System.telemetry.packet("SYSTEM","META")
+        time = Time.new(2020,1,31,12,15,30.5)
+        pkt.received_time = time
+        @interface.write(pkt)
+        expect($buffer[0..1]).to eql("\xDE\xAD")
+        expect($buffer[2..2].unpack('C')[0]).to eql 0
+        expect($buffer[3..6].unpack('N')[0]).to eql time.to_f.to_i
+        expect($buffer[7..10].unpack('N')[0]).to eql 500000
+        offset = 11
+        tgt_name_length = $buffer[offset].unpack('C')[0]
+        offset += 1 # for the length field
+        expect($buffer[offset...(offset+tgt_name_length)]).to eql 'SYSTEM'
+        offset += tgt_name_length
+        pkt_name_length = $buffer[offset].unpack('C')[0]
+        offset += 1 # for the length field
+        expect($buffer[offset...(offset+pkt_name_length)]).to eql 'META'
+        offset += pkt_name_length
+        expect($buffer[offset..(offset+3)].unpack('N')[0]).to eql pkt.buffer.length
+        offset += 4
+        expect($buffer[offset..-1]).to eql pkt.buffer
+      end
+
+      it "handles a sync pattern with stored and extra" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        @interface.add_protocol(PreidentifiedProtocol, ["DEAD", 5], :READ_WRITE)
+        pkt = System.telemetry.packet("SYSTEM","META").clone
+        time = Time.new(2020,1,31,12,15,30.5)
+        pkt.received_time = time
+        pkt.stored = true
+        extra_data = {"vcid" => 2}
+        pkt.extra = extra_data
+        @interface.write(pkt)
+        expect($buffer[0..1]).to eql("\xDE\xAD")
+        offset = 2
+        expect($buffer[2..2].unpack('C')[0]).to eql 0xC0
+        json_extra = extra_data.to_json
+        offset += 1
+        expect($buffer[offset..(offset+3)].unpack('N')[0]).to eql json_extra.length
+        offset += 4
+        expect($buffer[offset..(offset + json_extra.length - 1)]).to eql json_extra
+        offset += json_extra.length
+        expect($buffer[offset..(offset + 3)].unpack('N')[0]).to eql time.to_f.to_i
+        expect($buffer[(offset + 4)..(offset + 7)].unpack('N')[0]).to eql 500000
+        offset = offset += 8 # time fields
+        tgt_name_length = $buffer[offset].unpack('C')[0]
+        offset += 1 # for the length field
+        expect($buffer[offset...(offset+tgt_name_length)]).to eql 'SYSTEM'
+        offset += tgt_name_length
+        pkt_name_length = $buffer[offset].unpack('C')[0]
+        offset += 1 # for the length field
+        expect($buffer[offset...(offset+pkt_name_length)]).to eql 'META'
+        offset += pkt_name_length
+        expect($buffer[offset..(offset+3)].unpack('N')[0]).to eql pkt.buffer.length
+        offset += 4
+        expect($buffer[offset..-1]).to eql pkt.buffer
+      end
+    end
+
+    describe "read" do
+      it "handles a sync pattern" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        @interface.add_protocol(PreidentifiedProtocol, ["0x1234"], :READ_WRITE)
+        pkt = System.telemetry.packet("SYSTEM","META")
+        pkt.write("COSMOS_VERSION", "TEST")
+        time = Time.new(2020,1,31,12,15,30.5)
+        pkt.received_time = time
+        @interface.write(pkt)
+        expect($buffer[0]).to eql "\x12"
+        expect($buffer[1]).to eql "\x34"
+        packet = @interface.read
+        expect(packet.target_name).to eql 'SYSTEM'
+        expect(packet.packet_name).to eql 'META'
+        expect(packet.identified?).to be true
+        expect(packet.defined?).to be false
+
+        pkt2 = System.telemetry.update!("SYSTEM","META",packet.buffer)
+        expect(pkt2.read('COSMOS_VERSION')).to eql 'TEST'
+        expect(pkt2.identified?).to be true
+        expect(pkt2.defined?).to be true
+      end
+
+      it "returns a packet" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        @interface.add_protocol(PreidentifiedProtocol, [], :READ_WRITE)
+        pkt = System.telemetry.packet("SYSTEM","META")
+        pkt.write("COSMOS_VERSION", "TEST")
+        time = Time.new(2020,1,31,12,15,30.5)
+        pkt.received_time = time
+        @interface.write(pkt)
+        packet = @interface.read
+        expect(packet.target_name).to eql 'SYSTEM'
+        expect(packet.packet_name).to eql 'META'
+        expect(packet.identified?).to be true
+        expect(packet.defined?).to be false
+
+        pkt2 = System.telemetry.update!("SYSTEM","META",packet.buffer)
+        expect(pkt2.read('COSMOS_VERSION')).to eql 'TEST'
+        expect(pkt2.identified?).to be true
+        expect(pkt2.defined?).to be true
+      end
+    end
+
+    describe "write in mode 2" do
+      it "creates a packet header" do
+        @interface.instance_variable_set(:@stream, PreStream.new)
+        @interface.add_protocol(PreidentifiedProtocol, [nil, 5, 2], :READ_WRITE)
+        pkt = System.telemetry.packet("SYSTEM","META")
+        time = Time.new(2020,1,31,12,15,30.5)
+        pkt.received_time = time
+        @interface.write(pkt)
         expect($buffer[0..3].unpack('N')[0]).to eql time.to_f.to_i
         expect($buffer[4..7].unpack('N')[0]).to eql 500000
         offset = 8
@@ -74,7 +297,7 @@ module Cosmos
 
       it "handles a sync pattern" do
         @interface.instance_variable_set(:@stream, PreStream.new)
-        @interface.add_protocol(PreidentifiedProtocol, ["DEAD"], :READ_WRITE)
+        @interface.add_protocol(PreidentifiedProtocol, ["DEAD", nil, 2], :READ_WRITE)
         pkt = System.telemetry.packet("SYSTEM","META")
         time = Time.new(2020,1,31,12,15,30.5)
         pkt.received_time = time
@@ -97,10 +320,10 @@ module Cosmos
       end
     end
 
-    describe "read" do
+    describe "read in mode 2" do
       it "handles a sync pattern" do
         @interface.instance_variable_set(:@stream, PreStream.new)
-        @interface.add_protocol(PreidentifiedProtocol, ["0x1234"], :READ_WRITE)
+        @interface.add_protocol(PreidentifiedProtocol, ["0x1234", nil, 2], :READ_WRITE)
         pkt = System.telemetry.packet("SYSTEM","META")
         pkt.write("COSMOS_VERSION", "TEST")
         time = Time.new(2020,1,31,12,15,30.5)
@@ -122,7 +345,7 @@ module Cosmos
 
       it "returns a packet" do
         @interface.instance_variable_set(:@stream, PreStream.new)
-        @interface.add_protocol(PreidentifiedProtocol, [], :READ_WRITE)
+        @interface.add_protocol(PreidentifiedProtocol, [nil, nil, 2], :READ_WRITE)
         pkt = System.telemetry.packet("SYSTEM","META")
         pkt.write("COSMOS_VERSION", "TEST")
         time = Time.new(2020,1,31,12,15,30.5)