rspec-instructure 1.3.3

data/spec/spec/matchers/dsl_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+module Spec
+  module Matchers
+    module DSL
+      describe "#create" do
+        it "is deprecated" do
+          Spec.should_receive(:deprecate)
+          mod = Module.new
+          mod.extend Spec::Matchers::DSL
+          mod.create(:foo)
+        end
+      end
+      
+      describe "#define" do
+        it "creates a method that initializes a new matcher with the submitted name and expected arg" do
+          # FIXME - this expects new to be called, but we need something
+          # more robust - that expects new to be called with a specific
+          # block (lambda, proc, whatever)
+          mod = Module.new
+          mod.extend Spec::Matchers::DSL
+          mod.define(:foo)
+    
+          obj = Object.new
+          obj.extend mod
+    
+          Spec::Matchers::Matcher.should_receive(:new).with(:foo, 3)
+    
+          obj.foo(3)
+        end
+      end
+    end
+  end
+end

data/spec/spec/matchers/eql_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+
+module Spec
+  module Matchers
+    describe "eql" do
+      it "should match when actual.eql?(expected)" do
+        1.should eql(1)
+      end
+      
+      it "should not match when !actual.eql?(expected)" do
+        1.should_not eql(2)
+      end
+      
+      it "should describe itself" do
+        matcher = eql(1)
+        matcher.matches?(1)
+        matcher.description.should == "eql 1"
+      end
+      
+      it "should provide message, expected and actual on #failure_message" do
+        matcher = eql("1")
+        matcher.matches?(1)
+        matcher.failure_message_for_should.should == "\nexpected \"1\"\n     got 1\n\n(compared using eql?)\n"
+      end
+      
+      it "should provide message, expected and actual on #negative_failure_message" do
+        matcher = eql(1)
+        matcher.matches?(1)
+        matcher.failure_message_for_should_not.should == "\nexpected 1 not to equal 1\n\n(compared using eql?)\n"
+      end
+    end
+  end
+end

data/spec/spec/matchers/equal_spec.rb

@@ -0,0 +1,57 @@
+require 'spec_helper'
+module Spec
+  module Matchers
+    describe "equal" do
+      
+      def inspect_object(o)
+        "#<#{o.class}:#{o.object_id}> => #{o.inspect}"
+      end
+      
+      it "should match when actual.equal?(expected)" do
+        1.should equal(1)
+      end
+
+      it "should not match when !actual.equal?(expected)" do
+        1.should_not equal("1")
+      end
+      
+      it "should describe itself" do
+        matcher = equal(1)
+        matcher.matches?(1)
+        matcher.description.should == "equal 1"
+      end
+      
+      it "should provide message on #failure_message" do
+        expected, actual = "1", "1"
+        matcher = equal(expected)
+        matcher.matches?(actual)
+        
+        matcher.failure_message_for_should.should == <<-MESSAGE
+
+expected #{inspect_object(expected)}
+     got #{inspect_object(actual)}
+
+Compared using equal?, which compares object identity,
+but expected and actual are not the same object. Use
+'actual.should == expected' if you don't care about
+object identity in this example.
+
+MESSAGE
+      end
+      
+      it "should provide message on #negative_failure_message" do
+        expected = actual = "1"
+        matcher = equal(expected)
+        matcher.matches?(actual)
+        matcher.failure_message_for_should_not.should == <<-MESSAGE
+
+expected not #{inspect_object(expected)}
+         got #{inspect_object(actual)}
+
+Compared using equal?, which compares object identity.
+
+MESSAGE
+      end
+    end
+  end
+end

data/spec/spec/matchers/exist_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+
+class Substance
+  def initialize exists, description
+    @exists = exists
+    @description = description
+  end
+  def exist?(arg=nil)
+    @exists
+  end
+  def inspect
+    @description
+  end
+end
+  
+class SubstanceTester
+  include Spec::Matchers
+  def initialize substance
+    @substance = substance
+  end
+  def should_exist
+    @substance.should exist
+  end
+end
+
+describe "should exist" do
+  
+  before(:each) do
+    @real = Substance.new true, 'something real'
+    @imaginary = Substance.new false, 'something imaginary'
+  end
+
+  describe "within an example group" do
+  
+    it "passes if target exists" do
+      @real.should exist
+    end
+  
+    it "passes if target exists with args" do
+      @real.should exist('this arg')
+    end
+  
+    it "fails if target does not exist" do
+      lambda { @imaginary.should exist }.should fail
+    end
+    
+    it "describes itself" do
+      exist.description.should == "exist"
+    end
+    
+    it "passes should_not exist if target doesn't exist" do
+      lambda { @real.should_not exist }.should fail
+    end
+  end
+
+  describe "outside of an example group" do
+
+    it "should pass if target exists" do
+      real_tester = SubstanceTester.new @real
+      real_tester.should_exist
+    end
+
+  end
+
+end

data/spec/spec/matchers/has_spec.rb

@@ -0,0 +1,190 @@
+require 'spec_helper'
+
+describe "should have_sym(*args)" do
+  it "should pass if #has_sym?(*args) returns true" do
+    {:a => "A"}.should have_key(:a)
+  end
+
+  it "should fail if #has_sym?(*args) returns false" do
+    lambda {
+      {:b => "B"}.should have_key(:a)
+    }.should fail_with("expected #has_key?(:a) to return true, got false")
+  end
+
+  it "should fail if #has_sym?(*args) returns nil" do
+    klass = Class.new do
+      def has_foo?
+      end
+    end
+    lambda {
+      klass.new.should have_foo
+    }.should fail_with("expected #has_foo?(nil) to return true, got false")
+  end
+
+  it "should fail if target does not respond to #has_sym?" do
+    lambda {
+      Object.new.should have_key(:a)
+    }.should raise_error(NoMethodError)
+  end
+  
+  it "should reraise an exception thrown in #has_sym?(*args)" do
+    o = Object.new
+    def o.has_sym?(*args)
+      raise "Funky exception"
+    end
+    lambda { o.should have_sym(:foo) }.should raise_error("Funky exception")
+  end
+end
+
+describe "should_not have_sym(*args)" do
+  it "should pass if #has_sym?(*args) returns false" do
+    {:a => "A"}.should_not have_key(:b)
+  end
+
+  it "should pass if #has_sym?(*args) returns nil" do
+    klass = Class.new do
+      def has_foo?
+      end
+    end
+    klass.new.should_not have_foo
+  end
+
+  it "should fail if #has_sym?(*args) returns true" do
+    lambda {
+      {:a => "A"}.should_not have_key(:a)
+    }.should fail_with("expected #has_key?(:a) to return false, got true")
+  end
+
+  it "should fail if target does not respond to #has_sym?" do
+    lambda {
+      Object.new.should have_key(:a)
+    }.should raise_error(NoMethodError)
+  end
+  
+  it "should reraise an exception thrown in #has_sym?(*args)" do
+    o = Object.new
+    def o.has_sym?(*args)
+      raise "Funky exception"
+    end
+    lambda { o.should_not have_sym(:foo) }.should raise_error("Funky exception")
+  end
+end
+
+describe "should have_sym(&block)" do
+  it "should pass when actual returns true for :has_sym?(&block)" do
+    actual = mock("actual")
+    delegate = mock("delegate")
+    actual.should_receive(:has_foo?).and_yield
+    delegate.should_receive(:check_has_foo).and_return(true)
+    actual.should have_foo { delegate.check_has_foo }
+  end
+
+  it "should fail when actual returns false for :has_sym?(&block)" do
+    actual = mock("actual")
+    delegate = mock("delegate")
+    actual.should_receive(:has_foo?).and_yield
+    delegate.should_receive(:check_has_foo).and_return(false)
+    lambda {
+      actual.should have_foo { delegate.check_has_foo }
+    }.should fail_with("expected #has_foo?(nil) to return true, got false")
+  end
+
+  it "should fail when actual does not respond to :has_sym?" do
+    delegate = mock("delegate", :check_has_foo => true)
+    lambda {
+      Object.new.should have_foo { delegate.check_has_foo }
+    }.should raise_error(NameError)
+  end
+end
+
+describe "should_not have_sym(&block)" do
+  it "should pass when actual returns false for :has_sym?(&block)" do
+    actual = mock("actual")
+    delegate = mock("delegate")
+    actual.should_receive(:has_foo?).and_yield
+    delegate.should_receive(:check_has_foo).and_return(false)
+    actual.should_not have_foo { delegate.check_has_foo }
+  end
+
+  it "should fail when actual returns true for :has_sym?(&block)" do
+    actual = mock("actual")
+    delegate = mock("delegate")
+    actual.should_receive(:has_foo?).and_yield
+    delegate.should_receive(:check_has_foo).and_return(true)
+    lambda {
+      actual.should_not have_foo { delegate.check_has_foo }
+    }.should fail_with("expected #has_foo?(nil) to return false, got true")
+  end
+
+  it "should fail when actual does not respond to :has_sym?" do
+    delegate = mock("delegate", :check_has_foo => true)
+    lambda {
+      Object.new.should_not have_foo { delegate.check_has_foo }
+    }.should raise_error(NameError)
+  end
+end
+
+describe "should have_sym(*args, &block)" do
+  it "should pass when actual returns true for :has_sym?(*args, &block)" do
+    actual = mock("actual")
+    delegate = mock("delegate")
+    actual.should_receive(:has_foo?).with(:a).and_yield(:a)
+    delegate.should_receive(:check_has_foo).with(:a).and_return(true)
+    actual.should have_foo(:a) { |foo| delegate.check_has_foo(foo) }
+  end
+
+  it "should fail when actual returns false for :has_sym?(*args, &block)" do
+    actual = mock("actual")
+    delegate = mock("delegate")
+    actual.should_receive(:has_foo?).with(:a).and_yield(:a)
+    delegate.should_receive(:check_has_foo).with(:a).and_return(false)
+    lambda {
+      actual.should have_foo(:a) { |foo| delegate.check_has_foo(foo) }
+    }.should fail_with("expected #has_foo?(:a) to return true, got false")
+  end
+
+  it "should fail when actual does not respond to :has_sym?" do
+    delegate = mock("delegate", :check_has_foo => true)
+    lambda {
+      Object.new.should have_foo(:a) { |foo| delegate.check_has_foo(foo) }
+    }.should raise_error(NameError)
+  end
+end
+
+describe "should_not have_sym(*args, &block)" do
+  it "should pass when actual returns false for :has_sym?(*args, &block)" do
+    actual = mock("actual")
+    delegate = mock("delegate")
+    actual.should_receive(:has_foo?).with(:a).and_yield(:a)
+    delegate.should_receive(:check_has_foo).with(:a).and_return(false)
+    actual.should_not have_foo(:a) { |foo| delegate.check_has_foo(foo) }
+  end
+
+  it "should fail when actual returns true for :has_sym?(*args, &block)" do
+    actual = mock("actual")
+    delegate = mock("delegate")
+    actual.should_receive(:has_foo?).with(:a).and_yield(:a)
+    delegate.should_receive(:check_has_foo).with(:a).and_return(true)
+    lambda {
+      actual.should_not have_foo(:a) { |foo| delegate.check_has_foo(foo) }
+    }.should fail_with("expected #has_foo?(:a) to return false, got true")
+  end
+
+  it "should fail when actual does not respond to :has_sym?" do
+    delegate = mock("delegate", :check_has_foo => true)
+    lambda {
+      Object.new.should_not have_foo(:a) { |foo| delegate.check_has_foo(foo) }
+    }.should raise_error(NameError)
+  end
+end
+
+
+describe "has" do
+  it "should work when the target implements #send" do
+    o = {:a => "A"}
+    def o.send(*args); raise "DOH! Library developers shouldn't use #send!" end
+    lambda {
+      o.should have_key(:a)
+    }.should_not raise_error
+  end
+end

data/spec/spec/matchers/have_spec.rb

@@ -0,0 +1,381 @@
+require 'spec_helper'
+
+describe "have matcher" do
+
+  def create_collection_owner_with(n)
+    owner = Spec::Expectations::Helper::CollectionOwner.new
+    (1..n).each do |number|
+      owner.add_to_collection_with_length_method(number)
+      owner.add_to_collection_with_size_method(number)
+    end
+    owner
+  end
+
+  before(:each) do
+    if defined?(::ActiveSupport::Inflector)
+      @active_support_was_defined = true
+    else
+      @active_support_was_defined = false
+      module ::ActiveSupport
+        class Inflector
+          def self.pluralize(string)
+            string.to_s + 's'
+          end
+        end
+      end
+    end
+  end
+
+  describe "should have(n).items" do
+    it "should pass if target has a collection of items with n members" do
+      owner = create_collection_owner_with(3)
+      owner.should have(3).items_in_collection_with_length_method
+      owner.should have(3).items_in_collection_with_size_method
+    end
+
+    it "should convert :no to 0" do
+      owner = create_collection_owner_with(0)
+      owner.should have(:no).items_in_collection_with_length_method
+      owner.should have(:no).items_in_collection_with_size_method
+    end
+
+    it "should fail if target has a collection of items with < n members" do
+      owner = create_collection_owner_with(3)
+      lambda {
+        owner.should have(4).items_in_collection_with_length_method
+      }.should fail_with("expected 4 items_in_collection_with_length_method, got 3")
+      lambda {
+        owner.should have(4).items_in_collection_with_size_method
+      }.should fail_with("expected 4 items_in_collection_with_size_method, got 3")
+    end
+    
+    it "should fail if target has a collection of items with > n members" do
+      owner = create_collection_owner_with(3)
+      lambda {
+        owner.should have(2).items_in_collection_with_length_method
+      }.should fail_with("expected 2 items_in_collection_with_length_method, got 3")
+      lambda {
+        owner.should have(2).items_in_collection_with_size_method
+      }.should fail_with("expected 2 items_in_collection_with_size_method, got 3")
+    end
+  end
+
+  describe 'should have(1).item when ActiveSupport::Inflector is defined' do
+    it 'should pluralize the collection name' do
+      owner = create_collection_owner_with(1)
+      owner.should have(1).item
+    end
+    
+    after(:each) do
+      unless @active_support_was_defined
+        Object.__send__ :remove_const, :ActiveSupport
+      end
+    end
+  end
+
+  describe 'should have(1).item when Inflector is defined' do
+    before(:each) do
+      if defined?(Inflector)
+        @inflector_was_defined = true
+      else
+        @inflector_was_defined = false
+        class ::Inflector
+          def self.pluralize(string)
+            string.to_s + 's'
+          end
+        end
+      end
+    end
+    
+    it 'should pluralize the collection name' do
+      owner = create_collection_owner_with(1)
+      owner.should have(1).item
+    end
+
+    after(:each) do
+      unless @inflector_was_defined
+        Object.__send__ :remove_const, :Inflector
+      end
+    end
+  end
+
+  describe "should have(n).items where result responds to items but returns something other than a collection" do
+    it "should provide a meaningful error" do
+      owner = Class.new do
+        def items
+          Object.new
+        end
+      end.new
+      lambda do
+        owner.should have(3).items
+      end.should raise_error("expected items to be a collection but it does not respond to #length or #size")
+    end
+  end
+
+  describe "should_not have(n).items" do
+    it "should pass if target has a collection of items with < n members" do
+      owner = create_collection_owner_with(3)
+      owner.should_not have(4).items_in_collection_with_length_method
+      owner.should_not have(4).items_in_collection_with_size_method
+    end
+    
+    it "should pass if target has a collection of items with > n members" do
+      owner = create_collection_owner_with(3)
+      owner.should_not have(2).items_in_collection_with_length_method
+      owner.should_not have(2).items_in_collection_with_size_method
+    end
+
+    it "should fail if target has a collection of items with n members" do
+      owner = create_collection_owner_with(3)
+      lambda {
+        owner.should_not have(3).items_in_collection_with_length_method
+      }.should fail_with("expected target not to have 3 items_in_collection_with_length_method, got 3")
+      lambda {
+        owner.should_not have(3).items_in_collection_with_size_method
+        }.should fail_with("expected target not to have 3 items_in_collection_with_size_method, got 3")
+    end
+  end
+
+  describe "should have_exactly(n).items" do
+    it "should pass if target has a collection of items with n members" do
+      owner = create_collection_owner_with(3)
+      owner.should have_exactly(3).items_in_collection_with_length_method
+      owner.should have_exactly(3).items_in_collection_with_size_method
+    end
+
+    it "should convert :no to 0" do
+      owner = create_collection_owner_with(0)
+      owner.should have_exactly(:no).items_in_collection_with_length_method
+      owner.should have_exactly(:no).items_in_collection_with_size_method
+    end
+
+    it "should fail if target has a collection of items with < n members" do
+      owner = create_collection_owner_with(3)
+      lambda {
+        owner.should have_exactly(4).items_in_collection_with_length_method
+      }.should fail_with("expected 4 items_in_collection_with_length_method, got 3")
+      lambda {
+        owner.should have_exactly(4).items_in_collection_with_size_method
+      }.should fail_with("expected 4 items_in_collection_with_size_method, got 3")
+    end
+    
+    it "should fail if target has a collection of items with > n members" do
+      owner = create_collection_owner_with(3)
+      lambda {
+        owner.should have_exactly(2).items_in_collection_with_length_method
+      }.should fail_with("expected 2 items_in_collection_with_length_method, got 3")
+      lambda {
+        owner.should have_exactly(2).items_in_collection_with_size_method
+      }.should fail_with("expected 2 items_in_collection_with_size_method, got 3")
+    end
+  end
+
+  describe "should have_at_least(n).items" do
+    it "should pass if target has a collection of items with n members" do
+      owner = create_collection_owner_with(3)
+      owner.should have_at_least(3).items_in_collection_with_length_method
+      owner.should have_at_least(3).items_in_collection_with_size_method
+    end
+    
+    it "should pass if target has a collection of items with > n members" do
+      owner = create_collection_owner_with(3)
+      owner.should have_at_least(2).items_in_collection_with_length_method
+      owner.should have_at_least(2).items_in_collection_with_size_method
+    end
+
+    it "should fail if target has a collection of items with < n members" do
+      owner = create_collection_owner_with(3)
+      lambda {
+        owner.should have_at_least(4).items_in_collection_with_length_method
+      }.should fail_with("expected at least 4 items_in_collection_with_length_method, got 3")
+      lambda {
+        owner.should have_at_least(4).items_in_collection_with_size_method
+      }.should fail_with("expected at least 4 items_in_collection_with_size_method, got 3")
+    end
+    
+    it "should provide educational negative failure messages" do
+      #given
+      owner = create_collection_owner_with(3)
+      length_matcher = have_at_least(3).items_in_collection_with_length_method
+      size_matcher = have_at_least(3).items_in_collection_with_size_method
+      
+      #when
+      length_matcher.matches?(owner)
+      size_matcher.matches?(owner)
+      
+      #then
+      length_matcher.failure_message_for_should_not.should == <<-EOF
+Isn't life confusing enough?
+Instead of having to figure out the meaning of this:
+  should_not have_at_least(3).items_in_collection_with_length_method
+We recommend that you use this instead:
+  should have_at_most(2).items_in_collection_with_length_method
+EOF
+
+      size_matcher.failure_message_for_should_not.should == <<-EOF
+Isn't life confusing enough?
+Instead of having to figure out the meaning of this:
+  should_not have_at_least(3).items_in_collection_with_size_method
+We recommend that you use this instead:
+  should have_at_most(2).items_in_collection_with_size_method
+EOF
+    end
+  end
+
+  describe "should have_at_most(n).items" do
+    it "should pass if target has a collection of items with n members" do
+      owner = create_collection_owner_with(3)
+      owner.should have_at_most(3).items_in_collection_with_length_method
+      owner.should have_at_most(3).items_in_collection_with_size_method
+    end
+
+    it "should fail if target has a collection of items with > n members" do
+      owner = create_collection_owner_with(3)
+      lambda {
+        owner.should have_at_most(2).items_in_collection_with_length_method
+      }.should fail_with("expected at most 2 items_in_collection_with_length_method, got 3")
+      lambda {
+        owner.should have_at_most(2).items_in_collection_with_size_method
+      }.should fail_with("expected at most 2 items_in_collection_with_size_method, got 3")
+    end
+    
+    it "should pass if target has a collection of items with < n members" do
+      owner = create_collection_owner_with(3)
+      owner.should have_at_most(4).items_in_collection_with_length_method
+      owner.should have_at_most(4).items_in_collection_with_size_method
+    end
+
+    it "should provide educational negative failure messages" do
+      #given
+      owner = create_collection_owner_with(3)
+      length_matcher = have_at_most(3).items_in_collection_with_length_method
+      size_matcher = have_at_most(3).items_in_collection_with_size_method
+      
+      #when
+      length_matcher.matches?(owner)
+      size_matcher.matches?(owner)
+      
+      #then
+      length_matcher.failure_message_for_should_not.should == <<-EOF
+Isn't life confusing enough?
+Instead of having to figure out the meaning of this:
+  should_not have_at_most(3).items_in_collection_with_length_method
+We recommend that you use this instead:
+  should have_at_least(4).items_in_collection_with_length_method
+EOF
+      
+      size_matcher.failure_message_for_should_not.should == <<-EOF
+Isn't life confusing enough?
+Instead of having to figure out the meaning of this:
+  should_not have_at_most(3).items_in_collection_with_size_method
+We recommend that you use this instead:
+  should have_at_least(4).items_in_collection_with_size_method
+EOF
+    end
+  end
+
+  describe "have(n).items(args, block)" do
+    it "should pass args to target" do
+      target = mock("target")
+      target.should_receive(:items).with("arg1","arg2").and_return([1,2,3])
+      target.should have(3).items("arg1","arg2")
+    end
+
+    it "should pass block to target" do
+      target = mock("target")
+      block = lambda { 5 }
+      target.should_receive(:items).with("arg1","arg2", block).and_return([1,2,3])
+      target.should have(3).items("arg1","arg2", block)
+    end
+  end
+
+  describe "have(n).items where target IS a collection" do
+    it "should reference the number of items IN the collection" do
+      [1,2,3].should have(3).items
+    end
+
+    it "should fail when the number of items IN the collection is not as expected" do
+      lambda { [1,2,3].should have(7).items }.should fail_with("expected 7 items, got 3")
+    end
+  end
+
+  describe "have(n).characters where target IS a String" do
+    it "should pass if the length is correct" do
+      "this string".should have(11).characters
+    end
+
+    it "should fail if the length is incorrect" do
+      lambda { "this string".should have(12).characters }.should fail_with("expected 12 characters, got 11")
+    end
+  end
+
+  describe "have(n).things on an object which is not a collection nor contains one" do
+    it "should fail" do
+      lambda { Object.new.should have(2).things }.should raise_error(NoMethodError, /undefined method `things' for #<Object:/)
+    end
+  end
+
+  describe Spec::Matchers::Have, "for a collection owner that implements #send" do
+    before(:each) do
+      @collection = Object.new
+      def @collection.floozles; [1,2] end
+      def @collection.send(*args); raise "DOH! Library developers shouldn't use #send!" end
+    end
+    
+    it "should work in the straightforward case" do
+      lambda {
+        @collection.should have(2).floozles
+      }.should_not raise_error
+    end
+
+    it "should work when doing automatic pluralization" do
+      lambda {
+        @collection.should have_at_least(1).floozle
+      }.should_not raise_error
+    end
+
+    it "should blow up when the owner doesn't respond to that method" do
+      lambda {
+        @collection.should have(99).problems
+      }.should raise_error(NoMethodError, /problems/)
+    end
+  end
+
+  module Spec
+    module Matchers
+      describe Have do
+        treats_method_missing_as_private :noop => false
+        
+        describe "respond_to?" do
+          before :each do
+            @have = Have.new(:foo)
+            @a_method_which_have_defines = Have.instance_methods.first
+            @a_method_which_object_defines = Object.instance_methods.first
+          end
+          
+          it "should be true for a method which Have defines" do
+            @have.should respond_to(@a_method_which_have_defines)
+          end
+          
+          it "should be true for a method that it's superclass (Object) defines" do
+            @have.should respond_to(@a_method_which_object_defines)
+          end
+          
+          it "should be false for a method which neither Object nor nor Have defines" do
+            @have.should_not respond_to(:foo_bar_baz)
+          end
+          
+          it "should be false if the owner doesn't respond to the method" do
+            have = Have.new(99)
+            have.should_not respond_to(:problems)
+          end
+          
+          it "should be true if the owner responds to the method" do
+            have = Have.new(:a_symbol)
+            have.should respond_to(:to_sym)
+          end
+        end
+      end
+    end
+  end
+end