tree c94fccc6dea3ca946c2c9039f062d3814d61477f
parent 9aa7828cbff8e8d66e63909db5a222d1b90fa986
author banksean <banksean@gmail.com> 1750953925 +0000
committer Sean McCullough <banksean@gmail.com> 1750954037 -0700

fix: prevent nil pointer panic in codereview compareTestResults

Fix for https://github.com/boldsoftware/sketch/issues/175

Fix nil pointer dereference when comparing test results between commits
where a package exists in the after state but not the before state.

The panic occurred at line 867 when accessing beforeResult.TestStatus[test]
without checking if beforeResult was nil. This happens when testing new
packages that didn't exist in the base commit.

Now properly check for nil beforeResult and default to testStatusUnknown
for tests in packages that didn't exist in the before state.

Add comprehensive unit tests covering:
- New package with passing tests (the panic scenario)
- New package with failing tests (regression detection)
- Existing package regressions (ensure normal flow still works)
- End-to-end integration test for new package scenario

Co-Authored-By: sketch <hello@sketch.dev>
Change-ID: sab802d91eff08039k