Skip to content

Add isolated test coverage directories to prevent collisions#9

Merged
jtk18 merged 1 commit into
mainfrom
claude/isolated-test-coverage-dirs-U7jnb
Dec 31, 2025
Merged

Add isolated test coverage directories to prevent collisions#9
jtk18 merged 1 commit into
mainfrom
claude/isolated-test-coverage-dirs-U7jnb

Conversation

@jtk18

@jtk18 jtk18 commented Dec 31, 2025

Copy link
Copy Markdown
Owner

When running tests in parallel, multiple tests may cover the same source
file, causing coverage data collisions. This change gives each test file
its own isolated coverage directory (cover_db_N) and then merges them
all into a single cover_db after all tests complete.

Key changes:

  • runner.go: Each test gets its own isolated coverage directory (cover_db_N)
  • coverage.go: Added MergeCoverageDBs function to merge multiple coverage
    directories into a single output directory
  • cli.go: After running tests, collects and merges isolated coverage
    directories, then cleans up the temporary directories

Co-Authored-By: Claude claude-opus-4-5-20251101 noreply@anthropic.com

When running tests in parallel, multiple tests may cover the same source
file, causing coverage data collisions. This change gives each test file
its own isolated coverage directory (cover_db_N) and then merges them
all into a single cover_db after all tests complete.

Key changes:
- runner.go: Each test gets its own isolated coverage directory (cover_db_N)
- coverage.go: Added MergeCoverageDBs function to merge multiple coverage
  directories into a single output directory
- cli.go: After running tests, collects and merges isolated coverage
  directories, then cleans up the temporary directories

Co-Authored-By: Claude claude-opus-4-5-20251101 <noreply@anthropic.com>
@jtk18 jtk18 force-pushed the claude/isolated-test-coverage-dirs-U7jnb branch from 27c560e to cd1b9ca Compare December 31, 2025 23:36
@jtk18 jtk18 merged commit 6b13633 into main Dec 31, 2025
4 checks passed
@jtk18 jtk18 deleted the claude/isolated-test-coverage-dirs-U7jnb branch December 31, 2025 23:37
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.

1 participant