pio 0.30.1 → 0.30.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fdea8ee5cb716f9d6dd67083baf361c3b3b706b0
-  data.tar.gz: 76628fc9d6845d884d7c366adf51861572e8683e
+  metadata.gz: e707c4843ee4cc795399648cf314a7a5b7f5b901
+  data.tar.gz: 56d5b9d2451ca62a0fb622c2ec0ad319229bdd45
 SHA512:
-  metadata.gz: 2c8641f225a53391fa3c7a861ea52755d55e1a2e4f335af6da27eb4cb5a727e51eb10d7ec4cd2ce640cda4078d2f33006ba7bf8c5a8261040b055564c95a4241
-  data.tar.gz: 34af6da3d91113ea51c87725791d2d9a313988133e51d14aeb085dd8f09a7b497a713f7be09129bfa7018c8c5f89f7a5fd0e52666f9781fe1fda97b27142ad21
+  metadata.gz: e33f76294116b64b5202404a40d0ec39f5f38cef4453e6270cda83c129bf1e61b521c44987bca56357d6433f53e44a7202c2ec145a87b3791f19f80a39217cb9
+  data.tar.gz: ac8bb54c42fbde230c827e7b31394e9018d8d1ee64c825536ca679c382e9a0f11bf2eaf1b1e14ee2fe62af7749585fa2f4f4fef15a2ef5d91bc057f2fd0161c2

data/README.md

@@ -2,11 +2,7 @@
 
 <a href='https://rubygems.org/gems/pio'><img src='http://img.shields.io/gem/v/pio.svg?style=flat' alt='Gem Version' /></a>
 <a href='https://travis-ci.org/trema/pio'><img src='http://img.shields.io/travis/trema/pio/develop.svg?style=flat' alt='Build Status' /></a>
-<a href='https://codeclimate.com/github/trema/pio'><img src='http://img.shields.io/codeclimate/github/trema/pio.svg?style=flat' alt='Code Climate' /></a>
-<a href='https://coveralls.io/r/trema/pio?branch=develop'><img src='http://img.shields.io/coveralls/trema/pio/develop.svg?style=flat' alt='Coverage Status' /></a>
 <a href='https://gemnasium.com/trema/pio'><img src='http://img.shields.io/gemnasium/trema/pio.svg?style=flat' alt='Dependency Status' /></a>
-<a href='https://gitter.im/trema/pio'><img src='https://badges.gitter.im/Join Chat.svg?style=flat' alt='Gitter Chat' /></a>
-<a href="http://inch-pages.github.io/github/trema/pio"><img src="http://inch-pages.github.io/github/trema/pio.svg" alt="Inline docs"></a>
 
 Pio is a ruby gem to easily parse and generate network packets.
 

data/Rakefile

@@ -1,14 +1,16 @@
+# frozen_string_literal: true
+
 require 'bundler/gem_tasks'
 
-RELISH_PROJECT = 'trema/pio'.freeze
+RELISH_PROJECT = 'trema/pio'
 FLAY_THRESHOLD = 1446
 
 task default: :travis
-task test: [:spec, :cucumber]
-task travis: [:test, :quality, 'coveralls:push']
+task test: %i[spec cucumber]
+task travis: %i[test quality]
 
 desc 'Check for code quality'
-task quality: [:reek, :flog, :flay, :rubocop]
+task quality: %i[reek flog flay rubocop]
 
 Dir.glob('tasks/*.rake').each { |each| import each }
 

data/features/step_definitions/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 group :development, :test do

data/features/step_definitions/Guardfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 guard :rubocop do
   watch(/.+\.rb$/)
   watch(%r{(?:.+/)?\.rubocop\.yml$}) { |m| File.dirname(m[0]) }

data/features/step_definitions/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 task default: :rubocop
 
 begin

data/features/step_definitions/dump_flows_steps.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # rubocop:disable LineLength
 
 Then(/^the switch "(.*?)" has (\d+) flow entr(?:y|ies)$/) do |switch, num_entries|

data/features/step_definitions/open_flow_steps.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Given(/^I switch the Pio::OpenFlow version to "([^"]*)"$/) do |version|
   Pio::OpenFlow.version = version
 end

data/features/step_definitions/packet_data_steps.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 When(/^I parse a file named "(.*?\.raw)" with "(.*?)" class$/) do |path, klass|
   raw_data = IO.read(expand_path("%/#{path}"))
   parser_klass = Pio.const_get(klass)
@@ -31,7 +33,7 @@ end
 Then(/^the packet has the following fields and values:$/) do |table|
   table.hashes.each do |each|
     output = @result.instance_eval("self.#{each['field']}")
-    if /^:/ =~ output.inspect
+    if /^:/.match?(output.inspect)
       expect(output.inspect).to eq(each['value'])
     else
       expect(output.to_s).to eq(each['value'])

data/features/step_definitions/rest_api_steps.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'rack/test'
 

data/features/step_definitions/ruby_steps.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 When(/^I eval the following Ruby code:$/) do |ruby_code|
   @result = Pio.module_eval(ruby_code)
 end

data/features/step_definitions/show_stats_steps.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # rubocop:disable LineLength
 
 Then(/^the number of packets sent from "(.*?)" should be:$/) do |host_name, table|

data/features/step_definitions/trema_steps.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Given(/^I use OpenFlow 1\.0$/) do
   @open_flow_version = :open_flow10
 end

data/features/step_definitions/virtual_link_steps.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # rubocop:disable LineLength
 
 Then(/^virtual links should not exist$/) do

data/features/support/env.rb

@@ -1,5 +1,4 @@
-require 'coveralls'
-Coveralls.wear!
+# frozen_string_literal: true
 
 require 'aruba/cucumber'
 require 'pio'

data/features/support/hooks.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Before('@open_flow10') do
   Pio::OpenFlow.version = :OpenFlow10
 end

data/lib/pio.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/parse_error'
 
 require 'pio/arp'

data/lib/pio/arp.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/arp/format'
 require 'pio/arp/reply'
 require 'pio/arp/request'

data/lib/pio/arp/format.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bindata'
 require 'pio/ethernet_header'
 require 'pio/type/ip_address'

data/lib/pio/arp/message.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/arp/format'
 require 'pio/message'
 

data/lib/pio/arp/reply.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/arp/message'
 require 'pio/instance_inspector'
 require 'pio/mac'

data/lib/pio/arp/request.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/arp/message'
 require 'pio/instance_inspector'
 require 'pio/mac'
@@ -10,9 +12,9 @@ module Pio
 
       option :operation, value: 1
       option :source_mac
-      option :destination_mac, default: 'ff:ff:ff:ff:ff:ff'.freeze
+      option :destination_mac, default: 'ff:ff:ff:ff:ff:ff'
       option :sender_hardware_address, value: :source_mac
-      option :target_hardware_address, default: '00:00:00:00:00:00'.freeze
+      option :target_hardware_address, default: '00:00:00:00:00:00'
       option :sender_protocol_address
       option :target_protocol_address
     end

data/lib/pio/class_inspector.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_support/core_ext/string/inflections'
 
 module Pio

data/lib/pio/dhcp.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Packet parser and generator library.
 module Pio
   # Dhcp parser and generator.

data/lib/pio/dhcp/ack.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/dhcp/boot_reply'
 
 module Pio

data/lib/pio/dhcp/boot_reply.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/dhcp/message'
 require 'pio/dhcp/boot_reply_options'
 

data/lib/pio/dhcp/boot_reply_options.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/options'
 require 'pio/dhcp/common_options'
 require 'pio/dhcp/dhcp_tlv_options'

data/lib/pio/dhcp/boot_request.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/dhcp/message'
 require 'pio/dhcp/boot_request_options'
 

data/lib/pio/dhcp/boot_request_options.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/options'
 require 'pio/dhcp/common_options'
 require 'pio/dhcp/dhcp_tlv_options'

data/lib/pio/dhcp/client_id.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/type/mac_address'
 
 module Pio

data/lib/pio/dhcp/common_options.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Pio
   class Dhcp
     # DHCP Common Options.

data/lib/pio/dhcp/dhcp_field.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/dhcp/optional_tlv'
 require 'pio/type/ip_address'
 require 'pio/type/mac_address'

data/lib/pio/dhcp/dhcp_tlv_options.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/dhcp/common_options'
 
 module Pio
@@ -31,11 +33,13 @@ module Pio
       end
 
       def dhcp_server_identifier_hash
-        {
-          tlv_type: Dhcp::SERVER_IDENTIFIER_TLV,
-          tlv_info_length: 4,
-          tlv_value: server_identifier
-        } if server_identifier
+        if server_identifier
+          {
+            tlv_type: Dhcp::SERVER_IDENTIFIER_TLV,
+            tlv_info_length: 4,
+            tlv_value: server_identifier
+          }
+        end
       end
 
       def parameters_list_hash
@@ -47,19 +51,23 @@ module Pio
       end
 
       def renewal_time_value_hash
-        {
-          tlv_type: Dhcp::RENEWAL_TIME_VALUE_TLV,
-          tlv_info_length: 4,
-          tlv_value: renewal_time_value
-        } if renewal_time_value
+        if renewal_time_value
+          {
+            tlv_type: Dhcp::RENEWAL_TIME_VALUE_TLV,
+            tlv_info_length: 4,
+            tlv_value: renewal_time_value
+          }
+        end
       end
 
       def rebinding_time_value_hash
-        {
-          tlv_type: Dhcp::REBINDING_TIME_VALUE_TLV,
-          tlv_info_length: 4,
-          tlv_value: rebinding_time_value
-        } if rebinding_time_value
+        if rebinding_time_value
+          {
+            tlv_type: Dhcp::REBINDING_TIME_VALUE_TLV,
+            tlv_info_length: 4,
+            tlv_value: rebinding_time_value
+          }
+        end
       end
 
       def ip_address_lease_time_hash
@@ -71,11 +79,13 @@ module Pio
       end
 
       def subnet_mask_hash
-        {
-          tlv_type: Dhcp::SUBNET_MASK_TLV,
-          tlv_info_length: 4,
-          tlv_value: subnet_mask
-        } if subnet_mask
+        if subnet_mask
+          {
+            tlv_type: Dhcp::SUBNET_MASK_TLV,
+            tlv_info_length: 4,
+            tlv_value: subnet_mask
+          }
+        end
       end
     end
   end

data/lib/pio/dhcp/discover.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/dhcp/boot_request'
 
 module Pio

data/lib/pio/dhcp/field_util.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Pio
   class Dhcp
     # Dhcp Field Read Methods.
@@ -93,7 +95,7 @@ module Pio
 
       def get_tlv_field(tlv_type)
         tlv = get_tlv(tlv_type)
-        tlv.snapshot if tlv
+        tlv&.snapshot
       end
     end
   end

data/lib/pio/dhcp/frame.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/dhcp/dhcp_field'
 require 'pio/dhcp/field_util'
 require 'pio/ipv4_header'

data/lib/pio/dhcp/message.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/dhcp/frame'
 require 'forwardable'
 

data/lib/pio/dhcp/offer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/dhcp/boot_reply'
 
 module Pio

data/lib/pio/dhcp/optional_tlv.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/dhcp/parameter_list'
 require 'pio/dhcp/client_id'
 require 'pio/type/ip_address'
@@ -7,7 +9,7 @@ module Pio
   class Dhcp
     # DHCP Optional TLV
     class OptionalTlv < BinData::Record
-      DEFAULT = 'default'.freeze
+      DEFAULT = 'default'
 
       endian :big
 

data/lib/pio/dhcp/parameter_list.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bindata'
 
 module Pio

data/lib/pio/dhcp/request.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/dhcp/boot_request'
 
 module Pio

data/lib/pio/ethernet_frame.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bindata'
 require 'pio/ethernet_header'
 require 'pio/class_inspector'

data/lib/pio/ethernet_header.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/class_inspector'
 require 'pio/instance_inspector'
 require 'pio/ruby_dumper'

data/lib/pio/icmp.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/icmp/format'
 require 'pio/icmp/reply'
 require 'pio/icmp/request'

data/lib/pio/icmp/format.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pio/ethernet_header'
 require 'pio/ipv4_header'
 require 'pio/monkey_patch/bindata_record'