[macOS] Clean up resources in ViewController tests (#47792)
Wraps all FlutterViewController tests in an autorelease pool to ensure
resources are cleaned up.
Adds a MockFlutterEngineTest subclass of AutoreleasePoolTest that
creates an OCPartialMock FlutterEngine and shuts it down at the end of
the test. Previously we were not shutting down any FlutterEngine
instances we allocated, resulting in potentially thousands of threads
and graphics contexts being allocated.
Prior to this change, running these tests via:
../out/host_debug_unopt_arm64/flutter_desktop_darwin_unittests \
--gtest_filter='FlutterViewController.*' --gtest_repeat=1000
resulted in test failures and sometimes segfaults. This ensures
resources are cleaned up
Eventually all unit tests should configure their FlutterEngine via
either FlutterEngineTest (which should be an AutoreleasePoolTest) or
MockFlutterEngineTest, and the CreateMockFlutterEngine function moved to
a static used in the implementation of these.
Issue: https://github.com/flutter/flutter/issues/104789
Issue: https://github.com/flutter/flutter/issues/127441
Issue: https://github.com/flutter/flutter/issues/124840
## Pre-launch Checklist
- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide] and the [C++,
Objective-C, Java style guides].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I added new tests to check the change I am making or feature I am
adding, or the PR is [test-exempt]. See [testing the engine] for
instructions on writing and running engine tests.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I signed the [CLA].
- [x] All existing and new tests are passing.
If you need help, consider asking for advice on the #hackers-new channel
on [Discord].
<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
[test-exempt]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[C++, Objective-C, Java style guides]:
https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
[testing the engine]:
https://github.com/flutter/flutter/wiki/Testing-the-engine
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
[Discord]: https://github.com/flutter/flutter/wiki/Chatdiff --git a/shell/platform/darwin/macos/framework/Source/FlutterEngineTestUtils.h b/shell/platform/darwin/macos/framework/Source/FlutterEngineTestUtils.h
index b52524c..a0a58d4 100644
--- a/shell/platform/darwin/macos/framework/Source/FlutterEngineTestUtils.h
+++ b/shell/platform/darwin/macos/framework/Source/FlutterEngineTestUtils.h
@@ -5,6 +5,8 @@
#import "flutter/shell/platform/darwin/macos/framework/Headers/FlutterEngine.h"
#import <OCMock/OCMock.h>
+
+#include "flutter/testing/autoreleasepool_test.h"
#include "flutter/testing/test_dart_native_resolver.h"
#include "gtest/gtest.h"
@@ -36,6 +38,30 @@
// Returns a mock FlutterEngine that is able to work in environments
// without a real pasteboard.
+//
+// Callers MUST call [mockEngine shutDownEngine] when finished with the returned engine.
id CreateMockFlutterEngine(NSString* pasteboardString);
+class MockFlutterEngineTest : public AutoreleasePoolTest {
+ public:
+ MockFlutterEngineTest();
+
+ void SetUp() override;
+ void TearDown() override;
+
+ id GetMockEngine() { return engine_mock_; }
+
+ void ShutDownEngine();
+
+ ~MockFlutterEngineTest() {
+ [engine_mock_ shutDownEngine];
+ [engine_mock_ stopMocking];
+ }
+
+ private:
+ id engine_mock_;
+
+ FML_DISALLOW_COPY_AND_ASSIGN(MockFlutterEngineTest);
+};
+
} // namespace flutter::testing
diff --git a/shell/platform/darwin/macos/framework/Source/FlutterEngineTestUtils.mm b/shell/platform/darwin/macos/framework/Source/FlutterEngineTestUtils.mm
index 8a535aa..5e7ba44 100644
--- a/shell/platform/darwin/macos/framework/Source/FlutterEngineTestUtils.mm
+++ b/shell/platform/darwin/macos/framework/Source/FlutterEngineTestUtils.mm
@@ -67,4 +67,21 @@
}
}
+MockFlutterEngineTest::MockFlutterEngineTest() = default;
+
+void MockFlutterEngineTest::SetUp() {
+ engine_mock_ = CreateMockFlutterEngine(@"");
+}
+
+void MockFlutterEngineTest::TearDown() {
+ [engine_mock_ shutDownEngine];
+ [engine_mock_ stopMocking];
+ engine_mock_ = nil;
+}
+
+void MockFlutterEngineTest::ShutDownEngine() {
+ [engine_mock_ shutDownEngine];
+ engine_mock_ = nil;
+}
+
} // namespace flutter::testing
diff --git a/shell/platform/darwin/macos/framework/Source/FlutterViewControllerTest.mm b/shell/platform/darwin/macos/framework/Source/FlutterViewControllerTest.mm
index e5f5bef..93579f5 100644
--- a/shell/platform/darwin/macos/framework/Source/FlutterViewControllerTest.mm
+++ b/shell/platform/darwin/macos/framework/Source/FlutterViewControllerTest.mm
@@ -16,7 +16,8 @@
#import "flutter/shell/platform/darwin/macos/framework/Source/FlutterRenderer.h"
#import "flutter/shell/platform/darwin/macos/framework/Source/FlutterViewControllerTestUtils.h"
#include "flutter/shell/platform/embedder/test_utils/key_codes.g.h"
-#import "flutter/testing/testing.h"
+#include "flutter/testing/autoreleasepool_test.h"
+#include "flutter/testing/testing.h"
#pragma mark - Test Helper Classes
@@ -92,18 +93,18 @@
@end
@interface FlutterViewControllerTestObjC : NSObject
-- (bool)testKeyEventsAreSentToFramework;
-- (bool)testKeyEventsArePropagatedIfNotHandled;
-- (bool)testKeyEventsAreNotPropagatedIfHandled;
-- (bool)testCtrlTabKeyEventIsPropagated;
-- (bool)testKeyEquivalentIsPassedToTextInputPlugin;
-- (bool)testFlagsChangedEventsArePropagatedIfNotHandled;
-- (bool)testKeyboardIsRestartedOnEngineRestart;
-- (bool)testTrackpadGesturesAreSentToFramework;
-- (bool)testMouseDownUpEventsSentToNextResponder;
-- (bool)testModifierKeysAreSynthesizedOnMouseMove;
-- (bool)testViewWillAppearCalledMultipleTimes;
-- (bool)testFlutterViewIsConfigured;
+- (bool)testKeyEventsAreSentToFramework:(id)mockEngine;
+- (bool)testKeyEventsArePropagatedIfNotHandled:(id)mockEngine;
+- (bool)testKeyEventsAreNotPropagatedIfHandled:(id)mockEngine;
+- (bool)testCtrlTabKeyEventIsPropagated:(id)mockEngine;
+- (bool)testKeyEquivalentIsPassedToTextInputPlugin:(id)mockEngine;
+- (bool)testFlagsChangedEventsArePropagatedIfNotHandled:(id)mockEngine;
+- (bool)testKeyboardIsRestartedOnEngineRestart:(id)mockEngine;
+- (bool)testTrackpadGesturesAreSentToFramework:(id)mockEngine;
+- (bool)testMouseDownUpEventsSentToNextResponder:(id)mockEngine;
+- (bool)testModifierKeysAreSynthesizedOnMouseMove:(id)mockEngine;
+- (bool)testViewWillAppearCalledMultipleTimes:(id)mockEngine;
+- (bool)testFlutterViewIsConfigured:(id)mockEngine;
- (bool)testLookupKeyAssets;
- (bool)testLookupKeyAssetsWithPackage;
- (bool)testViewControllerIsReleased;
@@ -173,7 +174,27 @@
#pragma mark - gtest tests
-TEST(FlutterViewController, HasViewThatHidesOtherViewsInAccessibility) {
+// AutoreleasePoolTest subclass that exists simply to provide more specific naming.
+class FlutterViewControllerTest : public AutoreleasePoolTest {
+ public:
+ FlutterViewControllerTest() = default;
+ ~FlutterViewControllerTest() = default;
+
+ private:
+ FML_DISALLOW_COPY_AND_ASSIGN(FlutterViewControllerTest);
+};
+
+// MockFlutterEngineTest subclass that exists simply to provide more specific naming.
+class FlutterViewControllerMockEngineTest : public MockFlutterEngineTest {
+ public:
+ FlutterViewControllerMockEngineTest() = default;
+ ~FlutterViewControllerMockEngineTest() = default;
+
+ private:
+ FML_DISALLOW_COPY_AND_ASSIGN(FlutterViewControllerMockEngineTest);
+};
+
+TEST_F(FlutterViewControllerTest, HasViewThatHidesOtherViewsInAccessibility) {
FlutterViewController* viewControllerMock = CreateMockViewController();
[viewControllerMock loadView];
@@ -194,13 +215,13 @@
EXPECT_EQ(accessibilityChildren[0], viewControllerMock.flutterView);
}
-TEST(FlutterViewController, FlutterViewAcceptsFirstMouse) {
+TEST_F(FlutterViewControllerTest, FlutterViewAcceptsFirstMouse) {
FlutterViewController* viewControllerMock = CreateMockViewController();
[viewControllerMock loadView];
EXPECT_EQ([viewControllerMock.flutterView acceptsFirstMouse:nil], YES);
}
-TEST(FlutterViewController, ReparentsPluginWhenAccessibilityDisabled) {
+TEST_F(FlutterViewControllerTest, ReparentsPluginWhenAccessibilityDisabled) {
FlutterEngine* engine = CreateTestEngine();
FlutterViewController* viewController = [[FlutterViewController alloc] initWithEngine:engine
nibName:nil
@@ -226,7 +247,7 @@
EXPECT_TRUE(viewController.textInputPlugin.superview == viewController.view);
}
-TEST(FlutterViewController, CanSetMouseTrackingModeBeforeViewLoaded) {
+TEST_F(FlutterViewControllerTest, CanSetMouseTrackingModeBeforeViewLoaded) {
NSString* fixtures = @(testing::GetFixturesPath());
FlutterDartProject* project = [[FlutterDartProject alloc]
initWithAssetsPath:fixtures
@@ -236,64 +257,84 @@
ASSERT_EQ(viewController.mouseTrackingMode, kFlutterMouseTrackingModeInActiveApp);
}
-TEST(FlutterViewControllerTest, TestKeyEventsAreSentToFramework) {
- ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testKeyEventsAreSentToFramework]);
+TEST_F(FlutterViewControllerMockEngineTest, TestKeyEventsAreSentToFramework) {
+ id mockEngine = GetMockEngine();
+ ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testKeyEventsAreSentToFramework:mockEngine]);
}
-TEST(FlutterViewControllerTest, TestKeyEventsArePropagatedIfNotHandled) {
- ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testKeyEventsArePropagatedIfNotHandled]);
-}
-
-TEST(FlutterViewControllerTest, TestKeyEventsAreNotPropagatedIfHandled) {
- ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testKeyEventsAreNotPropagatedIfHandled]);
-}
-
-TEST(FlutterViewControllerTest, TestCtrlTabKeyEventIsPropagated) {
- ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testCtrlTabKeyEventIsPropagated]);
-}
-
-TEST(FlutterViewControllerTest, TestKeyEquivalentIsPassedToTextInputPlugin) {
- ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testKeyEquivalentIsPassedToTextInputPlugin]);
-}
-
-TEST(FlutterViewControllerTest, TestFlagsChangedEventsArePropagatedIfNotHandled) {
+TEST_F(FlutterViewControllerMockEngineTest, TestKeyEventsArePropagatedIfNotHandled) {
+ id mockEngine = GetMockEngine();
ASSERT_TRUE(
- [[FlutterViewControllerTestObjC alloc] testFlagsChangedEventsArePropagatedIfNotHandled]);
+ [[FlutterViewControllerTestObjC alloc] testKeyEventsArePropagatedIfNotHandled:mockEngine]);
}
-TEST(FlutterViewControllerTest, TestKeyboardIsRestartedOnEngineRestart) {
- ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testKeyboardIsRestartedOnEngineRestart]);
+TEST_F(FlutterViewControllerMockEngineTest, TestKeyEventsAreNotPropagatedIfHandled) {
+ id mockEngine = GetMockEngine();
+ ASSERT_TRUE(
+ [[FlutterViewControllerTestObjC alloc] testKeyEventsAreNotPropagatedIfHandled:mockEngine]);
}
-TEST(FlutterViewControllerTest, TestTrackpadGesturesAreSentToFramework) {
- ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testTrackpadGesturesAreSentToFramework]);
+TEST_F(FlutterViewControllerMockEngineTest, TestCtrlTabKeyEventIsPropagated) {
+ id mockEngine = GetMockEngine();
+ ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testCtrlTabKeyEventIsPropagated:mockEngine]);
}
-TEST(FlutterViewControllerTest, TestMouseDownUpEventsSentToNextResponder) {
- ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testMouseDownUpEventsSentToNextResponder]);
+TEST_F(FlutterViewControllerMockEngineTest, TestKeyEquivalentIsPassedToTextInputPlugin) {
+ id mockEngine = GetMockEngine();
+ ASSERT_TRUE([[FlutterViewControllerTestObjC alloc]
+ testKeyEquivalentIsPassedToTextInputPlugin:mockEngine]);
}
-TEST(FlutterViewControllerTest, TestModifierKeysAreSynthesizedOnMouseMove) {
- ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testModifierKeysAreSynthesizedOnMouseMove]);
+TEST_F(FlutterViewControllerMockEngineTest, TestFlagsChangedEventsArePropagatedIfNotHandled) {
+ id mockEngine = GetMockEngine();
+ ASSERT_TRUE([[FlutterViewControllerTestObjC alloc]
+ testFlagsChangedEventsArePropagatedIfNotHandled:mockEngine]);
}
-TEST(FlutterViewControllerTest, testViewWillAppearCalledMultipleTimes) {
- ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testViewWillAppearCalledMultipleTimes]);
+TEST_F(FlutterViewControllerMockEngineTest, TestKeyboardIsRestartedOnEngineRestart) {
+ id mockEngine = GetMockEngine();
+ ASSERT_TRUE(
+ [[FlutterViewControllerTestObjC alloc] testKeyboardIsRestartedOnEngineRestart:mockEngine]);
}
-TEST(FlutterViewControllerTest, testFlutterViewIsConfigured) {
- ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testFlutterViewIsConfigured]);
+TEST_F(FlutterViewControllerMockEngineTest, TestTrackpadGesturesAreSentToFramework) {
+ id mockEngine = GetMockEngine();
+ ASSERT_TRUE(
+ [[FlutterViewControllerTestObjC alloc] testTrackpadGesturesAreSentToFramework:mockEngine]);
}
-TEST(FlutterViewControllerTest, testLookupKeyAssets) {
+TEST_F(FlutterViewControllerMockEngineTest, TestMouseDownUpEventsSentToNextResponder) {
+ id mockEngine = GetMockEngine();
+ ASSERT_TRUE(
+ [[FlutterViewControllerTestObjC alloc] testMouseDownUpEventsSentToNextResponder:mockEngine]);
+}
+
+TEST_F(FlutterViewControllerMockEngineTest, TestModifierKeysAreSynthesizedOnMouseMove) {
+ id mockEngine = GetMockEngine();
+ ASSERT_TRUE(
+ [[FlutterViewControllerTestObjC alloc] testModifierKeysAreSynthesizedOnMouseMove:mockEngine]);
+}
+
+TEST_F(FlutterViewControllerMockEngineTest, testViewWillAppearCalledMultipleTimes) {
+ id mockEngine = GetMockEngine();
+ ASSERT_TRUE(
+ [[FlutterViewControllerTestObjC alloc] testViewWillAppearCalledMultipleTimes:mockEngine]);
+}
+
+TEST_F(FlutterViewControllerMockEngineTest, testFlutterViewIsConfigured) {
+ id mockEngine = GetMockEngine();
+ ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testFlutterViewIsConfigured:mockEngine]);
+}
+
+TEST_F(FlutterViewControllerTest, testLookupKeyAssets) {
ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testLookupKeyAssets]);
}
-TEST(FlutterViewControllerTest, testLookupKeyAssetsWithPackage) {
+TEST_F(FlutterViewControllerTest, testLookupKeyAssetsWithPackage) {
ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testLookupKeyAssetsWithPackage]);
}
-TEST(FlutterViewControllerTest, testViewControllerIsReleased) {
+TEST_F(FlutterViewControllerTest, testViewControllerIsReleased) {
ASSERT_TRUE([[FlutterViewControllerTestObjC alloc] testViewControllerIsReleased]);
}
@@ -303,8 +344,7 @@
@implementation FlutterViewControllerTestObjC
-- (bool)testKeyEventsAreSentToFramework {
- id engineMock = flutter::testing::CreateMockFlutterEngine(@"");
+- (bool)testKeyEventsAreSentToFramework:(id)engineMock {
id binaryMessengerMock = OCMProtocolMock(@protocol(FlutterBinaryMessenger));
OCMStub( // NOLINT(google-objc-avoid-throwing-exception)
[engineMock binaryMessenger])
@@ -342,8 +382,7 @@
}
// Regression test for https://github.com/flutter/flutter/issues/122084.
-- (bool)testCtrlTabKeyEventIsPropagated {
- id engineMock = flutter::testing::CreateMockFlutterEngine(@"");
+- (bool)testCtrlTabKeyEventIsPropagated:(id)engineMock {
__block bool called = false;
__block FlutterKeyEvent last_event;
OCMStub([[engineMock ignoringNonObjectArgs] sendKeyEvent:FlutterKeyEvent {}
@@ -387,8 +426,7 @@
return true;
}
-- (bool)testKeyEquivalentIsPassedToTextInputPlugin {
- id engineMock = flutter::testing::CreateMockFlutterEngine(@"");
+- (bool)testKeyEquivalentIsPassedToTextInputPlugin:(id)engineMock {
__block bool called = false;
__block FlutterKeyEvent last_event;
OCMStub([[engineMock ignoringNonObjectArgs] sendKeyEvent:FlutterKeyEvent {}
@@ -438,8 +476,7 @@
return true;
}
-- (bool)testKeyEventsArePropagatedIfNotHandled {
- id engineMock = flutter::testing::CreateMockFlutterEngine(@"");
+- (bool)testKeyEventsArePropagatedIfNotHandled:(id)engineMock {
id binaryMessengerMock = OCMProtocolMock(@protocol(FlutterBinaryMessenger));
OCMStub( // NOLINT(google-objc-avoid-throwing-exception)
[engineMock binaryMessenger])
@@ -494,9 +531,7 @@
return true;
}
-- (bool)testFlutterViewIsConfigured {
- id engineMock = flutter::testing::CreateMockFlutterEngine(@"");
-
+- (bool)testFlutterViewIsConfigured:(id)engineMock {
FlutterRenderer* renderer_ = [[FlutterRenderer alloc] initWithFlutterEngine:engineMock];
OCMStub([engineMock renderer]).andReturn(renderer_);
@@ -515,8 +550,7 @@
return true;
}
-- (bool)testFlagsChangedEventsArePropagatedIfNotHandled {
- id engineMock = flutter::testing::CreateMockFlutterEngine(@"");
+- (bool)testFlagsChangedEventsArePropagatedIfNotHandled:(id)engineMock {
id binaryMessengerMock = OCMProtocolMock(@protocol(FlutterBinaryMessenger));
OCMStub( // NOLINT(google-objc-avoid-throwing-exception)
[engineMock binaryMessenger])
@@ -569,8 +603,7 @@
return true;
}
-- (bool)testKeyEventsAreNotPropagatedIfHandled {
- id engineMock = flutter::testing::CreateMockFlutterEngine(@"");
+- (bool)testKeyEventsAreNotPropagatedIfHandled:(id)engineMock {
id binaryMessengerMock = OCMProtocolMock(@protocol(FlutterBinaryMessenger));
OCMStub( // NOLINT(google-objc-avoid-throwing-exception)
[engineMock binaryMessenger])
@@ -625,8 +658,7 @@
return true;
}
-- (bool)testKeyboardIsRestartedOnEngineRestart {
- id engineMock = flutter::testing::CreateMockFlutterEngine(@"");
+- (bool)testKeyboardIsRestartedOnEngineRestart:(id)engineMock {
id binaryMessengerMock = OCMProtocolMock(@protocol(FlutterBinaryMessenger));
OCMStub( // NOLINT(google-objc-avoid-throwing-exception)
[engineMock binaryMessenger])
@@ -687,8 +719,7 @@
}
}
-- (bool)testTrackpadGesturesAreSentToFramework {
- id engineMock = flutter::testing::CreateMockFlutterEngine(@"");
+- (bool)testTrackpadGesturesAreSentToFramework:(id)engineMock {
// Need to return a real renderer to allow view controller to load.
FlutterRenderer* renderer_ = [[FlutterRenderer alloc] initWithFlutterEngine:engineMock];
OCMStub([engineMock renderer]).andReturn(renderer_);
@@ -983,8 +1014,7 @@
return true;
}
-- (bool)testViewWillAppearCalledMultipleTimes {
- id engineMock = flutter::testing::CreateMockFlutterEngine(@"");
+- (bool)testViewWillAppearCalledMultipleTimes:(id)engineMock {
FlutterViewController* viewController = [[FlutterViewController alloc] initWithEngine:engineMock
nibName:@""
bundle:nil];
@@ -1020,7 +1050,7 @@
// See: https://github.com/flutter/flutter/issues/115015
// See: http://www.openradar.me/FB12050037
// See: https://developer.apple.com/documentation/appkit/nsresponder/1524634-mousedown
-- (bool)testMouseDownUpEventsSentToNextResponder {
+- (bool)testMouseDownUpEventsSentToNextResponder:(id)engineMock {
// The root cause of the above bug is NSResponder mouseDown/mouseUp methods that don't correctly
// walk the responder chain calling the appropriate method on the next responder under certain
// conditions. Simulate this by swizzling out the default implementations and replacing them with
@@ -1032,7 +1062,6 @@
IMP origMouseUp = method_setImplementation(mouseUp, noopImp);
// Verify that mouseDown/mouseUp trigger mouseDown/mouseUp calls on FlutterViewController.
- id engineMock = flutter::testing::CreateMockFlutterEngine(@"");
MouseEventFlutterViewController* viewController =
[[MouseEventFlutterViewController alloc] initWithEngine:engineMock nibName:@"" bundle:nil];
FlutterView* view = (FlutterView*)[viewController view];
@@ -1057,8 +1086,7 @@
return true;
}
-- (bool)testModifierKeysAreSynthesizedOnMouseMove {
- id engineMock = flutter::testing::CreateMockFlutterEngine(@"");
+- (bool)testModifierKeysAreSynthesizedOnMouseMove:(id)engineMock {
id binaryMessengerMock = OCMProtocolMock(@protocol(FlutterBinaryMessenger));
OCMStub( // NOLINT(google-objc-avoid-throwing-exception)
[engineMock binaryMessenger])
@@ -1177,10 +1205,11 @@
bundle:nil];
[viewController loadView];
weakController = viewController;
+
+ [engineMock shutDownEngine];
}
EXPECT_EQ(weakController, nil);
-
return true;
}
