Skip to content

flutter test leaks frontend_server.dart.snapshot on exception #34613

Closed

Description

@jmagman

Steps to Reproduce

frontend_server.dart.snapshot leaks after flutter test fails with engine exception.

$ cd packages/flutter
$ flutter test

Logs

[        ] test 178: error caught during test; sending to test framework
[        ] test 178: cleaning up...
[        ] test 178: deleting temporary directory
[   +1 ms] test 178: shutting down test harness socket server
00:58 +1175 ~25 -2: loading flutter/packages/flutter/test/cupertino/route_test.dart [E]                                                                                         
  Failed to load "flutter/packages/flutter/test/cupertino/route_test.dart":
  Too many open files
...
02:56 +1487 ~34 -339: Some tests failed.                                                                                                                                                                                          
thistle flutter$  ps -ef | grep frontend_server
670434 55506     1   0  4:23PM ttys000    3:07.85 flutter/bin/cache/dart-sdk/bin/dart flutter/bin/cache/artifacts/engine/darwin-x64/frontend_server.dart.snapshot --sdk-root flutter/bin/cache/artifacts/engine/common/flutter_patched_sdk/ --incremental --strong --target=flutter --output-dill /var/folders/mv/qlhc4vsj6tqct166rw3zwmth00mfq2/T/flutter_test_compiler.pbdWfe/output.dill --packages flutter/packages/flutter/.packages --initialize-from-dill flutter/packages/flutter/build/testfile.dill
670434 56385 91983   0  4:26PM ttys000    0:00.00 grep frontend_server
$ flutter doctor -v
[✓] Flutter (Channel master, v1.7.5-pre.16, on Mac OS X 10.14.5 18F203, locale en-US)
    • Flutter version 1.7.5-pre.16 at /Users/magder/Projects/flutter
    • Framework revision ac2f85bb63 (8 hours ago), 2019-06-17 11:31:58 -0400
    • Engine revision 308c11301a
    • Dart version 2.4.0

 
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/magder/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 10.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2.1, Build version 10E1001
    • CocoaPods version 1.7.1

[✓] iOS tools - develop for iOS devices
    • ios-deploy 1.9.4

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 36.1.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] VS Code (version 1.35.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.1.0

[✓] Connected device (1 available)
    • macOS • macOS • darwin-x64 • Mac OS X 10.14.5 18F203

• No issues found!

Activity

  1. jmagman commented on Jun 17, 2019

    @jmagman
    MemberAuthor

    Discovered this while investigating #34612. Note this issue is tracking the leak of flutter_test, NOT the test exception.
    I suspect this is #20949 but I want to track separately in case there's more work to be done to handle a SIGINT.

  2. self-assigned this
    on Jun 17, 2019
  3. added
    a: tests"flutter test", flutter_test, or one of our tests
    toolAffects the "flutter" command-line tool. See also t: labels.
    on Jun 17, 2019
  4. added this to the June 2019 milestone on Jun 17, 2019
  5. changed the title [-]flutter test leaks flutter_tester on exception[/-] [+]flutter test leaks frontend_server.dart.snapshot on exception[/+] on Jun 17, 2019
  6. jmagman commented on Jun 17, 2019

    @jmagman
    MemberAuthor

    Ignore the #20949 mention, this is tracking the leak of the frontend_server, not flutter_test.

  7. github-actions commented on Aug 29, 2021

    @github-actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

  8. locked as resolved and limited conversation to collaborators on Aug 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

a: tests"flutter test", flutter_test, or one of our teststoolAffects the "flutter" command-line tool. See also t: labels.

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions