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

Test Timeout Running Unit Tests on DDC on Linux #1113

Closed
todbachman-wf opened this issue Nov 26, 2019 · 3 comments
Closed

Test Timeout Running Unit Tests on DDC on Linux #1113

todbachman-wf opened this issue Nov 26, 2019 · 3 comments
Labels
needs-info Additional information needed from the issue author

Comments

@todbachman-wf
Copy link

I've run into this test timeout issue on DDC again: #1043 . I'm using the 1.9.4 release of the test package, so I should have the fix that that was put in place to addressed that issue. Once again, the issue is only happening in CI (where we run on Linux VMs) and not on a local Macbook pro. I've debugged this on a Linux VM and found that when the timeout occurs there is a requirejs error in the chrome dev console:

Uncaught Error: Load timeout for modules: language_test.dart.browser_test.dart.bootstrap
http://requirejs.org/docs/errors.html#timeout
    at makeError (:41593/RtPbIlR80P1hudA2GrzJ2EH6l95nHTOA/test/packages/build_web_compilers/src/dev_compiler/require.js:168)
    at checkLoaded (:41593/RtPbIlR80P1hudA2GrzJ2EH6l95nHTOA/test/packages/build_web_compilers/src/dev_compiler/require.js:698)
    at :41593/RtPbIlR80P1hudA2GrzJ2EH6l95nHTOA/test/packages/build_web_compilers/src/dev_compiler/require.js:719

I've tried setting the requirejs waitSeconds setting to 0. When I do that the suite loads consistently, but then I see a unit test timeout after ~30 seconds.

I have found a workaround, which is to set concurrency to 1, which surprised me a bit. I don't have good repro steps at this time, as we've only encountered this on one of our many, many internal Dart projects.

Dart version: 2.6.1
Chrome: 78

@kevmoo
Copy link
Member

kevmoo commented Feb 7, 2020

Still an issue? How are you running tests w/ DDC? (build_test, I'm guessing)

@kevmoo kevmoo added the needs-info Additional information needed from the issue author label Feb 7, 2020
@todbachman-wf
Copy link
Author

I just ran our unit tests in CI 30 times without the concurrency workaround applied and the tests passed every time. So it appears this is no longer an issue for us. Just fyi, we're running tests using build_test.

@grouma
Copy link
Member

grouma commented Feb 10, 2020

Thanks for the update. I'm going to close this one out.

@grouma grouma closed this as completed Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-info Additional information needed from the issue author
Projects
None yet
Development

No branches or pull requests

3 participants