ampproject / ampproject/amphtml

flaky e2e test for amp-consent

Open
#37,403 2 comments 0 reactions 0 assignees View on GitHub
Component: amp-consent Related to: Flaky Tests Type: Bug WG: components
Dominant language
JavaScript
Stars
14.9k
Forks
4.1k
PR merge metrics
No merged PRs in 30d

Description

### Description

The tests below are flaky:

amp-consent chrome Standalone environment "after each" hook for "should restrict fullscreen until user interaction"
amp-consent chrome Standalone environment "before each" hook for "should restrict fullscreen until user interaction"

### Reproduction Steps

3 failures so far:

Jan 11: https://github.com/ampproject/amphtml/commit/755700547d12669a649042f54bfb38d828e59bb6
Jan 12: https://github.com/ampproject/amphtml/commit/96bebbc98d1f79b185c5ff6c58442ddc2092547d
Jan 13: https://github.com/ampproject/amphtml/commit/e9b16ad7a5cee398ceaabf6ba7990488624e098b

Possibly related failures:

Jan 10
amp-consent chrome Standalone environment should work with client side decision
https://github.com/ampproject/amphtml/commit/f0b7ffe685c60b51cfbab92bada4e0c87e0377a1

Dec 8
amp-consent chrome Standalone environment should respect server side decision and clear on next visit
https://github.com/ampproject/amphtml/commit/f2f8ef91af1484c76c608185cbbe22f13864a190

### Relevant Logs

_No response_

### Browser(s) Affected

_No response_

### OS(s) Affected

_No response_

### Device(s) Affected

_No response_

### AMP Version Affected

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.