Skip to content

Improve tests for require_organization logic

What does this MR do and why?

This is an addition to !155732 (merged). I don't want to reset approvals on that MR so I created a new branch for this

These changes were part of !160073 (closed) but it was hard to rebase that branch on master. So I decided to cherry-pick the relevant commit from that MR and start over.

Based on the discussion in this comment, the following improvements are made:

  • For both enabled and disabled feature flag, the specs are now similar.
  • Request Store is enabled for all the specs for this validation
  • One test was wrong, it is now correct (see line 740)

Related to #411832 (closed)

Merge request reports

Loading