Repository navigation
fix: point webdriver CI tests at the live production URL - #25688
Open
akashreddy-nr wants to merge 3 commits into
Open
akashreddy-nr wants to merge 3 commits into
akashreddy-nr wants to merge 3 commits into
Conversation
develop--docs-website-netlify.netlify.app and main--docs-website-netlify.netlify.app both 404 (confirmed via direct fetch) — that branch-deploy alias no longer exists for this site, while the bare production alias (docs-website-netlify.netlify.app) is live. This made the "webdriver test" check fail on every PR to main regardless of content, going back at least two weeks.
Contributor
|
Hi @akashreddy-nr 👋 Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days. Please ensure the propsed changes look good by building it first in your local environment. Refer to this contribution guide to get the site up and running in your local. If you really require a preview url, reach out to one of the writers and they will generate one for you. |
Testing against the real production URL (previous commit) surfaced a genuine gap: on the iPhone 12 Pro viewport, the Osano cookie consent banner overlaps the homepage doc tile after scrollIntoView, so the click lands on the banner instead — ElementClickInterceptedError. Dismiss the banner first, same as a real user would.
The previous selector guessed at an osano-cm-accept* class that doesn't exist in this deployment. The rendered banner exposes an aria-label="Close this consent banner" on its close control, which is a stable, accessible selector to target instead of guessing markup.
akashreddy-nr
force-pushed
the
fix/webdriver-dead-url
branch
from
September 14, 2026 17:01
2c54335 to
3c8135d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.github/workflows/webdriver.yml, all failures.webdriver-desktop.mjs/webdriver-mobile.mjshardcodehttps://develop--docs-website-netlify.netlify.app/as the test target. That branch-deploy alias no longer resolves (confirmed 404 via direct fetch, as doesmain--docs-website-netlify.netlify.app), while the bare production aliashttps://docs-website-netlify.netlify.app/is live and serves the expected/docs/mdx-test-page/content (verified the collapser markup renders there).Test plan