Skip to content

Test quote-run-large.scala timing out #3827

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.

Sign up for GitHub

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

Jump to bottom
Closed
smarter opened this issue Jan 14, 2018 · 3 comments
Closed

Test quote-run-large.scala timing out #3827

smarter opened this issue Jan 14, 2018 · 3 comments
Assignees
@nicolasstucki
Labels
area:testing itype:performance prio:low

Comments

@smarter
Copy link
Member

smarter commented Jan 14, 2018

This test timed out at least once (http://dotty-ci.epfl.ch/lampepfl/dotty/2518/6):

failed because test ../tests/run-special/quote-run-large.scala timed out        
[error] Test dotty.tools.dotc.CompilationTests.runAll failed: java.lang.AssertionError: Run test failed, but should not, reasons:
[error]   - test '../tests/run-special/quote-run-large.scala' timed out, took 87.203 sec
[error]     at dotty.tools.vulpix.ParallelTesting$CompilationTest.checkRuns(ParallelTesting.scala:925)
[error]     at dotty.tools.dotc.CompilationTests.runAll(CompilationTests.scala:207)
[error] ...

The CI was under heavy stress at that time. We may need to increase the timeout value, either globally or just for this test?
@smarter smarter changed the title Test timing out Test quote-run-large.scala timing out Jan 14, 2018
@smarter smarter mentioned this issue Jan 14, 2018
Merged
@smarter
Copy link
Member Author

smarter commented Jan 14, 2018

Happened again in http://dotty-ci.epfl.ch/lampepfl/dotty/2525/4

@nicolasstucki nicolasstucki self-assigned this Jan 14, 2018
smarter added a commit that referenced this issue Jan 14, 2018
@smarter
Merge pull request #3829 from dotty-staging/fix-#3827
032d8e3 
Fix #3827: Reduce runtime of quote-run-large by half
smarter referenced this issue Jan 14, 2018
@smarter
Merge pull request #3832 from dotty-staging/fix-#3827
0266c2c 
Shave of more time from quote-run-large.scala
@smarter
Copy link
Member Author

smarter commented Jan 15, 2018

The timeout happened again: http://dotty-ci.epfl.ch/lampepfl/dotty/2603

@smarter smarter reopened this Jan 15, 2018
@nicolasstucki
Copy link
Contributor

Currently that tests is in tests/pending

nicolasstucki added a commit that referenced this issue Apr 3, 2018
@nicolasstucki
Merge pull request #4216 from dotty-staging/fix-#3827
1fdb94b 
Fix #3827: Reenable large quote test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nicolasstucki nicolasstucki

Labels
area:testing itype:performance prio:low
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@smarter @nicolasstucki