Use named subject in tests to avoid Rubocop errors
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.
-
I have evaluated the MR acceptance checklist for this merge request.