Skip to content

refactor: extract test helpers in e2e/bare_test.go#153

Merged
k1LoW merged 2 commits into
k1LoW:mainfrom
usadamasa:refactor-bare
Feb 24, 2026
Merged

refactor: extract test helpers in e2e/bare_test.go#153
k1LoW merged 2 commits into
k1LoW:mainfrom
usadamasa:refactor-bare

Conversation

@usadamasa

@usadamasa usadamasa commented Feb 21, 2026

Copy link
Copy Markdown
Contributor

Cleanup of #150 (closes part of #130).

  • Extract addRawWorktreeFromBare, assertWorktreeExists, assertWorktreeDeleted, and runGitWtWithShellIntegration helpers in e2e/helper_test.go to reduce boilerplate across bare_test.go.
  • Remove unused XXX function from internal/git/repo_context.go.

@usadamasa usadamasa closed this Feb 21, 2026
@usadamasa usadamasa deleted the refactor-bare branch February 21, 2026 04:41
@usadamasa usadamasa restored the refactor-bare branch February 21, 2026 04:42
@usadamasa usadamasa reopened this Feb 21, 2026
@usadamasa usadamasa marked this pull request as draft February 21, 2026 05:23
@usadamasa usadamasa changed the title refactor: remove unused func in repo_context refactor: extract test helpers in e2e/bare_test.go Feb 21, 2026
@usadamasa usadamasa marked this pull request as ready for review February 21, 2026 05:25

@k1LoW k1LoW left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@usadamasa LGTM!
Thank you!!

@k1LoW k1LoW merged commit a112091 into k1LoW:main Feb 24, 2026
3 checks passed
@github-actions github-actions Bot mentioned this pull request Feb 24, 2026
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.

2 participants