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.

Sign up for GitHub

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

Jump to bottom

have_secure_password cannot be used in the negative #1187

Closed
mcmire opened this issue Mar 10, 2019 · 1 comment · Fixed by #1593
Closed

have_secure_password cannot be used in the negative #1187

mcmire opened this issue Mar 10, 2019 · 1 comment · Fixed by #1593
Labels
🕷 Issue: Bug

Comments

@mcmire
Copy link
Collaborator

mcmire commented Mar 10, 2019

The have_secure_password is missing a failure_message_when_negated method, so it cannot be used like this:

it { should_not have_secure_password }
@Kutomore Kutomore mentioned this issue Mar 21, 2019
Closed
@mcmire
Copy link
Collaborator Author

mcmire commented Jun 9, 2019

Closing this in favor of the PR.

@mcmire mcmire closed this as completed Jul 12, 2020
@amalrik amalrik mentioned this issue Dec 20, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🕷 Issue: Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

adds failure_message_when_negated to have_secure_password_matcher amalrik/shoulda-matchers
1 participant
@mcmire