Skip to content

[draft] Introduce watchdog into ProcessGroupManager [4/N]: Add ProcessGroupManager unit tests for watchdog - #3

Draft
cameron-craig-etas wants to merge 2 commits into
feature/move-watchdog-to-launch-manager-part-3from
feature/move-watchdog-to-launch-manager-part-4
Draft

[draft] Introduce watchdog into ProcessGroupManager [4/N]: Add ProcessGroupManager unit tests for watchdog#3
cameron-craig-etas wants to merge 2 commits into
feature/move-watchdog-to-launch-manager-part-3from
feature/move-watchdog-to-launch-manager-part-4

Conversation

@cameron-craig-etas

@cameron-craig-etas cameron-craig-etas commented Jul 23, 2026

Copy link
Copy Markdown

eclipse-score#330

This is part 3 of a number of PRs, to move the watchdog ownership to ProcessGroupManager. Now that the watchdow is owned by ProcessGroupManager, we are adding some unit tests to ensure the watchdog is called as we would expect.

The following changes have been made:

  • Mocks and stubs have been implemented where needed, in order to test ProcessGroupManager.
  • Tests have been added to ensure the watchdog methods are called when expected.
  • The GivenWhenThen structure is being used for these tests, as suggested in the Test Specification Guide.
  • We do not plan to link these tests to detailed design at this stage.

@cameron-craig-etas

Copy link
Copy Markdown
Author

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.

1 participant