Skip to content

CCM-14195: Updating tests#100

Merged
damientobin1 merged 2 commits into
mainfrom
feature/CCM-14195_fail_release_test_pipeline_on_test_failures
Apr 10, 2026
Merged

CCM-14195: Updating tests#100
damientobin1 merged 2 commits into
mainfrom
feature/CCM-14195_fail_release_test_pipeline_on_test_failures

Conversation

@damientobin1

Copy link
Copy Markdown
Contributor

Added new test to prove out pipeline change

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the parallel send email release test to use a new, email-specific routing plan configuration constant.

Changes:

  • Update test_parallel_send_email.py to reference PARALLEL_SEND_ROUTING_CONFIGURATION_EMAIL.
  • Add PARALLEL_SEND_ROUTING_CONFIGURATION_EMAIL constant in helpers/constants.py.

Reviewed changes

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

File Description
tests/release_tests/test_parallel_send_email.py Switches the routing plan ID used by the parallel send email release test to an email-specific constant.
helpers/constants.py Introduces an email-specific parallel send routing configuration UUID constant.

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

Comment thread tests/release_tests/test_parallel_send_email.py
Comment thread helpers/constants.py Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

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


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

@damientobin1 damientobin1 merged commit 876dab2 into main Apr 10, 2026
5 checks passed
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