rspec 1.0.5 → 1.0.6

data/spec/spec/matchers/raise_error_spec.rb

@@ -24,6 +24,44 @@ describe "should_not raise_error" do
   end
 end
 
+describe "should raise_error(message)" do
+  it "should pass if RuntimeError is raised with the right message" do
+    lambda {raise 'blah'}.should raise_error('blah')
+  end
+  it "should pass if any other error is raised with the right message" do
+    lambda {raise NameError.new('blah')}.should raise_error('blah')
+  end
+  it "should fail if RuntimeError error is raised with the wrong message" do
+    lambda do
+      lambda {raise 'blarg'}.should raise_error('blah')
+    end.should fail_with("expected Exception with \"blah\", got #<RuntimeError: blarg>")
+  end
+  it "should fail if any other error is raised with the wrong message" do
+    lambda do
+      lambda {raise NameError.new('blarg')}.should raise_error('blah')
+    end.should fail_with("expected Exception with \"blah\", got #<NameError: blarg>")
+  end
+end
+
+describe "should_not raise_error(message)" do
+  it "should pass if RuntimeError error is raised with the different message" do
+    lambda {raise 'blarg'}.should_not raise_error('blah')
+  end
+  it "should pass if any other error is raised with the wrong message" do
+    lambda {raise NameError.new('blarg')}.should_not raise_error('blah')
+  end
+  it "should fail if RuntimeError is raised with message" do
+    lambda do
+      lambda {raise 'blah'}.should_not raise_error('blah')
+    end.should fail_with(%Q|expected no Exception with "blah", got #<RuntimeError: blah>|)
+  end
+  it "should fail if any other error is raised with message" do
+    lambda do
+      lambda {raise NameError.new('blah')}.should_not raise_error('blah')
+    end.should fail_with(%Q|expected no Exception with "blah", got #<NameError: blah>|)
+  end
+end
+
 describe "should raise_error(NamedError)" do
   it "should pass if named error is raised" do
     lambda { non_existent_method }.should raise_error(NameError)

data/spec/spec/mocks/argument_expectation_spec.rb

@@ -1,19 +1,23 @@
 require File.dirname(__FILE__) + '/../../spec_helper.rb'
 
-describe "An ArgumentExpectation" do
-  it "should consider an object that responds to #matches? and #description to be a matcher" do
-    argument_expecatation = Spec::Mocks::ArgumentExpectation.new([])
-    obj = mock("matcher")
-    obj.should_receive(:respond_to?).with(:matches?).and_return(true)
-    obj.should_receive(:respond_to?).with(:description).and_return(true)
-    argument_expecatation.is_matcher?(obj).should be_true
-  end
+module Spec
+  module Mocks
+    describe ArgumentExpectation do
+      it "should consider an object that responds to #matches? and #description to be a matcher" do
+        argument_expecatation = Spec::Mocks::ArgumentExpectation.new([])
+        obj = mock("matcher")
+        obj.should_receive(:respond_to?).with(:matches?).and_return(true)
+        obj.should_receive(:respond_to?).with(:description).and_return(true)
+        argument_expecatation.is_matcher?(obj).should be_true
+      end
 
-  it "should NOT consider an object that only responds to #matches? to be a matcher" do
-    argument_expecatation = Spec::Mocks::ArgumentExpectation.new([])
-    obj = mock("matcher")
-    obj.should_receive(:respond_to?).with(:matches?).and_return(true)
-    obj.should_receive(:respond_to?).with(:description).and_return(false)
-    argument_expecatation.is_matcher?(obj).should be_false
+      it "should NOT consider an object that only responds to #matches? to be a matcher" do
+        argument_expecatation = Spec::Mocks::ArgumentExpectation.new([])
+        obj = mock("matcher")
+        obj.should_receive(:respond_to?).with(:matches?).and_return(true)
+        obj.should_receive(:respond_to?).with(:description).and_return(false)
+        argument_expecatation.is_matcher?(obj).should be_false
+      end
+    end
   end
 end

data/spec/spec/mocks/at_most_spec.rb

@@ -2,7 +2,7 @@ require File.dirname(__FILE__) + '/../../spec_helper.rb'
 
 module Spec
   module Mocks
-    describe "AtMost" do
+    describe "at_most" do
       before(:each) do
         @mock = Mock.new("test mock")
       end

data/spec/spec/mocks/bug_report_11545_spec.rb

@@ -0,0 +1,31 @@
+require File.dirname(__FILE__) + '/../../spec_helper.rb'
+
+class LiarLiarPantsOnFire
+  def respond_to?(sym)
+    true
+  end
+  
+  def self.respond_to?(sym)
+    true
+  end
+end
+  
+describe 'should_receive' do
+  before(:each) do
+    @liar = LiarLiarPantsOnFire.new
+  end
+  
+  it "should work when object lies about responding to a method" do
+    @liar.should_receive(:something)
+    @liar.something
+  end
+
+  it 'should work when class lies about responding to a method' do
+    LiarLiarPantsOnFire.should_receive(:something)
+    LiarLiarPantsOnFire.something
+  end
+  
+  it 'should cleanup after itself' do
+    LiarLiarPantsOnFire.metaclass.instance_methods.should_not include("something")
+  end
+end

data/spec/spec/mocks/partial_mock_spec.rb

@@ -6,7 +6,20 @@ module Spec
       before(:each) do
         @object = Object.new
       end
-      
+
+      it "should name the class in the failure message" do
+        @object.should_receive(:foo)
+        lambda do
+          @object.rspec_verify
+        end.should raise_error(Spec::Mocks::MockExpectationError, /Object/)
+      end
+
+      it "should not conflict with @options in the object" do
+        @object.instance_eval { @options = Object.new }
+        @object.should_receive(:blah)
+        @object.blah
+      end
+            
       it "should_not_receive should mock out the method" do
         @object.should_not_receive(:fuhbar)
         @object.fuhbar
@@ -47,6 +60,25 @@ module Spec
         end.should raise_error(Spec::Mocks::MockExpectationError)
       end
 
+      it "should_receive should also take a String argument" do
+        @object.should_receive('foobar')
+        @object.foobar
+      end
+      
+      it "should_not_receive should also take a String argument" do
+        @object.should_not_receive('foobar')
+        @object.foobar
+        lambda do
+          @object.rspec_verify
+        end.should raise_error(Spec::Mocks::MockExpectationError)
+      end
+      
+      it "should use report nil in the error message" do
+        @this_will_resolve_to_nil.should_receive(:foobar)
+        lambda do
+          @this_will_resolve_to_nil.rspec_verify
+        end.should raise_error(Spec::Mocks::MockExpectationError, /NilClass.*expected :foobar with/)
+      end
     end
   end
 end

data/spec/spec/runner/behaviour_runner_spec.rb

@@ -4,7 +4,9 @@ module Spec
   module Runner
     describe BehaviourRunner, "#add_behaviour affecting passed in behaviour" do
       before do
-        @options = Options.new
+        @err = StringIO.new('')
+        @out = StringIO.new('')
+        @options = Options.new(@err,@out)
         @runner = BehaviourRunner.new(@options)
         class << @runner
           attr_reader :behaviours
@@ -49,7 +51,9 @@ module Spec
 
     describe BehaviourRunner, "#add_behaviour affecting behaviours" do
       before do
-        @options = Options.new
+        @err = StringIO.new('')
+        @out = StringIO.new('')
+        @options = Options.new(@err,@out)
         @runner = BehaviourRunner.new(@options)
         class << @runner
           attr_reader :behaviours
@@ -91,27 +95,32 @@ module Spec
     end
 
     describe BehaviourRunner do
+      before do
+        @err = StringIO.new('')
+        @out = StringIO.new('')
+        @options = Options.new(@err,@out)
+      end
 
       it "should only run behaviours with at least one example" do
-        desired_context = mock("desired context")
-        desired_context.should_receive(:run)
-        desired_context.should_receive(:retain_examples_matching!)
-        desired_context.should_receive(:number_of_examples).twice.and_return(1)
-        desired_context.should_receive(:shared?).and_return(false)
-
-        other_context = mock("other context")
-        other_context.should_receive(:run).never
-        other_context.should_receive(:retain_examples_matching!)
-        other_context.should_receive(:number_of_examples).and_return(0)
+        desired_behaviour = mock("desired behaviour")
+        desired_behaviour.should_receive(:run)
+        desired_behaviour.should_receive(:retain_examples_matching!)
+        desired_behaviour.should_receive(:number_of_examples).twice.and_return(1)
+        desired_behaviour.should_receive(:shared?).and_return(false)
+        desired_behaviour.should_receive(:set_sequence_numbers).with(0, anything)
+
+        other_behaviour = mock("other behaviour")
+        other_behaviour.should_receive(:run).never
+        other_behaviour.should_receive(:retain_examples_matching!)
+        other_behaviour.should_receive(:number_of_examples).and_return(0)
 
         reporter = mock("reporter")
-        options = Options.new
-        options.reporter = reporter
-        options.examples = ["desired context legal spec"]
+        @options.reporter = reporter
+        @options.examples = ["desired behaviour legal spec"]
 
-        runner = Spec::Runner::BehaviourRunner.new(options)
-        runner.add_behaviour(desired_context)
-        runner.add_behaviour(other_context)
+        runner = Spec::Runner::BehaviourRunner.new(@options)
+        runner.add_behaviour(desired_behaviour)
+        runner.add_behaviour(other_behaviour)
         reporter.should_receive(:start)
         reporter.should_receive(:end)
         reporter.should_receive(:dump)
@@ -119,7 +128,7 @@ module Spec
       end
 
       it "should dump even if Interrupt exception is occurred" do
-        behaviour = Spec::DSL::Behaviour.new("context") do
+        behaviour = Spec::DSL::Behaviour.new("behaviour") do
           it "no error" do
           end
 
@@ -130,7 +139,7 @@ module Spec
         
         reporter = mock("reporter")
         reporter.should_receive(:start)
-        reporter.should_receive(:add_behaviour).with("context")
+        reporter.should_receive(:add_behaviour)
         reporter.should_receive(:example_started).twice
         reporter.should_receive(:example_finished).twice
         reporter.should_receive(:rspec_verify)
@@ -138,18 +147,17 @@ module Spec
         reporter.should_receive(:end)
         reporter.should_receive(:dump)
 
-        options = Options.new
-        options.reporter = reporter
-        runner = Spec::Runner::BehaviourRunner.new(options)
+        @options.reporter = reporter
+        runner = Spec::Runner::BehaviourRunner.new(@options)
         runner.add_behaviour(behaviour)
         runner.run([], false)
       end
 
       it "should heckle when options have heckle_runner" do
-        context = mock("context", :null_object => true)
-        context.should_receive(:number_of_examples).twice.and_return(1)
-        context.should_receive(:run).and_return(0)
-        context.should_receive(:shared?).and_return(false)
+        behaviour = mock("behaviour", :null_object => true)
+        behaviour.should_receive(:number_of_examples).twice.and_return(1)
+        behaviour.should_receive(:run).and_return(0)
+        behaviour.should_receive(:shared?).and_return(false)
 
         reporter = mock("reporter")
         reporter.should_receive(:start).with(1)
@@ -159,39 +167,39 @@ module Spec
         heckle_runner = mock("heckle_runner")
         heckle_runner.should_receive(:heckle_with)
 
-        options = Options.new
-        options.reporter = reporter
-        options.heckle_runner = heckle_runner
+        @options.reporter = reporter
+        @options.heckle_runner = heckle_runner
 
-        runner = Spec::Runner::BehaviourRunner.new(options)
-        runner.add_behaviour(context)
+        runner = Spec::Runner::BehaviourRunner.new(@options)
+        runner.add_behaviour(behaviour)
         runner.run([], false)
       end
 
-      it "should run specs backward if options.reverse is true" do
-        options = Options.new
-        options.reverse = true
+      it "should run examples backwards if options.reverse is true" do
+        @options.reverse = true
 
         reporter = mock("reporter")
         reporter.should_receive(:start).with(3)
         reporter.should_receive(:end)
         reporter.should_receive(:dump).and_return(0)
-        options.reporter = reporter
-
-        runner = Spec::Runner::BehaviourRunner.new(options)
-        c1 = mock("c1")
-        c1.should_receive(:number_of_examples).twice.and_return(1)
-        c1.should_receive(:shared?).and_return(false)
-
-        c2 = mock("c2")
-        c2.should_receive(:number_of_examples).twice.and_return(2)
-        c2.should_receive(:shared?).and_return(false)
-        c2.should_receive(:run) do
-          c1.should_receive(:run)
+        @options.reporter = reporter
+
+        runner = Spec::Runner::BehaviourRunner.new(@options)
+        b1 = mock("b1")
+        b1.should_receive(:number_of_examples).twice.and_return(1)
+        b1.should_receive(:shared?).and_return(false)
+        b1.should_receive(:set_sequence_numbers).with(12, true).and_return(18)
+
+        b2 = mock("b2")
+        b2.should_receive(:number_of_examples).twice.and_return(2)
+        b2.should_receive(:shared?).and_return(false)
+        b2.should_receive(:set_sequence_numbers).with(0, true).and_return(12)
+        b2.should_receive(:run) do
+          b1.should_receive(:run)
         end
 
-        runner.add_behaviour(c1)
-        runner.add_behaviour(c2)
+        runner.add_behaviour(b1)
+        runner.add_behaviour(b2)
     
         runner.run([], false)
       end
@@ -201,6 +209,21 @@ module Spec
           config.should equal(Spec::Runner.configuration)
         end
       end
+
+      it "should pass its Description to the reporter" do
+        behaviour = Spec::DSL::Behaviour.new("behaviour") do
+          it "should" do
+          end
+        end
+        
+        reporter = mock("reporter", :null_object => true)
+        reporter.should_receive(:add_behaviour).with(an_instance_of(Spec::DSL::Description))
+
+        @options.reporter = reporter
+        runner = Spec::Runner::BehaviourRunner.new(@options)
+        runner.add_behaviour(behaviour)
+        runner.run([], false)
+      end
     end
   end
 end

data/spec/spec/runner/command_line_spec.rb

@@ -8,7 +8,7 @@ describe "CommandLine" do
     Spec::Runner::CommandLine.run([file], err, out, false, true)
     
     out.rewind
-    out.read.should =~ /77 examples, 0 failures/n
+    out.read.should =~ /78 examples, 0 failures, 3 pending/n
   end
 
   it "should run file" do

data/spec/spec/runner/context_matching_spec.rb

@@ -2,24 +2,24 @@ require File.dirname(__FILE__) + '/../../spec_helper.rb'
 
 module Spec
   module DSL
-    describe "ContextMatching" do
+    describe Behaviour do
 
       before(:each) do
         @formatter = Spec::Mocks::Mock.new("formatter")
-        @behaviour = Behaviour.new("context") {}
+        @behaviour = Behaviour.new("behaviour") {}
       end
 
-      it "run all specs when spec is not specified" do
-        @behaviour.it("spec1") {}
-        @behaviour.it("spec2") {}
-        @behaviour.retain_examples_matching!(["context"])
+      it "should retain examples that don't match" do
+        @behaviour.it("example1") {}
+        @behaviour.it("example2") {}
+        @behaviour.retain_examples_matching!(["behaviour"])
         @behaviour.number_of_examples.should == 2
       end
 
-      it "should only run specified examples when specified" do
-        @behaviour.it("spec1") {}
-        @behaviour.it("spec2") {}
-        @behaviour.retain_examples_matching!(["context spec1"])
+      it "should remove examples that match" do
+        @behaviour.it("example1") {}
+        @behaviour.it("example2") {}
+        @behaviour.retain_examples_matching!(["behaviour example1"])
         @behaviour.number_of_examples.should == 1
       end
     end

data/spec/spec/runner/drb_command_line_spec.rb

@@ -1,81 +1,84 @@
 require File.dirname(__FILE__) + '/../../spec_helper.rb'
 
-describe "DrbCommandLine without running local server" do
+module Spec
+  module Runner
+    describe DrbCommandLine, "without running local server" do
   
-  unless Config::CONFIG['ruby_install_name'] == 'jruby'
-    it "should print error when there is no running local server" do
-      err = StringIO.new
-      out = StringIO.new
-      Spec::Runner::DrbCommandLine.run(['--version'], err, out, false)
+      unless Config::CONFIG['ruby_install_name'] == 'jruby'
+        it "should print error when there is no running local server" do
+          err = StringIO.new
+          out = StringIO.new
+          DrbCommandLine.run(['--version'], err, out, false)
     
-      err.rewind
-      err.read.should =~ /No server is running/
+          err.rewind
+          err.read.should =~ /No server is running/
+        end
+      end    
     end
-  end    
-end
 
-describe "DrbCommandLine with local server" do
+    describe DrbCommandLine, "with local server" do
 
-  unless Config::CONFIG['ruby_install_name'] == 'jruby'  
-    before(:all) do
-      DRb.start_service("druby://localhost:8989", Spec::Runner::CommandLine)
-      $drb_example_file_counter = 0
-    end
+      unless Config::CONFIG['ruby_install_name'] == 'jruby'  
+        before(:all) do
+          DRb.start_service("druby://localhost:8989", Spec::Runner::CommandLine)
+          $drb_example_file_counter = 0
+        end
     
-    before(:each) do
-      create_dummy_spec_file
-      $drb_example_file_counter = $drb_example_file_counter + 1
-    end
+        before(:each) do
+          create_dummy_spec_file
+          $drb_example_file_counter = $drb_example_file_counter + 1
+        end
     
-    after(:each) do
-      File.delete(@dummy_spec_filename)
-    end
+        after(:each) do
+          File.delete(@dummy_spec_filename)
+        end
     
-    after(:all) do
-      DRb.stop_service      
-    end
+        after(:all) do
+          DRb.stop_service      
+        end
 
-    it "should run against local server" do
-      out = run_spec_via_druby(['--version'])
-      out.should =~ /RSpec/n
-    end
+        it "should run against local server" do
+          out = run_spec_via_druby(['--version'])
+          out.should =~ /RSpec/n
+        end
     
-    it "should output green colorized text when running with --colour option" do
-      out = run_spec_via_druby(["--colour", @dummy_spec_filename])
-      out.should =~ /\e\[32m/n
-    end
+        it "should output green colorized text when running with --colour option" do
+          out = run_spec_via_druby(["--colour", @dummy_spec_filename])
+          out.should =~ /\e\[32m/n
+        end
     
-    it "should output red colorized text when running with -c option" do      
-      out = run_spec_via_druby(["-c", @dummy_spec_filename])
-      out.should =~ /\e\[31m/n
-    end
+        it "should output red colorized text when running with -c option" do      
+          out = run_spec_via_druby(["-c", @dummy_spec_filename])
+          out.should =~ /\e\[31m/n
+        end
 
-    def create_dummy_spec_file
-      @dummy_spec_filename = File.expand_path(File.dirname(__FILE__)) + "/_dummy_spec#{$drb_example_file_counter}.rb"
-      File.open(@dummy_spec_filename, 'w') do |f|
-        f.write %{
-describe "DUMMY CONTEXT for 'DrbCommandLine with -c option'" do
-  it "should be output with green bar" do
-    true.should be_true
-  end
+        def create_dummy_spec_file
+          @dummy_spec_filename = File.expand_path(File.dirname(__FILE__)) + "/_dummy_spec#{$drb_example_file_counter}.rb"
+          File.open(@dummy_spec_filename, 'w') do |f|
+            f.write %{
+    describe "DUMMY CONTEXT for 'DrbCommandLine with -c option'" do
+      it "should be output with green bar" do
+        true.should be_true
+      end
 
-  it "should be output with red bar" do
-    violated("I want to see a red bar!")
-  end
-end
-}
+      it "should be output with red bar" do
+        violated("I want to see a red bar!")
       end
     end
+    }
+          end
+        end
     
-    def run_spec_via_druby(args)
-      err, out = StringIO.new, StringIO.new
-      out.instance_eval do
-        def tty?; true end
+        def run_spec_via_druby(args)
+          err, out = StringIO.new, StringIO.new
+          out.instance_eval do
+            def tty?; true end
+          end
+          Spec::Runner::DrbCommandLine.run(args, err, out, false, true)
+          out.rewind; out.read
+        end
       end
-      Spec::Runner::DrbCommandLine.run(args, err, out, false, true)
-      out.rewind; out.read
+  
     end
   end
-  
 end
-