Merge branch 'fix-todo-cops' into 'master'

Fix todo cops

See merge request gitlab-org/gitlab-ce!21850

.rubocop_todo.yml

@@ -10,24 +10,6 @@
 Capybara/CurrentPathExpectation:
   Enabled: false
 
-# Offense count: 23
-FactoryBot/DynamicAttributeDefinedStatically:
-  Exclude:
-    - 'spec/factories/broadcast_messages.rb'
-    - 'spec/factories/ci/builds.rb'
-    - 'spec/factories/ci/runners.rb'
-    - 'spec/factories/clusters/applications/helm.rb'
-    - 'spec/factories/clusters/platforms/kubernetes.rb'
-    - 'spec/factories/emails.rb'
-    - 'spec/factories/gpg_keys.rb'
-    - 'spec/factories/group_members.rb'
-    - 'spec/factories/merge_requests.rb'
-    - 'spec/factories/notes.rb'
-    - 'spec/factories/oauth_access_grants.rb'
-    - 'spec/factories/project_members.rb'
-    - 'spec/factories/todos.rb'
-    - 'spec/factories/uploads.rb'
-
 # Offense count: 167
 # Cop supports --auto-correct.
 Layout/EmptyLinesAroundArguments:
@@ -53,20 +35,6 @@ Layout/IndentArray:
 Layout/IndentHash:
   Enabled: false
 
-# Offense count: 11
-# Cop supports --auto-correct.
-# Configuration parameters: AllowForAlignment.
-Layout/SpaceBeforeFirstArg:
-  Exclude:
-    - 'config/routes/project.rb'
-    - 'db/migrate/20170506185517_add_foreign_key_pipeline_schedules_and_pipelines.rb'
-    - 'features/steps/project/source/browse_files.rb'
-    - 'features/steps/project/source/markdown_render.rb'
-    - 'lib/api/runners.rb'
-    - 'spec/features/search/user_uses_search_filters_spec.rb'
-    - 'spec/routing/project_routing_spec.rb'
-    - 'spec/services/system_note_service_spec.rb'
-
 # Offense count: 93
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle.
@@ -74,15 +42,6 @@ Layout/SpaceBeforeFirstArg:
 Layout/SpaceInLambdaLiteral:
   Enabled: false
 
-# Offense count: 1
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, EnforcedStyleForEmptyBrackets.
-# SupportedStyles: space, no_space, compact
-# SupportedStylesForEmptyBrackets: space, no_space
-Layout/SpaceInsideArrayLiteralBrackets:
-  Exclude:
-    - 'spec/lib/gitlab/import_export/relation_factory_spec.rb'
-
 # Offense count: 327
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, EnforcedStyleForEmptyBraces, SpaceBeforeBlockParameters.
@@ -96,14 +55,6 @@ Layout/SpaceInsideBlockBraces:
 Layout/SpaceInsideParens:
   Enabled: false
 
-# Offense count: 14
-# Cop supports --auto-correct.
-Layout/SpaceInsidePercentLiteralDelimiters:
-  Exclude:
-    - 'lib/gitlab/git_access.rb'
-    - 'lib/gitlab/health_checks/fs_shards_check.rb'
-    - 'spec/lib/gitlab/health_checks/fs_shards_check_spec.rb'
-
 # Offense count: 26
 Lint/DuplicateMethods:
   Exclude:
@@ -135,31 +86,11 @@ Lint/InterpolationCheck:
 Lint/MissingCopEnableDirective:
   Enabled: false
 
-# Offense count: 2
-Lint/NestedPercentLiteral:
-  Exclude:
-    - 'lib/gitlab/git/repository.rb'
-    - 'spec/support/shared_examples/email_format_shared_examples.rb'
-
 # Offense count: 1
 Lint/ReturnInVoidContext:
   Exclude:
     - 'app/models/project.rb'
 
-# Offense count: 1
-# Configuration parameters: IgnoreImplicitReferences.
-Lint/ShadowedArgument:
-  Exclude:
-    - 'lib/gitlab/database/sha_attribute.rb'
-
-# Offense count: 3
-# Cop supports --auto-correct.
-Lint/UnneededRequireStatement:
-  Exclude:
-    - 'db/post_migrate/20161221153951_rename_reserved_project_names.rb'
-    - 'db/post_migrate/20170313133418_rename_more_reserved_project_names.rb'
-    - 'lib/declarative_policy.rb'
-
 # Offense count: 9
 Lint/UriEscapeUnescape:
   Exclude:
@@ -199,16 +130,6 @@ Naming/HeredocDelimiterCase:
 Naming/HeredocDelimiterNaming:
   Enabled: false
 
-# Offense count: 1
-Performance/UnfreezeString:
-  Exclude:
-    - 'features/steps/project/commits/commits.rb'
-
-# Offense count: 1
-# Cop supports --auto-correct.
-Performance/UriDefaultParser:
-  Exclude:
-    - 'lib/gitlab/url_sanitizer.rb'
 
 # Offense count: 3821
 # Configuration parameters: Prefixes.

config/routes/project.rb

@@ -365,7 +365,7 @@
           get :discussions, format: :json
         end
         collection do
-          post  :bulk_update
+          post :bulk_update
         end
       end
 

db/migrate/20170506185517_add_foreign_key_pipeline_schedules_and_pipelines.rb

@@ -13,7 +13,7 @@ def up
         'SET NULL'
       end
 
-    add_concurrent_foreign_key  :ci_pipelines, :ci_pipeline_schedules,
+    add_concurrent_foreign_key :ci_pipelines, :ci_pipeline_schedules,
       column: :pipeline_schedule_id, on_delete: on_delete
   end
 

db/post_migrate/20161221153951_rename_reserved_project_names.rb

-require 'thread'
-
 class RenameReservedProjectNames < ActiveRecord::Migration
   include Gitlab::Database::MigrationHelpers
   include Gitlab::ShellAdapter

db/post_migrate/20170313133418_rename_more_reserved_project_names.rb

-require 'thread'
-
 class RenameMoreReservedProjectNames < ActiveRecord::Migration
   include Gitlab::Database::MigrationHelpers
   include Gitlab::ShellAdapter

lib/api/runners.rb

@@ -113,7 +113,7 @@ class Runners < Grape::API
         optional :status, type: String, desc: 'Status of the job', values: Ci::Build::AVAILABLE_STATUSES
         use :pagination
       end
-      get  ':id/jobs' do
+      get ':id/jobs' do
         runner = get_runner(params[:id])
         authenticate_list_runners_jobs!(runner)
 

lib/declarative_policy.rb

@@ -10,8 +10,6 @@
 
 require_dependency 'declarative_policy/base'
 
-require 'thread'
-
 module DeclarativePolicy
   CLASS_CACHE_MUTEX = Mutex.new
   CLASS_CACHE_IVAR = :@__DeclarativePolicy_CLASS_CACHE

lib/gitlab/git_access.rb

@@ -24,8 +24,8 @@ class GitAccess
       cannot_push_to_read_only: "You can't push code to a read-only GitLab instance."
     }.freeze
 
-    DOWNLOAD_COMMANDS = %w{ git-upload-pack git-upload-archive }.freeze
-    PUSH_COMMANDS = %w{ git-receive-pack }.freeze
+    DOWNLOAD_COMMANDS = %w{git-upload-pack git-upload-archive}.freeze
+    PUSH_COMMANDS = %w{git-receive-pack}.freeze
     ALL_COMMANDS = DOWNLOAD_COMMANDS + PUSH_COMMANDS
 
     attr_reader :actor, :project, :protocol, :authentication_abilities, :namespace_path, :project_path, :redirected_path, :auth_result_type, :changes

lib/gitlab/url_sanitizer.rb

@@ -3,7 +3,7 @@ class UrlSanitizer
     ALLOWED_SCHEMES = %w[http https ssh git].freeze
 
     def self.sanitize(content)
-      regexp = URI::Parser.new.make_regexp(ALLOWED_SCHEMES)
+      regexp = URI::DEFAULT_PARSER.make_regexp(ALLOWED_SCHEMES)
 
       content.gsub(regexp) { |url| new(url).masked_url }
     rescue Addressable::URI::InvalidURIError

spec/factories/broadcast_messages.rb

 FactoryBot.define do
   factory :broadcast_message do
     message "MyText"
-    starts_at 1.day.ago
-    ends_at 1.day.from_now
+    starts_at { 1.day.ago }
+    ends_at { 1.day.from_now }
 
     trait :expired do
-      starts_at 5.days.ago
-      ends_at 3.days.ago
+      starts_at { 5.days.ago }
+      ends_at { 3.days.ago }
     end
 
     trait :future do
-      starts_at 5.days.from_now
-      ends_at 6.days.from_now
+      starts_at { 5.days.from_now }
+      ends_at { 6.days.from_now }
     end
   end
 end

spec/factories/ci/builds.rb

@@ -159,12 +159,12 @@
     end
 
     trait :erased do
-      erased_at Time.now
+      erased_at { Time.now }
       erased_by factory: :user
     end
 
     trait :queued do
-      queued_at Time.now
+      queued_at { Time.now }
       runner factory: :ci_runner
     end
 
@@ -194,7 +194,7 @@
     end
 
     trait :expired do
-      artifacts_expire_at 1.minute.ago
+      artifacts_expire_at { 1.minute.ago }
     end
 
     trait :with_commit do

spec/factories/ci/runners.rb

@@ -9,7 +9,7 @@
     runner_type :instance_type
 
     trait :online do
-      contacted_at Time.now
+      contacted_at { Time.now }
     end
 
     trait :instance do

spec/factories/clusters/applications/helm.rb

@@ -42,7 +42,7 @@
 
     trait :timeouted do
       installing
-      updated_at ClusterWaitForAppInstallationWorker::TIMEOUT.ago
+      updated_at { ClusterWaitForAppInstallationWorker::TIMEOUT.ago }
     end
 
     factory :clusters_applications_ingress, class: Clusters::Applications::Ingress do

spec/factories/clusters/platforms/kubernetes.rb

@@ -3,11 +3,10 @@
     cluster
     namespace nil
     api_url 'https://kubernetes.example.com'
-    token 'a' * 40
+    token { 'a' * 40 }
 
     trait :configured do
       api_url 'https://kubernetes.example.com'
-      token 'a' * 40
       username 'xxxxxx'
       password 'xxxxxx'
 

spec/factories/emails.rb

@@ -3,7 +3,7 @@
     user
     email { generate(:email_alias) }
 
-    trait(:confirmed) { confirmed_at Time.now }
+    trait(:confirmed) { confirmed_at { Time.now } }
     trait(:skip_validate) { to_create {|instance| instance.save(validate: false) } }
   end
 end

spec/factories/gpg_keys.rb

@@ -2,11 +2,11 @@
 
 FactoryBot.define do
   factory :gpg_key do
-    key GpgHelpers::User1.public_key
+    key { GpgHelpers::User1.public_key }
     user
 
     factory :gpg_key_with_subkeys do
-      key GpgHelpers::User1.public_key_with_extra_signing_key
+      key { GpgHelpers::User1.public_key_with_extra_signing_key }
     end
   end
 end

spec/factories/group_members.rb

@@ -9,7 +9,7 @@
     trait(:developer) { access_level GroupMember::DEVELOPER }
     trait(:maintainer) { access_level GroupMember::MAINTAINER }
     trait(:owner)     { access_level GroupMember::OWNER }
-    trait(:access_request) { requested_at Time.now }
+    trait(:access_request) { requested_at { Time.now } }
 
     trait(:invited) do
       user_id nil

spec/factories/merge_requests.rb

@@ -80,7 +80,7 @@
 
     trait :merge_when_pipeline_succeeds do
       merge_when_pipeline_succeeds true
-      merge_user author
+      merge_user { author }
     end
 
     trait :remove_source_branch do

spec/factories/notes.rb

@@ -90,7 +90,7 @@
       noteable nil
       noteable_type 'Commit'
       noteable_id nil
-      commit_id RepoHelpers.sample_commit.id
+      commit_id { RepoHelpers.sample_commit.id }
     end
 
     trait :legacy_diff_note do

spec/factories/oauth_access_grants.rb

@@ -3,7 +3,7 @@
     resource_owner_id { create(:user).id }
     application
     token { Doorkeeper::OAuth::Helpers::UniqueToken.generate }
-    expires_in 2.hours
+    expires_in { 2.hours }
 
     redirect_uri { application.redirect_uri }
     scopes { application.scopes }