Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce test on Error#full_message for attribute with underscores; Fix... #9864

Merged
merged 1 commit into from
Mar 25, 2013

Conversation

vipulnsward
Copy link
Member

  1. Introduce test on Error#full_message for attribute with underscores
  2. Fix typo in test names

@senny
Copy link
Member

senny commented Mar 22, 2013

Why is the new test relevant? Was there a change in behavior regarding attributes with underscores? Is it to prevent regressions?

Also it would be great if you could shorten your commit message. Github does not like long commit messages. You can always do a short description on the first line, then a blank line and then add a detailed description.

@vipulnsward
Copy link
Member Author

About the comments: Will keep in mind the next time

While scanning through Error model, I found the tr(".","_") and humanize on the result in full_messages.

The behaviour for having _ in the name is different than having a .

Ex:
Model.nestedattr => Model nestedattr
but model_test maps to model_test itself

Test for previous behaviour is present the latter isn't, so added the test.

@senny
Copy link
Member

senny commented Mar 22, 2013

I see.

Regarding the commit message, you can easily update it by using git commit --amend or git rebase -i and then issue a force push on your branch fix_test_AR_v2. The PR on Github will reflect the changes automatically.

@vipulnsward
Copy link
Member Author

@senny updating in some time

Introduce test on Error#full_message for attribute with underscores; Fix
some typos
@vipulnsward
Copy link
Member Author

Fixed message

@vipulnsward
Copy link
Member Author

rafaelfranca added a commit that referenced this pull request Mar 25, 2013
Introduce test on Error#full_message for attribute with underscores; Fix...
@rafaelfranca rafaelfranca merged commit ba012fa into rails:master Mar 25, 2013
@vipulnsward vipulnsward deleted the fix_test_AR_v2 branch February 22, 2016 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants