Skip to content

build: fix webdriver tests #28915

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

Merged
merged 1 commit into from
Apr 22, 2024
Merged

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Apr 19, 2024

Reworks the Webdriver tests to avoid the frequent timouts that we've been seeing. Working theory is that refreshing the page on each test and then waiting for Angular to stabizilize causes Webdriver to eventually time out. These changes switch to only loading the page once and resetting its state within the same session.

@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Apr 19, 2024
@crisbeto crisbeto force-pushed the webdriver-no-stable branch 5 times, most recently from 80eebe6 to 5a88610 Compare April 22, 2024 07:43
Reworks the Webdriver tests to avoid the frequent timouts that we've been seeing. Working theory is that refreshing the page on each test and then waiting for Angular to stabizilize causes Webdriver to eventually time out. These changes switch to only loading the page once and resetting its state within the same session.
@crisbeto crisbeto force-pushed the webdriver-no-stable branch from 5a88610 to da1a89a Compare April 22, 2024 07:51
@crisbeto crisbeto changed the title build: test webdriver build: fix webdriver tests Apr 22, 2024
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Apr 22, 2024
@crisbeto crisbeto marked this pull request as ready for review April 22, 2024 07:59
@crisbeto crisbeto self-assigned this Apr 22, 2024
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Apr 22, 2024
@crisbeto crisbeto merged commit 971f83f into angular:main Apr 22, 2024
crisbeto added a commit that referenced this pull request Apr 22, 2024
Reworks the Webdriver tests to avoid the frequent timouts that we've been seeing. Working theory is that refreshing the page on each test and then waiting for Angular to stabizilize causes Webdriver to eventually time out. These changes switch to only loading the page once and resetting its state within the same session.

(cherry picked from commit 971f83f)
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators May 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants