Skip to content

Conversation

@officialasishkumar
Copy link
Member

What has changed?

Please include a summary of the change.

This PR Resolves keploy/keploy#2919

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue).
  • New feature (non-breaking change which adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to not work as expected).
  • Documentation update (if none of the other choices apply).

How Has This Been Tested?

Please run npm run build and npm run serve to check if the changes are working as expected. Please include screenshots of the output of both the commands. Add screenshots/gif of the changes if possible.

Checklist:

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.

Copilot AI review requested due to automatic review settings August 27, 2025 08:19
@keploy
Copy link

keploy bot commented Aug 27, 2025

To generate Unit Tests for this PR, please click here.

Signed-off-by: Asish Kumar <officialasishkumar@gmail.com>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds comprehensive documentation for the report command in Keploy CLI, addressing issue #2919. The documentation explains how to use the command to display detailed test result summaries and human-readable diffs for failed test cases.

  • Added report command to the commands table with available flags
  • Documented usage, purpose, and all available flags with examples
  • Provided practical examples for each flag option

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

| `gen` | `--sourceFilePath`, `--testFilePath`,`--coverageReportPath`,`--testCommand`,`--coverageFormat`,`--expectedCoverage`,`--maxIterations`,`--testDir`,`--llmBaseUrl`,`--model`,`--llmApiVersion` |
| `normailze` | `-p, --path`, `--test-run`, `--tests` |
| `rerecord` | `--test-sets`, `-t` |
| `report` | `--test-sets, -t`, `-p, --path`, `--report-path, -r`, `--full-body` |
Copy link

Copilot AI Aug 27, 2025

Choose a reason for hiding this comment

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

The flag documentation is inconsistent. The -r alias for --report-path is shown in the table but not mentioned in the detailed flag description at line 351. Either add the -r alias to the detailed description or remove it from the table for consistency.

Copilot uses AI. Check for mistakes.

<b> Available flags: </b>

- `-t, --test-sets strings` - Testsets to report, e.g., `--test-sets "test-set-1, test-set-2"`.
Copy link

Copilot AI Aug 27, 2025

Choose a reason for hiding this comment

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

The term 'Testsets' should be 'Test sets' (two words) for proper grammar and consistency with the rest of the documentation.

Suggested change
- `-t, --test-sets strings` - Testsets to report, e.g., `--test-sets "test-set-1, test-set-2"`.
- `-t, --test-sets strings` - Test sets to report, e.g., `--test-sets "test-set-1, test-set-2"`.

Copilot uses AI. Check for mistakes.
Signed-off-by: Asish Kumar <officialasishkumar@gmail.com>
@Sarthak160 Sarthak160 merged commit 390b521 into main Aug 28, 2025
1 check passed
@Sarthak160 Sarthak160 deleted the feat/report branch August 28, 2025 11:43
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.

[feature]: Only show the changed diffs in report command

3 participants