pio 0.6.0 → 0.7.0

data/spec/pio/features_spec.rb

@@ -0,0 +1,78 @@
+# encoding: utf-8
+
+require 'pio'
+
+describe Pio::Features do
+  describe '.read' do
+    context 'with a Features Request message' do
+      Given(:features_request_dump) { [1, 5, 0, 8, 0, 0, 0, 0].pack('C*') }
+
+      When(:features_request) { Pio::Features.read(features_request_dump) }
+
+      Then { features_request.class == Pio::Features::Request }
+      Then { features_request.version == 1 }
+      Then { features_request.message_type == Pio::Features::REQUEST }
+      Then { features_request.message_length == 8 }
+      Then { features_request.transaction_id == 0 }
+      Then { features_request.xid == 0 }
+      Then { features_request.body.empty? }
+    end
+
+    context 'with a Features Reply message' do
+      Given(:features_reply_dump) do
+        [0x01, 0x06, 0x01, 0x10, 0x00, 0x00, 0x00, 0x02, 0x00, 0x00,
+         0xe2, 0xfc, 0x0f, 0xae, 0x66, 0x4d, 0x00, 0x00, 0x01, 0x00,
+         0xfe, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xc7, 0x00, 0x00,
+         0x0f, 0xff, 0x00, 0x03, 0x82, 0xb0, 0xff, 0x6e, 0x51, 0x44,
+         0x76, 0x65, 0x74, 0x68, 0x30, 0x00, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xc0, 0x00, 0x00,
+         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0x01, 0x9e, 0x49, 0x7d, 0x7c, 0x06, 0x8d, 0x76, 0x65,
+         0x74, 0x68, 0x32, 0x62, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0x00, 0x00, 0x00, 0x00, 0xc0, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x04,
+         0x12, 0x13, 0xb4, 0x13, 0x4c, 0x19, 0x76, 0x65, 0x74, 0x68,
+         0x33, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0x00, 0x00, 0xc0, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x02, 0x86, 0xbd,
+         0xe3, 0xad, 0x3d, 0x73, 0x76, 0x65, 0x74, 0x68, 0x31, 0x62,
+         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0xc0, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0x00, 0x00, 0x00, 0xff, 0xfe, 0xe2, 0xfc, 0x0f, 0xae,
+         0x66, 0x4d, 0x62, 0x72, 0x30, 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, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+         0x00, 0x00].pack('C*')
+      end
+
+      When(:features_reply) { Pio::Features.read(features_reply_dump) }
+
+      Then { features_reply.class == Pio::Features::Reply }
+      Then { features_reply.version == 1 }
+      Then { features_reply.message_type == Pio::Features::REPLY }
+      Then { features_reply.message_length == 272 }
+      Then { features_reply.transaction_id == 2 }
+      Then { features_reply.xid == 2 }
+      Then { !features_reply.body.empty? }
+      Then { features_reply.datapath_id == 0xe2fc0fae664d }
+      Then { features_reply.n_buffers == 0x100 }
+      Then { features_reply.n_tables == 0xfe }
+      Then { features_reply.capabilities == 0xc7 }
+      Then { features_reply.actions == 0xfff }
+      Then { features_reply.ports.size == 7 }
+    end
+
+    context 'with a Hello message' do
+      Given(:hello_dump) { [1, 0, 0, 8, 0, 0, 0, 0].pack('C*') }
+
+      When(:result) { Pio::Features.read(hello_dump) }
+
+      Then { result == Failure(Pio::ParseError) }
+    end
+  end
+end

data/spec/pio/hello_spec.rb

@@ -4,7 +4,7 @@ require 'pio'
 
 describe Pio::Hello do
   describe '.read' do
-    context 'with an hello message' do
+    context 'with a hello message' do
       Given(:hello_dump) { [1, 0, 0, 8, 0, 0, 0, 0].pack('C*') }
 
       When(:hello) { Pio::Hello.read(hello_dump) }
@@ -13,15 +13,17 @@ describe Pio::Hello do
       Then { hello.version == 1 }
       Then { hello.message_type == 0 }
       Then { hello.message_length == 8 }
+      Then { hello.transaction_id == 0 }
       Then { hello.xid == 0 }
       Then { hello.body.empty? }
     end
 
-    context 'with an features-request message' do
+    context 'with a features-request message' do
       Given(:features_request_dump) { [1, 5, 0, 8, 0, 0, 0, 0].pack('C*') }
 
       When(:result) { Pio::Hello.read(features_request_dump) }
-      Then { result == Failure(BinData::ValidityError) }
+
+      Then { result == Failure(Pio::ParseError) }
     end
   end
 
@@ -29,22 +31,43 @@ describe Pio::Hello do
     context 'with no arguments' do
       When(:hello) { Pio::Hello.new }
 
-      Then { hello.class == Pio::Hello }
       Then { hello.version == 1 }
       Then { hello.message_type == 0 }
       Then { hello.message_length == 8 }
+      Then { hello.transaction_id == 0 }
       Then { hello.xid == 0 }
       Then { hello.body.empty? }
       Then { hello.to_binary == [1, 0, 0, 8, 0, 0, 0, 0].pack('C*') }
     end
 
+    context 'with 123' do
+      When(:hello) { Pio::Hello.new(123) }
+
+      Then { hello.version == 1 }
+      Then { hello.message_type == 0 }
+      Then { hello.message_length == 8 }
+      Then { hello.transaction_id == 123 }
+      Then { hello.xid == 123 }
+      Then { hello.body.empty? }
+      Then { hello.to_binary == [1, 0, 0, 8, 0, 0, 0, 123].pack('C*') }
+    end
+
+    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
+    end
+
     context 'with transaction_id: 123' do
       When(:hello) { Pio::Hello.new(transaction_id: 123) }
 
-      Then { hello.class == Pio::Hello }
       Then { hello.version == 1 }
       Then { hello.message_type == 0 }
       Then { hello.message_length == 8 }
+      Then { hello.transaction_id == 123 }
       Then { hello.xid == 123 }
       Then { hello.body.empty? }
       Then { hello.to_binary == [1, 0, 0, 8, 0, 0, 0, 123].pack('C*') }
@@ -53,13 +76,19 @@ describe Pio::Hello do
     context 'with xid: 123' do
       When(:hello) { Pio::Hello.new(xid: 123) }
 
-      Then { hello.class == Pio::Hello }
       Then { hello.version == 1 }
       Then { hello.message_type == 0 }
       Then { hello.message_length == 8 }
+      Then { hello.transaction_id == 123 }
       Then { hello.xid == 123 }
       Then { hello.body.empty? }
       Then { hello.to_binary == [1, 0, 0, 8, 0, 0, 0, 123].pack('C*') }
     end
+
+    context 'with :INVALID_ARGUMENT' do
+      When(:result) { Pio::Hello.new(:INVALID_ARGUMENT) }
+
+      Then { result == Failure(TypeError) }
+    end
   end
 end

data/spec/spec_helper.rb

@@ -2,6 +2,9 @@
 
 $LOAD_PATH.unshift File.join(File.dirname(__FILE__), '..', 'lib')
 
+require 'codeclimate-test-reporter'
+CodeClimate::TestReporter.start
+
 require 'simplecov'
 SimpleCov.start
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pio
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.0
 platform: ruby
 authors:
 - Yasuhito Takamiya
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-15 00:00:00.000000000 Z
+date: 2014-04-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bindata
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: 2.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: 2.1.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.6.0
+        version: 1.6.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.6.0
+        version: 1.6.2
 description: Pure ruby packet parser and generator.
 email:
 - yasuhito@gmail.com
@@ -60,6 +60,8 @@ files:
 - examples/dhcp_read.rb
 - examples/echo_new.rb
 - examples/echo_read.rb
+- examples/features_new.rb
+- examples/features_read.rb
 - examples/hello_new.rb
 - examples/hello_read.rb
 - examples/icmp_new.rb
@@ -68,19 +70,27 @@ files:
 - examples/lldp_read.rb
 - features/arp_read.feature
 - features/dhcp_read.feature
+- features/echo_read.feature
+- features/features_read.feature
+- features/hello_read.feature
 - features/icmp_read.feature
 - features/lldp_read.feature
-- features/pcap/arp-storm.pcap
-- features/pcap/arp.pcap
-- features/pcap/dhcp.pcap
-- features/pcap/icmp.pcap
-- features/pcap/lldp.detailed.pcap
-- features/pcap/lldp.minimal.pcap
-- features/step_definitions/pcap_steps.rb
+- features/packet_data/arp-storm.pcap
+- features/packet_data/arp.pcap
+- features/packet_data/dhcp.pcap
+- features/packet_data/echo.raw
+- features/packet_data/features_reply.raw
+- features/packet_data/features_request.raw
+- features/packet_data/hello.raw
+- features/packet_data/icmp.pcap
+- features/packet_data/lldp.detailed.pcap
+- features/packet_data/lldp.minimal.pcap
+- features/step_definitions/packet_data_steps.rb
+- features/step_definitions/pending_steps.rb
 - features/support/env.rb
 - lib/pio.rb
 - lib/pio/arp.rb
-- lib/pio/arp/frame.rb
+- lib/pio/arp/format.rb
 - lib/pio/arp/message.rb
 - lib/pio/arp/reply.rb
 - lib/pio/arp/request.rb
@@ -108,10 +118,15 @@ files:
 - lib/pio/echo/message.rb
 - lib/pio/echo/reply.rb
 - lib/pio/echo/request.rb
+- lib/pio/features.rb
+- lib/pio/features/format.rb
+- lib/pio/features/message.rb
+- lib/pio/features/reply.rb
+- lib/pio/features/request.rb
 - lib/pio/hello.rb
 - lib/pio/hello/format.rb
 - lib/pio/icmp.rb
-- lib/pio/icmp/frame.rb
+- lib/pio/icmp/format.rb
 - lib/pio/icmp/message.rb
 - lib/pio/icmp/options.rb
 - lib/pio/icmp/reply.rb
@@ -140,6 +155,7 @@ files:
 - lib/pio/type/ip_address.rb
 - lib/pio/type/ipv4_header.rb
 - lib/pio/type/mac_address.rb
+- lib/pio/type/open_flow.rb
 - lib/pio/type/udp_header.rb
 - lib/pio/version.rb
 - pio.gemspec
@@ -156,6 +172,9 @@ files:
 - spec/pio/echo/reply_spec.rb
 - spec/pio/echo/request_spec.rb
 - spec/pio/echo_spec.rb
+- spec/pio/features/reply_spec.rb
+- spec/pio/features/request_spec.rb
+- spec/pio/features_spec.rb
 - spec/pio/hello_spec.rb
 - spec/pio/icmp/reply_spec.rb
 - spec/pio/icmp/request_spec.rb
@@ -190,38 +209,49 @@ signing_key:
 specification_version: 4
 summary: Packet parser and generator.
 test_files:
-- spec/pio/ipv4_address_spec.rb
-- spec/pio/hello_spec.rb
-- spec/pio/dhcp/request_spec.rb
-- spec/pio/dhcp/offer_spec.rb
+- spec/pio/arp/reply/options_spec.rb
+- spec/pio/arp/reply_spec.rb
+- spec/pio/arp/request/options_spec.rb
+- spec/pio/arp/request_spec.rb
+- spec/pio/arp_spec.rb
 - spec/pio/dhcp/ack_spec.rb
 - spec/pio/dhcp/discover_spec.rb
+- spec/pio/dhcp/offer_spec.rb
+- spec/pio/dhcp/request_spec.rb
+- spec/pio/dhcp_spec.rb
+- spec/pio/echo/reply_spec.rb
+- spec/pio/echo/request_spec.rb
 - spec/pio/echo_spec.rb
-- spec/pio/arp/request_spec.rb
-- spec/pio/arp/reply/options_spec.rb
-- spec/pio/arp/request/options_spec.rb
-- spec/pio/arp/reply_spec.rb
-- spec/pio/lldp/options_spec.rb
-- spec/pio/mac_spec.rb
-- spec/pio/icmp/request_spec.rb
+- spec/pio/features/reply_spec.rb
+- spec/pio/features/request_spec.rb
+- spec/pio/features_spec.rb
+- spec/pio/hello_spec.rb
 - spec/pio/icmp/reply_spec.rb
+- spec/pio/icmp/request_spec.rb
 - spec/pio/icmp_spec.rb
-- spec/pio/dhcp_spec.rb
+- spec/pio/ipv4_address_spec.rb
+- spec/pio/lldp/options_spec.rb
 - spec/pio/lldp_spec.rb
-- spec/pio/arp_spec.rb
-- spec/pio/echo/request_spec.rb
-- spec/pio/echo/reply_spec.rb
+- spec/pio/mac_spec.rb
 - spec/spec_helper.rb
-- features/pcap/dhcp.pcap
-- features/pcap/arp.pcap
-- features/pcap/lldp.minimal.pcap
-- features/pcap/icmp.pcap
-- features/pcap/arp-storm.pcap
-- features/pcap/lldp.detailed.pcap
-- features/support/env.rb
-- features/icmp_read.feature
 - features/arp_read.feature
-- features/step_definitions/pcap_steps.rb
-- features/lldp_read.feature
 - features/dhcp_read.feature
+- features/echo_read.feature
+- features/features_read.feature
+- features/hello_read.feature
+- features/icmp_read.feature
+- features/lldp_read.feature
+- features/packet_data/arp-storm.pcap
+- features/packet_data/arp.pcap
+- features/packet_data/dhcp.pcap
+- features/packet_data/echo.raw
+- features/packet_data/features_reply.raw
+- features/packet_data/features_request.raw
+- features/packet_data/hello.raw
+- features/packet_data/icmp.pcap
+- features/packet_data/lldp.detailed.pcap
+- features/packet_data/lldp.minimal.pcap
+- features/step_definitions/packet_data_steps.rb
+- features/step_definitions/pending_steps.rb
+- features/support/env.rb
 has_rdoc: 

data/features/step_definitions/pcap_steps.rb

@@ -1,18 +0,0 @@
-# encoding: utf-8
-
-Given(/^a pcap file "(.*?)"$/) do |pcap|
-  @pcap = File.join(File.dirname(__FILE__), '..', 'pcap', pcap)
-end
-
-When(/^I try to parse the pcap file with "(.*?)" class$/) do |parser|
-  File.open(@pcap) do |file|
-    pcap = Pio::Pcap::Frame.read(file)
-    pcap.records.each do |each|
-      Pio.const_get(parser).__send__ :read, each.data
-    end
-  end
-end
-
-Then(/^it should finish successfully$/) do
-  # Noop.
-end