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 You can disable this status message by setting the
✨ 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 |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #24 +/- ##
=======================================
Coverage ? 85.97%
Complexity ? 234
=======================================
Files ? 9
Lines ? 699
Branches ? 54
=======================================
Hits ? 601
Misses ? 61
Partials ? 37 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Adds TestNG retry detection support to mark ReportPortal scenario items as retries (with retryOf linkage), plus updates the ReportPortal client dependency and extends parameter-reporting tests.
Changes:
- Add a TestNG listener (
TestNgRetriesListener) and wire ScenarioReporter to mark retries on scenario start - Add new integration tests/feature/steps to validate retry detection and scenario outline parameter reporting
- Bump
com.epam.reportportal:client-javato5.4.8and addcucumber-testngas a compileOnly dependency
Reviewed changes
Copilot reviewed 11 out of 13 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/main/java/com/epam/reportportal/cucumber/ScenarioReporter.java | Uses TestNG retry info to set retry/retryOf on scenario start requests |
| src/main/java/com/epam/reportportal/cucumber/Utils.java | Exposes KEY_VALUE_SEPARATOR for reuse by retry listener logic |
| src/main/java/com/epam/reportportal/cucumber/testng/TestNgRetriesListener.java | New TestNG listener that tracks whether a Cucumber scenario execution is a retry |
| src/main/resources/META-INF/services/org.testng.ITestNGListener | Registers the retry listener via Java ServiceLoader |
| src/test/resources/features/TestNgRetry.feature | New Cucumber feature to simulate a flaky scenario for retry testing |
| src/test/java/com/epam/reportportal/cucumber/TestNgRetryDetectionTest.java | New test validating retry flags and retryOf linkage |
| src/test/java/com/epam/reportportal/cucumber/integration/feature/RetrySteps.java | Step definitions to fail N times then pass |
| src/test/java/com/epam/reportportal/cucumber/integration/testng/RetryAnalyzer.java | Simple TestNG retry analyzer used by the new retry test |
| src/test/java/com/epam/reportportal/cucumber/integration/util/TestUtils.java | Adds a helper to run TestNG tests with an optional listener |
| src/test/java/com/epam/reportportal/cucumber/ParameterTest.java | Adds scenario-outline parameter verification and adjusts IDs to cover added cases |
| build.gradle | Updates client dependency and adds compileOnly cucumber-testng |
| CHANGELOG.md | Documents added retry handling and client version bump |
| .gitignore | Ignores bin/ output directory |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.