Skip to content

Commit 835d3b0

Browse files
committed
Add manual test doc for .quarto_ipynb cleanup on ungraceful termination
1 parent 0dc5414 commit 835d3b0

1 file changed

Lines changed: 42 additions & 0 deletions

File tree

Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
# Manual Test: .quarto_ipynb cleanup on ungraceful termination (#14359)
2+
3+
## Purpose
4+
5+
Verify that `.quarto_ipynb` files are cleaned up after render even when preview
6+
is killed ungracefully (e.g., Positron terminal bin icon). This tests the fix
7+
that restores immediate deletion in `cleanupNotebook()`.
8+
9+
**Why manual?** Ungraceful termination bypasses normal cleanup handlers.
10+
The fix ensures deletion happens right after execution, before any signal
11+
could interrupt the process.
12+
13+
## Automated coverage
14+
15+
The smoke-all test `tests/docs/smoke-all/2025/05/21/keep_ipynb_single-file/14359.qmd`
16+
verifies the core behavior (file deleted after render). This manual test
17+
covers the interactive preview scenario.
18+
19+
## Steps
20+
21+
1. Open Positron (or VS Code with Quarto extension)
22+
2. Open `tests/docs/manual/preview/14281-quarto-ipynb-accumulation.qmd`
23+
3. Start Quarto Preview (terminal or Quarto extension)
24+
4. Wait for first render to complete
25+
5. Check directory: `ls *.quarto_ipynb*` — should be zero files (file was deleted after render)
26+
6. Edit and save the file to trigger a re-render
27+
7. After re-render completes, check again — still zero `.quarto_ipynb` files
28+
8. Kill preview ungracefully (Positron terminal bin icon / close terminal)
29+
9. Check directory — still zero `.quarto_ipynb` files
30+
31+
## Expected
32+
33+
- Zero `.quarto_ipynb` files at all times (the file is deleted immediately
34+
after each execution, not deferred to cleanup handlers)
35+
- Ungraceful termination does not leave stale files because deletion already
36+
happened
37+
38+
## Related
39+
40+
- #14281 — within-session accumulation (fixed by PR #14287)
41+
- #12780`keep-ipynb` support (PR #12793 introduced this regression)
42+
- posit-dev/positron#13006 — Killing Quarto Preview should exit process

0 commit comments

Comments
 (0)