Skip to content

Run single integration test#86

Merged
carole-lavillonniere merged 1 commit intomainfrom
run-single-test
Mar 9, 2026
Merged

Run single integration test#86
carole-lavillonniere merged 1 commit intomainfrom
run-single-test

Conversation

@carole-lavillonniere
Copy link
Copy Markdown
Collaborator

@carole-lavillonniere carole-lavillonniere commented Mar 9, 2026 •
edited
Loading

Added a RUN variable to the test-integration Makefile target so a single integration test can be run from the repo root without having to cd into test/integration:

make test-integration RUN=TestStartCommandSucceedsWithValidToken

Without RUN, the target behaves exactly as before and runs all integration tests.

@carole-lavillonniere carole-lavillonniere changed the title Add RUN filter to test-integration Makefile target Run single integration test Mar 9, 2026
@carole-lavillonniere carole-lavillonniere marked this pull request as ready for review March 9, 2026 08:48
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: a5565941-3782-4a09-90c6-a0c36842dcfe

📥 Commits

Reviewing files that changed from the base of the PR and between b6fe59a and 9621842.

📒 Files selected for processing (2)
  • CLAUDE.md
  • Makefile
📝 Walkthrough

Walkthrough

This change enables test filtering for the integration test target by adding conditional support for a RUN parameter in the Makefile, allowing selective test execution. The documentation example has been updated to demonstrate using the Make target with this new filtering capability.

Changes

Cohort / File(s) Summary
Documentation
CLAUDE.md		 Updated integration test invocation example to use the Make target with the RUN parameter for test filtering instead of direct Go command.
Build Configuration
Makefile		 Modified test-integration target to conditionally append -run $(RUN) flag when the RUN variable is set, enabling selective test execution across both Darwin and non-Darwin branches.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Suggested reviewers

  • silv-io
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Run single integration test' directly and clearly describes the main change, which is enabling a single integration test to be run via the test-integration target.
Description check ✅ Passed The description clearly explains the feature added: a RUN variable for the test-integration Makefile target, with example usage and backward compatibility details.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch run-single-test

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

silv-io
silv-io approved these changes Mar 9, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@silv-io silv-io left a comment

Choose a reason for hiding this comment

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

LGTM!

anisaoshafi
anisaoshafi approved these changes Mar 9, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@anisaoshafi anisaoshafi left a comment

Choose a reason for hiding this comment

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

Nice :D

@carole-lavillonniere carole-lavillonniere merged commit 4eb7d9b into main Mar 9, 2026
8 checks passed
@carole-lavillonniere carole-lavillonniere deleted the run-single-test branch March 9, 2026 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@silv-io silv-io silv-io approved these changes

@anisaoshafi anisaoshafi anisaoshafi approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@carole-lavillonniere @silv-io @anisaoshafi