-
Notifications
You must be signed in to change notification settings - Fork 192
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
Functional tests fail starting with a certain test #1569
Comments
@miketaylr @brizental |
I don't think this reproduces for me. @brizental does it for you? |
I just changed from Ubuntu to MacOS and this has started happening to me too... I'll have a look, now that I can reproduce it. |
Failing test starts here: https://github.com/webcompat/webcompat.com/blob/master/tests/functional/issue-list-non-auth.js#L290 |
The problem is that the I tried to look and see if the problem is in our code, but I didn't find anything wrong, so I think that this might be a problem in Leadfoot, it only happens for the newest Firefox versions, like 54 and forward. I also tried to mess around with Leadfoot's code, but didn't have success. Apparently everything is working fine, the URL changes right as it should and everything, it just gets stuck in some Promise limbo and never gets to the next line (https://github.com/webcompat/webcompat.com/blob/master/tests/functional/issue-list-non-auth.js#L321). Also if you manually refresh the page when it gets stuck it gets going again. I'll keep on investigating, but just wanted to see if anybody has any ideia of what might be happening :) |
It's also possible it's a Geckodriver bug or regression: https://github.com/mozilla/geckodriver |
Fixes #1569 - Gets 'Going back in history' test to work again.
The tests fail on all branches. It would appear that the tests fail from:
Going back in history updates issue list and URL state
Fx ANY: [✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓×××××✓✓✓✓×××✓\ ] 65/78, 15 fail
The text was updated successfully, but these errors were encountered: