Remove redundant docker executor integration tests
What does this MR do?
The assertion of the after_script
being executed was introduced in !1963 (diffs) with TestDockerCommandMultistepBuild
.
This MR removes the tests TestDockerCommandBuildFail
and TestDockerCommandSuccessRun
which are included in the test mentioned above.
Closes #25425 (closed).
Why was this MR needed?
TestDockerCommandMultistepBuild
runs the same assertions as TestDockerCommandBuildFail
and TestDockerCommandSuccessRun
so those tests are redundant.
What's the best way to test this MR?
What are the relevant issue numbers?
Edited by Steve Xuereb