Skip to content

Verify background migration arguments in specs [RUN ALL RSPEC]

Markus Koller requested to merge validate-background-migration-arguments into master

What does this MR do?

This extends the existing matchers to also verify that we're passing the correct number of arguments to background migrations.

This should avoid mistakes like !51475 (merged) 😅

Screenshots (strongly suggested)

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 Markus Koller

Merge request reports