|
30 | 30 | #import "OCMExceptionReturnValueProvider.h" |
31 | 31 | #import "OCMExpectationRecorder.h" |
32 | 32 |
|
| 33 | +@class XCTestCase; |
| 34 | +@class XCTest; |
| 35 | + |
| 36 | +// gMocksToStopRecorders is a stack of recorders that gets added to and removed from |
| 37 | +// as we enter test suite/case scopes. |
| 38 | +// Controlled by OCMockXCTestObserver. |
| 39 | +static NSMutableArray<NSHashTable<OCMockObject *> *> *gMocksToStopRecorders; |
| 40 | + |
| 41 | +// Flag that controls whether we should be asserting after stopmocking is called. |
| 42 | +// Controlled by OCMockXCTestObserver. |
| 43 | +static BOOL gAssertOnCallsAfterStopMocking; |
33 | 44 |
|
34 | 45 | @implementation OCMockObject |
35 | 46 |
|
36 | 47 | #pragma mark Class initialisation |
37 | 48 |
|
38 | 49 | + (void)initialize |
39 | 50 | { |
40 | | - if([[NSInvocation class] instanceMethodSignatureForSelector:@selector(getArgumentAtIndexAsObject:)] == NULL) |
41 | | - [NSException raise:NSInternalInconsistencyException format:@"** Expected method not present; the method getArgumentAtIndexAsObject: is not implemented by NSInvocation. If you see this exception it is likely that you are using the static library version of OCMock and your project is not configured correctly to load categories from static libraries. Did you forget to add the -ObjC linker flag?"]; |
| 51 | + if([[NSInvocation class] instanceMethodSignatureForSelector:@selector(getArgumentAtIndexAsObject:)] == NULL) |
| 52 | + { |
| 53 | + [NSException raise:NSInternalInconsistencyException format:@"** Expected method not present; the method getArgumentAtIndexAsObject: is not implemented by NSInvocation. If you see this exception it is likely that you are using the static library version of OCMock and your project is not configured correctly to load categories from static libraries. Did you forget to add the -ObjC linker flag?"]; |
| 54 | + } |
42 | 55 | } |
43 | 56 |
|
| 57 | +#pragma mark Mock cleanup recording |
| 58 | + |
| 59 | ++ (void)recordAMockToStop:(OCMockObject *)mock |
| 60 | +{ |
| 61 | + @synchronized(self) |
| 62 | + { |
| 63 | + [[gMocksToStopRecorders lastObject] addObject:mock]; |
| 64 | + } |
| 65 | +} |
| 66 | + |
| 67 | ++ (void)removeAMockToStop:(OCMockObject *)mock |
| 68 | +{ |
| 69 | + @synchronized(self) |
| 70 | + { |
| 71 | + [[gMocksToStopRecorders lastObject] removeObject:mock]; |
| 72 | + } |
| 73 | +} |
| 74 | + |
| 75 | ++ (void)stopAllCurrentMocks |
| 76 | +{ |
| 77 | + @synchronized(self) { |
| 78 | + NSHashTable<OCMockObject *> *recorder = [gMocksToStopRecorders lastObject]; |
| 79 | + for (OCMockObject *mock in recorder) |
| 80 | + { |
| 81 | + [mock stopMocking]; |
| 82 | + } |
| 83 | + [recorder removeAllObjects]; |
| 84 | + } |
| 85 | +} |
44 | 86 |
|
45 | 87 | #pragma mark Factory methods |
46 | 88 |
|
@@ -108,6 +150,7 @@ - (instancetype)init |
108 | 150 | expectations = [[NSMutableArray alloc] init]; |
109 | 151 | exceptions = [[NSMutableArray alloc] init]; |
110 | 152 | invocations = [[NSMutableArray alloc] init]; |
| 153 | + [OCMockObject recordAMockToStop:self]; |
111 | 154 | return self; |
112 | 155 | } |
113 | 156 |
|
@@ -146,10 +189,20 @@ - (void)assertInvocationsArrayIsPresent |
146 | 189 | { |
147 | 190 | if(invocations == nil) |
148 | 191 | { |
149 | | - [NSException raise:NSInternalInconsistencyException format:@"** Cannot use mock object %@ at %p. This error usually occurs when a mock object is used after stopMocking has been called on it. In most cases it is not necessary to call stopMocking. If you know you have to, please make sure that the mock object is not used afterwards.", [self description], (void *)self]; |
| 192 | + [OCMockObject logMatcherIssue:@"** Cannot use mock object %@ at %p. This error usually occurs when a mock object is used after stopMocking has been called on it. In most cases it is not necessary to call stopMocking. If you know you have to, please make sure that the mock object is not used afterwards.", [self description], (void *)self]; |
150 | 193 | } |
151 | 194 | } |
152 | 195 |
|
| 196 | ++ (void)logMatcherIssue:(NSString *)format, ... |
| 197 | +{ |
| 198 | + if(gAssertOnCallsAfterStopMocking) |
| 199 | + { |
| 200 | + va_list args; |
| 201 | + va_start(args, format); |
| 202 | + [NSException raise:NSInternalInconsistencyException format:format arguments:args]; |
| 203 | + va_end(args); |
| 204 | + } |
| 205 | +} |
153 | 206 |
|
154 | 207 | #pragma mark Public API |
155 | 208 |
|
@@ -459,7 +512,7 @@ - (void)handleUnRecordedInvocation:(NSInvocation *)anInvocation |
459 | 512 | { |
460 | 513 | if(isNice == NO) |
461 | 514 | { |
462 | | - [NSException raise:NSInternalInconsistencyException format:@"%@: unexpected method invoked: %@ %@", |
| 515 | + [OCMockObject logMatcherIssue:@"%@: unexpected method invoked: %@ %@", |
463 | 516 | [self description], [anInvocation invocationDescription], [self _stubDescriptions:NO]]; |
464 | 517 | } |
465 | 518 | } |
@@ -519,4 +572,83 @@ - (NSString *)_stubDescriptions:(BOOL)onlyExpectations |
519 | 572 | } |
520 | 573 |
|
521 | 574 |
|
| 575 | +@end |
| 576 | + |
| 577 | +/** |
| 578 | + * The observer gets installed the first time a mock object is created (see +[OCMockObject initialize] |
| 579 | + * It stops all the mocks that are still active when the testcase has finished. |
| 580 | + * In many cases this should break a lot of retain loops and allow mocks to be freed. |
| 581 | + * More importantly this will remove mocks that have mocked a class method and persist across testcases. |
| 582 | + * It intentionally turns off the assert that fires when calling a mock after stopMocking has been |
| 583 | + * called on it, because when we are doing cleanup there are cases in dealloc methods where a mock |
| 584 | + * may be called. We allow the "assert off" state to persist beyond the end of -testCaseDidFinish |
| 585 | + * because objects may be destroyed by the autoreleasepool that wraps the entire test and this may |
| 586 | + * cause mocks to be called. The state is global (instead of per mock) because we want to be able |
| 587 | + * to catch the case where a mock is trapped by some global state (e.g. a non-mock singleton) and |
| 588 | + * then that singleton is used in a later test and attempts to call a stopped mock. |
| 589 | + **/ |
| 590 | +@interface OCMockXCTestObserver : NSObject |
| 591 | +@end |
| 592 | + |
| 593 | +// "Fake" Protocol so we can avoid having to link to XCTest, but not get warnings about |
| 594 | +// methods not being declared. |
| 595 | +@protocol OCMockXCTestObservation |
| 596 | ++ (id)sharedTestObservationCenter; |
| 597 | +- (void)addTestObserver:(id)observer; |
| 598 | +@end |
| 599 | + |
| 600 | +@implementation OCMockXCTestObserver |
| 601 | + |
| 602 | ++ (void)load |
| 603 | +{ |
| 604 | + gMocksToStopRecorders = [[NSMutableArray alloc] init]; |
| 605 | + gAssertOnCallsAfterStopMocking = YES; |
| 606 | + Class xctest = NSClassFromString(@"XCTestObservationCenter"); |
| 607 | + if (xctest) |
| 608 | + { |
| 609 | + // If XCTest is available, we set up an observer to stop our mocks for us. |
| 610 | + [[xctest sharedTestObservationCenter] addTestObserver:[[OCMockXCTestObserver alloc] init]]; |
| 611 | + } |
| 612 | +} |
| 613 | + |
| 614 | +- (BOOL)conformsToProtocol:(Protocol *)aProtocol |
| 615 | +{ |
| 616 | + // This allows us to avoid linking XCTest into OCMock. |
| 617 | + return strcmp(protocol_getName(aProtocol), "XCTestObservation") == 0; |
| 618 | +} |
| 619 | + |
| 620 | +- (void)addRecorder |
| 621 | +{ |
| 622 | + gAssertOnCallsAfterStopMocking = YES; |
| 623 | + NSHashTable<OCMockObject *> *recorder = [NSHashTable weakObjectsHashTable]; |
| 624 | + [gMocksToStopRecorders addObject:recorder]; |
| 625 | +} |
| 626 | + |
| 627 | +- (void)finalizeRecorder |
| 628 | +{ |
| 629 | + gAssertOnCallsAfterStopMocking = NO; |
| 630 | + [OCMockObject stopAllCurrentMocks]; |
| 631 | + [gMocksToStopRecorders removeLastObject]; |
| 632 | +} |
| 633 | + |
| 634 | +- (void)testSuiteWillStart:(XCTestCase *)testCase |
| 635 | +{ |
| 636 | + [self addRecorder]; |
| 637 | +} |
| 638 | + |
| 639 | +- (void)testSuiteDidFinish:(XCTestCase *)testCase |
| 640 | +{ |
| 641 | + [self finalizeRecorder]; |
| 642 | +} |
| 643 | + |
| 644 | +- (void)testCaseWillStart:(XCTestCase *)testCase |
| 645 | +{ |
| 646 | + [self addRecorder]; |
| 647 | +} |
| 648 | + |
| 649 | +- (void)testCaseDidFinish:(XCTestCase *)testCase |
| 650 | +{ |
| 651 | + [self finalizeRecorder]; |
| 652 | +} |
| 653 | + |
522 | 654 | @end |
0 commit comments