Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

OCMock and NSFileManager

I am trying to unit test a method which uses NSFileManager to check if a file exists and delete the file. I'm not sure I am correctly setting up the mock object.

I'm getting "Method was not invoked" even though I'm certain it was. Am I missing something in the way I set it up?

I set the mock object up as follows:

- (void)testdeleteFileWithPath {

    id fileManagerMock = OCMClassMock([NSFileManager class]);

    OCMStub([NSFileManager defaultManager]).andReturn(fileManagerMock);

    NSString *filePath = @"testPDF.pdf";
    NSURL *fileURL = [self.documentsDirectory URLByAppendingPathComponent:filePath];

    [self.fileDAO deleteFileWithPath:filePath];

    OCMVerify([fileManagerMock fileExistsAtPath:[fileURL path]]);
}

- (void)deleteFileWithPath:(NSString *)filePath
{    
    if (filePath) {
        NSURL *fileURL = [self.documentsDirectory URLByAppendingPathComponent:filePath];

        NSError *error = nil;
        if (![[NSFileManager defaultManager] fileExistsAtPath:[fileURL path]]) {
            NSLog(@"File to delete does not exist. \nPath: %@", [fileURL absoluteString]);

        } else
        {
            [[NSFileManager defaultManager] removeItemAtURL:fileURL error:&error];

            NSLog(@"Error: %@", [error description]);
        }
    }
}
like image 961
Caroline Avatar asked Mar 25 '15 16:03

Caroline


2 Answers

Got this working now. I was not stubbing the method on the mock object. The code should read as follows:

id fileManagerMock = OCMClassMock([NSFileManager class]); 
OCMStub([fileManagerMock defaultManager]).andReturn(fileManagerMock);
like image 52
Caroline Avatar answered Oct 27 '22 04:10

Caroline


You really should be doing a partial mock of the Object instance [fileManagerMock defaultManager] instead of the class: [NSFileManager class]:

id fileManagerMock = [OCMockObject partialMockForObject[fileManagerMock defaultManager]];

Seems likely that would affect the results

like image 36
bplattenburg Avatar answered Oct 27 '22 04:10

bplattenburg