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

Cypress test for execution environments #1138

Merged

Conversation

ShaiahWren
Copy link
Contributor

Issue: https://issues.redhat.com/browse/AAH-1001

This pr tests Execution Environments - list view, detail view, and readme file.

As it is the tests are fairly simple. Lmk what other details you would like me to include.

});

it('checks the EE detail view', () => {
cy.get('a[href="/ui/containers/alpine338"]').click();
Copy link
Member

Choose a reason for hiding this comment

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

We should be testing all tabs and their content. But given the deadline (and vague issue description O:)) it can be added in followup PR :)

@ZitaNemeckova
Copy link
Member

@ShaiahWren Nice work! I'm leaving it up to you if you want to address all reviews here or just the one with saving readme :)

@ShaiahWren ShaiahWren force-pushed the cypress-test-for-execution-environments branch 4 times, most recently from 1d59e8d to bbe4361 Compare October 29, 2021 19:15
@himdel himdel added the backport-4.4 This PR should be backported to stable-4.4 (2.1) label Nov 1, 2021
@ShaiahWren ShaiahWren force-pushed the cypress-test-for-execution-environments branch from bbe4361 to 6aec628 Compare November 2, 2021 16:22
@ShaiahWren ShaiahWren force-pushed the cypress-test-for-execution-environments branch from 6aec628 to d93a948 Compare November 2, 2021 16:28
Copy link
Collaborator

@himdel himdel left a comment

Choose a reason for hiding this comment

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

LGTM standing on its own 👍

There's some code duplication with the functions added both here and in #1141 , but we can deal with that separately once both are merged, and move them to helpers :)

@ShaiahWren
Copy link
Contributor Author

Finally closing this pr. :)

@ShaiahWren ShaiahWren merged commit e237e53 into ansible:master Nov 3, 2021
@ShaiahWren ShaiahWren deleted the cypress-test-for-execution-environments branch November 3, 2021 21:26
@patchback
Copy link

patchback bot commented Nov 3, 2021

Backport to stable-4.4: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4.4/e237e531262b5a1850a6b0a73d300643d3dde473/pr-1138

Backported as #1192

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Nov 3, 2021
* testing execution exvironments

* testing execution environments

* remove registries/containers before testing

(cherry picked from commit e237e53)
himdel pushed a commit that referenced this pull request Nov 3, 2021
* testing execution exvironments

* testing execution environments

* remove registries/containers before testing

(cherry picked from commit e237e53)

Co-authored-by: Shaiah Emigh-Doyle <64337863+ShaiahWren@users.noreply.github.com>
@github-actions github-actions bot added the backported-4.4 This PR has been backported to stable-4.4 (2.1) label Nov 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-4.4 This PR should be backported to stable-4.4 (2.1) backported-4.4 This PR has been backported to stable-4.4 (2.1)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants