Skip to content

Do not use factories to populate security_training_providers

Michał Zającrequested to merge
exclude-security_training_providers-factory-from-data-seeder into master

What does this MR do and why?

Do not use factories to populate security_training_providers because:

  1. Our CI uses https://gitlab.com/gitlab-org/quality/pg-dump-generator to populate the test db, which, in turn, uses all factories to create rows.
  2. We recently decided to remove that table in preparation for Protocells.
  3. We opted to replace that table by a static model with the same record set as production database and executed a migration to populate the faux-foreign-key.
  4. CI now fails on that migration because we have test data present in the table which are not accounted for in the static model

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports