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

api-test: failure on 'navigator' suite #8812

Closed
vince-fugnitto opened this issue Dec 1, 2020 · 5 comments
Closed

api-test: failure on 'navigator' suite #8812

vince-fugnitto opened this issue Dec 1, 2020 · 5 comments
Labels
ci issues related to CI / tests test issues related to unit and api tests

Comments

@vince-fugnitto
Copy link
Member

Bug Description:

The navigator @theia/api-tests suite is currently failing on recent builds:

root INFO   Navigator
root INFO     1) "before each" hook for "copy file"
root INFO     2) "after each" hook for "copy file"

Steps to reproduce:

The error is reproduced in recent builds (github actions), but locally the tests pass (using npx run test @theia/example-browser)

@vince-fugnitto vince-fugnitto added ci issues related to CI / tests test issues related to unit and api tests labels Dec 1, 2020
@tsmaeder
Copy link
Contributor

tsmaeder commented Feb 8, 2021

@vince-fugnitto for me, they fail locally as well, but only when running all tests. Running `navigator-spec.js' in isolation always succeeds.
#9027 first failed on the windows run, rerunning them made them fail on ubuntu. This smells like timing issues.

@vince-fugnitto
Copy link
Member Author

@tsmaeder I've noticed that as well, alone the test-suite is fine but with the rest it fails intermittently (and pretty often on github actions). I'll have to try and find the time to see what can be done to resolve the issue unless someone else wants to try their hand at fixing it.

@tsmaeder
Copy link
Contributor

tsmaeder commented Feb 8, 2021

I have a suspicion that the underlying issue is #7953 making the IDE super-slow.

@vince-fugnitto
Copy link
Member Author

I have a suspicion that the underlying issue is #7953 making the IDE super-slow.

@tsmaeder I'm not sure, the following PR #9045 is mentioned as fixing #7953 but the CI still fails for it with the navigator test-suite.

@vince-fugnitto
Copy link
Member Author

Closing for now thanks to #9123.
I'll re-open the issue if the navigator test-suite regresses again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci issues related to CI / tests test issues related to unit and api tests
Projects
None yet
Development

No branches or pull requests

2 participants