19

I'm trying to mock a method that has the equivalent of the following signature:

- (NSDictionary *) uploadValues:(BOOL)doSomething error:(NSError **)error

I want it to return a small dictionary so that my test can make sure the code uses the dictionary properly. however, no matter what i do OCMock always returns nil from the method, regardless of how i stub it out. The error begins as nil in the code i'm testing, and these are the different ways i've tried stubbing it:

NSError * error = nil;
[[[mock stub] andReturn:someDict] uploadValues:YES error:&error];

[[[mock stub] andReturn:someDict] uploadValues:YES error:nil];

[[[mock stub] andReturn:someDict] uploadValues:YES error:[OCMArg any]];

and none of them work. Does OCMock support handles as stubbed message arguments and if so, what's the correct way to do it?

Kevlar
  • 8,804
  • 9
  • 55
  • 81

5 Answers5

49
[[[mock stub] andReturn:someDict] uploadValues:YES error:[OCMArg setTo:nil]];

or

NSError* someError = ...
[[[mock stub] andReturn:someDict] uploadValues:YES error:[OCMArg setTo:someError]];

You could also do

[[[mock stub] andReturn:someDict] uploadValues:YES error:[OCMArg anyPointer]];

but this might cause your code to incorrectly think that you passed back a real NSError.

Andreas Järliden
  • 1,884
  • 1
  • 17
  • 16
  • 7
    In recent versions of OCMock, you can use `[OCMArg anyObjectRef]` instead of `[OCMArg anyPointer]` to avoid errors under ARC when you are dealing with pointers-to-pointers of Objective-C object types (such as `NSError**`). – Tim Yates May 22 '14 at 16:04
3

With ARC, your method declaration will probably look like this:

- (NSDictionary *) uploadValues:(BOOL)doSomething error:(NSError *__autoreleasing *)error;

Here is how I mock these types of methods:

BOOL mockDoSomething = YES;

NSError __autoreleasing *error = nil;

[[[mock stub] andReturn:someDict] uploadValues:OCMOCK_VALUE(mockDoSomething) error:&error];
Eric
  • 2,045
  • 17
  • 24
  • 1
    Specifically, the following seems to work just fine in an expect: error:((NSError __autoreleasing **)[OCMArg anyPointer]) – Mark Pauley Dec 14 '12 at 19:40
3
NSError *__autoreleasing *err = (NSError *__autoreleasing *) [OCMArg anyPointer];

[[[_mockReporter stub] andReturnValue:OCMOCK_VALUE((BOOL){YES})]
   yourMethodCall:err];
David
  • 2,429
  • 24
  • 15
3

I created a category on OCMArg to aid with this situation.

OCMArg+Helpers.h:

@interface OCMArg (Helpers)

+ (NSError *__autoreleasing *)anyError;

@end

OCMArg+Helpers.m:

@implementation OCMArg (Helpers)

+ (NSError *__autoreleasing *)anyError {
    return (NSError *__autoreleasing *)[OCMArg anyPointer];
}

@end

Then, whenever I have an error param I need to mock, use anyError, like so:

[[myMock stub] someMethodWithArg:anArg error:[OCMArg anyError]];
Nicholas Hart
  • 1,734
  • 13
  • 22
  • 1
    Also, it looks like the latest version of OCMock on GitHub has something along these lines in OCMarg, `+ (id __autoreleasing *)anyObjectRef`. – Nicholas Hart Jan 13 '14 at 17:13
-4

Sadly, I haven't found a good solution for this either. The best I can say is to try to make the use of the NSError** as small as possible, and then put it in an isolated function that you can completely mock out on your partial mock.

I'm finding that any code that uses anything other than an NSObject* (or derived) or primitive values (NSInteger, BOOL, etc) is pretty much impossible to test using OCMock.

Chris Phillips
  • 11,607
  • 3
  • 34
  • 45
  • 4
    Using and providing methods that accept NSError ** is the preferred error handling mechanism in Cocoa. I suggest to stick with this paradigm and have a look at the answer from Andreas Järliden to mock these methods. – rluba Aug 17 '11 at 06:40
  • 3
    See Andreas Järliden's answer below. Your answer is incorrect. – markshiz Feb 08 '13 at 18:54