-
-
Notifications
You must be signed in to change notification settings - Fork 906
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
[Nimble] Addressed issues #30, 42, 47, 50 #51
Conversation
The screenshot will need to be updated to reflect the change from exampleGroups() to spec(). |
describe("failureMessage") { | ||
it("says it expected the subject to be false") { | ||
let message = matcher.failureMessage("Antoine Roquentin") | ||
expect(message).to.equal("expected 'Antoine Roquentin' to be false") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haha, not a showstopper or anything, but most of the other specs reference Game of Thrones 👑 ✨
@endersstocker Cool, I see you've already taken care of the typo! My only other comment is on I'd also merge a pull request to change the test strings to Game of Thrones references, wink wink 😉 |
…-nil was redundant”
…redundant. Changed references to Game of Thrones.
@modocache That should do it. |
} | ||
|
||
context("and is not false") { | ||
beforeEach { subject = "Petyr Baelish" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@endersstocker Excellent! Thanks a ton! |
[Nimble] Addressed issues #30, 42, 47, 50
contains() for objc matches strings correctly.
No description provided.