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

Fix flakiness with stale-while-revalidate tests. #15362

Merged
merged 1 commit into from Feb 15, 2019

Conversation

dtapuska
Copy link
Contributor

Using cookies didn't seem reliable in the wpt test runner when it was
re-run. Stop using cookies and use the server side stash instead
like the rest of the fetch tests do.

Increase timeout on staleness.

Using cookies didn't seem reliable in the wpt test runner when it was
re-run. Stop using cookies and use the server side stash instead
like the rest of the fetch tests do.

Increase timeout on staleness.
@dtapuska
Copy link
Contributor Author

@foolip PTAL. This fixes the tests encountered on the chrome bot.

@dtapuska
Copy link
Contributor Author

@Hexcles PTAL

Copy link
Member

@Hexcles Hexcles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (assuming you've done some sanity checks by of the test results)

FWIW, I think the cookie approach might work if you added a cleanup hook to clear the cookies, but I'd prefer using stash, too.

Copy link
Member

@foolip foolip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @dtapuska!

@foolip foolip merged commit 2801597 into web-platform-tests:master Feb 15, 2019
marcoscaceres pushed a commit that referenced this pull request Jul 23, 2019
Using cookies didn't seem reliable in the wpt test runner when it was
re-run. Stop using cookies and use the server side stash instead
like the rest of the fetch tests do.

Increase timeout on staleness.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants