Skip to content

RuboCop: Fix warnings about wrong namespace

Peter Leitzen requested to merge pl-rubocop-warning-namespace into master

What does this MR do and why?

This MR fixes the following RuboCop warnings (introduced in !105772 (merged)):

ee/spec/lib/gitlab/geo/log_helpers_spec.rb: Rspec/DescribedClass has the wrong namespace - should be RSpec
ee/spec/controllers/concerns/routable_actions_spec.rb: Rspec/DescribedClass has the wrong namespace - should be RSpec
ee/spec/controllers/concerns/registrations/verification_spec.rb: Rspec/DescribedClass has the wrong namespace - should be RSpec
ee/spec/controllers/concerns/gitlab_subscriptions/seat_count_alert_spec.rb: Rspec/DescribedClass has the wrong namespace - should be RSpec

How to set up and validate locally

# On this MR

changed_files=$(git diff --name-only origin/master)

bundle exec rubocop -C false $changed_files

Inspecting 4 files
....

4 files inspected, no offenses detected


# On master

bundle exec rubocop -C false $changed_files

Inspecting 4 files
ee/spec/controllers/concerns/gitlab_subscriptions/seat_count_alert_spec.rb: Rspec/DescribedClass has the wrong namespace - should be RSpec
.ee/spec/controllers/concerns/registrations/verification_spec.rb: Rspec/DescribedClass has the wrong namespace - should be RSpec
.ee/spec/controllers/concerns/routable_actions_spec.rb: Rspec/DescribedClass has the wrong namespace - should be RSpec
.ee/spec/lib/gitlab/geo/log_helpers_spec.rb: Rspec/DescribedClass has the wrong namespace - should be RSpec
.

4 files inspected, no offenses detected

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Peter Leitzen

Merge request reports