Add Autotests #283
Add Autotests #283
Conversation
There was a problem hiding this comment.
Great job on structuring the test script and following the Page Object Model pattern! 🎉 While there are minor improvements suggested for the assertions and the expect function, they don't impact the core functionality. Keep up the good work and consider these suggestions for future enhancements!
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| cy.get('.modal-body').should('contain', 'Test Name'); | ||
| cy.get('.modal-body').should('contain', 'Test Country'); | ||
| cy.get('.modal-body').should('contain', 'Test City'); |
There was a problem hiding this comment.
The assertions in assertOrderDetails method are currently using hardcoded values ('Test Name', 'Test Country', 'Test City'). Consider using dynamic values or parameters to make the test more flexible and applicable to different scenarios.
|
|
||
| assertAlert(expectedMessage) { | ||
| cy.on('window:alert', (text) => { | ||
| expect(text).to.contains(expectedMessage); |
There was a problem hiding this comment.
There is a minor issue here. The expect function should use to.contain instead of to.contains. The correct method is to.contain for checking if a string contains a substring.
No description provided.