Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[macOS] Clean up allocations in menu plugin test #48093

Merged
merged 1 commit into from
Nov 16, 2023

Commits on Nov 15, 2023

  1. [macOS] Clean up allocations in menu plugin test

    Runs all FlutterMenuPlugin tests in an AutoReleasepoolTest, which
    ensures all allocations are cleaned up.
    
    Also extracts out some hackery into the FlutterMenuPluginTest fixture to
    ensure that NSApplication instantiates everything necessary to do menu
    bar manipulation.
    
    Also replaces the use of OCMock in FlutterMenuPluginTest.mm with a fake
    FakePluginRegistrar class. This avoids unnecessary use of OCMock in the
    tests, which has been responsible for flakiness in some tests, in
    particular where the mock is used across threads. This test was not
    problematic, but the fake makes the tests more readable.
    
    Also fixes linter warnings about using NSLocalizedString for user-facing
    strings.
    
    Issue: flutter/flutter#104789
    Issue: flutter/flutter#127441
    Issue: flutter/flutter#124840
    cbracken committed Nov 15, 2023
    Configuration menu
    Copy the full SHA
    53cb871 View commit details
    Browse the repository at this point in the history