-
Notifications
You must be signed in to change notification settings - Fork 361
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
test: [M3-8526] - Update remaining Linode Create Cypress tests run against Linode Create v2 #10889
test: [M3-8526] - Update remaining Linode Create Cypress tests run against Linode Create v2 #10889
Conversation
…inst Linode Create v2
Coverage Report: ✅ |
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.
@jdamore-linode Are we good to delete this? Do I need to move any of these tests out into new ones?
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.
@bnussman-akamai I think we might want to yank out the region select
test into its own spec, if we can. This should only be temporary since we have a component test for the region select, but as long as we don't have component tests running in CI I'd like to hang onto that
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.
Looks good overall - tests pass and are using v2. I left a couple comments in the legacy test file but will defer to Joe for final thoughts on what, if anything, is necessary to keep.
Okay, I essentially extracted out some tests from |
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.
Nice work @bnussman-akamai!
I did get one failure the first time I ran create-linode-with-firewall.spec.ts
where it apparently failed to find the "Firewall Assigned" text, but not sure why and I didn't have any luck reproducing after that -- no reason to think it was caused by this PR, but I'll keep an eye on that once this is merged!
Description 📝
How to test 🧪
As an Author I have considered 🤔