Default unset step when to on_success in concrete builder

What does this MR do?

Default unset step when to on_success in concrete builder

The concrete builder mapped a step with an empty when to both OnSuccess and OnFailure being false, so step-runner skipped the step while the job still reported success.

Add OnSuccess/OnFailure methods to StepWhen, mirroring ArtifactWhen and CacheWhen, so an unset when defaults to on_success in one place. The concrete builder now derives step, cache, and artifact flags through these helpers instead of inline comparisons.

Why was this MR needed?

There was a mismatch in concrete for step when, where the wrong default was used.

What are the relevant issue numbers?

Closes #39539 (closed)

Merge request reports

Loading