Added missing feature_category for CurrentUserMode tests
requested to merge gitlab-community/gitlab:gerardo-navarro-chore-add-feature-categeory-to-current-user-mode-tests into master
What does this MR do and why?
- Applying the GitLab guideline to some tests I worked on during another MR, i.e. !142158 (merged)
- This MR removes distracting console output when executing the tests, i.e.
Missing metadata feature_category: ./spec/lib/gitlab/auth/current_user_mode_spec.rb See https://docs.gitlab.com/ee/development/testing_guide/best_practices.html#feature-category-metadata
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
MR Checklist (@gerardo-navarro)
-
Changelog entry added, if necessary -
Documentation created/updated via this MR -
Documentation reviewed by technical writer or follow-up review issue created -
Tests added for this feature/bug -
Tested in all supported browsers -
Conforms to the code review guidelines -
Conforms to the merge request performance guidelines -
Conforms to the style guides -
Conforms to the javascript style guides -
Conforms to the database guides
Screenshots or screen recordings
No frontend . Only test -related changes.
How to set up and validate locally
- Execute the changed tests
bundle exec rspec spec/lib/gitlab/auth/current_user_mode_spec.rb ee/spec/lib/ee/gitlab/auth/current_user_mode_spec.rb- You should not see a console output related to missing feature category
Edited by Gerardo Navarro