Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
|
Tick the box to add this pull request to the merge queue (same as
|
The backend exposes PUT /api/v1/test_run_executions/{id}/rename to
rename a test run execution, but no CLI command called it.
Adds `test-run-execution rename --id <id> --name <new-name>`, calling
rename_test_run_execution_api_v1_test_run_executions__id__rename_put
and reporting success or surfacing the API error, mirroring the
`delete` command added for #1112.
5048a4f to
570a067
Compare
The backend strips whitespace from new_execution_name before persisting, so echoing the raw --name input could show a different value than what was actually saved (e.g. --name " New Name " persists as "New Name"). Use the response's .title instead, matching how repeat/import already handle this. Addresses review feedback from @oxesoft on project-chip#121.
3 participants
Summary
The backend exposes
PUT /api/v1/test_run_executions/{id}/renameto rename a test run execution, but the CLI had no command that called it.Changes
renamesubcommand to thetest_run_executioncommand group (th_cli/commands/test_run_execution.py), taking--idand--name.rename_test_run_execution_api_v1_test_run_executions__id__rename_put(already present in the autogenerated API client).--id/--name, and--helpoutput.Usage
```
th_cli test-run-execution rename --id 1 --name "New Name"
```
Note on stacking
This branch is built on top of #120's branch (
feature/1112-cli-delete-test-run-execution), so this PR's diff currently includes #120'sdeletecommand commit too. No rebase needed on our side — once #120 merges intov2.16-cli-develop, this diff will automatically shrink to just therenamecommand changes.Closes #1113