-
Notifications
You must be signed in to change notification settings - Fork 332
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-7395] Add new cypress tests to longview landing page #10321
Conversation
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.
Great work @cliu-akamai, really nicely done! Approved pending removal of that stray .skip()
(edit: and changeset!)
@@ -103,7 +111,7 @@ describe('longview', () => { | |||
* - Creates a Linode, connects to it via SSH, and installs Longview using the given cURL command. | |||
* - Confirms that Cloud Manager UI updates to reflect Longview installation and data. | |||
*/ | |||
it('can install Longview client on a Linode', () => { | |||
it.skip('can install Longview client on a Linode', () => { |
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.
it.skip('can install Longview client on a Linode', () => { | |
it('can install Longview client on a Linode', () => { |
cy.wait('@ceateLongviewClient'); | ||
|
||
// Confirms that UI updates to show the new client when creating one. | ||
cy.findByText(`longview-client-${client.id}`).should('be.visible'); |
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.
cy.findByText(`longview-client-${client.id}`).should('be.visible'); | |
cy.findByText(`${client.label}`).should('be.visible'); |
Very minor change, but this way if the Longview client factory label ever changes the test will still pass 👍
5af58f5
to
65f1951
Compare
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.
Tests pass ✅
Code review ✅
"@linode/manager": Tests | ||
--- | ||
|
||
Add new cypress tests to longview landing page ([#10321](https://github.com/linode/manager/pull/10321)) |
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.
Add new cypress tests to longview landing page ([#10321](https://github.com/linode/manager/pull/10321)) | |
Add new Cypress tests for Longview landing page ([#10321](https://github.com/linode/manager/pull/10321)) |
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!
Description 📝
Add new tests to check the empty state message in Longview landing page.
Major Changes 🔄
How to test 🧪