Skip to content

Refactor CI processable delegation specs

drew stachon requested to merge refactor-ci-processable-delegation-specs into master

What does this MR do?

This MR is a follow-up from !27968 (comment 323086163) to more simply test the delegation of methods from Ci::Processable to Ci::Pipeline. Repeating the outcome requires a bunch of setup for no real additional value over the specs that exist for the method on Ci::Pipeline.

Merging !27968 (merged) is a requirement for this, and this MR will be rebased on master at that time. After that this MR will consist of a single commit: !29605 (ed826ae2)

https://about.gitlab.com/handbook/values/#cleanup-over-sign-off

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • ~~Label as security and @ mention @gitlab-com/gl-security/appsec~~
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by drew stachon

Merge request reports