Skip to content

test: integration test PR (DO NOT MERGE)#7

Draft
sQVe wants to merge 2 commits into
mainfrom
integration-test-pr-branch
Draft

test: integration test PR (DO NOT MERGE)#7
sQVe wants to merge 2 commits into
mainfrom
integration-test-pr-branch

Conversation

@sQVe

@sQVe sQVe commented Dec 5, 2025

Copy link
Copy Markdown
Owner

This PR exists solely for integration testing of grove's PR features.

DO NOT MERGE - This is test infrastructure.

The PR tests in add_pr_integration.txt use this PR to verify:

  • PR number creates worktree
  • PR URL creates worktree
  • PR with --switch flag
  • Error handling for deleted branches

This PR should remain open indefinitely as a draft.

sQVe added 2 commits December 5, 2025 08:53
Grove users are git users - their muscle memory expects -v for verbose,
-f for force, -d for detach. Fighting that is stupid.

Add: -d (detach), -v (verbose), -f (force), -a (all)
Remove: -d (debug), -r (reason) - non-standard
This branch and associated PR are used by grove's integration
tests to verify PR functionality. DO NOT MERGE.
sQVe added a commit that referenced this pull request Dec 5, 2025
- Add PR integration tests using dedicated test PR #7
- Fix case-insensitive GitHub username comparison for fork detection
- Add tests for lock cleanup, remote handling, and error messages
- Auto-detect GH_TOKEN from gh CLI in mage test:integration
- Add SCENARIOS.md documenting test coverage for grove add

The fork detection bug caused same-repo PRs to be misidentified as
forks when the URL used different casing than the GitHub API returns
(e.g., sqve vs sQVe). Fixed with strings.EqualFold().
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