Skip to content

alerts test cases#16

Open
kshinde2512 wants to merge 6 commits intomainfrom
alerts-test-case
Open

alerts test cases#16
kshinde2512 wants to merge 6 commits intomainfrom
alerts-test-case

Conversation

@kshinde2512
Copy link
Owner

Describe your changes :

I worked on the ..... because ...

Type of change :

  • Bug fix
  • Improvement
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

Frontend Preview (Screenshots) :

For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

  • I have read the CONTRIBUTING document.
  • I have commented on my code, particularly in hard-to-understand areas.
  • I have added tests that prove my fix is effective or that my feature works.
  • All new and existing tests passed.

Reviewers

});

it('Create new alert for all data assets', () => {
interceptURL('GET', '/api/v1/alerts/*', 'createAlert');

Choose a reason for hiding this comment

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

Suggested change
interceptURL('GET', '/api/v1/alerts/*', 'createAlert');
interceptURL('POST', '/api/v1/alerts/*', 'createAlert');

Choose a reason for hiding this comment

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

Or it's not what I am thinking change alias name

Copy link
Owner Author

Choose a reason for hiding this comment

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

Checking

.click()
.type('Engineering');
cy.get('[title="Engineering"]').should('be.visible').click();
cy.get('#description').should('be.visible').click();

Choose a reason for hiding this comment

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

Why description in case of owner?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Its to close the dropdown once the selection is done

cy.get('table').should('contain', ALERTS.name);
});

it.only('Create new alert for Test case data asset', () => {

Choose a reason for hiding this comment

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

We should remove this to run all the tests right

Copy link
Owner Author

Choose a reason for hiding this comment

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

Yes my bad

@kshinde2512 kshinde2512 temporarily deployed to cypress December 26, 2022 12:29 — with GitHub Actions Inactive
@kshinde2512 kshinde2512 temporarily deployed to cypress December 26, 2022 12:29 — with GitHub Actions Inactive
interceptURL('POST', '/api/v1/alerts', 'createAlert');
//Click on create alert button
cy.get('button').contains('Create alert').should('be.visible').click();
verifyResponseStatusCode('@createAlert', 200);

Choose a reason for hiding this comment

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

we need to remove this right?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Removed

cy.get('.ant-select-item-option-content').contains('Email').click();
cy.wait(500);
//Enter email
cy.get('.ant-select-selector > .ant-select-selection-overflow')

Choose a reason for hiding this comment

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

Can you add one more alert with Webhook, MSTeams, Slack as well?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Adding these tests

@cypress
Copy link

cypress bot commented Dec 26, 2022



Test summary

174 5 8 0


Run details

Project OMD
Status Failed
Commit 455c2c8
Started Dec 26, 2022 3:45 PM
Ended Dec 26, 2022 4:30 PM
Duration 44:32 💡
OS Linux Ubuntu - 22.04
Browser Chrome 108

View run in Cypress Dashboard ➡️

Failures are unavailable for this run. For more information, see the Cypress Dashboard


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@kshinde2512 kshinde2512 temporarily deployed to cypress December 26, 2022 14:36 — with GitHub Actions Inactive
@kshinde2512 kshinde2512 temporarily deployed to cypress December 26, 2022 14:37 — with GitHub Actions Inactive
@kshinde2512 kshinde2512 temporarily deployed to cypress December 26, 2022 15:25 — with GitHub Actions Inactive
@kshinde2512 kshinde2512 temporarily deployed to cypress December 26, 2022 15:25 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants