Follow-up from "Sync approval rules for group policies when transferring a project"

The following discussion from !127123 (merged) should be addressed:

• @group_9970_bot_58e69a7d2ace73c7373ac2bc2a5cabd0 started a discussion: (+4 comments)

          let_it_be(:project) { create(:project, :repository, group: sub_group) }

  Project creations are very slow. To improve test performance, consider using let_it_be, build, or build_stubbed instead.

  ⚠️ Warning: If your test modifies data, let_it_be may be unsuitable, and cause state leaks! Use let_it_be_with_reload or let_it_be_with_refind instead.

  Unsure which method to use? See the testing best practices for background information and alternative options for optimizing factory usage.

  If you're concerned about causing state leaks, or if you know let or let! are the better options, ignore this comment.

  (Improve this message?)