pio 0.9.0 → 0.10.0

data/spec/pio/echo/reply_spec.rb

@@ -29,10 +29,7 @@ describe Pio::Echo::Reply do
     context 'with 2**32' do
       When(:result) { Pio::Echo::Reply.new(2**32) }
 
-      Then do
-        pending 'check if xid is within 32bit range.'
-        result == Failure(ArgumentError)
-      end
+      Then { result == Failure(ArgumentError) }
     end
 
     context 'with transaction_id: 123' do

data/spec/pio/echo/request_spec.rb

@@ -10,7 +10,7 @@ describe Pio::Echo::Request do
       Then { echo_request.message_length == 8 }
       Then { echo_request.transaction_id == 0 }
       Then { echo_request.xid == 0 }
-      Then { echo_request.user_data == '' }
+      Then { echo_request.user_data.empty? }
       Then { echo_request.to_binary == [1, 2, 0, 8, 0, 0, 0, 0].pack('C*') }
     end
 
@@ -22,17 +22,14 @@ describe Pio::Echo::Request do
       Then { echo_request.message_length == 8 }
       Then { echo_request.transaction_id == 123 }
       Then { echo_request.xid == 123 }
-      Then { echo_request.user_data == '' }
+      Then { echo_request.user_data.empty? }
       Then { echo_request.to_binary == [1, 2, 0, 8, 0, 0, 0, 123].pack('C*') }
     end
 
     context 'with 2**32' do
       When(:result) { Pio::Echo::Request.new(2**32) }
 
-      Then do
-        pending 'check if xid is within 32bit range.'
-        result == Failure(ArgumentError)
-      end
+      Then { result == Failure(ArgumentError) }
     end
 
     context 'with transaction_id: 123' do
@@ -43,7 +40,7 @@ describe Pio::Echo::Request do
       Then { echo_request.message_length == 8 }
       Then { echo_request.transaction_id == 123 }
       Then { echo_request.xid == 123 }
-      Then { echo_request.user_data == '' }
+      Then { echo_request.user_data.empty? }
       Then { echo_request.to_binary == [1, 2, 0, 8, 0, 0, 0, 123].pack('C*') }
     end
 
@@ -55,7 +52,7 @@ describe Pio::Echo::Request do
       Then { echo_request.message_length == 8 }
       Then { echo_request.transaction_id == 123 }
       Then { echo_request.xid == 123 }
-      Then { echo_request.user_data == '' }
+      Then { echo_request.user_data.empty? }
       Then { echo_request.to_binary == [1, 2, 0, 8, 0, 0, 0, 123].pack('C*') }
     end
 

data/spec/pio/enqueue_spec.rb

@@ -0,0 +1,67 @@
+require 'pio/enqueue'
+
+describe Pio::Enqueue do
+  describe '.new' do
+    context 'with port_number: 1, queue_id: 2' do
+      When(:enqueue) { Pio::Enqueue.new(port_number: 1, queue_id: 2) }
+
+      describe '#port_number' do
+        Then { enqueue.port_number == 1 }
+      end
+
+      describe '#queue_id' do
+        Then { enqueue.queue_id == 2 }
+      end
+
+      describe '#type' do
+        Then { enqueue.type == 11 }
+      end
+
+      describe '#message_length' do
+        Then { enqueue.message_length == 16 }
+      end
+
+      describe '#to_binary' do
+        Then { enqueue.to_binary.length == 16 }
+      end
+    end
+
+    context 'with port_number: :in_port, queue_id: 2' do
+      When(:enqueue) { Pio::Enqueue.new(port_number: :in_port, queue_id: 2) }
+
+      describe '#port_number' do
+        Then { enqueue.port_number == :in_port }
+      end
+    end
+
+    context 'with port_number: :local, queue_id: 2' do
+      When(:enqueue) { Pio::Enqueue.new(port_number: :local, queue_id: 2) }
+      Then { enqueue == Failure(ArgumentError) }
+    end
+
+    context 'with port_number: -1, queue_id: 2' do
+      When(:enqueue) { Pio::Enqueue.new(port_number: -1, queue_id: 2) }
+      Then { enqueue == Failure(ArgumentError) }
+    end
+
+    context 'with port_number: 0xff00, queue_id: 2' do
+      When(:enqueue) { Pio::Enqueue.new(port_number: 0xff00, queue_id: 2) }
+      Then { enqueue == Failure(ArgumentError) }
+    end
+
+    context 'with port_number: 1, queue_id: -2' do
+      When(:enqueue) { Pio::Enqueue.new(port_number: 1, queue_id: -2) }
+      Then { enqueue == Failure(ArgumentError) }
+    end
+
+    context 'with port_number: 1' do
+      When(:enqueue) { Pio::Enqueue.new(port_number: 1) }
+      Then { enqueue == Failure(ArgumentError) }
+    end
+
+    context 'with queue_id: 2' do
+      When(:enqueue) { Pio::Enqueue.new(queue_id: 2) }
+      Then { enqueue == Failure(ArgumentError) }
+    end
+  end
+end

data/spec/pio/features/request_spec.rb

@@ -10,7 +10,7 @@ describe Pio::Features::Request do
       Then { request.message_length == 8 }
       Then { request.transaction_id == 0 }
       Then { request.xid == 0 }
-      Then { request.body == '' }
+      Then { request.body.empty? }
       Then { request.to_binary == [1, 5, 0, 8, 0, 0, 0, 0].pack('C*') }
     end
 
@@ -22,7 +22,7 @@ describe Pio::Features::Request do
       Then { request.message_length == 8 }
       Then { request.transaction_id == 123 }
       Then { request.xid == 123 }
-      Then { request.body == '' }
+      Then { request.body.empty? }
       Then { request.to_binary == [1, 5, 0, 8, 0, 0, 0, 123].pack('C*') }
     end
 
@@ -40,7 +40,7 @@ describe Pio::Features::Request do
       Then { request.message_length == 8 }
       Then { request.transaction_id == 123 }
       Then { request.xid == 123 }
-      Then { request.body == '' }
+      Then { request.body.empty? }
       Then { request.to_binary == [1, 5, 0, 8, 0, 0, 0, 123].pack('C*') }
     end
 
@@ -52,7 +52,7 @@ describe Pio::Features::Request do
       Then { request.message_length == 8 }
       Then { request.transaction_id == 123 }
       Then { request.xid == 123 }
-      Then { request.body == '' }
+      Then { request.body.empty? }
       Then { request.to_binary == [1, 5, 0, 8, 0, 0, 0, 123].pack('C*') }
     end
 

data/spec/pio/features_spec.rb

@@ -89,8 +89,7 @@ describe Pio::Features do
       When(:result) { Pio::Features.read(hello_dump) }
 
       Then do
-        result == Failure(Pio::ParseError,
-                          'Invalid features request/reply message.')
+        result == Failure(Pio::ParseError, 'Invalid OpenFlow message.')
       end
     end
   end

data/spec/pio/hello_spec.rb

@@ -53,10 +53,7 @@ describe Pio::Hello do
     context 'with 2**32' do
       When(:result) { Pio::Hello.new(2**32) }
 
-      Then do
-        pending 'check if xid is within 32bit range.'
-        result == Failure(ArgumentError)
-      end
+      Then { result == Failure(ArgumentError) }
     end
 
     context 'with transaction_id: 123' do

data/spec/pio/packet_out_spec.rb

@@ -0,0 +1,290 @@
+require 'pio/packet_out'
+
+describe Pio::PacketOut do
+  Given(:header_dump) do
+    [
+      0x01,
+      0x0d,
+      0x00, 0x58,
+      0x00, 0x00, 0x00, 0x16
+    ].pack('C*')
+  end
+
+  Given(:data_dump) do
+    [
+      0x01, 0x80, 0xc2, 0x00, 0x00, 0x0e, 0x01, 0x02, 0x03, 0x04,
+      0x05, 0x06, 0x88, 0xcc, 0x02, 0x09, 0x07, 0x00, 0x00, 0x00,
+      0x00, 0x00, 0x00, 0x01, 0x23, 0x04, 0x05, 0x07, 0x00, 0x00,
+      0x00, 0x0c, 0x06, 0x02, 0x00, 0x78, 0x00, 0x00, 0x00, 0x00,
+      0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+      0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+      0x00, 0x00, 0x00, 0x00
+    ].pack('C*')
+  end
+
+  Given(:body_dump) do
+    [
+      0xff, 0xff, 0xff, 0xff,
+      0xff, 0xff,
+      0x00, 0x08,
+      0x00, 0x00, 0x00, 0x08, 0x00, 0x02, 0xff, 0xff
+    ].pack('C*') + data_dump
+  end
+
+  Given(:packet_out_dump) { header_dump + body_dump }
+
+  describe '.read' do
+    context 'with a Packet-Out message' do
+      When(:packet_out) do
+        Pio::PacketOut.read(packet_out_dump)
+      end
+
+      Then { packet_out.class == Pio::PacketOut }
+      Then { packet_out.ofp_version == 0x1 }
+      Then { packet_out.message_type == 0xd }
+      Then { packet_out.message_length == 0x58 }
+      Then { packet_out.transaction_id == 0x16 }
+      Then { packet_out.xid == 0x16 }
+
+      Then { !packet_out.body.empty? }
+      Then { packet_out.buffer_id == 0xffffffff }
+      Then { packet_out.in_port == 0xffff }
+      Then { packet_out.actions_len == 0x8 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::SendOutPort }
+      Then { packet_out.actions[0].port_number == 2 }
+      Then { packet_out.actions[0].max_len == 2**16 - 1 }
+      Then { packet_out.data.length == 64 }
+    end
+  end
+
+  describe '.new' do
+    context 'with a SendOutPort action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: Pio::SendOutPort.new(2),
+                           data: data_dump)
+      end
+
+      Then { packet_out.ofp_version == 0x1 }
+      Then { packet_out.message_type == 0xd }
+      Then { packet_out.message_length == 0x58 }
+      Then { packet_out.transaction_id == 0x16 }
+      Then { packet_out.xid == 0x16 }
+
+      Then { !packet_out.body.empty? }
+      Then { packet_out.buffer_id == 0xffffffff }
+      Then { packet_out.in_port == 0xffff }
+      Then { packet_out.actions_len == 0x8 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::SendOutPort }
+      Then { packet_out.actions[0].port_number == 2 }
+      Then { packet_out.actions[0].max_len == 2**16 - 1 }
+      Then { packet_out.data.length == 64 }
+
+      context '#to_binary' do
+        When(:binary) { packet_out.to_binary }
+        Then { binary == packet_out_dump }
+      end
+    end
+
+    context 'with a SetVlanVid action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: Pio::SetVlanVid.new(10),
+                           data: data_dump)
+      end
+
+      Then { packet_out.message_length == 0x58 }
+      Then { packet_out.actions_len == 0x8 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::SetVlanVid }
+      Then { packet_out.actions[0].vlan_id == 10 }
+    end
+
+    context 'with a SetVlanPriority action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: Pio::SetVlanPriority.new(3),
+                           data: data_dump)
+      end
+
+      Then { packet_out.message_length == 0x58 }
+      Then { packet_out.actions_len == 0x8 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::SetVlanPriority }
+      Then { packet_out.actions[0].vlan_priority == 3 }
+    end
+
+    context 'with a StripVlanHeader action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: Pio::StripVlanHeader.new,
+                           data: data_dump)
+      end
+
+      Then { packet_out.message_length == 0x58 }
+      Then { packet_out.actions_len == 0x8 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::StripVlanHeader }
+    end
+
+    context 'with a SetEthSrcAddr action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: Pio::SetEthSrcAddr.new('11:22:33:44:55:66'),
+                           data: data_dump)
+      end
+
+      Then { packet_out.message_length == 0x60 }
+      Then { packet_out.actions_len == 0x10 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::SetEthSrcAddr }
+      Then { packet_out.actions[0].mac_address == '11:22:33:44:55:66' }
+    end
+
+    context 'with a SetEthDstAddr action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: Pio::SetEthDstAddr.new('11:22:33:44:55:66'),
+                           data: data_dump)
+      end
+
+      Then { packet_out.message_length == 0x60 }
+      Then { packet_out.actions_len == 0x10 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::SetEthDstAddr }
+      Then { packet_out.actions[0].mac_address == '11:22:33:44:55:66' }
+    end
+
+    context 'with a SetIpSrcAddr action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: Pio::SetIpSrcAddr.new('1.2.3.4'),
+                           data: data_dump)
+      end
+
+      Then { packet_out.message_length == 0x58 }
+      Then { packet_out.actions_len == 0x8 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::SetIpSrcAddr }
+      Then { packet_out.actions[0].ip_address == '1.2.3.4' }
+    end
+
+    context 'with a SetIpDstAddr action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: Pio::SetIpDstAddr.new('1.2.3.4'),
+                           data: data_dump)
+      end
+
+      Then { packet_out.message_length == 0x58 }
+      Then { packet_out.actions_len == 0x8 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::SetIpDstAddr }
+      Then { packet_out.actions[0].ip_address == '1.2.3.4' }
+    end
+
+    context 'with a SetIpTos action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: Pio::SetIpTos.new(32),
+                           data: data_dump)
+      end
+
+      Then { packet_out.message_length == 0x58 }
+      Then { packet_out.actions_len == 0x8 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::SetIpTos }
+      Then { packet_out.actions[0].type_of_service == 32 }
+    end
+
+    context 'with a SetTransportSrcPort action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: Pio::SetTransportSrcPort.new(100),
+                           data: data_dump)
+      end
+
+      Then { packet_out.message_length == 0x58 }
+      Then { packet_out.actions_len == 0x8 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::SetTransportSrcPort }
+      Then { packet_out.actions[0].port_number == 100 }
+    end
+
+    context 'with a SetTransportDstPort action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: Pio::SetTransportDstPort.new(100),
+                           data: data_dump)
+      end
+
+      Then { packet_out.message_length == 0x58 }
+      Then { packet_out.actions_len == 0x8 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::SetTransportDstPort }
+      Then { packet_out.actions[0].port_number == 100 }
+    end
+
+    context 'with a Enqueue action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: Pio::Enqueue.new(port_number: 1,
+                                                     queue_id: 2),
+                           data: data_dump)
+      end
+
+      Then { packet_out.message_length == 0x60 }
+      Then { packet_out.actions_len == 0x10 }
+      Then { packet_out.actions.length == 1 }
+      Then { packet_out.actions[0].is_a? Pio::Enqueue }
+      Then { packet_out.actions[0].port_number == 1 }
+      Then { packet_out.actions[0].queue_id == 2 }
+    end
+
+    context 'with SendOutPort and SetVlanVid action' do
+      Given(:packet_out) do
+        Pio::PacketOut.new(transaction_id: 0x16,
+                           buffer_id: 0xffffffff,
+                           in_port: 0xffff,
+                           actions: [Pio::SendOutPort.new(2),
+                                     Pio::SetVlanVid.new(10)],
+                           data: data_dump)
+      end
+
+      Then { packet_out.message_length == 0x60 }
+      Then { packet_out.actions_len == 0x10 }
+      Then { packet_out.actions.length == 2 }
+      Then { packet_out.actions[0].is_a? Pio::SendOutPort }
+      Then { packet_out.actions[0].port_number == 2 }
+      Then { packet_out.actions[0].max_len == 2**16 - 1 }
+      Then { packet_out.actions[1].is_a? Pio::SetVlanVid }
+      Then { packet_out.actions[1].vlan_id == 10 }
+    end
+  end
+end