Skip to content

[7.x] [Upgrade Assistant] Fixes to Upgrade Assistant Tests and Page Objects To Stop CI Failures (#89942)#90387

Merged
cuff-links merged 2 commits intoelastic:7.xfrom
cuff-links:backport/7.x/pr-89942
Feb 5, 2021
Merged

[7.x] [Upgrade Assistant] Fixes to Upgrade Assistant Tests and Page Objects To Stop CI Failures (#89942)#90387
cuff-links merged 2 commits intoelastic:7.xfrom
cuff-links:backport/7.x/pr-89942

Conversation

@cuff-links
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

… To Stop CI Failures (elastic#89942)

* Updated test to use no hard coded waits which could be contributing to CI failures. Expect functions should be in the test file itself so I changed the page objects for page actions and extracted out the expect calls to the test files.

* Added retry within the test file to retry the assertion if it doesn't pass on the first attempt. Simplified the toggle function to just click the toggle.

* Fixed type check issue.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
# Conflicts:
#	x-pack/test/functional/apps/upgrade_assistant/upgrade_assistant.ts
@cuff-links cuff-links added the backport This PR is a backport of another PR label Feb 5, 2021
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@cuff-links cuff-links merged commit a22d189 into elastic:7.x Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants