test: add unit tests for context menu drop

[AlexAndBear]

Description

Related Issue

• Part of Extend "open" option as context menu for files #773

How Has This Been Tested?

• test environment:
• test case 1:
• test case 2:
• ...

Types of changes

• Bugfix
• Enhancement (a change that doesn't break existing code or deployments)
• Breaking change (a modification that affects current functionality)
• Technical debt (addressing code that needs refactoring or improvements)
• Tests (adding or improving tests)
• Documentation (updates or additions to documentation)
• Maintenance (like dependency updates or tooling adjustments)

[Copilot]

Pull Request Overview

This PR adds comprehensive unit tests for context menu drop functionality and refactors the related components to support the new drop-menu API.

• Snapshots for ContextActionMenu and ActionMenuDropItem updated to include nested drop menus.
• Tests migrated from shallowMount to mount and prop renamed to menuSectionDrop.
• Component code refactored: prop renames, ID and class adjustments, and removal of unused imports.

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
packages/web-pkg/tests/unit/components/ContextActions/ContextActionMenu.spec.ts	Switched to mount, updated test cases and snapshots to cover drop menus.
packages/web-pkg/tests/unit/components/ContextActions/ActionMenuDropItem.spec.ts	New tests for drop-item rendering and empty-state behavior.
packages/web-pkg/src/composables/actions/files/useFileActionsNavigate.ts	Changed the user-facing label from “Open folder” to “Open”.
packages/web-pkg/src/components/ContextActions/ContextActionMenu.vue	Prop for drop component renamed to menu-section-drop.
packages/web-pkg/src/components/ContextActions/ActionMenuItem.vue	Removed unused useGettext import.
packages/web-pkg/src/components/ContextActions/ActionMenuDropItem.vue	Renamed props to menuSectionDrop, updated HTML ID generation logic, empty-state class, and iteration keys.

Comments suppressed due to low confidence (3)

packages/web-pkg/tests/unit/components/ContextActions/ContextActionMenu.spec.ts:17

• The new empty-state behavior (renderOnEmpty) is tested in ActionMenuDropItem but not in ContextActionMenu. Consider adding a test case in this spec to verify that empty-drop sections render the empty message correctly.

it('renders the menu with drop menu items', async () => {

packages/web-pkg/src/components/ContextActions/ActionMenuDropItem.vue:4

• Generating HTML IDs from the dynamic label can lead to invalid or unstable IDs (e.g., spaces, translated text). Consider using a stable property such as section.name or a sanitized slug for the ID.

:id="`oc-files-context-actions-${menuSectionDrop.label}-toggle`"

packages/web-pkg/src/composables/actions/files/useFileActionsNavigate.ts:43

• [nitpick] Changing the label from 'Open folder' to 'Open' may affect user expectations and translation keys. Ensure all translation resources are updated and any documentation or UI guidelines reflect this change.

label: () => $gettext('Open'),

[AlexAndBear]

@JammingBen this also fixed:

• "open with..." drop opens randomly by default not 😡
• drop disappears when moving mouse out of the drop
• don't close Open with... drop, when the toggle button is clicked
• Move check if nested drop toggle button is clicked to oc-drop, and don't close the drop, if so @JammingBen this is nice, as this was the onliest implementation in the former PR, I despised

[AlexAndBear]

this is for e2e tests

[AlexAndBear]

Reverted to original

[JammingBen]

🏅