Better message if #and_raise exception type requires constructor params #100

Merged
merged 2 commits into from Jan 30, 2012

Conversation

Projects
None yet
3 participants
Contributor

coreyhaines commented Jan 29, 2012

Made message so very nice.
#99

Add at the end of the message " arguments"?

Owner

coreyhaines replied Jan 29, 2012

Yeah, yeah. :) Tried to wordsmith it to not need to implement a pluralize method. :) 1 argument, 2 arguments. Ugh!
And, yes, it could be as simple as (["s"]*(arity-1)).uniq.to_s (confusing enough?) HAHA!

Owner

coreyhaines commented on 6fe4168 Jan 29, 2012

Looks like it committed a bunch of "get rid of ending space" changes, too. :)

dchelimsky added a commit that referenced this pull request Jan 30, 2012

Merge pull request #100 from coreyhaines/and_raise_big_arity_construc…
…tor_better_message

Better message if #and_raise exception type requires constructor params

- Fixes #99.

@dchelimsky dchelimsky merged commit 7ce8c6f into rspec:master Jan 30, 2012

dchelimsky added a commit that referenced this pull request Jan 30, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment