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 a flakiness observed when using master.runScript() #618

Merged
merged 1 commit into from
Nov 6, 2020

Conversation

jtnord
Copy link
Member

@jtnord jtnord commented Nov 4, 2020

The script console is added by Javascript and not completely ready with
the page load.
Therefore we need to wait for the parts we want to manipulate to be
ready.

Fixes a flaky test observed when using master.runStript() with the following failure:

org.openqa.selenium.NoSuchElementException:
Unable to locate element: //*[@path='/script']

The script console is added by Javascript and not completely ready with
the page load.
Therefore we need to wait for the parts we want to manipulate to be
ready.

Fixes a flaky test observed when using master.runStript() with the following failure:

```
org.openqa.selenium.NoSuchElementException:
Unable to locate element: //*[@path='/script']

```
@jtnord jtnord added the bug label Nov 4, 2020
@jtnord jtnord changed the title Fixes a flakyness observed when using master.runStript() Fixes a flakyness observed when using master.runScript() Nov 4, 2020
@jtnord jtnord changed the title Fixes a flakyness observed when using master.runScript() Fixe a flakiness observed when using master.runScript() Nov 4, 2020
@jtnord jtnord changed the title Fixe a flakiness observed when using master.runScript() Fix a flakiness observed when using master.runScript() Nov 4, 2020
Copy link
Member

@timja timja left a comment

Choose a reason for hiding this comment

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

looks sensible

@jtnord jtnord marked this pull request as ready for review November 6, 2020 13:26
@jtnord
Copy link
Member Author

jtnord commented Nov 6, 2020

verified failures are not new :(

@jtnord jtnord merged commit 2f748dc into jenkinsci:master Nov 6, 2020
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.

3 participants