Skip to content

fix: remove duplicate TestCompactTier1_DryRun#2631

Open
maphew wants to merge 1 commit intosteveyegge:mainfrom
maphew:fix-duplicate-test
Open

fix: remove duplicate TestCompactTier1_DryRun#2631
maphew wants to merge 1 commit intosteveyegge:mainfrom
maphew:fix-duplicate-test

Conversation

@maphew
Copy link
Collaborator

@maphew maphew commented Mar 16, 2026

Summary

  • Remove duplicate TestCompactTier1_DryRun function from internal/compact/compactor_unit_test.go
  • Fixes CI build failure from duplicate test function

@codecov-commenter
Copy link

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
4804 1 4803 548
View the top 1 failed test(s) by shortest run time
github.com/steveyegge/beads/cmd/bd::TestGetVersionsSinceOrder
Stack Traces | 0s run time
=== RUN   TestGetVersionsSinceOrder
    version_tracking_test.go:80: Versions not in chronological order: 0.61.0 (2026-03-15) should come before 0.57.0 (2026-03-01)
--- FAIL: TestGetVersionsSinceOrder (0.00s)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

Copy link

@DreadPirateRobertz DreadPirateRobertz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean fix — removes the duplicate test function. LGTM.

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.

3 participants