Skip to content

tests: add unit tests for notebook execution CLI#10

Merged
anish749 merged 1 commit into
mainfrom
add-notebook-exec-tests
Mar 30, 2026
Merged

tests: add unit tests for notebook execution CLI#10
anish749 merged 1 commit into
mainfrom
add-notebook-exec-tests

Conversation

@anish749

Copy link
Copy Markdown
Owner

Problem

No tests existed for the notebook execution feature (_exec_notebook, --from-cell/--to-cell, etc.), making it hard to verify behavior before adding the upcoming --output flag.

Solution

Add 8 unit tests covering the notebook execution CLI path:

  • Basic multi-cell execution
  • Markdown/empty cell filtering
  • Fail-fast on cell error
  • Progress marker output
  • --from-cell / --to-cell range selection
  • No-code-cells graceful exit
  • Cell-range flags rejected for non-notebook files
  • Multi-line output capture

All tests mock send_to_daemon so no running daemon or Jupyter server is needed.

@anish749 anish749 merged commit 20481c5 into main Mar 30, 2026
1 check passed
@anish749 anish749 deleted the add-notebook-exec-tests branch March 30, 2026 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant