Test-only speedup: Shave 2 mins off test runtime

[computermode]

https://entire.io/gh/entireio/cli/trails/382

Removes t.Parallel() in two slow tests in favor of t.Chdir(...).

For these two tests specifically, the working directory mattered because the code under test shells out to git without -C . The tests create temp repos, but without t.Chdir(tempRepo), those fallback git calls run from the real project checkout, hit its partial-clone promisor remotes, and burned about a minute.

Before:

➜  cli git:(main) ✗ go test -run '^TestV2ReadSessionMetadata_FetchesMissingMetadataBlob$' -v -count=1 ./cmd/entire/cli/checkpoint
=== RUN   TestV2ReadSessionMetadata_FetchesMissingMetadataBlob
=== PAUSE TestV2ReadSessionMetadata_FetchesMissingMetadataBlob
=== CONT  TestV2ReadSessionMetadata_FetchesMissingMetadataBlob
2026/05/14 15:54:01 WARN FetchingTree.readFileViaGit: cat-file failed path=metadata.json hash=7fbc2440559c error="exit status 128"
--- PASS: TestV2ReadSessionMetadata_FetchesMissingMetadataBlob (58.68s)
PASS
ok  	github.com/entireio/cli/cmd/entire/cli/checkpoint	59.071s

After:

➜  cli git:(test-speedups) ✗ go test -run '^TestV2ReadSessionMetadata_FetchesMissingMetadataBlob$' -v -count=1 ./cmd/entire/cli/checkpoint
=== RUN   TestV2ReadSessionMetadata_FetchesMissingMetadataBlob
2026/05/14 15:52:33 WARN FetchingTree.readFileViaGit: cat-file failed path=metadata.json hash=28e8b85d8618 error="exit status 128"
--- PASS: TestV2ReadSessionMetadata_FetchesMissingMetadataBlob (0.15s)
PASS
ok  	github.com/entireio/cli/cmd/entire/cli/checkpoint	0.554s

Before:

➜  cli git:(main) ✗ go test -run '^TestCheckV2CheckpointCounts_ReturnsErrorForCorruptRef$' -v -count=1 ./cmd/entire/cli
=== RUN   TestCheckV2CheckpointCounts_ReturnsErrorForCorruptRef
=== PAUSE TestCheckV2CheckpointCounts_ReturnsErrorForCorruptRef
=== CONT  TestCheckV2CheckpointCounts_ReturnsErrorForCorruptRef
--- PASS: TestCheckV2CheckpointCounts_ReturnsErrorForCorruptRef (62.24s)
PASS
ok  	github.com/entireio/cli/cmd/entire/cli	62.754s

After:

➜  cli git:(test-speedups) ✗ go test -run '^TestCheckV2CheckpointCounts_ReturnsErrorForCorruptRef$' -v -count=1 ./cmd/entire/cli
=== RUN   TestCheckV2CheckpointCounts_ReturnsErrorForCorruptRef
--- PASS: TestCheckV2CheckpointCounts_ReturnsErrorForCorruptRef (0.05s)
PASS
ok  	github.com/entireio/cli/cmd/entire/cli	0.520s

---

Note

Low Risk
Test-only changes that adjust working-directory and parallelism; low functional risk but could slightly reduce overall test concurrency.

Overview
Speeds up two previously slow tests by removing t.Parallel() and explicitly t.Chdir(...) into the temp repo root so any shell-outs to git run against the intended fixture repo.

This avoids accidental interaction with the developer checkout (e.g., partial-clone promisor remotes) and keeps the tests focused on corrupt/missing-object scenarios (TestV2ReadSessionMetadata_FetchesMissingMetadataBlob and TestCheckV2CheckpointCounts_ReturnsErrorForCorruptRef).

^{Reviewed by Cursor Bugbot for commit d5b684d. Configure here.}

[Copilot]

Pull request overview

Removes t.Parallel() in two slow tests and adds t.Chdir(...) so that git subprocesses invoked by the code under test run inside the test's temp repo rather than the project checkout, eliminating expensive promisor remote fetches.

Changes:

• Replaces t.Parallel() with t.Chdir(dir) in TestCheckV2CheckpointCounts_ReturnsErrorForCorruptRef.
• Replaces t.Parallel() with t.Chdir(repoRoot) in TestV2ReadSessionMetadata_FetchesMissingMetadataBlob.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
cmd/entire/cli/doctor_test.go	Drops parallelism, chdirs into the temp repo to avoid stray git calls hitting the real repo.
cmd/entire/cli/checkpoint/v2_read_test.go	Same treatment for the missing-metadata-blob fetch test.