@@ -125,18 +125,22 @@ EOF # employee.develop_great_new_social_networking_app # }.should change(employee, :title).from("Mail Clerk").to("CEO") # - # Evaluates +receiver.message+ or +block+ before and - # after it evaluates the c object (generated by the lambdas in the examples above). + # Evaluates <tt>receiver.message</tt> or <tt>block</tt> before and after + # it evaluates the c object (generated by the lambdas in the examples + # above). # - # Then compares the values before and after the +receiver.message+ and - # evaluates the difference compared to the expected difference. + # Then compares the values before and after the <tt>receiver.message</tt> + # and evaluates the difference compared to the expected difference. # - # == Warning - # +should_not+ +change+ only supports the form with no subsequent calls to - # +by+, +by_at_least+, +by_at_most+, +to+ or +from+. + # == WARNING + # <tt>should_not change</tt> only supports the form with no + # subsequent calls to <tt>by</tt>, <tt>by_at_least</tt>, + # <tt>by_at_most</tt>, <tt>to</tt> or <tt>from</tt>. + # + # blocks passed to <tt>should</tt> <tt>change</tt> and <tt>should_not</tt> + # <tt>change</tt> must use the <tt>{}</tt> form (<tt>do/end</tt> is not + # supported). # - # blocks passed to +should+ +change+ and +should_not+ +change+ - # must use the <tt>{}</tt> form (<tt>do/end</tt> is not supported) def change(target=nil, message=nil, &block) Matchers::Change.new(target, message, &block) end lib/spec/matchers/change.rb @@ -55,10 +55,9 @@ describe "should change { block }" do end.should fail_with("result should have changed, but is still 5") end - it "should warn if passed a block using do/end" do + it "should warn if passed a block using do/end instead of {}" do lambda do - lambda {}.should change do - end + lambda {}.should change do; end end.should raise_error(Spec::Matchers::MatcherError, /block passed to should or should_not/) end end @@ -79,10 +78,9 @@ describe "should_not change { block }" do end.should fail_with("result should not have changed, but did change from 5 to 6") end - it "should warn if passed a block using do/end" do + it "should warn if passed a block using do/end instead of {}" do lambda do - lambda {}.should_not change do - end + lambda {}.should_not change do; end end.should raise_error(Spec::Matchers::MatcherError, /block passed to should or should_not/) end end spec/spec/matchers/change_spec.rb b9ca5c1e06a6805a3229d2f4169f9832161c2c0f David Chelimsky dchelimsky@gmail.com http://github.com/dchelimsky/rspec/commit/b171b70245e4eaa1505c8bcb792ab2460fe17ab1 b171b70245e4eaa1505c8bcb792ab2460fe17ab1 2008-06-22T08:19:20-07:00 2008-06-22T08:19:20-07:00 tweak rdoc for should change eba2b5c91f5a9a7aae022f93fa142fea891a7d64 David Chelimsky dchelimsky@gmail.com