pio 0.27.2 → 0.28.0

data/spec/pio/open_flow10/packet_out_spec.rb

@@ -1,6 +1,7 @@
 require 'pio/open_flow10/packet_out'
 require 'pio/parse_error'
 
+# rubocop:disable LineLength
 describe Pio::OpenFlow10::PacketOut do
   Given(:header_dump) do
     [
@@ -137,7 +138,7 @@ describe Pio::OpenFlow10::PacketOut do
           transaction_id: 0x16,
           buffer_id: 0xffffffff,
           in_port: 0xffff,
-          actions: Pio::SetVlanVid.new(10),
+          actions: Pio::OpenFlow10::SetVlanVid.new(10),
           raw_data: data_dump
         }
       end
@@ -145,7 +146,7 @@ describe Pio::OpenFlow10::PacketOut do
       Then { result.message_length == 0x58 }
       Then { result.actions_len == 0x8 }
       Then { result.actions.length == 1 }
-      Then { result.actions[0].is_a? Pio::SetVlanVid }
+      Then { result.actions[0].is_a? Pio::OpenFlow10::SetVlanVid }
       Then { result.actions[0].vlan_id == 10 }
     end
 
@@ -155,7 +156,7 @@ describe Pio::OpenFlow10::PacketOut do
           transaction_id: 0x16,
           buffer_id: 0xffffffff,
           in_port: 0xffff,
-          actions: Pio::SetVlanPriority.new(3),
+          actions: Pio::OpenFlow10::SetVlanPriority.new(3),
           raw_data: data_dump
         }
       end
@@ -163,7 +164,7 @@ describe Pio::OpenFlow10::PacketOut do
       Then { result.message_length == 0x58 }
       Then { result.actions_len == 0x8 }
       Then { result.actions.length == 1 }
-      Then { result.actions[0].is_a? Pio::SetVlanPriority }
+      Then { result.actions[0].is_a? Pio::OpenFlow10::SetVlanPriority }
       Then { result.actions[0].vlan_priority == 3 }
     end
 
@@ -173,7 +174,7 @@ describe Pio::OpenFlow10::PacketOut do
           transaction_id: 0x16,
           buffer_id: 0xffffffff,
           in_port: 0xffff,
-          actions: Pio::StripVlanHeader.new,
+          actions: Pio::OpenFlow10::StripVlanHeader.new,
           raw_data: data_dump
         }
       end
@@ -181,16 +182,16 @@ describe Pio::OpenFlow10::PacketOut do
       Then { result.message_length == 0x58 }
       Then { result.actions_len == 0x8 }
       Then { result.actions.length == 1 }
-      Then { result.actions[0].is_a? Pio::StripVlanHeader }
+      Then { result.actions[0].is_a? Pio::OpenFlow10::StripVlanHeader }
     end
 
-    context 'with a SetEtherSourceAddress action' do
+    context 'with a SetSourceMacAddress action' do
       When(:user_options) do
         {
           transaction_id: 0x16,
           buffer_id: 0xffffffff,
           in_port: 0xffff,
-          actions: Pio::SetEtherSourceAddress.new('11:22:33:44:55:66'),
+          actions: Pio::OpenFlow10::SetSourceMacAddress.new('11:22:33:44:55:66'),
           raw_data: data_dump
         }
       end
@@ -198,17 +199,17 @@ describe Pio::OpenFlow10::PacketOut do
       Then { result.message_length == 0x60 }
       Then { result.actions_len == 0x10 }
       Then { result.actions.length == 1 }
-      Then { result.actions[0].is_a? Pio::SetEtherSourceAddress }
+      Then { result.actions[0].is_a? Pio::OpenFlow10::SetSourceMacAddress }
       Then { result.actions[0].mac_address == '11:22:33:44:55:66' }
     end
 
-    context 'with a SetEtherDestinationAddress action' do
+    context 'with a SetDestinationMacAddress action' do
       When(:user_options) do
         {
           transaction_id: 0x16,
           buffer_id: 0xffffffff,
           in_port: 0xffff,
-          actions: Pio::SetEtherDestinationAddress.new('11:22:33:44:55:66'),
+          actions: Pio::OpenFlow10::SetDestinationMacAddress.new('11:22:33:44:55:66'),
           raw_data: data_dump
         }
       end
@@ -216,17 +217,17 @@ describe Pio::OpenFlow10::PacketOut do
       Then { result.message_length == 0x60 }
       Then { result.actions_len == 0x10 }
       Then { result.actions.length == 1 }
-      Then { result.actions[0].is_a? Pio::SetEtherDestinationAddress }
+      Then { result.actions[0].is_a? Pio::OpenFlow10::SetDestinationMacAddress }
       Then { result.actions[0].mac_address == '11:22:33:44:55:66' }
     end
 
-    context 'with a SetIpSourceAddress action' do
+    context 'with a SetSourceIpAddress action' do
       When(:user_options) do
         {
           transaction_id: 0x16,
           buffer_id: 0xffffffff,
           in_port: 0xffff,
-          actions: Pio::SetIpSourceAddress.new('1.2.3.4'),
+          actions: Pio::OpenFlow10::SetSourceIpAddress.new('1.2.3.4'),
           raw_data: data_dump
         }
       end
@@ -234,17 +235,17 @@ describe Pio::OpenFlow10::PacketOut do
       Then { result.message_length == 0x58 }
       Then { result.actions_len == 0x8 }
       Then { result.actions.length == 1 }
-      Then { result.actions[0].is_a? Pio::SetIpSourceAddress }
+      Then { result.actions[0].is_a? Pio::OpenFlow10::SetSourceIpAddress }
       Then { result.actions[0].ip_address == '1.2.3.4' }
     end
 
-    context 'with a SetIpDestinationAddress action' do
+    context 'with a SetDestinationIpAddress action' do
       When(:user_options) do
         {
           transaction_id: 0x16,
           buffer_id: 0xffffffff,
           in_port: 0xffff,
-          actions: Pio::SetIpDestinationAddress.new('1.2.3.4'),
+          actions: Pio::OpenFlow10::SetDestinationIpAddress.new('1.2.3.4'),
           raw_data: data_dump
         }
       end
@@ -252,17 +253,17 @@ describe Pio::OpenFlow10::PacketOut do
       Then { result.message_length == 0x58 }
       Then { result.actions_len == 0x8 }
       Then { result.actions.length == 1 }
-      Then { result.actions[0].is_a? Pio::SetIpDestinationAddress }
+      Then { result.actions[0].is_a? Pio::OpenFlow10::SetDestinationIpAddress }
       Then { result.actions[0].ip_address == '1.2.3.4' }
     end
 
-    context 'with a SetIpTos action' do
+    context 'with a SetTos action' do
       When(:user_options) do
         {
           transaction_id: 0x16,
           buffer_id: 0xffffffff,
           in_port: 0xffff,
-          actions: Pio::SetIpTos.new(32),
+          actions: Pio::OpenFlow10::SetTos.new(32),
           raw_data: data_dump
         }
       end
@@ -270,7 +271,7 @@ describe Pio::OpenFlow10::PacketOut do
       Then { result.message_length == 0x58 }
       Then { result.actions_len == 0x8 }
       Then { result.actions.length == 1 }
-      Then { result.actions[0].is_a? Pio::SetIpTos }
+      Then { result.actions[0].is_a? Pio::OpenFlow10::SetTos }
       Then { result.actions[0].type_of_service == 32 }
     end
 
@@ -280,7 +281,7 @@ describe Pio::OpenFlow10::PacketOut do
           transaction_id: 0x16,
           buffer_id: 0xffffffff,
           in_port: 0xffff,
-          actions: Pio::SetTransportSourcePort.new(100),
+          actions: Pio::OpenFlow10::SetTransportSourcePort.new(100),
           raw_data: data_dump
         }
       end
@@ -288,7 +289,7 @@ describe Pio::OpenFlow10::PacketOut do
       Then { result.message_length == 0x58 }
       Then { result.actions_len == 0x8 }
       Then { result.actions.length == 1 }
-      Then { result.actions[0].is_a? Pio::SetTransportSourcePort }
+      Then { result.actions[0].is_a? Pio::OpenFlow10::SetTransportSourcePort }
       Then { result.actions[0].port == 100 }
     end
 
@@ -298,7 +299,7 @@ describe Pio::OpenFlow10::PacketOut do
           transaction_id: 0x16,
           buffer_id: 0xffffffff,
           in_port: 0xffff,
-          actions: Pio::SetTransportDestinationPort.new(100),
+          actions: Pio::OpenFlow10::SetTransportDestinationPort.new(100),
           raw_data: data_dump
         }
       end
@@ -306,7 +307,7 @@ describe Pio::OpenFlow10::PacketOut do
       Then { result.message_length == 0x58 }
       Then { result.actions_len == 0x8 }
       Then { result.actions.length == 1 }
-      Then { result.actions[0].is_a? Pio::SetTransportDestinationPort }
+      Then { result.actions[0].is_a? Pio::OpenFlow10::SetTransportDestinationPort }
       Then { result.actions[0].port == 100 }
     end
 
@@ -316,7 +317,7 @@ describe Pio::OpenFlow10::PacketOut do
           transaction_id: 0x16,
           buffer_id: 0xffffffff,
           in_port: 0xffff,
-          actions: Pio::Enqueue.new(port: 1, queue_id: 2),
+          actions: Pio::OpenFlow10::Enqueue.new(port: 1, queue_id: 2),
           raw_data: data_dump
         }
       end
@@ -324,7 +325,7 @@ describe Pio::OpenFlow10::PacketOut do
       Then { result.message_length == 0x60 }
       Then { result.actions_len == 0x10 }
       Then { result.actions.length == 1 }
-      Then { result.actions[0].is_a? Pio::Enqueue }
+      Then { result.actions[0].is_a? Pio::OpenFlow10::Enqueue }
       Then { result.actions[0].port == 1 }
       Then { result.actions[0].queue_id == 2 }
     end
@@ -336,7 +337,7 @@ describe Pio::OpenFlow10::PacketOut do
           buffer_id: 0xffffffff,
           in_port: 0xffff,
           actions: [Pio::OpenFlow10::SendOutPort.new(2),
-                    Pio::SetVlanVid.new(10)],
+                    Pio::OpenFlow10::SetVlanVid.new(10)],
           raw_data: data_dump
         }
       end
@@ -347,8 +348,9 @@ describe Pio::OpenFlow10::PacketOut do
       Then { result.actions[0].is_a? Pio::OpenFlow10::SendOutPort }
       Then { result.actions[0].port == 2 }
       Then { result.actions[0].max_length == 2**16 - 1 }
-      Then { result.actions[1].is_a? Pio::SetVlanVid }
+      Then { result.actions[1].is_a? Pio::OpenFlow10::SetVlanVid }
       Then { result.actions[1].vlan_id == 10 }
     end
   end
 end
+# rubocop:enable LineLength

data/spec/pio/open_flow10/set_destination_ip_address_spec.rb

@@ -0,0 +1,27 @@
+require 'pio/open_flow10/set_destination_ip_address'
+
+describe Pio::OpenFlow10::SetDestinationIpAddress do
+  describe '.new' do
+    context "with '1.2.3.4'" do
+      When(:set_destination_ip_addr) do
+        Pio::OpenFlow10::SetDestinationIpAddress.new('1.2.3.4')
+      end
+
+      describe '#ip_address' do
+        Then { set_destination_ip_addr.ip_address == '1.2.3.4' }
+      end
+
+      describe '#action_type' do
+        Then { set_destination_ip_addr.action_type == 7 }
+      end
+
+      describe '#action_length' do
+        Then { set_destination_ip_addr.action_length == 8 }
+      end
+
+      describe '#to_binary' do
+        Then { set_destination_ip_addr.to_binary.length == 8 }
+      end
+    end
+  end
+end

data/spec/pio/open_flow10/set_destination_mac_address_spec.rb

@@ -0,0 +1,28 @@
+require 'pio/open_flow10/set_destination_mac_address'
+
+describe Pio::OpenFlow10::SetDestinationMacAddress do
+  describe '.new' do
+    Given(:set_destination_mac_address) do
+      Pio::OpenFlow10::SetDestinationMacAddress.new(mac_address)
+    end
+
+    context "with '11:22:33:44:55:66'" do
+      When(:mac_address) { '11:22:33:44:55:66' }
+      Then { set_destination_mac_address.mac_address == '11:22:33:44:55:66' }
+
+      describe '#to_binary' do
+        Then { set_destination_mac_address.to_binary.length == 16 }
+      end
+    end
+
+    context 'with 0x112233445566' do
+      When(:mac_address) { 0x112233445566 }
+      Then { set_destination_mac_address.mac_address == '11:22:33:44:55:66' }
+    end
+
+    context "with Pio::Mac.new('11:22:33:44:55:66')" do
+      When(:mac_address) { Pio::Mac.new('11:22:33:44:55:66') }
+      Then { set_destination_mac_address.mac_address == '11:22:33:44:55:66' }
+    end
+  end
+end

data/spec/pio/open_flow10/set_source_ip_address_spec.rb

@@ -0,0 +1,27 @@
+require 'pio/open_flow10/set_source_ip_address'
+
+describe Pio::OpenFlow10::SetSourceIpAddress do
+  describe '.new' do
+    context "with '1.2.3.4'" do
+      When(:set_source_ip_addr) do
+        Pio::OpenFlow10::SetSourceIpAddress.new('1.2.3.4')
+      end
+
+      describe '#ip_address' do
+        Then { set_source_ip_addr.ip_address == '1.2.3.4' }
+      end
+
+      describe '#action_type' do
+        Then { set_source_ip_addr.action_type == 6 }
+      end
+
+      describe '#action_length' do
+        Then { set_source_ip_addr.action_length == 8 }
+      end
+
+      describe '#to_binary' do
+        Then { set_source_ip_addr.to_binary.length == 8 }
+      end
+    end
+  end
+end

data/spec/pio/open_flow10/set_source_mac_address_spec.rb

@@ -0,0 +1,28 @@
+require 'pio/open_flow10/set_source_mac_address'
+
+describe Pio::OpenFlow10::SetSourceMacAddress do
+  describe '.new' do
+    Given(:set_source_mac_address) do
+      Pio::OpenFlow10::SetSourceMacAddress.new(mac_address)
+    end
+
+    context "with '11:22:33:44:55:66'" do
+      When(:mac_address) { '11:22:33:44:55:66' }
+      Then { set_source_mac_address.mac_address == '11:22:33:44:55:66' }
+
+      describe '#to_binary' do
+        Then { set_source_mac_address.to_binary.length == 16 }
+      end
+    end
+
+    context 'with 0x112233445566' do
+      When(:mac_address) { 0x112233445566 }
+      Then { set_source_mac_address.mac_address == '11:22:33:44:55:66' }
+    end
+
+    context "with Pio::Mac.new('11:22:33:44:55:66')" do
+      When(:mac_address) { Pio::Mac.new('11:22:33:44:55:66') }
+      Then { set_source_mac_address.mac_address == '11:22:33:44:55:66' }
+    end
+  end
+end

data/spec/pio/open_flow10/set_tos_spec.rb

@@ -0,0 +1,30 @@
+require 'pio/open_flow10/set_tos'
+
+describe Pio::OpenFlow10::SetTos do
+  describe '.new' do
+    context 'with 32' do
+      When(:set_tos) { Pio::OpenFlow10::SetTos.new(32) }
+
+      describe '#type_of_service' do
+        Then { set_tos.type_of_service == 32 }
+      end
+
+      describe '#action_type' do
+        Then { set_tos.action_type == 8 }
+      end
+
+      describe '#action_length' do
+        Then { set_tos.action_length == 8 }
+      end
+
+      describe '#to_binary' do
+        Then { set_tos.to_binary.length == 8 }
+      end
+    end
+
+    context 'with 1' do
+      When(:set_tos) { Pio::OpenFlow10::SetTos.new(1) }
+      Then { set_tos == Failure(ArgumentError) }
+    end
+  end
+end

data/spec/pio/open_flow10/set_transport_destination_port_spec.rb

@@ -1,9 +1,9 @@
 require 'pio/open_flow10/set_transport_port'
 
-describe Pio::SetTransportDestinationPort do
+describe Pio::OpenFlow10::SetTransportDestinationPort do
   describe '.new' do
     When(:set_transport_destination_port) do
-      Pio::SetTransportDestinationPort.new(port)
+      Pio::OpenFlow10::SetTransportDestinationPort.new(port)
     end
 
     context 'with 100' do
@@ -17,8 +17,8 @@ describe Pio::SetTransportDestinationPort do
         Then { set_transport_destination_port.action_type == 10 }
       end
 
-      describe '#length' do
-        Then { set_transport_destination_port.length == 8 }
+      describe '#action_length' do
+        Then { set_transport_destination_port.action_length == 8 }
       end
 
       describe '#to_binary' do

data/spec/pio/open_flow10/set_transport_source_port_spec.rb

@@ -1,9 +1,9 @@
 require 'pio/open_flow10/set_transport_port'
 
-describe Pio::SetTransportSourcePort do
+describe Pio::OpenFlow10::SetTransportSourcePort do
   describe '.new' do
     When(:set_transport_source_port) do
-      Pio::SetTransportSourcePort.new port
+      Pio::OpenFlow10::SetTransportSourcePort.new port
     end
 
     context 'with 100' do
@@ -17,8 +17,8 @@ describe Pio::SetTransportSourcePort do
         Then { set_transport_source_port.action_type == 9 }
       end
 
-      describe '#length' do
-        Then { set_transport_source_port.length == 8 }
+      describe '#action_length' do
+        Then { set_transport_source_port.action_length == 8 }
       end
 
       describe '#to_binary' do

data/spec/pio/open_flow10/set_vlan_priority_spec.rb

@@ -1,8 +1,8 @@
 require 'pio/open_flow10/set_vlan_priority'
 
-describe Pio::SetVlanPriority do
+describe Pio::OpenFlow10::SetVlanPriority do
   describe '.new' do
-    When(:set_vlan_priority) { Pio::SetVlanPriority.new(priority) }
+    When(:set_vlan_priority) { Pio::OpenFlow10::SetVlanPriority.new(priority) }
 
     context 'with 3' do
       When(:priority) { 3 }
@@ -15,8 +15,8 @@ describe Pio::SetVlanPriority do
         Then { set_vlan_priority.action_type == 2 }
       end
 
-      describe '#length' do
-        Then { set_vlan_priority.length == 8 }
+      describe '#action_length' do
+        Then { set_vlan_priority.action_length == 8 }
       end
 
       describe '#to_binary' do

data/spec/pio/open_flow10/set_vlan_vid_spec.rb

@@ -1,8 +1,8 @@
 require 'pio/open_flow10/set_vlan_vid'
 
-describe Pio::SetVlanVid do
+describe Pio::OpenFlow10::SetVlanVid do
   describe '.new' do
-    When(:set_vlan_vid) { Pio::SetVlanVid.new(vlan_id) }
+    When(:set_vlan_vid) { Pio::OpenFlow10::SetVlanVid.new(vlan_id) }
 
     context 'with 10' do
       When(:vlan_id) { 10 }
@@ -15,8 +15,8 @@ describe Pio::SetVlanVid do
         Then { set_vlan_vid.action_type == 1 }
       end
 
-      describe '#length' do
-        Then { set_vlan_vid.length == 8 }
+      describe '#action_length' do
+        Then { set_vlan_vid.action_length == 8 }
       end
 
       describe '#to_binary' do