Skip to content

Fix integration test sending mail#6709

Merged
oliver-sanders merged 2 commits intocylc:8.4.xfrom
MetRonnie:tests
Apr 7, 2025
Merged

Fix integration test sending mail#6709
oliver-sanders merged 2 commits intocylc:8.4.xfrom
MetRonnie:tests

Conversation

@MetRonnie
Copy link
Copy Markdown
Member

@MetRonnie MetRonnie commented Apr 7, 2025 •
edited
Loading

Follow-up to #6639

Also consolidate tests that were spread across different files

Check List

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.
  • Contains logically grouped changes (else tidy your branch by rebase).
  • Does not contain off-topic changes (use other PRs for other changes).
  • No dependency changes
  • Tests, changelog, docs N/A
  • If this is a bug fix, PR should be raised against the relevant ?.?.x branch.

@MetRonnie MetRonnie added the small label Apr 7, 2025
@MetRonnie MetRonnie added this to the 8.4.2 milestone Apr 7, 2025
@MetRonnie MetRonnie self-assigned this Apr 7, 2025
@MetRonnie MetRonnie requested a review from wxtim April 7, 2025 11:56
@wxtim wxtim requested a review from oliver-sanders April 7, 2025 12:02
@oliver-sanders oliver-sanders merged commit 43fe2ed into cylc:8.4.x Apr 7, 2025
11 of 12 checks passed
@MetRonnie MetRonnie deleted the tests branch April 7, 2025 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@oliver-sanders oliver-sanders oliver-sanders approved these changes

+1 more reviewer

@wxtim wxtim wxtim approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@MetRonnie MetRonnie

Labels

small

Projects

None yet

Milestone

8.4.2

Development

Successfully merging this pull request may close these issues.

3 participants

@MetRonnie @oliver-sanders @wxtim