Skip to content

chore: use demo-repo as neutral test fixture name#23

Merged
fank merged 1 commit into
mainfrom
claude/neutral-test-fixtures
Apr 11, 2026
Merged

chore: use demo-repo as neutral test fixture name#23
fank merged 1 commit into
mainfrom
claude/neutral-test-fixtures

Conversation

@fank
Copy link
Copy Markdown
Member

@fank fank commented Apr 11, 2026

Summary

  • Replaces negsoft-gui in test fixtures with the neutral demo-repo placeholder as part of open-sourcing readiness
  • Owner stays enthus-appdev — this project ships under that org
  • Pure identifier swap: internal/gh/repo_test.go + internal/gh/comment_test.go, 10 lines each direction, zero functional change

Test plan

  • go test -count=1 ./... passes
  • go vet ./... clean
  • grep -r negsoft returns no matches in working tree

Drops the lingering negsoft-gui reference from test fixtures as
part of open-sourcing readiness. Owner stays enthus-appdev; this
is a pure identifier swap with no functional change.
@github-actions
Copy link
Copy Markdown

Merging this branch will not change overall coverage

Impacted Packages Coverage Δ 🤖
github.com/enthus-appdev/gh-attach/internal/gh 96.67% (ø)

Coverage by file

Changed unit test files

  • github.com/enthus-appdev/gh-attach/internal/gh/comment_test.go
  • github.com/enthus-appdev/gh-attach/internal/gh/repo_test.go

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

The pull request updates the test suites in internal/gh/comment_test.go and internal/gh/repo_test.go to replace the repository name "negsoft-gui" with "demo-repo" across various test cases. I have no feedback to provide.

@fank fank merged commit 143d144 into main Apr 11, 2026
3 checks passed
@fank fank deleted the claude/neutral-test-fixtures branch April 11, 2026 11:10
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