Skip to content

Implement resilience test plan: Phase 2–5 E2E tests#62

Merged
pinodeca merged 2 commits into
pinodeca/breakitfrom
copilot/sub-pr-52
Mar 14, 2026
Merged

Implement resilience test plan: Phase 2–5 E2E tests#62
pinodeca merged 2 commits into
pinodeca/breakitfrom
copilot/sub-pr-52

Conversation

Copilot AI commented Mar 14, 2026

Copy link
Copy Markdown
Contributor

PR #56 merged 13 Phase 1 resilience tests (infinite loops, truthiness bugs, empty SQL, recursive start, crafted JSON, concurrency, signal edge cases) into pinodeca/breakit. This PR continues from there, adding 7 more tests covering stress limits, misuse patterns, and variable edge cases.

New tests

File Plan ID What it covers
51_wide_parallel.sql A3 9-branch parallel graph via nested join3
52_long_loop_history.sql A4 100-iteration loop — execution history bloat
53_large_result_sets.sql A5 10,000-row cross-join — exercises fetch_all() OOM risk
54_large_query_text.sql A6 ~10KB query via 500-element VALUES list
55_large_variables.sql A8 5KB variable value via df.setvar() + {var} substitution
56_rapid_status_poll.sql C5 Tight-loop df.status() polling — resource exhaustion
57_variable_name_conflict.sql B4/B14 Step result overwrites same-named binding; user var shadowed by result var

Not yet implemented

Chaos/fault-injection tests (D1 kill worker, D2 crash PG, D3 disk full) require shell-level process control and cannot be expressed as SQL E2E tests. These remain documented in docs/resilience-testing.md.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…lts, large query, large vars, rapid polling, variable name conflicts

Co-authored-by: pinodeca <32303022+pinodeca@users.noreply.github.com>
Copilot AI changed the title [WIP] Add resilience testing plan Implement resilience test plan: Phase 2–5 E2E tests Mar 14, 2026
Copilot AI requested a review from pinodeca March 14, 2026 00:42
@pinodeca pinodeca merged commit 4157364 into pinodeca/breakit Mar 14, 2026
3 of 5 checks passed
@pinodeca pinodeca deleted the copilot/sub-pr-52 branch March 14, 2026 14:24
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.

2 participants