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

[SEVERE] test_api v0.4.6 causes "Timed out waiting for Chrome to connect." #1620

Closed
felangel opened this issue Oct 23, 2021 · 9 comments · Fixed by #1623
Closed

[SEVERE] test_api v0.4.6 causes "Timed out waiting for Chrome to connect." #1620

felangel opened this issue Oct 23, 2021 · 9 comments · Fixed by #1623
Assignees
Labels
P0 A serious issue requiring immediate resolution type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@felangel
Copy link

felangel commented Oct 23, 2021

Running dart run build_runner test --fail-on-severe with test_api 0.4.5 consistently runs the tests within 2-3 seconds whereas test_api 0.4.6 consistently exceeds the 30s timeout with:

Failed to load "test/my_test.dart": Timed out waiting for Chrome to connect.
  package:test/src/runner/browser/browser_manager.dart 132:7  BrowserManager.start.<fn>
  dart:async/zone.dart 1420:47                                _rootRun
  dart:async/zone.dart 1328:19                                _CustomZone.run
  dart:async/future_impl.dart 865:34                          Future.timeout.<fn>
  dart:async/zone.dart 1420:47                                _rootRun
  dart:async/zone.dart 1328:19                                _CustomZone.run
  dart:async/zone.dart 1236:7                                 _CustomZone.runGuarded
  dart:async/zone.dart 1276:23                                _CustomZone.bindCallbackGuarded.<fn>
  dart:async/zone.dart 1428:13                                _rootRun
  dart:async/zone.dart 1328:19                                _CustomZone.run
  dart:async/zone.dart 1260:23                                _CustomZone.bindCallback.<fn>
  dart:async-patch/timer_patch.dart 18:15                     Timer._createTimer.<fn>
  dart:isolate-patch/timer_impl.dart 395:19                   _Timer._runTimers
  dart:isolate-patch/timer_impl.dart 426:5                    _Timer._handleMessage
  dart:isolate-patch/isolate_patch.dart 184:12                _RawReceivePortImpl._handleMessage

Adding:

test_api: ">=0.4.0 <0.4.6"

Resolves the timeouts.

Tests Failing with timeout: https://github.com/felangel/bloc/runs/3982800253
Commit with fix: felangel/bloc@fc591f6
Tests Passing: https://github.com/felangel/bloc/runs/3982912418

Related Issues:

I'm guessing the bug was introduced in #1604 but I'm not totally sure.

Originally posted by @felangel in #1610 (comment)

@gmpassos
Copy link
Contributor

Just to highlight for investigation:

I speculate that this is where the bug was introduced:

https://github.com/dart-lang/test/blob/master/pkgs/test_core/lib/src/runner.dart#L261

I can't confirm it yet, but it's one of the significant changes from 0.4.5 to 0.4.6.

@ivoleitao
Copy link

I have the same exact error. All my browser tests cases started to fail

@d-markey
Copy link

Same here.

@gmpassos
Copy link
Contributor

Browser tests are definitely broken!

The title should have a "SEVERE" prefix.

@felangel felangel changed the title test_api v0.4.6 causes "Timed out waiting for Chrome to connect." [SEVERE] test_api v0.4.6 causes "Timed out waiting for Chrome to connect." Oct 23, 2021
@marandaneto
Copy link

marandaneto commented Oct 24, 2021

same for https://github.com/getsentry/sentry-dart/runs/3984790347?check_suite_focus=true

dev_dependencies:
  test: ">=1.18.2 <1.19.0"

is a workaround

Skycoder42 added a commit to Skycoder42/firebase_sync that referenced this issue Oct 25, 2021
@athomas
Copy link
Member

athomas commented Oct 25, 2021

This also affects dart-lang packages, e.g.:
https://github.com/dart-lang/characters/actions/runs/1376610144

@athomas athomas added the type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) label Oct 25, 2021
@jakemac53 jakemac53 self-assigned this Oct 25, 2021
@jakemac53 jakemac53 added the P0 A serious issue requiring immediate resolution label Oct 25, 2021
@jakemac53
Copy link
Contributor

Ah yes, looks like I got hit by dart-lang/pub#3126 again :(. We need to commit a local .pubignore or something to prevent this in the future until a fix is out for the pub validator :(.

@jakemac53
Copy link
Contributor

This should be fixed in test version 1.19.2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 A serious issue requiring immediate resolution type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants