Solution #280
Solution #280
Conversation
There was a problem hiding this comment.
Great job on the implementation! 🎉 While there are a few minor issues like typos and a missing semicolon, they don't critically impact the functionality. Just make sure to address these in future updates to keep your code clean and maintainable. Keep up the good work! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| @@ -1,11 +1,39 @@ | |||
| /// <reference types='cypress' /> | |||
| import { faker } from '@faker-js/faker'; | |||
| import HomeAndCataloguePageObject | |||
There was a problem hiding this comment.
There is a missing semicolon at the end of the import statement for HomeAndCataloguePageObject. This might cause issues depending on your project's configuration.
| homeCatalogue.clickOnProduct('Sony vaio i7'); | ||
| homeCatalogue.clickOnButton('Add to cart'); | ||
|
|
||
| homeCatalogue.verifyAddingToCart('Product added'); |
There was a problem hiding this comment.
The method verifyAddingToCart is being called with the argument 'Product added'. Ensure that this message matches exactly what is expected from the application after adding a product to the cart.
| homeCatalogue.verifyAddingToCart('Product added'); | ||
|
|
||
| homeCatalogue.clickOnLink('Cart'); | ||
| homeCatalogue.verifyProductinCart('Sony vaio i7'); |
There was a problem hiding this comment.
The method verifyProductinCart is being called. Double-check the spelling of the method name and ensure it matches the actual method name in the HomeAndCataloguePageObject class. It should be verifyProductInCart if following camelCase conventions.
| verifyAddingToCart() { | ||
| return this.assertAllert('Product added'); |
There was a problem hiding this comment.
There seems to be a typo in the method name assertAllert. It should likely be assertAlert if you are trying to assert an alert message.
No description provided.