Skip to content

Use named subject in tests to avoid Rubocop errors

Evan Read requested to merge eread/user-named-subject-in-tests into main

What does this MR do and why?

With this new major update to gitlab-styles: !4388 (merged), some of our tests now violate https://docs.rubocop.org/rubocop-rspec/cops_rspec.html#rspecnamedsubject.

I've simply updated the tests to use a named subject. That will unblock !4388 (merged).

Merge request 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