Skip to content

Refactor e2e RHCOS tests - #1050

Merged
vikin91 merged 2 commits into
masterfrom
piotr/refactor-e2e-rhcos-tests
Feb 1, 2023
Merged

Refactor e2e RHCOS tests#1050
vikin91 merged 2 commits into
masterfrom
piotr/refactor-e2e-rhcos-tests

Conversation

@vikin91

@vikin91 vikin91 commented Jan 11, 2023

Copy link
Copy Markdown
Contributor

This PR aims at refactoring the e2e tests for node scanning, so that the assertions (equals vs. is subset) are clearer to read.

This is a follow-up to the discussion in #1045 (comment)

@vikin91
vikin91 requested a review from RTann January 11, 2023 14:55
@ghost

ghost commented Jan 11, 2023

Copy link
Copy Markdown

Images are ready for the commit at 12ad330.

To use the images, use the tag 2.27.x-39-g12ad330892.

@vikin91
vikin91 requested a review from jvdm January 26, 2023 14:24
@vikin91

vikin91 commented Jan 26, 2023

Copy link
Copy Markdown
Contributor Author

Ping @RTann and @jvdm - This is a pure refactor PR, but I don't want it to be forgotten. What do you think?

@vikin91
vikin91 force-pushed the piotr/refactor-e2e-rhcos-tests branch from 4225d77 to 12ad330 Compare January 27, 2023 15:05

@jvdm jvdm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@vikin91
vikin91 merged commit 48a96d4 into master Feb 1, 2023
@vikin91
vikin91 deleted the piotr/refactor-e2e-rhcos-tests branch February 1, 2023 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants