Fix HTTP method mismatch and remove unnecessary intercepts in navigation menu Cypress tests

[Copilot]

Three navigation menu test files were using POST instead of PUT for save operations, despite the API endpoint being defined as [HttpPut] in MenuController.cs. Additionally, these tests had unnecessary intercepts before editItemSave() calls that don't exist in the reference implementation.

Changes

• Updated cy.intercept() calls from POST to PUT for main save operations in:
  • navigation-menu.delete-item.spec.cy.ts (3 occurrences)
  • navigation-menu.drag-item.spec.cy.ts (3 occurrences)
  • navigation-menu.edit-item.spec.cy.ts (9 occurrences)
• Removed unnecessary saveMenuEdit intercepts and waits before editItemSave() calls in:
  • navigation-menu.drag-item.spec.cy.ts (1 occurrence)
  • navigation-menu.edit-item.spec.cy.ts (4 occurrences)

Example

// Before
cy.intercept('POST', '**/api/navigation-menu').as('saveMenu');
navigationMenuPage.clickSaveMenuBtn();
cy.wait('@saveMenu', { timeout: 30000 });

// ... later in test ...
cy.intercept('PUT', '**/api/navigation-menu').as('saveMenuEdit');
navigationMenuPage.editItemSave();
cy.wait('@saveMenuEdit', { timeout: 30000 });

// After
cy.intercept('PUT', '**/api/navigation-menu').as('saveMenu');
navigationMenuPage.clickSaveMenuBtn();
cy.wait('@saveMenu', { timeout: 30000 });

// ... later in test ...
navigationMenuPage.editItemSave();

Tests now fully align with both the API controller definition and the existing navigation-menu.create-item.spec.cy.ts reference implementation.

• Fixes Fix delete,drag, edit and use create as reference #7272

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Fix delete,drag, edit and use create as reference</issue_title>
<issue_description>eform-client/cypress/e2e/c needs to be updated to test correctly</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Fix delete,drag, edit and use create as reference #7272

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[renemadsen]

@copilot look more closely at the reference create tests and adjust the other tests, so they don't do:

cy.intercept('PUT', '**/api/navigation-menu').as('saveMenuEdit');

[Copilot]

@copilot look more closely at the reference create tests and adjust the other tests, so they don't do:

cy.intercept('PUT', '**/api/navigation-menu').as('saveMenuEdit');

Removed all saveMenuEdit intercepts and waits before editItemSave() calls to match the reference implementation in navigation-menu.create-item.spec.cy.ts. Changes in commit 26575df.