Skip to content

Commit bea5562

Browse files
Copilotrenemadsen
andcommitted
Fix test assertion: should expect initialCount + 3 elements (1 dropdown + 2 template items)
Co-authored-by: renemadsen <[email protected]>
1 parent 0e91e4f commit bea5562

File tree

1 file changed

+24
-13
lines changed

1 file changed

+24
-13
lines changed

eform-client/cypress/e2e/c/navigation-menu.delete-item.spec.cy.ts

Lines changed: 24 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -21,15 +21,21 @@ describe('Navigation menu - Delete item', function () {
2121

2222
navigationMenuPage.createCustomDropdown(dropdown);
2323

24-
// Verify 1 element was created (the dropdown)
25-
navigationMenuPage.getMenuItems().should('have.length', initialCount + 1);
24+
// Create 2 items from templates menu
25+
navigationMenuPage.collapseTemplates(0);
26+
navigationMenuPage.createMenuItemFromTemplate(2);
27+
navigationMenuPage.createMenuItemFromTemplate(3);
28+
navigationMenuPage.collapseTemplates(0);
2629

27-
// Drag 3 template items into dropdown
30+
// Verify 3 elements were created (1 dropdown + 2 template items)
31+
navigationMenuPage.getMenuItems().should('have.length', initialCount + 3);
32+
33+
// Drag template items into dropdown
2834
navigationMenuPage.getMenuItems().its('length').then(currentCount => {
2935
navigationMenuPage.collapseMenuItemDropdown(currentCount - 1);
30-
navigationMenuPage.dragTemplateOnElementInCreatedDropdown(1, 0);
31-
navigationMenuPage.dragTemplateOnElementInCreatedDropdown(2, 0);
32-
navigationMenuPage.dragTemplateOnElementInCreatedDropdown(3, 0);
36+
navigationMenuPage.dragTemplateOnElementInCreatedDropdown(1, 1);
37+
navigationMenuPage.dragTemplateOnElementInCreatedDropdown(2, 1);
38+
navigationMenuPage.dragTemplateOnElementInCreatedDropdown(3, 1);
3339

3440
// Verify 3 items in dropdown
3541
navigationMenuPage.getDropdownBodyChilds(currentCount - 1).should('have.length', 3);
@@ -59,15 +65,20 @@ describe('Navigation menu - Delete item', function () {
5965
// Verify dropdown is empty
6066
navigationMenuPage.getDropdownBodyChilds(menuItemsCount - 1).should('have.length', dropdownCount - 3);
6167

62-
// Delete the dropdown itself
63-
navigationMenuPage.deleteElementFromMenuItems(menuItemsCount - 1);
68+
// Delete menu items
69+
navigationMenuPage.deleteElementFromMenuItems(0);
70+
navigationMenuPage.deleteElementFromMenuItems(0); // delete 2 template elements
6471

65-
cy.intercept('PUT', '**/api/navigation-menu').as('saveMenu2');
66-
navigationMenuPage.clickSaveMenuBtn();
67-
cy.wait('@saveMenu2', { timeout: 30000 });
72+
navigationMenuPage.getMenuItems().its('length').then(currentCount => {
73+
navigationMenuPage.deleteElementFromMenuItems(currentCount - 1); // delete created dropdown
74+
75+
cy.intercept('PUT', '**/api/navigation-menu').as('saveMenu2');
76+
navigationMenuPage.clickSaveMenuBtn();
77+
cy.wait('@saveMenu2', { timeout: 30000 });
6878

69-
// Verify dropdown was deleted
70-
navigationMenuPage.getMenuItems().should('have.length', menuItemsCount - 1);
79+
// Verify items were deleted
80+
navigationMenuPage.getMenuItems().should('have.length', menuItemsCount - 3);
81+
});
7182
});
7283
});
7384
});

0 commit comments

Comments
 (0)