chore: remove panic assertions from tarball integrity tests

Per review feedback, tests should not assert absence of "panic" - they
should rely on exit code and content assertions instead.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Claude Bot
2025-12-01 04:57:05 +00:00
parent 2f10b71f50
commit a42c326de6

View File

@@ -45,8 +45,6 @@ describe("tarball integrity", () => {
},
});
expect(installResult.stdout.toString()).not.toContain("panic");
expect(installResult.stderr.toString()).not.toContain("panic");
expect(installResult.exitCode).toBe(0);
// Read the lockfile and verify it contains the integrity hash
@@ -96,7 +94,6 @@ describe("tarball integrity", () => {
},
});
expect(firstInstall.stdout.toString()).not.toContain("panic");
expect(firstInstall.exitCode).toBe(0);
// Verify lockfile has integrity