pio 0.1.1 → 0.2.1

data/spec/pio/arp/request_spec.rb

@@ -0,0 +1,131 @@
+require File.join( File.dirname( __FILE__ ), "..", "..", "spec_helper" )
+require "pio/arp/request"
+
+
+module Pio
+  class Arp
+    describe Request do
+      context ".new" do
+        subject {
+          Arp::Request.new(
+            :source_mac => source_mac,
+            :sender_protocol_address => sender_protocol_address,
+            :target_protocol_address => target_protocol_address
+          )
+        }
+
+        let( :arp_request_dump ) {
+          [
+           0xff, 0xff, 0xff, 0xff, 0xff, 0xff, # destination mac address
+           0x00, 0x26, 0x82, 0xeb, 0xea, 0xd1, # source mac address
+           0x08, 0x06,                         # ethernet type
+           0x00, 0x01,                         # arp hardware type
+           0x08, 0x00,                         # arp protocol type
+           0x06,                               # hardware address size
+           0x04,                               # protocol address size
+           0x00, 0x01,                         # operational code
+           0x00, 0x26, 0x82, 0xeb, 0xea, 0xd1, # sender hardware address
+           0xc0, 0xa8, 0x53, 0x03,             # sender protocol address
+           0x00, 0x00, 0x00, 0x00, 0x00, 0x00, # target hardware address
+           0xc0, 0xa8, 0x53, 0xfe,             # target protocol address
+           0x00, 0x00, 0x00, 0x00, 0x00, 0x00, # padding
+           0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+           0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+           0x00, 0x00, 0x00, 0x00,
+          ].pack( "C*" )
+        }
+
+
+        context "with Integer MAC and IP address" do
+          let( :source_mac ) { 0x002682ebead1 }
+          let( :sender_protocol_address ) { 0xc0a85303 }
+          let( :target_protocol_address ) { 0xc0a853fe }
+
+          context "#to_binary" do
+            it "returns an Arp Request binary String" do
+              expect( subject.to_binary ).to eq arp_request_dump
+            end
+
+            it "returns a valid ether frame with size = 64" do
+              expect( subject.to_binary.size ).to eq 64
+            end
+          end
+        end
+
+
+        context "with String MAC and IP address" do
+          let( :source_mac ) { "00:26:82:eb:ea:d1" }
+          let( :sender_protocol_address ) { 0xc0a85303 }
+          let( :target_protocol_address ) { 0xc0a853fe }
+
+          context "#to_binary" do
+            it "returns an Arp Request binary String" do
+              expect( subject.to_binary ).to eq arp_request_dump
+            end
+
+            it "returns a valid ether frame with size = 64" do
+              expect( subject.to_binary.size ).to eq 64
+            end
+          end
+        end
+
+
+        context "with Integer MAC and String IP address" do
+          let( :source_mac ) { 0x002682ebead1 }
+          let( :sender_protocol_address ) { "192.168.83.3" }
+          let( :target_protocol_address ) { "192.168.83.254" }
+
+          context "#to_binary" do
+            it "returns an Arp Request binary String" do
+              expect( subject.to_binary ).to eq arp_request_dump
+            end
+
+            it "returns a valid ether frame with size = 64" do
+              expect( subject.to_binary.size ).to eq 64
+            end
+          end
+        end
+
+
+        context "when :source_mac is not set" do
+          let( :source_mac ) { nil }
+          let( :sender_protocol_address ) { "192.168.83.3" }
+          let( :target_protocol_address ) { "192.168.83.254" }
+
+          it "raises an invalid MAC address error" do
+            expect { subject }.to raise_error( "Invalid MAC address: nil" )
+          end
+        end
+
+
+        context "when :sender_protocol_address is not set" do
+          let( :source_mac ) { 0x002682ebead1 }
+          let( :sender_protocol_address ) { nil }
+          let( :target_protocol_address ) { "192.168.83.254" }
+
+          it "raises an invalid IP address error" do
+            expect { subject }.to raise_error( "Invalid IP address: nil" )
+          end
+        end
+
+
+        context "when :target_protocol_address is not set" do
+          let( :source_mac ) { 0x002682ebead1 }
+          let( :sender_protocol_address ) { "192.168.83.254" }
+          let( :target_protocol_address ) { nil }
+          
+          it "raises an invalid IP address error" do
+            expect { subject }.to raise_error( "Invalid IP address: nil" )
+          end
+        end
+      end
+    end
+  end
+end
+
+
+### Local variables:
+### mode: Ruby
+### coding: utf-8-unix
+### indent-tabs-mode: nil
+### End:

data/spec/pio/arp_spec.rb

@@ -0,0 +1,136 @@
+require File.join( File.dirname( __FILE__ ), "..", "spec_helper" )
+require "pio"
+
+
+module Pio
+  describe Arp do
+    context ".read" do
+      subject { Arp.read( data.pack( "C*" ) ) }
+
+      context "with an ARP Request packet" do
+        let( :data ) {
+          [
+           0xff, 0xff, 0xff, 0xff, 0xff, 0xff, # destination mac address
+           0x00, 0x26, 0x82, 0xeb, 0xea, 0xd1, # source mac address
+           0x08, 0x06,                         # ethernet type
+           0x00, 0x01,                         # arp hardware type
+           0x08, 0x00,                         # arp protocol type
+           0x06,                               # hardware address size
+           0x04,                               # protocol address size
+           0x00, 0x01,                         # operational code
+           0x00, 0x26, 0x82, 0xeb, 0xea, 0xd1, # sender hardware address
+           0xc0, 0xa8, 0x53, 0x03,             # sender protocol address
+           0x00, 0x00, 0x00, 0x00, 0x00, 0x00, # target hardware address
+           0xc0, 0xa8, 0x53, 0xfe,             # target protocol address
+          ]
+        }
+
+        its( :class ) { should be Arp::Request }
+        its( "destination_mac.to_s" ) { should eq "ff:ff:ff:ff:ff:ff" }
+        its( "source_mac.to_s" ) { should eq "00:26:82:eb:ea:d1" }
+        its( :ether_type ) { should eq 0x0806 }
+        its( :hardware_type ) { should eq 1 }
+        its( :protocol_type ) { should eq 0x800 }
+        its( :hardware_length ) { should eq 6 }
+        its( :protocol_length ) { should eq 4 }
+        its( :operation ) { should eq 1 }
+        its( "sender_hardware_address.to_s" ) { should eq "00:26:82:eb:ea:d1" }
+        its( "sender_protocol_address.to_s" ) { should eq "192.168.83.3" }
+        its( "target_hardware_address.to_s" ) { should eq "00:00:00:00:00:00" }
+        its( "target_protocol_address.to_s" ) { should eq "192.168.83.254" }
+      end
+
+
+      context "with an ARP Reply packet" do
+        let( :data ) {
+          [
+           0x00, 0x26, 0x82, 0xeb, 0xea, 0xd1, # destination mac address
+           0x00, 0x16, 0x9d, 0x1d, 0x9c, 0xc4, # source mac address
+           0x08, 0x06,                         # ethernet type
+           0x00, 0x01,                         # arp hardware type
+           0x08, 0x00,                         # arp protocol type
+           0x06,                               # hardware address size
+           0x04,                               # protocol address size
+           0x00, 0x02,                         # operational code
+           0x00, 0x16, 0x9d, 0x1d, 0x9c, 0xc4, # sender hardware address
+           0xc0, 0xa8, 0x53, 0xfe,             # sender protocol address
+           0x00, 0x26, 0x82, 0xeb, 0xea, 0xd1, # target hardware address
+           0xc0, 0xa8, 0x53, 0x03,             # target protocol address
+          ]
+        }
+
+        its( :class ) { should be Arp::Reply }
+        its( "destination_mac.to_s" ) { should eq "00:26:82:eb:ea:d1" }
+        its( "source_mac.to_s" ) { should eq "00:16:9d:1d:9c:c4" }
+        its( :ether_type ) { should eq 0x0806 }
+        its( :hardware_type ) { should eq 1 }
+        its( :protocol_type ) { should eq 0x800 }
+        its( :hardware_length ) { should eq 6 }
+        its( :protocol_length ) { should eq 4 }
+        its( :operation ) { should eq 2 }
+        its( "sender_hardware_address.to_s" ) { should eq "00:16:9d:1d:9c:c4" }
+        its( "sender_protocol_address.to_s" ) { should eq "192.168.83.254" }
+        its( "target_hardware_address.to_s" ) { should eq "00:26:82:eb:ea:d1" }
+        its( "target_protocol_address.to_s" ) { should eq "192.168.83.3" }
+      end
+
+
+      context "with an ARP Request captured in real environment" do
+        let( :data ) {
+          [
+           0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0x5c, 0x9a, 0xd8, 0xea,
+           0x37, 0x32, 0x08, 0x06, 0x00, 0x01, 0x08, 0x00, 0x06, 0x04,
+           0x00, 0x01, 0x5c, 0x9a, 0xd8, 0xea, 0x37, 0x32, 0xc0, 0xa8,
+           0x00, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xc0, 0xa8,
+           0x00, 0xfe
+          ]
+          it { expect { subject }.not_to raise_error }
+        }
+      end
+
+
+      context "with an ARP Request from a real OpenFlow switch" do
+        let( :data ) {
+          [
+           0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0x11, 0x22, 0x33, 0x44,
+           0x55, 0x66, 0x08, 0x06, 0x00, 0x01, 0x08, 0x00, 0x06, 0x04,
+           0x00, 0x01, 0x11, 0x22, 0x33, 0x44, 0x55, 0x66, 0xc0, 0xa8,
+           0x00, 0xfe, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xc0, 0xa8,
+           0x00, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+           0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+           0x00, 0x00, 0x00, 0x00
+          ]
+          it { expect { subject }.not_to raise_error }
+        }
+      end
+
+
+      context "with an ARP Reply captured in real environment" do
+        let( :data ) {
+          [
+           0x11, 0x22, 0x33, 0x44, 0x55, 0x66, 0x5c, 0x9a, 0xd8, 0xea,
+           0x37, 0x32, 0x08, 0x06, 0x00, 0x01, 0x08, 0x00, 0x06, 0x04,
+           0x00, 0x02, 0x5c, 0x9a, 0xd8, 0xea, 0x37, 0x32, 0xc0, 0xa8,
+           0x00, 0x01, 0x11, 0x22, 0x33, 0x44, 0x55, 0x66, 0xc0, 0xa8,
+           0x00, 0xfe
+          ]
+          it { expect { subject }.not_to raise_error }
+        }
+      end
+
+
+      context "with an invalid ARP packet" do
+        let( :data ) { [] }
+
+        it { expect { subject }.to raise_error( Pio::ParseError ) }
+      end
+    end
+  end
+end
+
+
+### Local variables:
+### mode: Ruby
+### coding: utf-8-unix
+### indent-tabs-mode: nil
+### End:

data/spec/pio/ip_spec.rb

@@ -0,0 +1,38 @@
+require File.join( File.dirname( __FILE__ ), "..", "spec_helper" )
+require "pio/ip"
+
+
+module Pio
+  describe IP do
+    context ".new" do
+      subject { IP.new( ip_address, prefixlen ) }
+
+
+      context %{with "192.168.1.1", 32} do
+        let( :ip_address ) { "192.168.1.1" }
+        let( :prefixlen ) { 32 }
+
+        its( :to_s ) { should eq "192.168.1.1" }
+        its( :to_i ) { should eq 3232235777 }
+        its( :to_a ) { should eq [ 0xc0, 0xa8, 0x01, 0x01 ] }
+      end
+
+
+      context %{with "10.1.1.1", 8} do
+        let( :ip_address ) { "10.1.1.1" }
+        let( :prefixlen ) { 8 }
+
+        its( :to_s ) { should eq "10.0.0.0" }
+        its( :to_i ) { should eq 10 * 256 * 256 * 256 }
+        its( :to_a ) { should eq [ 0x0a, 0x00, 0x00, 0x00 ] }
+      end
+    end
+  end
+end
+
+
+### Local variables:
+### mode: Ruby
+### coding: utf-8-unix
+### indent-tabs-mode: nil
+### End:

data/spec/pio/lldp_spec.rb

@@ -1,55 +1,212 @@
 require File.join( File.dirname( __FILE__ ), "..", "spec_helper" )
-require "pio/lldp"
+require "pio"
 
 
 module Pio
   describe Lldp do
-    subject { Lldp.read( data.pack( "C*" ) ) }
-
-    context "parsing a minimal LLDP frame" do
-      let( :data ) {
-        [
-          0x01, 0x80, 0xc2, 0x00, 0x00, 0x0e,  # Destination MAC
-          0x00, 0x19, 0x2f, 0xa7, 0xb2, 0x8d,  # Source MAC
-          0x88, 0xcc,  # Ethertype
-          0x02, 0x07, 0x04, 0x00, 0x19, 0x2f, 0xa7, 0xb2, 0x8d,  # Chassis ID TLV
-          0x04, 0x0d, 0x01, 0x55, 0x70, 0x6c, 0x69, 0x6e, 0x6b, 0x20, 0x74, 0x6f, 0x20, 0x53, 0x31,  # Port ID TLV
-          0x06, 0x02, 0x00, 0x78,  # Time to live TLV
-          0x00, 0x00  # End of LLDPDU TLV
-        ]
+    context ".new" do
+      subject {
+        Lldp.new(
+          :dpid => dpid,
+          :port_number => port_number,
+          :source_mac => source_mac,
+          :destination_mac => destination_mac
+        )
       }
 
-      its( :dpid ) { should eq 108173701773 }
 
-      it "should successfully parse the data" do
-        lambda { subject }.should_not raise_error
+      context "with :dpid and :port_number" do
+        let( :dpid ) { 108173701773 }
+        let( :port_number ) { 1 }
+        let( :source_mac ) { nil }
+        let( :destination_mac ) { nil }
+        let( :lldp_dump ) {
+          [
+           0x01, 0x80, 0xc2, 0x00, 0x00, 0x0e,  # Destination MAC
+           0x01, 0x02, 0x03, 0x04, 0x05, 0x06,  # Source MAC
+           0x88, 0xcc,  # Ethertype
+           0x02, 0x09, 0x07, 0x00, 0x00, 0x00, 0x19, 0x2f, 0xa7, 0xb2, 0x8d,  # Chassis ID TLV
+           0x04, 0x05, 0x07, 0x00, 0x00, 0x00, 0x01,  # Port ID TLV
+           0x06, 0x02, 0x00, 0x78,  # Time to live TLV
+           0x00, 0x00,  # End of LLDPDU TLV
+           0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,   # Padding
+           0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+           0x00, 0x00
+          ]
+        }
+
+        context "#to_binary" do
+          it "returns an LLDP binary string" do
+            expect( subject.to_binary.unpack( "C*" ) ).to eq lldp_dump
+          end
+
+          it "returns a valid ether frame with size = 64" do
+            expect( subject.to_binary.size ).to eq 64
+          end
+        end
+      end
+
+
+      context "with :dpid, :port_number and :source_mac" do
+        let( :dpid ) { 108173701773 }
+        let( :port_number ) { 1 }
+        let( :source_mac ) { "06:05:04:03:02:01" }
+        let( :destination_mac ) { nil }
+        let( :lldp_dump ) {
+          [
+           0x01, 0x80, 0xc2, 0x00, 0x00, 0x0e,  # Destination MAC
+           0x06, 0x05, 0x04, 0x03, 0x02, 0x01,  # Source MAC
+           0x88, 0xcc,  # Ethertype
+           0x02, 0x09, 0x07, 0x00, 0x00, 0x00, 0x19, 0x2f, 0xa7, 0xb2, 0x8d,  # Chassis ID TLV
+           0x04, 0x05, 0x07, 0x00, 0x00, 0x00, 0x01,  # Port ID TLV
+           0x06, 0x02, 0x00, 0x78,  # Time to live TLV
+           0x00, 0x00,  # End of LLDPDU TLV
+           0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,   # Padding
+           0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+           0x00, 0x00
+          ]
+        }
+
+        context "#to_binary" do
+          it "returns an LLDP binary string" do
+            expect( subject.to_binary.unpack( "C*" ) ).to eq lldp_dump
+          end
+
+          it "returns a valid ether frame with size = 64" do
+            expect( subject.to_binary.size ).to eq 64
+          end
+        end
+      end
+
+
+      context "with :dpid, :port_number, :source_mac and :destination_mac" do
+        let( :dpid ) { 108173701773 }
+        let( :port_number ) { 1 }
+        let( :source_mac ) { "06:05:04:03:02:01" }
+        let( :destination_mac ) { "01:02:03:04:05:06" }
+        let( :lldp_dump ) {
+          [
+           0x01, 0x02, 0x03, 0x04, 0x05, 0x06,  # Destination MAC
+           0x06, 0x05, 0x04, 0x03, 0x02, 0x01,  # Source MAC
+           0x88, 0xcc,  # Ethertype
+           0x02, 0x09, 0x07, 0x00, 0x00, 0x00, 0x19, 0x2f, 0xa7, 0xb2, 0x8d,  # Chassis ID TLV
+           0x04, 0x05, 0x07, 0x00, 0x00, 0x00, 0x01,  # Port ID TLV
+           0x06, 0x02, 0x00, 0x78,  # Time to live TLV
+           0x00, 0x00,  # End of LLDPDU TLV
+           0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,   # Padding
+           0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+           0x00, 0x00
+          ]
+        }
+
+        context "#to_binary" do
+          it "returns an LLDP binary string" do
+            expect( subject.to_binary.unpack( "C*" ) ).to eq lldp_dump
+          end
+
+          it "returns a valid ether frame with size = 64" do
+            expect( subject.to_binary.size ).to eq 64
+          end
+        end
+      end
+
+
+      context "when :dpid is not set" do
+        let( :dpid ) { nil }
+        let( :port_number ) { 1 }
+        let( :source_mac ) { nil }
+        let( :destination_mac ) { nil }
+
+        it { expect { subject }.to raise_error( "Invalid DPID: nil" ) }
+      end
+
+
+      context "when :port_number is not set" do
+        let( :dpid ) { 108173701773 }
+        let( :port_number ) { nil }
+        let( :source_mac ) { nil }
+        let( :destination_mac ) { nil }
+
+        it { expect { subject }.to raise_error( "Invalid port number: nil" ) }
       end
     end
 
 
-    context "parsing a detailed LLDP frame" do
-      let( :data ) {
-        [
-          0x01, 0x80, 0xc2, 0x00, 0x00, 0x0e,  # Destination MAC
-          0x00, 0x19, 0x2f, 0xa7, 0xb2, 0x8d,  # Source MAC
-          0x88, 0xcc,  # Ethertype
-          0x02, 0x07, 0x04, 0x00, 0x19, 0x2f, 0xa7, 0xb2, 0x8d,  # Chassis ID TLV
-          0x04, 0x0d, 0x01, 0x55, 0x70, 0x6c, 0x69, 0x6e, 0x6b, 0x20, 0x74, 0x6f, 0x20, 0x53, 0x31,  # Port ID TLV
-          0x06, 0x02, 0x00, 0x78,  # Time to live TLV
-          0x08, 0x17, 0x53, 0x75, 0x6d, 0x6d, 0x69, 0x74, 0x33, 0x30, 0x30, 0x2d, 0x34, 0x38, 0x2d, 0x50, 0x6f, 0x72, 0x74, 0x20, 0x31, 0x30, 0x30, 0x31, 0x00,  # Port Description
-          0x0a, 0x0d, 0x53, 0x75, 0x6d, 0x6d, 0x69, 0x74, 0x33, 0x30, 0x30, 0x2d, 0x34, 0x38, 0x00,  # System Name
-          0x0c, 0x4c, 0x53, 0x75, 0x6d, 0x6d, 0x69, 0x74, 0x33, 0x30, 0x30, 0x2d, 0x34, 0x38, 0x20, 0x2d, 0x20, 0x56, 0x65, 0x72, 0x73, 0x69, 0x6f, 0x6e, 0x20, 0x37, 0x2e, 0x34, 0x65, 0x2e, 0x31, 0x20, 0x28, 0x42, 0x75, 0x69, 0x6c, 0x64, 0x20, 0x35, 0x29, 0x20, 0x62, 0x79, 0x20, 0x52, 0x65, 0x6c, 0x65, 0x61, 0x73, 0x65, 0x5f, 0x4d, 0x61, 0x73, 0x74, 0x65, 0x72, 0x20, 0x30, 0x35, 0x2f, 0x32, 0x37, 0x2f, 0x30, 0x35, 0x20, 0x30, 0x34, 0x3a, 0x35, 0x33, 0x3a, 0x31, 0x31, 0x00,  # System Description
-          0x0e, 0x04, 0x00, 0x14, 0x00, 0x14, # System Capabilities
-          0x10, 0x0e, 0x07, 0x06, 0x00, 0x01, 0x30, 0xf9, 0xad, 0xa0, 0x02, 0x00, 0x00, 0x03, 0xe9, 0x00,
-          0xfe, 0x07, 0x00, 0x12, 0x0f, 0x02, 0x07, 0x01, 0x00,
-          0x00, 0x00  # End of LLDPDU TLV
-        ]
-      }
+    context ".read" do
+      subject { Lldp.read( data.pack( "C*" ) ) }
+
+      context "with a minimal LLDP frame" do
+        let( :data ) {
+          [
+           0x01, 0x80, 0xc2, 0x00, 0x00, 0x0e,  # Destination MAC
+           0x00, 0x19, 0x2f, 0xa7, 0xb2, 0x8d,  # Source MAC
+           0x88, 0xcc,  # Ethertype
+           0x02, 0x07, 0x04, 0x00, 0x19, 0x2f, 0xa7, 0xb2, 0x8d,  # Chassis ID TLV
+           0x04, 0x0d, 0x01, 0x55, 0x70, 0x6c, 0x69, 0x6e, 0x6b, 0x20, 0x74, 0x6f, 0x20, 0x53, 0x31,  # Port ID TLV
+           0x06, 0x02, 0x00, 0x78,  # Time to live TLV
+           0x00, 0x00  # End of LLDPDU TLV
+          ]
+        }
+
+        its( "destination_mac.to_s" ) { should eq "01:80:c2:00:00:0e" }
+        its( "source_mac.to_s" ) { should eq "00:19:2f:a7:b2:8d" }
+        its( :ether_type ) { should eq 0x88cc }
+        its( :dpid ) { should eq 108173701773 }
+        its( :chassis_id ) { should eq 108173701773 }
+        its( :port_id ) { should eq "Uplink to S1" }
+        its( :port_number ) { should eq "Uplink to S1" }
+        its( :ttl ) { should eq 120 }
+        its( :port_description ) { should be_nil }
+        its( :system_name ) { should be_nil }
+        its( :system_description ) { should be_nil }
+        its( :system_capabilities ) { should be_nil }
+        its( :management_address ) { should be_nil }
+      end
+
+
+      context "with a detailed LLDP frame" do
+        let( :data ) {
+          [
+           0x01, 0x80, 0xc2, 0x00, 0x00, 0x0e,  # Destination MAC
+           0x00, 0x19, 0x2f, 0xa7, 0xb2, 0x8d,  # Source MAC
+           0x88, 0xcc,  # Ethertype
+           0x02, 0x07, 0x04, 0x00, 0x19, 0x2f, 0xa7, 0xb2, 0x8d,  # Chassis ID TLV
+           0x04, 0x0d, 0x01, 0x55, 0x70, 0x6c, 0x69, 0x6e, 0x6b, 0x20, 0x74, 0x6f, 0x20, 0x53, 0x31,  # Port ID TLV
+           0x06, 0x02, 0x00, 0x78,  # Time to live TLV
+           0x08, 0x17, 0x53, 0x75, 0x6d, 0x6d, 0x69, 0x74, 0x33, 0x30, 0x30, 0x2d, 0x34, 0x38, 0x2d, 0x50, 0x6f, 0x72, 0x74, 0x20, 0x31, 0x30, 0x30, 0x31, 0x00,  # Port Description
+           0x0a, 0x0d, 0x53, 0x75, 0x6d, 0x6d, 0x69, 0x74, 0x33, 0x30, 0x30, 0x2d, 0x34, 0x38, 0x00,  # System Name
+           0x0c, 0x4c, 0x53, 0x75, 0x6d, 0x6d, 0x69, 0x74, 0x33, 0x30, 0x30, 0x2d, 0x34, 0x38, 0x20, 0x2d, 0x20, 0x56, 0x65, 0x72, 0x73, 0x69, 0x6f, 0x6e, 0x20, 0x37, 0x2e, 0x34, 0x65, 0x2e, 0x31, 0x20, 0x28, 0x42, 0x75, 0x69, 0x6c, 0x64, 0x20, 0x35, 0x29, 0x20, 0x62, 0x79, 0x20, 0x52, 0x65, 0x6c, 0x65, 0x61, 0x73, 0x65, 0x5f, 0x4d, 0x61, 0x73, 0x74, 0x65, 0x72, 0x20, 0x30, 0x35, 0x2f, 0x32, 0x37, 0x2f, 0x30, 0x35, 0x20, 0x30, 0x34, 0x3a, 0x35, 0x33, 0x3a, 0x31, 0x31, 0x00,  # System Description
+           0x0e, 0x04, 0x00, 0x14, 0x00, 0x14, # System Capabilities
+           0x10, 0x0e, 0x07, 0x06, 0x00, 0x01, 0x30, 0xf9, 0xad, 0xa0, 0x02, 0x00, 0x00, 0x03, 0xe9, 0x00, # Management Address
+           0xfe, 0x07, 0x00, 0x12, 0x0f, 0x02, 0x07, 0x01, 0x00, # Organizationally Specific
+           0x00, 0x00  # End of LLDPDU TLV
+          ]
+        }
+
+        its( "destination_mac.to_s" ) { should eq "01:80:c2:00:00:0e" }
+        its( "source_mac.to_s" ) { should eq "00:19:2f:a7:b2:8d" }
+        its( :ether_type ) { should eq 0x88cc }
+        its( :dpid ) { should eq 108173701773 }
+        its( :chassis_id ) { should eq 108173701773 }
+        its( :port_id ) { should eq "Uplink to S1" }
+        its( :port_number ) { should eq "Uplink to S1" }
+        its( :ttl ) { should eq 120 }
+        its( :port_description ) { should eq "Summit300-48-Port 1001" }
+        its( :system_name ) { should eq "Summit300-48" }
+        its( :system_description ) { should eq "Summit300-48 - Version 7.4e.1 (Build 5) by Release_Master 05/27/05 04:53:11" }
+        its( "system_capabilities.system_capabilities" ) { should eq 20 }
+        its( "system_capabilities.enabled_capabilities" ) { should eq 20 }
+        its( :management_address ) { should eq [ 0x00, 0x01, 0x30, 0xf9, 0xad, 0xa0 ].pack( "C*" ) }
+        its( "organizationally_specific.oui" ) { should eq 4623 }
+        its( "organizationally_specific.subtype" ) { should eq 2 }
+        its( "organizationally_specific.information" ) { should eq "\a\x01" }
+      end
+
 
-      its( :dpid ) { should eq 108173701773 }
+      context "with an invalid Lldp frame" do
+        let( :data ) { [] }
 
-      it "should successfully parse the data" do
-        lambda { subject }.should_not raise_error
+        it { expect { subject }.to raise_error( Pio::ParseError ) }
       end
     end
   end