test: make overwrite confirmations deterministic

[peter-gy]

Replaces timing-sensitive overwrite confirmation subprocesses with deterministic CliRunner tests that exercise the prompt and verify rewritten output.

Addresses marimo-team/ci-agent#74.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
marimo-docs	Ready	Preview, Comment	Jul 22, 2026 6:22pm

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[cubic-dev-ai]

No issues found across 1 file

Architecture diagram

sequenceDiagram
    participant Test as Test Function
    participant CliRunner as CliRunner
    participant main as marimo._cli.cli.main
    participant utils as marimo._cli.utils
    participant FS as File System

    Note over Test,FS: Deterministic overwrite test (CliRunner)

    Test->>utils: mock.patch('marimo._cli.utils.stdout')
    Test->>utils: mock_stdout.isatty.return_value = True

    Test->>CliRunner: invoke(main, args, input="y\n")
    activate CliRunner
    CliRunner->>main: run CLI command with stdin="y\n"
    activate main

    main->>utils: check stdout.isatty()
    utils-->>main: True (mocked)

    main->>main: prompt "Overwrite?"
    main->>CliRunner: read stdin (simulated)
    CliRunner-->>main: "y"

    main->>FS: overwrite output file
    FS-->>main: OK

    main-->>CliRunner: exit code 0
    deactivate main
    CliRunner-->>Test: result
    deactivate CliRunner

    Test->>Test: assert result.exit_code == 0
    Test->>Test: assert "Overwrite?" in result.output
    Test->>FS: read output file
    FS-->>Test: file contents
    Test->>Test: assert expected content

Loading

_{Re-trigger cubic}

[github-actions]

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