Skip to content

test(Bowl): add end-to-end crash pipeline integration tests - #278

Merged
JusterZhu merged 2 commits into
masterfrom
fix/bowl-copilot-fixes-and-tests
May 23, 2026
Merged

test(Bowl): add end-to-end crash pipeline integration tests#278
JusterZhu merged 2 commits into
masterfrom
fix/bowl-copilot-fixes-and-tests

Conversation

@JusterZhu

Copy link
Copy Markdown
Collaborator

Closes #277

Summary

Adds integration tests that simulate the full crash → dump → report → callback pipeline with mock strategies producing real output files users can inspect.

New tests (3)

Test Scenario
\SimulatedCrash_ProducesDumpAndReportFiles\ Upgrade mode: writes dump → generates crash JSON → restores backup → fires OnCrash callback
\SimulatedCrash_NormalMode_DoesNotRestore\ Normal mode: no restore
\NoCrash_NoDumpFile_ReturnsSuccess\ Healthy process exit

Changes

  • \BowlCrashPipelineTests.cs\ — uses \ITestOutputHelper\ to print full file paths and report contents
  • \GeneralUpdate.Bowl.csproj\ — added \InternalsVisibleTo\ for test access to DI constructor
  • All 56 tests pass, 0 build errors

JusterZhu added 2 commits May 23, 2026 17:47
- BowlCrashPipelineTests: simulates full crash -> dump -> report -> callback flow
- MockCrashStrategy writes fake dump file, real CrashReporter creates JSON
- Verifies DumpCaptured, file existence, report content, OnCrash callback data
- Tests Normal mode (no restore) and healthy exit (no crash) scenarios
- Added InternalsVisibleTo to Bowl.csproj for test access to internal DI constructor
- Outputs all file paths and report contents via ITestOutputHelper for inspection
Copilot AI review requested due to automatic review settings May 23, 2026 09:48
@JusterZhu
JusterZhu removed the request for review from Copilot May 23, 2026 09:50
@JusterZhu
JusterZhu merged commit e60a3fc into master May 23, 2026
1 check failed
@JusterZhu
JusterZhu deleted the fix/bowl-copilot-fixes-and-tests branch May 23, 2026 09:50
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.

Bowl: add end-to-end crash pipeline integration tests

1 participant