Use `ProcessTree.killAll` from `InboundAgentRule`

jenkinsci / jenkins-test-harness

Unit test framework for Jenkins core and its plugins

https://www.jenkins.io/doc/developer/testing/

MIT License

109 stars 126 forks source link

Closed jglick closed 3 months ago

jglick commented 3 months ago

Ensures that even if an inbound agent starts some subprocesses, they are reliably killed off when the test completes.

basil commented 3 months ago

Breaks the following tests (and thus my EE 9 test runs):

org.jenkinsci.plugins.workflow.support.steps.AgentErrorConditionTest#retryNodeBlockSynch
org.jenkinsci.plugins.workflow.support.steps.ExecutorStepTest#buildShellScriptAcrossDisconnect
org.jenkinsci.plugins.workflow.support.steps.ExecutorStepTest#contextualizeFreshFilePathAfterAgentReconnection