Skip to content

Remove redundant docker executor integration tests

Jaime Martinez requested to merge 25425-integration-test-docker-steps into master

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?

#25425 (closed)

Edited by Steve Xuereb

Merge request reports