ampproject / ampproject/amphtml
Fix tests broken in updated ChromeDriver version
- Dominant language
- JavaScript
- Stars
- 14.9k
- Forks
- 4.1k
- PR merge metrics
- No merged PRs in 30d
Description
PRs #38965 and #39074 updated `chromedriver` from v96 to v113, the following tests were broken and skipped as a result:
* All tests in extensions/amp-powr-player/0.1/test/test-amp-powr-player.js
* test `should execute next and prev actions` in extensions/amp-base-carousel/1.0/test/test-amp-base-carousel.js
* test `should execute next and prev actions` in extensions/amp-stream-gallery/1.0/test/test-amp-stream-gallery.js
* tests `PreactBaseElement> shadow container rendering > SSR > *` in test/unit/preact/test-base-element-mapping.js
Contributor guide
Research direction
Start with the skipped tests in extensions/amp-powr-player/0.1/test/test-amp-powr-player.js, extensions/amp-base-carousel/1.0/test/test-amp-base-carousel.js, extensions/amp-stream-gallery/1.0/test/test-amp-stream-gallery.js, and test/unit/preact/test-base-element-mapping.js. Run these tests with the updated ChromeDriver and restore them so the listed cases pass without skips.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- testing-qa
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100