Skip to content

expose test report id on pull command#367

Merged
Germandrummer92 merged 1 commit intomainfrom
expose-the-test-report-id
Feb 3, 2026
Merged

expose test report id on pull command#367
Germandrummer92 merged 1 commit intomainfrom
expose-the-test-report-id

Conversation

@Germandrummer92
Copy link
Member

@Germandrummer92 Germandrummer92 commented Feb 3, 2026

Summary by CodeRabbit

  • New Features

    • Added --test-report-id option to the pull command to filter results by test report.
  • Chores

    • Updated CLI version to 4.7.0.
    • Updated biome schema reference.

@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

📝 Walkthrough

Walkthrough

The pull request bumps the CLI version from 4.6.1 to 4.7.0 and introduces a new optional filter parameter (-r, --test-report-id) to the pull command, enabling users to filter test case auto-fixes by a specific test report ID.

Changes

Cohort / File(s) Summary
Version Bumps
README.md, package.json
Bumped CLI version from 4.6.1 to 4.7.0 across package metadata and documentation.
Configuration
biome.json
Updated Biome schema reference from 2.3.11 to 2.3.13.
Feature Implementation
src/cli.ts, src/tools/test-targets.ts
Added new optional testReportId parameter to the pull command. Updated pullTestTarget method signature to accept testReportId and adjusted query construction to always include both testPlanId and testReportId keys.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • PR #344: Both PRs modify the same pullTestTarget function in src/tools/test-targets.ts to add optional filter parameters for enhanced query capability.

Suggested reviewers

  • fabianhinz

Poem

🐰 A filter for reports, so keen and so bright,
New flags in the CLI make testing just right,
With -r and --test-report-id in place,
We pull only what we need—case by case! ✨

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: exposing a test report ID option on the pull command, which aligns with the modifications across README, CLI, and test-targets files.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch expose-the-test-report-id

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

@Germandrummer92 Germandrummer92 merged commit c093312 into main Feb 3, 2026
4 checks passed
@Germandrummer92 Germandrummer92 deleted the expose-the-test-report-id branch February 3, 2026 17: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.

2 participants

Comments