Skip to content

Conversation

@rconner46
Copy link
Contributor

What changed?

When reporting to an applause internal test cycle, support custom test run names

@rconner46 rconner46 requested a review from Copilot November 5, 2025 16:15
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 support for custom test run names when reporting to Applause internal test cycles. Previously, only the test cycle ID could be specified; now users can also provide an optional custom name for the test run.

Key changes:

  • Added applauseTestRunName parameter to ApplauseTestCycleReportingConfig
  • Updated both TestNG and Cucumber listeners to pass the custom test run name
  • Renamed itwTestCycleId to applauseTestCycleId for consistency

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
ApplauseTestCycleReportingConfig.java Added nullable applauseTestRunName parameter to the record
ApplauseSdkConfigBean.java Added applauseTestRunName() configuration method
TestRunConfigurationParamDto.java Renamed field from itwTestCycleId to applauseTestCycleId and added applauseTestRunName
SessionInitializer.java Updated to use renamed setter method and pass custom test run name
ReportingSuiteListener.java Updated TestNG listener to pass custom test run name to reporting config
ApplauseReporterPlugin.java Updated Cucumber plugin to pass custom test run name to reporting config

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rconner46 rconner46 merged commit df686c2 into master Nov 7, 2025
6 checks passed
@rconner46 rconner46 deleted the custom-applause-internal-run-name branch November 7, 2025 15:41
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.

3 participants