EPMRPP-111752 || Add info about 'Change status' on Step and Log level#1061
EPMRPP-111752 || Add info about 'Change status' on Step and Log level#1061maria-hambardzumian merged 1 commit intodevelopfrom
Conversation
WalkthroughDocumentation files were reorganized by updating sidebar position metadata for two files, and a new section describing manual status changes for Steps and Logs was added to the ViewLaunches documentation. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@docs/work-with-reports/ViewLaunches.mdx`:
- Around line 147-150: Add a brief clarifying sentence under the "You can change
the status of a Step or Log" list explaining that the Interrupted status is not
user-settable: insert a note after the list stating something like "Note: The
Interrupted status is automatically applied by the system when test execution is
interrupted and cannot be manually set." Reference the existing list (Passed,
Failed, Skipped) and the term "Interrupted" so readers understand the
distinction.
🧹 Nitpick comments (1)
docs/work-with-reports/ViewLaunches.mdx (1)
144-162: Tighten grammar and casing in the new status-change section.Minor wording fixes improve clarity (Line 144, Line 160).
✍️ Suggested edit
-ReportPortal allows to manually change the execution status of Logs and Steps. +ReportPortal allows you to manually change the execution status of logs and steps. ... -Changing the status on Log or Step level affects launch statistics and aggregated counters +Changing the status on log or step level affects launch statistics and aggregated counters
| You can change the status of a Step or Log to one of the following values: | ||
| - Passed | ||
| - Failed | ||
| - Skipped |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
# First, let's examine the documentation file
head -160 docs/work-with-reports/ViewLaunches.mdx | tail -20Repository: reportportal/docs
Length of output: 714
🏁 Script executed:
# Search for status definitions and validations in the codebase
rg -i "passed|failed|skipped|interrupted" --type json --type ts --type tsx --type js --type jsx -l | head -20Repository: reportportal/docs
Length of output: 88
🏁 Script executed:
# Look for status enums or constants
rg -i "status.*enum|enum.*status|const.*status" --type ts --type tsx --type js -A 5Repository: reportportal/docs
Length of output: 88
🏁 Script executed:
# Search for API routes or endpoints that handle status changes
rg -i "step.*status|log.*status" --type ts --type tsx --type js -l | head -15Repository: reportportal/docs
Length of output: 88
🏁 Script executed:
# Search for "Interrupted" in the documentation
rg -i "interrupted" docs/Repository: reportportal/docs
Length of output: 5669
🏁 Script executed:
# Look for status reference or tables that mention multiple statuses
rg -i "status.*table|status.*reference" docs/ -A 3Repository: reportportal/docs
Length of output: 3106
🏁 Script executed:
# Check if there are any API documentation files mentioning status values
fd -e md -e mdx | xargs rg -l "status" | head -10Repository: reportportal/docs
Length of output: 475
🏁 Script executed:
# Search for any comprehensive status documentation or tables
rg -i "passed.*failed.*skipped" docs/ -B 2 -A 2Repository: reportportal/docs
Length of output: 25111
Add clarifying note that manual status changes are limited to Passed/Failed/Skipped.
The documentation correctly lists only Passed, Failed, and Skipped as manually changeable statuses for Steps and Logs. However, since Interrupted appears throughout the system (in statistics, widgets, and API responses), readers may be confused about why it's not included. Add a brief note clarifying that Interrupted is automatically set by the system and cannot be changed manually—for example: "Note: The Interrupted status is automatically applied by the system when test execution is interrupted and cannot be manually set."
🤖 Prompt for AI Agents
In `@docs/work-with-reports/ViewLaunches.mdx` around lines 147 - 150, Add a brief
clarifying sentence under the "You can change the status of a Step or Log" list
explaining that the Interrupted status is not user-settable: insert a note after
the list stating something like "Note: The Interrupted status is automatically
applied by the system when test execution is interrupted and cannot be manually
set." Reference the existing list (Passed, Failed, Skipped) and the term
"Interrupted" so readers understand the distinction.
* Merge pull request #1059 from reportportal/EPMRPP-111091 EPMRPP-111091 || Reorganize Log level docs * Update deploy-dev.yml to support OIDC auth for AWS * Update permissions in deploy-dev workflow for OIDC * EPMRPP-111752 || Add info about 'Change status' on Log and Step level (#1061) Co-authored-by: Yuliya_Prihodko <Yuliya_Prihodko@epam.com> --------- Co-authored-by: Yuliya Prihodko <107035036+pressayuliya@users.noreply.github.com> Co-authored-by: Ilya <ilya_hancharyk@epam.com> Co-authored-by: Yuliya_Prihodko <Yuliya_Prihodko@epam.com>
* Merge pull request #1059 from reportportal/EPMRPP-111091 EPMRPP-111091 || Reorganize Log level docs * Update deploy-dev.yml to support OIDC auth for AWS * Update permissions in deploy-dev workflow for OIDC * EPMRPP-111752 || Add info about 'Change status' on Log and Step level (#1061) Co-authored-by: Yuliya_Prihodko <Yuliya_Prihodko@epam.com> --------- Co-authored-by: Yuliya Prihodko <107035036+pressayuliya@users.noreply.github.com> Co-authored-by: Ilya <ilya_hancharyk@epam.com> Co-authored-by: Yuliya_Prihodko <Yuliya_Prihodko@epam.com> 9390ed1
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.