openflow-protocol 0.1.6 → 0.1.7

data/spec/messages/features_request_spec.rb

@@ -1,13 +1,19 @@
 require 'spec_helper'
 
 describe OFFeaturesRequest do
+  let(:data) { [1, 5, 0, 8, 0, 0, 0, 1].pack('C*') }
+
   it 'should read binary' do
-    msg = OFFeaturesRequest.read [1, 5, 0, 8, 0, 0, 0, 1].pack('C*')
+    msg = OFFeaturesRequest.read(data)
     expect(msg.version).to eq(OFMessage::OFP_VERSION)
     expect(msg.type).to eq(:features_request)
     expect(msg.len).to eq(8)
     expect(msg.xid).to eq(1)
   end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFFeaturesRequest)
+  end
   it 'should initialize with default values' do
     msg = OFFeaturesRequest.new
     expect(msg.version).to eq(OFMessage::OFP_VERSION)

data/spec/messages/flow_mod_spec.rb

@@ -1,8 +1,8 @@
 require 'spec_helper'
 
 describe OFFlowMod do
-  it 'should read binary' do
-    msg = OFFlowMod.read [
+  let(:data) {
+    [
       1, 14, 0, 80, 0, 0, 0, 1, # header
 
       # match
@@ -37,6 +37,10 @@ describe OFFlowMod do
       0, 1,       # port
       0xff, 0xff, # max_length
     ].pack('C*')
+  }
+
+  it 'should read binary' do
+    msg = OFFlowMod.read(data)
     expect(msg.version).to eq(OFMessage::OFP_VERSION)
     expect(msg.type).to eq(:flow_mod)
     expect(msg.len).to eq(80)
@@ -53,6 +57,10 @@ describe OFFlowMod do
     expect(msg.actions.length).to eq(1)
     expect(msg.actions.first.type).to eq(:output)
   end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFFlowMod)
+  end
   it 'should initialize with default values' do
     msg = OFFlowMod.new
     expect(msg.version).to eq(OFMessage::OFP_VERSION)

data/spec/messages/flow_removed_spec.rb

@@ -1,8 +1,8 @@
 require 'spec_helper'
 
 describe OFFlowRemoved do
-  it 'should read binary' do
-    msg = OFFlowRemoved.read [
+  let(:data) {
+    [
       1, 11, 0, 88, 0, 0, 0, 1, # header
 
       # match
@@ -33,6 +33,10 @@ describe OFFlowRemoved do
       0, 0, 0, 0, 0, 0, 0, 10, # packet_count
       0, 0, 0, 0, 0, 0, 0, 80  # byte_count
     ].pack('C*')
+  }
+
+  it 'should read binary' do
+    msg = OFFlowRemoved.read(data)
     expect(msg.version).to eq(OFMessage::OFP_VERSION)
     expect(msg.type).to eq(:flow_removed)
     expect(msg.len).to eq(88)
@@ -47,6 +51,10 @@ describe OFFlowRemoved do
     expect(msg.packet_count).to eq(10)
     expect(msg.byte_count).to eq(80)
   end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFFlowRemoved)
+  end
   it 'should initialize with default values' do
     msg = OFFlowRemoved.new
     expect(msg.version).to eq(OFMessage::OFP_VERSION)

data/spec/messages/get_config_reply_spec.rb

@@ -1,12 +1,16 @@
 require 'spec_helper'
 
 describe OFGetConfigReply do
-  it 'should read binary' do
-    msg = OFGetConfigReply.read [
+  let(:data) {
+    [
       1, 8, 0, 12, 0, 0, 0, 1, # header
       0, 0,                    # flags
       0, 0xff                  # miss_send_length
     ].pack('C*')
+  }
+
+  it 'should read binary' do
+    msg = OFGetConfigReply.read(data)
     expect(msg.version).to eq(OFMessage::OFP_VERSION)
     expect(msg.type).to eq(:get_config_reply)
     expect(msg.len).to eq(12)
@@ -14,6 +18,10 @@ describe OFGetConfigReply do
     expect(msg.flags).to eq(:fragments_normal)
     expect(msg.miss_send_length).to eq(0xff)
   end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFGetConfigReply)
+  end
   it 'should initialize with default values' do
     msg = OFGetConfigReply.new
     expect(msg.version).to eq(OFMessage::OFP_VERSION)

data/spec/messages/get_config_request_spec.rb

@@ -1,13 +1,19 @@
 require 'spec_helper'
 
 describe OFGetConfigRequest do
+  let(:data) { [1, 7, 0, 8, 0, 0, 0, 1].pack('C*') }
+
   it 'should read binary' do
-    msg = OFGetConfigRequest.read [1, 7, 0, 8, 0, 0, 0, 1].pack('C*')
+    msg = OFGetConfigRequest.read(data)
     expect(msg.version).to eq(OFMessage::OFP_VERSION)
     expect(msg.type).to eq(:get_config_request)
     expect(msg.len).to eq(8)
     expect(msg.xid).to eq(1)
   end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFGetConfigRequest)
+  end
   it 'should initialize with default values' do
     msg = OFGetConfigRequest.new
     expect(msg.version).to eq(OFMessage::OFP_VERSION)

data/spec/messages/hello_spec.rb

@@ -1,13 +1,19 @@
 require 'spec_helper'
 
 describe OFHello do
+  let(:data) { [1, 0, 0, 8, 0, 0, 0, 1].pack('C*') }
+
   it 'should read binary' do
-    msg = OFHello.read [1, 0, 0, 8, 0, 0, 0, 1].pack('C*')
+    msg = OFHello.read(data)
     expect(msg.version).to eq(OFMessage::OFP_VERSION)
     expect(msg.type).to eq(:hello)
     expect(msg.len).to eq(8)
     expect(msg.xid).to eq(1)
   end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFHello)
+  end
   it 'should initialize with default values' do
     msg = OFHello.new
     expect(msg.version).to eq(OFMessage::OFP_VERSION)

data/spec/messages/packet_in_spec.rb

@@ -1,8 +1,8 @@
 require 'spec_helper'
 
 describe OFPacketIn do
-  it 'should read binary' do
-    msg = OFPacketIn.read [
+  let(:data) {
+    [
       1, 10, 0, 64, 0, 0, 0, 1, # header
       0, 0, 0, 1,               # buffer_id
       0, 46,                    # total_length
@@ -31,6 +31,10 @@ describe OFPacketIn do
       0, 0, 0, 0, 0, 2, # mac_destination
       192, 168, 0, 2    # ip_destination
     ].pack('C*')
+  }
+
+  it 'should read binary' do
+    msg = OFPacketIn.read(data)
     expect(msg.version).to eq(OFMessage::OFP_VERSION)
     expect(msg.type).to eq(:packet_in)
     expect(msg.len).to eq(64)
@@ -43,6 +47,10 @@ describe OFPacketIn do
     expect(msg.parsed_data.length).to eq(46)
     expect(msg.parsed_data.mac_destination).to eq('00:00:00:00:00:02')
   end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFPacketIn)
+  end
   it 'should initialize with default values' do
     msg = OFPacketIn.new
     expect(msg.version).to eq(OFMessage::OFP_VERSION)

data/spec/messages/packet_out_spec.rb

@@ -1,8 +1,8 @@
 require 'spec_helper'
 
 describe OFPacketOut do
-  it 'should read binary' do
-    msg = OFPacketOut.read [
+  let(:data) {
+    [
       1, 13, 0, 26, 0, 0, 0, 1, # header
       0xff, 0xff, 0xff, 0xff,   # buffer_id
       0, 1,                     # in_port
@@ -15,6 +15,10 @@ describe OFPacketOut do
       # data
       1, 2
     ].pack('C*')
+  }
+
+  it 'should read binary' do
+    msg = OFPacketOut.read(data)
     expect(msg.version).to eq(OFMessage::OFP_VERSION)
     expect(msg.type).to eq(:packet_out)
     expect(msg.len).to eq(26)
@@ -25,6 +29,10 @@ describe OFPacketOut do
     expect(msg.actions.first.type).to eq(:output)
     expect(msg.data.length).to eq(2)
   end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFPacketOut)
+  end
   it 'should initialize with default values' do
     msg = OFPacketOut.new
     expect(msg.version).to eq(OFMessage::OFP_VERSION)

data/spec/messages/port_mod_spec.rb

@@ -1,8 +1,8 @@
 require 'spec_helper'
 
 describe OFPortMod do
-  it 'should read binary' do
-    msg = OFPortMod.read [
+  let(:data) {
+    [
       1, 15, 0, 32, 0, 0, 0, 1, # header
       0, 1,                     # port_number
       0, 0, 0, 0, 0, 1,         # hardware_address
@@ -11,6 +11,10 @@ describe OFPortMod do
       0, 0, 0, 3,               # advertise
       0, 0, 0, 0                # padding
     ].pack('C*')
+  }
+
+  it 'should read binary' do
+    msg = OFPortMod.read(data)
     expect(msg.version).to eq(OFMessage::OFP_VERSION)
     expect(msg.type).to eq(:port_mod)
     expect(msg.len).to eq(32)
@@ -24,6 +28,10 @@ describe OFPortMod do
       :port_10mb_full_duplex
     ])
   end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFPortMod)
+  end
   it 'should initialize with default values' do
     msg = OFPortMod.new
     expect(msg.version).to eq(OFMessage::OFP_VERSION)

data/spec/messages/port_status_spec.rb

@@ -1,8 +1,8 @@
 require 'spec_helper'
 
 describe OFPortStatus do
-  it 'should read binary' do
-    msg = OFPortStatus.read [
+  let(:data) {
+    [
       1, 12, 0, 64, 0, 0, 0, 1, # header
       1,                        # reason
       0, 0, 0, 0, 0, 0, 0,      # padding
@@ -20,6 +20,10 @@ describe OFPortStatus do
       0, 0, 0x0f, 0xff,   # supported_features
       0, 0, 0x0f, 0xff    # peer_features
     ].pack('C*')
+  }
+
+  it 'should read binary' do
+    msg = OFPortStatus.read(data)
     expect(msg.version).to eq(OFMessage::OFP_VERSION)
     expect(msg.type).to eq(:port_status)
     expect(msg.len).to eq(64)
@@ -27,6 +31,10 @@ describe OFPortStatus do
     expect(msg.reason).to eq(:delete)
     expect(msg.desc.port_number).to eq(1)
   end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFPortStatus)
+  end
   it 'should initialize with default values' do
     msg = OFPortStatus.new
     expect(msg.version).to eq(OFMessage::OFP_VERSION)

data/spec/messages/queue_get_config_reply_spec.rb

@@ -0,0 +1,68 @@
+require 'spec_helper'
+
+describe OFQueueGetConfigReply do
+  let(:data) {
+    [
+      1, 21, 0, 48, 0, 0, 0, 1, # header
+      0, 1,                     # port
+      0, 0, 0, 0, 0, 0,         # padding
+      # queue 1
+      0, 0, 0, 1, # queue_id
+      0, 24,      # len
+      0, 0,       # padding
+      #   queue_properties
+      #     queue_property_min_rate
+      0, 1, 0, 16, 0, 0, 0, 0, # header
+      0, 100,                  # rate
+      0, 0, 0, 0, 0, 0,        # padding
+      # queue 2
+      0, 0, 0, 2, # queue_id
+      0, 8,       # len
+      0, 0        # padding
+    ].pack('C*')
+  }
+
+  it 'should read binary' do
+    msg = OFQueueGetConfigReply.read(data)
+    expect(msg.version).to eq(OFMessage::OFP_VERSION)
+    expect(msg.type).to eq(:queue_get_config_reply)
+    expect(msg.len).to eq(48)
+    expect(msg.xid).to eq(1)
+    expect(msg.port).to eq(1)
+    expect(msg.queues.length).to eq(2)
+    expect(msg.queues.first.queue_id).to eq(1)
+    expect(msg.queues.first.properties.length).to eq(1)
+    expect(msg.queues.last.queue_id).to eq(2)
+  end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFQueueGetConfigReply)
+  end
+  it 'should initialize with default values' do
+    msg = OFQueueGetConfigReply.new
+    expect(msg.version).to eq(OFMessage::OFP_VERSION)
+    expect(msg.type).to eq(:queue_get_config_reply)
+    expect(msg.len).to eq(16)
+    expect(msg.xid).to eq(0)
+    expect(msg.port).to eq(0)
+    expect(msg.queues).to be_empty
+  end
+  it 'should initialize with some values' do
+    msg = OFQueueGetConfigReply.new(
+      xid: 1,
+      port: 2,
+      queues: [
+        { queue_id: 1 },
+        { queue_id: 2 },
+      ]
+    )
+    expect(msg.version).to eq(OFMessage::OFP_VERSION)
+    expect(msg.type).to eq(:queue_get_config_reply)
+    expect(msg.len).to eq(32)
+    expect(msg.xid).to eq(1)
+    expect(msg.port).to eq(2)
+    expect(msg.queues.length).to eq(2)
+    expect(msg.queues.first.queue_id).to eq(1)
+    expect(msg.queues.last.queue_id).to eq(2)
+  end
+end

data/spec/messages/queue_get_config_request_spec.rb

@@ -0,0 +1,40 @@
+require 'spec_helper'
+
+describe OFQueueGetConfigRequest do
+  let(:data) {
+    [
+      1, 20, 0, 12, 0, 0, 0, 1, # header
+      0, 1,                     # port
+      0, 0                      # padding
+    ].pack('C*')
+  }
+
+  it 'should read binary' do
+    msg = OFQueueGetConfigRequest.read(data)
+    expect(msg.version).to eq(OFMessage::OFP_VERSION)
+    expect(msg.type).to eq(:queue_get_config_request)
+    expect(msg.len).to eq(12)
+    expect(msg.xid).to eq(1)
+    expect(msg.port).to eq(1)
+  end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFQueueGetConfigRequest)
+  end
+  it 'should initialize with default values' do
+    msg = OFQueueGetConfigRequest.new
+    expect(msg.version).to eq(OFMessage::OFP_VERSION)
+    expect(msg.type).to eq(:queue_get_config_request)
+    expect(msg.len).to eq(12)
+    expect(msg.xid).to eq(0)
+    expect(msg.port).to eq(0)
+  end
+  it 'should initialize with some values' do
+    msg = OFQueueGetConfigRequest.new(xid: 1, port: 2)
+    expect(msg.version).to eq(OFMessage::OFP_VERSION)
+    expect(msg.type).to eq(:queue_get_config_request)
+    expect(msg.len).to eq(12)
+    expect(msg.xid).to eq(1)
+    expect(msg.port).to eq(2)
+  end
+end

data/spec/messages/set_config_spec.rb

@@ -1,12 +1,16 @@
 require 'spec_helper'
 
 describe OFSetConfig do
-  it 'should read binary' do
-    msg = OFSetConfig.read [
+  let(:data) {
+    [
       1, 9, 0, 12, 0, 0, 0, 1, # header
       0, 0,                    # flags
       0, 0xff                  # miss_send_length
     ].pack('C*')
+  }
+
+  it 'should read binary' do
+    msg = OFSetConfig.read(data)
     expect(msg.version).to eq(OFMessage::OFP_VERSION)
     expect(msg.type).to eq(:set_config)
     expect(msg.len).to eq(12)
@@ -14,6 +18,10 @@ describe OFSetConfig do
     expect(msg.flags).to eq(:fragments_normal)
     expect(msg.miss_send_length).to eq(0xff)
   end
+  it 'should be parsable' do
+    msg = OFParser.read(data)
+    expect(msg.class).to eq(OFSetConfig)
+  end
   it 'should initialize with default values' do
     msg = OFSetConfig.new
     expect(msg.version).to eq(OFMessage::OFP_VERSION)