Test Pio::Hello13.new().

trema · Apr 6, 2015 · 9ecd4f1 · 9ecd4f1
1 parent 6f1e9c9
commit 9ecd4f1
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 3 deletions.
diff --git a/lib/pio/hello13.rb b/lib/pio/hello13.rb
@@ -9,14 +9,16 @@ class HelloElement < BinData::Record
   class Hello13 < BinData::Record
     endian :big
 
-    uint8 :ofp_version, value: 4
+    uint8 :ofp_version, initial_value: 4
     uint8 :message_type
-    uint16 :message_length
+    uint16 :message_length, value: 8
     uint32 :transaction_id
     array :elements, type: :hello_element, length: 0
 
     def xid
       transaction_id
     end
+
+    alias_method :to_binary, :to_binary_s
   end
 end
diff --git a/spec/pio/hello13_spec.rb b/spec/pio/hello13_spec.rb
@@ -5,7 +5,7 @@
     When(:result) { Pio::Hello13.read(binary) }
 
     context 'with a hello message' do
-      Given(:binary) { [3, 0, 0, 8, 0, 0, 0, 0].pack('C*') }
+      Given(:binary) { [4, 0, 0, 8, 0, 0, 0, 0].pack('C*') }
 
       Then { result.class == Pio::Hello13 }
       Then { result.ofp_version == 4 }
@@ -16,4 +16,18 @@
       Then { result.elements.empty? }
     end
   end
+
+  describe '.new' do
+    context 'with no arguments' do
+      When(:result) { Pio::Hello13.new }
+
+      Then { result.ofp_version == 4 }
+      Then { result.message_type == 0 }
+      Then { result.message_length == 8 }
+      Then { result.transaction_id == 0 }
+      Then { result.xid == 0 }
+      Then { result.elements.empty? }
+      Then { result.to_binary == [4, 0, 0, 8, 0, 0, 0, 0].pack('C*') }
+    end
+  end
 end