Matty Cross
Matty Cross
An interesting observation, thanks for reporting it! We should definitely add this case to the tests and fix it.
This is a great addition! I can't believe we missed this during initial implementation. I agree with merging this, though I'd like to ask you to include some tests with...
Hmm, I see. That's a very good point. Unfortunately I don't have the analytics to see how much of an impact that would make. But yeah, we shouldn't include this...
> * A: setting the default value to `.atLeast(1)` I suppose that this is a nice enough solution to deliver the core feature of this PR. I'd then immediately create...
@ajpallares I've rebased the relevant changes according to @rolandkakonyi's suggestion while leaving the PR that makes the quantifier consistent with Cuckoo's verify for when the next major version is released....
Hey, nice find! I'd welcome a PR that fixes this.
I'm afraid you're not using the correct methods for ObjC mocking. Please post the whole test, so I can verify how you create mocks, stub them, and verify them.
Hey, I just played around a bit with this. We were missing a `NSObjectProtocol` support. Please try branch `fix/objc-nsobjectprotocol`, no need to rebuild/redownload the generator, this change affects only `Cuckoo`...
Thanks for the feedback, I'll take a look at it, hopefully it's nothing too major.
Hey, @quetool. Your issue is not quite relevant to @s-hocking's. Your `mockApiClient` doesn't stub the `expireSession` method, so OCMock tells you that in the error message. As for calling the...