-
Notifications
You must be signed in to change notification settings - Fork 4
Automate bundle tests #337
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Satyajit Bulage <[email protected]>
Signed-off-by: Satyajit Bulage <[email protected]>
cd809a7
to
2e27bfc
Compare
Newly written tests are skipped because tests should not run on 2.11 and older versions... |
CI for 2.12.0-alpha6
|
On PR CI, Fleet-127 test failed...apart from that the tests in this PR should skip on 2.11 and are skipping: https://github.com/rancher/fleet-e2e/actions/runs/15532999979/job/43733799480?pr=337#step:9:505 |
@@ -14,3 +16,35 @@ export const isK8sVersion = (version: string) => { | |||
export const isRancherManagerVersion = (version: string) => { | |||
return (new RegExp(version)).test(Cypress.env("rancher_version")); | |||
} | |||
|
|||
// Run `It()` block based on allowedVersions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Per my understanding not exclusively to It()
blocks. Suggest change: It() / Describe() blocks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will try and make it describe level version mapping.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Small consideration change
No description provided.