Merge pull request #4967 from consul/dependabot/bundler/master/rubocop-performance-1.14.3

Bump rubocop-performance from 1.11.4 to 1.14.3

.rubocop.yml

@@ -224,6 +224,9 @@ Performance/EndWith:
 Performance/StartWith:
   Enabled: true
 
+Performance/StringIdentifierArgument:
+  Enabled: true
+
 Performance/Sum:
   Enabled: true
 

Gemfile

@@ -108,7 +108,7 @@ group :development do
   gem "pronto-rubocop", "~> 0.11.2", require: false
   gem "pronto-scss", "~> 0.11.0", require: false
   gem "rubocop", "~> 1.35.1", require: false
-  gem "rubocop-performance", "~> 1.11.4", require: false
+  gem "rubocop-performance", "~> 1.14.3", require: false
   gem "rubocop-rails", "~> 2.15.2", require: false
   gem "rubocop-rspec", "~> 2.4.0", require: false
   gem "rvm1-capistrano3", "~> 1.4.0", require: false

Gemfile.lock

@@ -551,7 +551,7 @@ GEM
       unicode-display_width (>= 1.4.0, < 3.0)
     rubocop-ast (1.21.0)
       parser (>= 3.1.1.0)
-    rubocop-performance (1.11.4)
+    rubocop-performance (1.14.3)
       rubocop (>= 1.7.0, < 2.0)
       rubocop-ast (>= 0.4.0)
     rubocop-rails (2.15.2)
@@ -770,7 +770,7 @@ DEPENDENCIES
   ros-apartment (~> 2.11.0)
   rspec-rails (~> 5.1.2)
   rubocop (~> 1.35.1)
-  rubocop-performance (~> 1.11.4)
+  rubocop-performance (~> 1.14.3)
   rubocop-rails (~> 2.15.2)
   rubocop-rspec (~> 2.4.0)
   rvm1-capistrano3 (~> 1.4.0)

app/controllers/concerns/flag_actions.rb

@@ -17,9 +17,9 @@ module FlagActions
 
     def flaggable
       if resource_model.to_s == "Budget::Investment"
-        instance_variable_get("@investment")
+        instance_variable_get(:@investment)
       elsif resource_model.to_s == "Legislation::Proposal"
-        instance_variable_get("@proposal")
+        instance_variable_get(:@proposal)
       else
         instance_variable_get("@#{resource_model.to_s.downcase}")
       end

app/controllers/concerns/polymorphic.rb

@@ -3,9 +3,9 @@ module Polymorphic
 
     def resource
       if resource_model.to_s == "Budget::Investment"
-        @resource ||= instance_variable_get("@investment")
+        @resource ||= instance_variable_get(:@investment)
       elsif resource_model.to_s == "Legislation::Proposal"
-        @resource ||= instance_variable_get("@proposal")
+        @resource ||= instance_variable_get(:@proposal)
       else
         @resource ||= instance_variable_get("@#{resource_name}")
       end

app/models/concerns/skip_validation.rb

@@ -13,7 +13,7 @@ module SkipValidation
 
       _validate_callbacks.each do |callback|
         if callback.raw_filter.is_a?(validator_class)
-          callback.raw_filter.instance_variable_set("@attributes", callback.raw_filter.attributes - [field])
+          callback.raw_filter.instance_variable_set(:@attributes, callback.raw_filter.attributes - [field])
         end
       end
     end

app/models/direct_upload.rb

@@ -22,7 +22,7 @@ class DirectUpload
       # Refactor
       @relation = if @resource.respond_to?(:images) &&
                      ((@attachment.present? && !@attachment.content_type.match(/pdf/)) || @cached_attachment.present?)
-                    @resource.images.send("build", relation_attributtes)
+                    @resource.images.send(:build, relation_attributtes)
                   elsif @resource.class.reflections[@resource_relation].macro == :has_one
                     @resource.send("build_#{resource_relation}", relation_attributtes)
                   else

lib/comment_tree.rb

@@ -19,7 +19,7 @@ class CommentTree
   end
 
   def base_comments
-    if @valuations && commentable.respond_to?("valuations")
+    if @valuations && commentable.respond_to?(:valuations)
       commentable.valuations
     else
       commentable.comments

spec/models/related_content_spec.rb

@@ -79,11 +79,11 @@ describe RelatedContent do
       create(:related_content, parent_relationable: parent_relationable, child_relationable: child_relationable, author: build(:user))
 
       2.times do
-        related_content.send("score_positive", build(:user))
+        related_content.send(:score_positive, build(:user))
       end
 
       6.times do
-        related_content.send("score_negative", build(:user))
+        related_content.send(:score_negative, build(:user))
       end
     end
 

spec/shared/system/relationable.rb

@@ -173,11 +173,11 @@ shared_examples "relationable" do |relationable_model_name|
     related_content = create(:related_content, parent_relationable: relationable, child_relationable: related1, author: build(:user))
 
     2.times do
-      related_content.send("score_positive", build(:user))
+      related_content.send(:score_positive, build(:user))
     end
 
     6.times do
-      related_content.send("score_negative", build(:user))
+      related_content.send(:score_negative, build(:user))
     end
 
     login_as(user)