Although Graham Lee might not approve, I occasionally use the singleton pattern, particularly for managing access to my data models.
Singletons in Cocoa are accessed using the following convention:
+(UserModelManager *)sharedManager {
if (sharedManager == nil) {
sharedManager = [[super allocWithZone:NULL] init];
}
return sharedManager;
}
This makes it challenging to unit test classes that depend on the singleton. Normally when you test class A which depends on class B, you can pass A a mocked instance of B, configured appropriately for the test. But if B is a singleton, A’s only way to instantiate it is through that sharedB instantiator, so your test gets the one and only real instance of the singleton.
There are certainly singleton-ish ways to design around this constraint, but even if your classes don’t use the singleton pattern, you almost certainly depend on one of the singleton classes in the iOS API, like UIApplication or NSNotificationCenter.
Enter categories
The simplest way around this problem is to create a category on the singleton in your test code that overrides the singleton instantiator. In Objective-C, a category is mechanism for adding or overriding methods in existing classes. Using a category, we can modify the behavior of the defaultCenter method just within the scope of the unit tests to return a mock version:
@implementation NSNotificationCenter (UnitTests)
+ (id)defaultCenter {
return mockNotificationCenter;
}
@end
With this category, any time the code under test calls [NSNotificationCenter defaultCenter], it will get value of mockMotificationCenter instead. If you do nothing, it’ll be nil, so it effectively just becomes a message sink. But your test can create a mock using OCMock that expects the notification and verifies that it is issued:
mockNotificationCenter = [OCMockObject mockForClass:[NSNotificationCenter class]];
[[mockNotificationCenter expect] postNotificationName:@"some notification"];
[myObject doSomethingThatPostsNotification];
[mockNotificationCenter verify];
mockNotificationCenter = nil;
}
Note that with this approach, you need to be sure to set the mock instance to nil at the end of your test. OCMocks are autoreleased, so if the test case still has a reference to the released version, you’ll get an EXC_BAD_ACCESS error the next time [NSNotificationCenter defaultCenter] is called. A good place to do that is in your tearDown method, so it’s set back to nil after each test.
Making the mock available to other tests
Now that we can mock the singleton in a single test case, we should make it available to all of our unit tests. I have a base test case class that all of my test classes inherit from, which is where I import OCMock and OCHamcrest and do other global testing setup. We can move the static variable and category up to that class, and create static methods for setting the variable:
mockNotificationCenter = [OCMockObject mockForClass:[NSNotificationCenter class]];
return mockNotificationCenter;
}
+(id)createNiceMockNotificationCenter {
mockNotificationCenter = [OCMockObject niceMockForClass:[NSNotificationCenter class]];
return mockNotificationCenter;
}
These methods both set the static variable and return the mock to the caller, for setting expectations and verifying. The “nice” version will silently swallow unexpected messages, where the regular version will throw an exception if it receives an unexpected message. Don’t forget to set it back to nil in tearDown, and call [super tearDown] in the test class’s tearDown method. Now we can update the test case above to:
mockNotificationCenter = [BaseTestCase createMockNotificationCenter];
[[mockNotificationCenter expect] postNotificationName:@"some notification"];
[myObject doSomethingThatPostsNotification];
[mockNotificationCenter verify];
}
But sometimes I want the real NSNotificationCenter
The main drawback to this category is that it will always override the original. That may not be an issue for notifications, but when testing your own singletons, you’ll need to be able to test the actual class. But now that you’ve overridden the instantiator in a category, you can’t just call the original instantiator if the static variable is nil. Or can you? Fortunately, Matt Gallagher created some nice macros that you can use to conditionally invoke the original.
Using Matt’s invokeSupersequentNoArgs macro, we can change the category method to the following:
+(id)defaultCenter {
if ([BaseTestCase mockNotificationCenter] != nil) {
return [BaseTestCase mockNotificationCenter];
}
return invokeSupersequentNoArgs();
}
@end
Note we also have to add a class method to retrieve the mock:
return mockNotificationCenter;
}
Now when [NSNotificationCenter defaultCenter] is invoked, the test case first checks to see if a mock has been set. If it has, the mock is returned. Otherwise, the original, overridden method is invoked.
Conclusion
The singleton is a design pattern you should use with care, as it certainly makes you jump through a few more hurdles when testing. But with a bit of careful setup, your singleton dependencies can (and should) be tested and verified.
Two questions;
1. Mock for mockNotificationCenter, why not use method observerMock available in OCMock? It seems more easy.
id aMock = [OCMockObject observerMock]
2. mockNotificationCenter = nil; doesn’t work, dues to mockNotificationCenter is a static variable, need to use static method to set it.
My implementation:
@interface NSNotificationCenter (UnitTests)
+(id)createMockNotificationCenter;
+(id)createNiceMockNotificationCenter
+(void)releaseInstance;
@end
//———————————
static NSNotificationCenter * mockNotificationCenter = nil;
@implementation NSNotificationCenter (UnitTests)
+ (id)defaultCenter {
if(mockNotificationCenter != nil){
return mockNotificationCenter;
}
return invokeSupersequentNoArgs();
}
+(id)createMockNotificationCenter {
mockNotificationCenter = [OCMockObject mockForClass:[NSNotificationCenter class]];
return mockNotificationCenter;
}
+(id)createNiceMockNotificationCenter {
mockNotificationCenter = [OCMockObject niceMockForClass:[NSNotificationCenter class]];
return mockNotificationCenter;
}
+(void)releaseInstance {
mockNotificationCenter = nil;
}
@end
Thanks for the feedback!
1) I do sometimes use observerMock, but generally more in functional tests. In a unit test I generally want to avoid side effects, like other observers’ methods being called, so I feel like it’s cleaner to just mock the NSNotificationCenter. This way you can also verify your addObserver: and removeObserver: calls.
2) It depends how you set up your test classes. In my example, the static variable is defined inside my test class, and I can assign to it directly both from the NSNotificationCenter implementation and my tests. If it were defined in a separate scope, then you’re correct that you’d need a method on that class to set it from your test.
What I actually do is define all my static singletons in a base test class, which all of my tests extend. Then the tearDown method in the base test class looks like this, so I know the state is clean at the end of each test:
-(void)tearDown {
mockApplication = nil;
mockNotificationCenter = nil;
mockPaymentQueue = nil;
mockGanTracker = nil;
[super tearDown];
}
Thanks you kindly response.
For #1, agree with you, id aMock = [OCMockObject observerMock] can only mock a small part of actions.
For #2, now I see your design, it is cool, clean. I like it.
Plus with another question.
For singleton mock, how do you think of partial mocks?
id aMock = [OCMockObject partialMockForObject:anObject]. It is particularly usefully if you don’t want to mock your instance methods completely.
Do experiments and find that partialMockForObject also has its own defect, when used for static variable, it will remember original stub or expect.
Really tough, no better solution found so far.
Interesting. I hadn’t thought about using partial mocks for singletons, though I see how it could be a problem since the singleton instance is generally going to be instantiated once. I suppose you could fork OCMock and add a “removeAllExpectations” method or something.
Thanks Pickslay’s kindly response.
I have fired a question to OCMock dev group, try to discuss this issue with them.
No response from OCMock development side, fire a question on stackoverflow. http://stackoverflow.com/questions/8556527/using-partialmockforobject-to-do-singleton-class-mock-how-to-create-a-method-li
Hey Pickslay;
Sorry to trouble you again.
I still don’t understand how the static variable is defined inside your test class, what’s up in BaseTestCase? How do they communicate with category ones? Could you give me more tips?
Thanks very much.
Sure, take a look at http://dl.dropbox.com/u/3911390/BaseTestCase.m. I don’t actually have statics in my test classes–I wrote it that way in the post to make it more clear. All the statics are in my base test case, which all my tests extend.
Hey Pickslay;
Got it by another PC. One question, why do you declare it as a static method? I think (-) interface method will be easier to use, such as [self create....], how do you think of it?
+(id)createMockNotificationCenter;
+(id)createNiceMockNotificationCenter;
Finally appreciate your help very much.
Hey Pickslay;
Plus another question. Why not do mock verify on dealloc?
-(void)tearDown {
[mockApplication verify];
mockApplication = nil;
…
}
I think tearDown should be reserved for cleanup. If you want to verify a mock, you should do it explicitly in your test. The test is documenting what you think the code should do, so the verify is an explicit statement of what you expect.