Skip to content

The tests are absurdly slow #161

Closed
Closed

Description

@FlutterIssues

Issue by Hixie
Tuesday Sep 15, 2015 at 19:37 GMT
Originally opened as https://github.com/flutter/engine/issues/1172


The tests take too long to run. I don't understand why; they should take mere milliseconds.

Activity

  1. added
    a: tests"flutter test", flutter_test, or one of our tests
    on Nov 9, 2015
  2. FlutterIssues commented on Nov 9, 2015

    @FlutterIssues
    Author

    Comment by abarth
    Tuesday Sep 15, 2015 at 19:57 GMT


    How long do they take to run on your machine?

  3. FlutterIssues commented on Nov 9, 2015

    @FlutterIssues
    Author

    Comment by Hixie
    Tuesday Sep 15, 2015 at 20:16 GMT


    43s wall-clock (1m18s user, presumably across two CPUs?).
    For comparison, the analyzer takes 6 seconds (which is also too long), and telling the device to start the app takes around 2 seconds (which is also too long).

  4. FlutterIssues commented on Nov 9, 2015

    @FlutterIssues
    Author

    Comment by abarth
    Tuesday Sep 15, 2015 at 20:25 GMT


    Sounds like the same issue @apwilson was having with the test harness crawling symlinks.

  5. FlutterIssues commented on Nov 9, 2015

    @FlutterIssues
    Author

    Comment by Hixie
    Tuesday Sep 15, 2015 at 20:44 GMT


    yeah, though i wasn't able to figure out which version of the test package it's using so i could change it. :-/

  6. FlutterIssues commented on Nov 9, 2015

    @FlutterIssues
    Author

    Comment by Hixie
    Wednesday Sep 16, 2015 at 19:02 GMT


    Oh hey I did a gclient sync and now it does pick up my hacked copy of test. Now it runs in 6 seconds!

  7. FlutterIssues commented on Nov 9, 2015

    @FlutterIssues
    Author

    Comment by apwilson
    Thursday Sep 17, 2015 at 20:29 GMT


    Awesome!

    Just for reference, the workaround is this: apwilson/test@368cf30

  8. FlutterIssues commented on Nov 9, 2015

    @FlutterIssues
    Author

    Comment by apwilson
    Thursday Sep 17, 2015 at 20:31 GMT


    And for reasons why dart/test don't want to implement that fix, see: dart-lang/test#337

    TLDR: This issue will fix itself in a few months when the dart packages link goes away.

  9. abarth commented on Nov 10, 2015

    @abarth
    Contributor

    @Hixie I think this is fixed now that flutter test does it's own walk to find the files.

  10. github-actions commented on Sep 7, 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.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    a: tests"flutter test", flutter_test, or one of our tests

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions