LiteSun commented on a change in pull request #1093:
URL: https://github.com/apache/apisix-dashboard/pull/1093#discussion_r547104522
##########
File path: web/cypress/integration/route/create-edit-delete-route.spec.js
##########
@@ -80,11 +81,28 @@ context('Create and Delete Route', () => {
cy.url().should('contains', 'routes/list');
});
+ it('edit the route', () => {
+ cy.visit('/');
+ cy.contains('Route').click();
+ cy.contains('Edit').click();
+ // input new name and newdescription
+ cy.get('#name').clear().type(newname);
+ cy.get('#desc').clear().type('new desc');
+ cy.contains('Next').click();
+ cy.contains('Next').click();
+ cy.contains('Next').click();
+ cy.contains('Submit').click();
+ cy.contains('SubmitSuccessfully');
Review comment:
> no space after “submit”？
refer to cypress‘ documents: ` It’s automatically waiting and retrying
because it expects the content to eventually be found in the DOM. It doesn’t
immediately fail!`
https://docs.cypress.io/guides/getting-started/writing-your-first-test.html#Step-2-Query-for-an-element
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
