Refactor SetVlanVid class

Delete SetVlanVid::Format class Using Pio::OpenFlow::Action.
Refs #253

features/open_flow10/flow_mod.feature

@@ -76,7 +76,7 @@ Feature: Pio::FlowMod
       | out_port                               |                                                 65535 |
       | flags                                  |                                      [:send_flow_rem] |
       | actions.length                         |                                                    12 |
-      | actions.first.class                    |                                       Pio::SetVlanVid |
+      | actions.first.class                    |                           Pio::OpenFlow10::SetVlanVid |
       | actions.first.vlan_id                  |                                                    10 |
 
   Scenario: read (Flow Mod Modify)
@@ -113,7 +113,7 @@ Feature: Pio::FlowMod
       | out_port                               |                                                 65535 |
       | flags                                  |                                      [:send_flow_rem] |
       | actions.length                         |                                                    12 |
-      | actions.first.class                    |                                       Pio::SetVlanVid |
+      | actions.first.class                    |                           Pio::OpenFlow10::SetVlanVid |
       | actions.first.vlan_id                  |                                                    10 |
 
   Scenario: read (Flow Mod Modify Strict)
@@ -150,7 +150,7 @@ Feature: Pio::FlowMod
       | out_port                               |                                                 65535 |
       | flags                                  |                                      [:send_flow_rem] |
       | actions.length                         |                                                    12 |
-      | actions.first.class                    |                                       Pio::SetVlanVid |
+      | actions.first.class                    |                           Pio::OpenFlow10::SetVlanVid |
       | actions.first.vlan_id                  |                                                    10 |
 
   Scenario: read (Flow Mod Delete)

features/open_flow10/set_vlan_vid.feature

@@ -1,10 +1,10 @@
 @open_flow10
-Feature: Pio::SetVlanVid
+Feature: Pio::OpenFlow10::SetVlanVid
 
   Scenario: new(1)
     When I try to create an OpenFlow action with:
       """
-      Pio::SetVlanVid.new(1)
+      Pio::OpenFlow10::SetVlanVid.new(1)
       """
     Then it should finish successfully
     And the action has the following fields and values:

lib/pio/open_flow10/actions.rb

@@ -16,7 +16,7 @@ module OpenFlow
     class Actions < BinData::Primitive
       ACTION_CLASS = {
         0 => Pio::OpenFlow10::SendOutPort,
-        1 => Pio::SetVlanVid,
+        1 => Pio::OpenFlow10::SetVlanVid,
         2 => Pio::SetVlanPriority,
         3 => Pio::StripVlanHeader,
         4 => Pio::SetEtherSourceAddress,

lib/pio/open_flow10/set_vlan_vid.rb

@@ -1,18 +1,24 @@
+require 'pio/open_flow/action'
 require 'pio/open_flow10/set_vlan'
 
 module Pio
-  # An action to modify the VLAN ID of a packet.
-  class SetVlanVid < SetVlan
-    def_format :vlan_id, 1
+  module OpenFlow10
+    # An action to modify the VLAN ID of a packet.
+    class SetVlanVid < OpenFlow::Action
+      action_header action_type: 1, action_length: 8
+      uint16 :vlan_id
+      string :padding, length: 2
+      hide :padding
 
-    def initialize(number)
-      vlan_id = number.to_i
-      unless vlan_id >= 1 && vlan_id <= 4095
-        fail ArgumentError, 'VLAN ID must be between 1 and 4095 inclusive'
+      def initialize(number)
+        vlan_id = number.to_i
+        unless vlan_id >= 1 && vlan_id <= 4095
+          fail ArgumentError, 'VLAN ID must be between 1 and 4095 inclusive'
+        end
+        super(vlan_id: vlan_id)
+      rescue NoMethodError
+        raise TypeError, 'VLAN ID must be an Integer.'
       end
-      @format = Format.new(vlan_id: vlan_id)
-    rescue NoMethodError
-      raise TypeError, 'VLAN ID must be an Integer.'
     end
   end
 end

spec/pio/open_flow10/packet_out_spec.rb

@@ -137,15 +137,15 @@
           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
 
       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
 
@@ -336,7 +336,7 @@
           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,7 +347,7 @@
       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

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 }