Skip to content

Add cop to prevent the usage of TestProf methods in migration specs

Mario Celi requested to merge mcelicalderonG-add-cop-testprof-migration-specs into master

What does this MR do and why?

We recently introduced !105728 (merged) since we were getting a failure in master due to a stuck transaction in migration specs. More context in !105375 (comment 1193483755).

We disabled transactions when using TestProf methods like let_it_be and before_all, so we should not use those in migration specs as they might not work properly.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports