improvement: add data-testids to file-explorer dropdown menu items

[mscolnick]

Add data-test IDs to menu items

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
marimo-docs	Ready	Preview, Comment	Apr 30, 2026 5:37am

[cubic-dev-ai]

No issues found across 2 files

Architecture diagram

sequenceDiagram
    participant Test as Test Suite (Playwright/Cypress)
    participant Explorer as File Explorer Component
    participant Ops as File Operations (Sub-components)
    participant DOM as Browser DOM
    participant App as App State / Backend

    Note over Test, DOM: UI Selection & Action Flow

    Explorer->>Ops: NEW: Pass testId to Rename/Duplicate/Delete
    Ops->>DOM: NEW: Render DropdownMenuItem with [data-testid]
    Explorer->>DOM: NEW: Render file/folder items with [data-testid]

    Test->>DOM: NEW: Locate element via [data-testid]
    Test->>DOM: Perform click/select action

    alt File Action
        DOM->>Explorer: onSelect() triggered
        Explorer->>App: handleCreateNotebook() / handleCreateFile()
    else File Operation (Rename/Delete)
        DOM->>Ops: onSelect() triggered
        Ops->>Explorer: Callback executed
        Explorer->>App: handleDeleteFile() / handleDuplicate()
    end

    App-->>Explorer: Update file tree
    Explorer-->>DOM: Re-render updated state

Loading

[github-actions]

🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.23.5-dev7