trema 0.4.6 → 0.4.7

data/spec/trema/barrier-reply_spec.rb

@@ -16,22 +16,22 @@
 #
 
 
-require File.join( File.dirname( __FILE__ ), "..", "spec_helper" )
-require "trema"
+require File.join(File.dirname(__FILE__), '..', 'spec_helper')
+require 'trema'
 
 
-describe BarrierReply, ".new( VALID OPTIONS )" do
-  subject { BarrierReply.new( 0xabc, 1234 ) }
-  its ( :datapath_id ) { should == 0xabc }
-  its( :transaction_id ) { should == 1234 }
+describe BarrierReply, '.new( VALID OPTIONS )' do
+  subject { BarrierReply.new(0xabc, 1234) }
+  its ( :datapath_id) { should == 0xabc }
+  its(:transaction_id) { should == 1234 }
 end
 
 
-describe BarrierReply, ".new( MANDATORY OPTIONS MISSING )" do
-  it "should raise ArgumentError" do
-    expect {
+describe BarrierReply, '.new( MANDATORY OPTIONS MISSING )' do
+  it 'should raise ArgumentError' do
+    expect do
       BarrierReply.new
-    }.to raise_error( ArgumentError )
+    end.to raise_error(ArgumentError)
   end
 end
 

data/spec/trema/barrier-request_spec.rb

@@ -16,71 +16,71 @@
 #
 
 
-require File.join( File.dirname( __FILE__ ), "..", "spec_helper" )
-require "trema"
+require File.join(File.dirname(__FILE__), '..', 'spec_helper')
+require 'trema'
 
 
-shared_examples_for "barrier request message" do
+shared_examples_for 'barrier request message' do
   class BarrierRequestController < Controller; end
 
   it "should be logged to the switch's log", :sudo => true do
-    network {
-      vswitch( "barrier-request" ) { datapath_id 0xabc }
-    }.run( BarrierRequestController ) {
-      controller( "BarrierRequestController" ).send_message( 0xabc, subject )
+    network do
+      vswitch('barrier-request') { datapath_id 0xabc }
+    end.run(BarrierRequestController) do
+      controller('BarrierRequestController').send_message(0xabc, subject)
       sleep 2 # FIXME: wait to send_message
-      expect( IO.read( File.join( Trema.log, "openflowd.barrier-request.log" ) ) ).to include( "OFPT_BARRIER_REQUEST" )
-    }
+      expect(IO.read(File.join(Trema.log, 'openflowd.barrier-request.log'))).to include('OFPT_BARRIER_REQUEST')
+    end
   end
 end
 
 
 module Trema
-  describe BarrierRequest, ".new" do
-    it_should_behave_like "any Openflow message with default transaction ID"
-    it_should_behave_like "barrier request message"
+  describe BarrierRequest, '.new' do
+    it_should_behave_like 'any Openflow message with default transaction ID'
+    it_should_behave_like 'barrier request message'
   end
 
 
-  describe BarrierRequest, ".new(nil)" do
-    subject { BarrierRequest.new( nil ) }
-    it_should_behave_like "any Openflow message with default transaction ID"
-    it_should_behave_like "barrier request message"
+  describe BarrierRequest, '.new(nil)' do
+    subject { BarrierRequest.new(nil) }
+    it_should_behave_like 'any Openflow message with default transaction ID'
+    it_should_behave_like 'barrier request message'
   end
 
 
-  describe BarrierRequest, ".new(transaction_id)" do
-    subject { BarrierRequest.new( transaction_id ) }
-    it_should_behave_like "any Openflow message with transaction ID"
-    context "when sent to a switch" do
-      let( :transaction_id ) { 123 }
-      it_should_behave_like "barrier request message"
+  describe BarrierRequest, '.new(transaction_id)' do
+    subject { BarrierRequest.new(transaction_id) }
+    it_should_behave_like 'any Openflow message with transaction ID'
+    context 'when sent to a switch' do
+      let(:transaction_id) { 123 }
+      it_should_behave_like 'barrier request message'
     end
   end
 
 
-  describe BarrierRequest, ".new(:transaction_id => value)" do
-    subject { BarrierRequest.new( :transaction_id => transaction_id ) }
-    it_should_behave_like "any Openflow message with transaction ID"
-    context "when sent to a switch" do
-      let( :transaction_id ) { 123 }
-      it_should_behave_like "barrier request message"
+  describe BarrierRequest, '.new(:transaction_id => value)' do
+    subject { BarrierRequest.new(:transaction_id => transaction_id) }
+    it_should_behave_like 'any Openflow message with transaction ID'
+    context 'when sent to a switch' do
+      let(:transaction_id) { 123 }
+      it_should_behave_like 'barrier request message'
     end
   end
 
 
-  describe BarrierRequest, ".new(:xid => value)" do
-    subject { BarrierRequest.new( :xid => xid ) }
-    it_should_behave_like "any Openflow message with xid"
-    context "when sent to a switch" do
-      let( :xid ) { 123 }
-      it_should_behave_like "barrier request message"
+  describe BarrierRequest, '.new(:xid => value)' do
+    subject { BarrierRequest.new(:xid => xid) }
+    it_should_behave_like 'any Openflow message with xid'
+    context 'when sent to a switch' do
+      let(:xid) { 123 }
+      it_should_behave_like 'barrier request message'
     end
   end
 
 
   describe BarrierRequest, '.new("INVALID OPTION")', :nosudo => true do
-    it { expect { BarrierRequest.new "INVALID OPTION" }.to raise_error( TypeError ) }
+    it { expect { BarrierRequest.new 'INVALID OPTION' }.to raise_error(TypeError) }
   end
 end
 

data/spec/trema/cli_spec.rb

@@ -16,132 +16,132 @@
 #
 
 
-require File.join( File.dirname( __FILE__ ), "..", "spec_helper" )
-require "trema/cli"
+require File.join(File.dirname(__FILE__), '..', 'spec_helper')
+require 'trema/cli'
 
 
 module Trema
   describe Cli do
     before :each do
-      @source = mock( "source", :interface => "trema-0", :ip => "192.168.0.1" )
-      @dest = mock( "dest", :interface => "trema-1", :ip => "192.168.0.2" )
-      @cli = Cli.new( @source )
+      @source = mock('source', :interface => 'trema-0', :ip => '192.168.0.1')
+      @dest = mock('dest', :interface => 'trema-1', :ip => '192.168.0.2')
+      @cli = Cli.new(@source)
     end
 
 
-    context "when sending packets" do
-      it "should send packets (default)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22$/ )
-        @cli.send_packets( @dest )
+    context 'when sending packets' do
+      it 'should send packets (default)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22$/)
+        @cli.send_packets(@dest)
       end
 
 
-      it "should send packets (inc_ip_src)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_ip_src$/ )
-        @cli.send_packets( @dest, :inc_ip_src => true )
+      it 'should send packets (inc_ip_src)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_ip_src$/)
+        @cli.send_packets(@dest, :inc_ip_src => true)
       end
 
 
-      it "should send packets (inc_ip_src = 1)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_ip_src=1$/ )
-        @cli.send_packets( @dest, :inc_ip_src => 1 )
+      it 'should send packets (inc_ip_src = 1)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_ip_src=1$/)
+        @cli.send_packets(@dest, :inc_ip_src => 1)
       end
 
 
-      it "should send packets (inc_ip_dst)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_ip_dst$/ )
-        @cli.send_packets( @dest, :inc_ip_dst => true )
+      it 'should send packets (inc_ip_dst)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_ip_dst$/)
+        @cli.send_packets(@dest, :inc_ip_dst => true)
       end
 
 
-      it "should send packets (inc_ip_dst = 1)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_ip_dst=1$/ )
-        @cli.send_packets( @dest, :inc_ip_dst => 1 )
+      it 'should send packets (inc_ip_dst = 1)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_ip_dst=1$/)
+        @cli.send_packets(@dest, :inc_ip_dst => 1)
       end
 
 
-      it "should send_packets (tp_src = 60000)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 60000 --tp_dst 1 --pps 1 --duration 1 --length 22$/ )
-        @cli.send_packets( @dest, :tp_src => 60000 )
+      it 'should send_packets (tp_src = 60000)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 60000 --tp_dst 1 --pps 1 --duration 1 --length 22$/)
+        @cli.send_packets(@dest, :tp_src => 60000)
       end
 
 
-      it "should send_packets (tp_dst = 10000)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 10000 --pps 1 --duration 1 --length 22$/ )
-        @cli.send_packets( @dest, :tp_dst => 10000 )
+      it 'should send_packets (tp_dst = 10000)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 10000 --pps 1 --duration 1 --length 22$/)
+        @cli.send_packets(@dest, :tp_dst => 10000)
       end
 
 
-      it "should send packets (inc_tp_src)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_tp_src$/ )
-        @cli.send_packets( @dest, :inc_tp_src => true )
+      it 'should send packets (inc_tp_src)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_tp_src$/)
+        @cli.send_packets(@dest, :inc_tp_src => true)
       end
 
 
-      it "should send packets (inc_tp_src = 1)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_tp_src=1$/ )
-        @cli.send_packets( @dest, :inc_tp_src => 1 )
+      it 'should send packets (inc_tp_src = 1)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_tp_src=1$/)
+        @cli.send_packets(@dest, :inc_tp_src => 1)
       end
 
 
-      it "should send_packets (pps = 100)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 100 --duration 1 --length 22$/ )
-        @cli.send_packets( @dest, :pps => 100 )
+      it 'should send_packets (pps = 100)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 100 --duration 1 --length 22$/)
+        @cli.send_packets(@dest, :pps => 100)
       end
 
 
-      it "should send packets (duration = 10)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 10 --length 22$/ )
-        @cli.send_packets( @dest, :duration => 10 )
+      it 'should send packets (duration = 10)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 10 --length 22$/)
+        @cli.send_packets(@dest, :duration => 10)
       end
 
 
-      it "should send packets (length = 1000)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 1000$/ )
-        @cli.send_packets( @dest, :length => 1000 )
+      it 'should send packets (length = 1000)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 1000$/)
+        @cli.send_packets(@dest, :length => 1000)
       end
 
 
-      it "should send_packets (inc_tp_dst)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_tp_dst$/ )
-        @cli.send_packets( @dest, :inc_tp_dst => true )
+      it 'should send_packets (inc_tp_dst)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_tp_dst$/)
+        @cli.send_packets(@dest, :inc_tp_dst => true)
       end
 
 
-      it "should send_packets (inc_tp_dst = 65534)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_tp_dst=65534$/ )
-        @cli.send_packets( @dest, :inc_tp_dst => 65534 )
+      it 'should send_packets (inc_tp_dst = 65534)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_tp_dst=65534$/)
+        @cli.send_packets(@dest, :inc_tp_dst => 65534)
       end
 
 
-      it "should send_packets (inc_payload)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_payload$/ )
-        @cli.send_packets( @dest, :inc_payload => true )
+      it 'should send_packets (inc_payload)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_payload$/)
+        @cli.send_packets(@dest, :inc_payload => true)
       end
 
 
-      it "should send_packets (inc_payload = 1000)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_payload=1000$/ )
-        @cli.send_packets( @dest, :inc_payload => 1000 )
+      it 'should send_packets (inc_payload = 1000)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --duration 1 --length 22 --inc_payload=1000$/)
+        @cli.send_packets(@dest, :inc_payload => 1000)
       end
 
 
-      it "should send_packets (n_pkts = 10)" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --length 22 --n_pkts=10$/ )
-        @cli.send_packets( @dest, :n_pkts => 10 )
+      it 'should send_packets (n_pkts = 10)' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 send_packets --ip_src 192.168.0.1 --ip_dst 192.168.0.2 --tp_src 1 --tp_dst 1 --pps 1 --length 22 --n_pkts=10$/)
+        @cli.send_packets(@dest, :n_pkts => 10)
       end
 
 
-      it "should raise if both --duration and --n_pkts are specified" do
-        expect { @cli.send_packets( @dest, :duration => 10, :n_pkts => 10 ) }.to raise_error( "--duration and --n_pkts are exclusive." )
+      it 'should raise if both --duration and --n_pkts are specified' do
+        expect { @cli.send_packets(@dest, :duration => 10, :n_pkts => 10) }.to raise_error('--duration and --n_pkts are exclusive.')
       end
     end
 
 
-    context "when resetting stats" do
-      it "should reset_stats" do
-        @cli.should_receive( :sh ).with( /cli -i trema-0 reset_stats --tx$/ )
-        @cli.should_receive( :sh ).with( /cli -i trema-0 reset_stats --rx$/ )
+    context 'when resetting stats' do
+      it 'should reset_stats' do
+        @cli.should_receive(:sh).with(/cli -i trema-0 reset_stats --tx$/)
+        @cli.should_receive(:sh).with(/cli -i trema-0 reset_stats --rx$/)
         @cli.reset_stats
       end
     end

data/spec/trema/controller_spec.rb

@@ -16,8 +16,8 @@
 #
 
 
-require File.join( File.dirname( __FILE__ ), "..", "spec_helper" )
-require "trema"
+require File.join(File.dirname(__FILE__), '..', 'spec_helper')
+require 'trema'
 
 
 module Trema
@@ -35,7 +35,7 @@ module Trema
     it { should have_constant :OFPP_NONE }
 
 
-    context ".new" do
+    context '.new' do
       subject { Controller.new }
 
       it { should respond_to :critical }

data/spec/trema/default-logger_spec.rb

@@ -16,18 +16,18 @@
 #
 
 
-require File.join( File.dirname( __FILE__ ), "..", "spec_helper" )
-require "trema"
+require File.join(File.dirname(__FILE__), '..', 'spec_helper')
+require 'trema'
 
 
 module Trema
   describe DefaultLogger do
-    subject {
+    subject do
       class LoggingObject
         include Trema::DefaultLogger
       end
       LoggingObject.new
-    }
+    end
 
     it { should respond_to :critical }
     it { should respond_to :error }

data/spec/trema/dsl/configuration_spec.rb

@@ -16,8 +16,8 @@
 #
 
 
-require File.join( File.dirname( __FILE__ ), "..", "..", "spec_helper" )
-require "trema/dsl/configuration"
+require File.join(File.dirname(__FILE__), '..', '..', 'spec_helper')
+require 'trema/dsl/configuration'
 
 
 module Trema
@@ -28,84 +28,84 @@ module Trema
       end
 
 
-      context "when parsing trema configurations" do
-        it "should remember apps" do
-          expect( @context ).to have( 0 ).apps
+      context 'when parsing trema configurations' do
+        it 'should remember apps' do
+          expect(@context).to have(0).apps
 
-          Trema::App.add mock( "app #0", :name => "app #0" )
-          Trema::App.add mock( "app #1", :name => "app #1" )
-          Trema::App.add mock( "app #2", :name => "app #2" )
+          Trema::App.add mock('app #0', :name => 'app #0')
+          Trema::App.add mock('app #1', :name => 'app #1')
+          Trema::App.add mock('app #2', :name => 'app #2')
 
-          expect( @context ).to have( 3 ).apps
+          expect(@context).to have(3).apps
 
-          expect( @context.apps[ "app #0" ].name ).to eq( "app #0" )
-          expect( @context.apps[ "app #1" ].name ).to eq( "app #1" )
-          expect( @context.apps[ "app #2" ].name ).to eq( "app #2" )
+          expect(@context.apps[ 'app #0'].name).to eq('app #0')
+          expect(@context.apps[ 'app #1'].name).to eq('app #1')
+          expect(@context.apps[ 'app #2'].name).to eq('app #2')
         end
 
 
-        it "should remember hosts" do
-          expect( @context ).to have( 0 ).hosts
+        it 'should remember hosts' do
+          expect(@context).to have(0).hosts
 
-          Trema::Host.add mock( "host #0", :name => "host #0" )
-          Trema::Host.add mock( "host #1", :name => "host #1" )
-          Trema::Host.add mock( "host #2", :name => "host #2" )
+          Trema::Host.add mock('host #0', :name => 'host #0')
+          Trema::Host.add mock('host #1', :name => 'host #1')
+          Trema::Host.add mock('host #2', :name => 'host #2')
 
-          expect( @context ).to have( 3 ).hosts
+          expect(@context).to have(3).hosts
 
-          expect( @context.hosts[ "host #0" ].name ).to eq( "host #0" )
-          expect( @context.hosts[ "host #1" ].name ).to eq( "host #1" )
-          expect( @context.hosts[ "host #2" ].name ).to eq( "host #2" )
+          expect(@context.hosts[ 'host #0'].name).to eq('host #0')
+          expect(@context.hosts[ 'host #1'].name).to eq('host #1')
+          expect(@context.hosts[ 'host #2'].name).to eq('host #2')
         end
 
 
-        it "should remember links" do
-          expect( @context ).to have( 0 ).links
+        it 'should remember links' do
+          expect(@context).to have(0).links
 
-          Trema::Link.add mock( "link #0", :name => "link #0" )
-          Trema::Link.add mock( "link #1", :name => "link #1" )
-          Trema::Link.add mock( "link #2", :name => "link #2" )
+          Trema::Link.add mock('link #0', :name => 'link #0')
+          Trema::Link.add mock('link #1', :name => 'link #1')
+          Trema::Link.add mock('link #2', :name => 'link #2')
 
-          expect( @context ).to have( 3 ).links
+          expect(@context).to have(3).links
 
-          expect( @context.links[ "link #0" ].name ).to eq( "link #0" )
-          expect( @context.links[ "link #1" ].name ).to eq( "link #1" )
-          expect( @context.links[ "link #2" ].name ).to eq( "link #2" )
+          expect(@context.links[ 'link #0'].name).to eq('link #0')
+          expect(@context.links[ 'link #1'].name).to eq('link #1')
+          expect(@context.links[ 'link #2'].name).to eq('link #2')
         end
 
 
-        it "should remember filter settings" do
-          expect( @context.packetin_filter ).to be_nil
+        it 'should remember filter settings' do
+          expect(@context.packetin_filter).to be_nil
 
-          packetin_filter = mock( "filter", :name => "filter" )
+          packetin_filter = mock('filter', :name => 'filter')
           Trema::PacketinFilter.add packetin_filter
 
-          expect( @context.packetin_filter ).to eq( packetin_filter )
+          expect(@context.packetin_filter).to eq(packetin_filter)
         end
 
 
-        it "should remember switch manager" do
-          expect( @context.switch_manager ).to be_nil
+        it 'should remember switch manager' do
+          expect(@context.switch_manager).to be_nil
 
-          switch_manager = mock( "switch manager", :name => "switch manager" )
+          switch_manager = mock('switch manager', :name => 'switch manager')
           Trema::SwitchManager.add switch_manager
 
-          expect( @context.switch_manager ).to eq( switch_manager )
+          expect(@context.switch_manager).to eq(switch_manager)
         end
 
 
-        it "should remember switches" do
-          expect( @context ).to have( 0 ).switches
+        it 'should remember switches' do
+          expect(@context).to have(0).switches
 
-          Trema::OpenflowSwitch.add mock( "switch #0", :name => "switch #0" )
-          Trema::OpenflowSwitch.add mock( "switch #1", :name => "switch #1" )
-          Trema::OpenflowSwitch.add mock( "switch #2", :name => "switch #2" )
+          Trema::OpenflowSwitch.add mock('switch #0', :name => 'switch #0')
+          Trema::OpenflowSwitch.add mock('switch #1', :name => 'switch #1')
+          Trema::OpenflowSwitch.add mock('switch #2', :name => 'switch #2')
 
-          expect( @context ).to have( 3 ).switches
+          expect(@context).to have(3).switches
 
-          expect( @context.switches[ "switch #0" ].name ).to eq( "switch #0" )
-          expect( @context.switches[ "switch #1" ].name ).to eq( "switch #1" )
-          expect( @context.switches[ "switch #2" ].name ).to eq( "switch #2" )
+          expect(@context.switches[ 'switch #0'].name).to eq('switch #0')
+          expect(@context.switches[ 'switch #1'].name).to eq('switch #1')
+          expect(@context.switches[ 'switch #2'].name).to eq('switch #2')
         end
       end
     end