rails_best_practices 1.9.1 → 1.10.0

data/spec/rails_best_practices/reviews/use_before_filter_review_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 module RailsBestPractices
   module Reviews
     describe UseBeforeFilterReview do
-      let(:runner) { Core::Runner.new(:reviews => UseBeforeFilterReview.new('customize_count' => 2)) }
+      let(:runner) { Core::Runner.new(reviews: UseBeforeFilterReview.new('customize_count' => 2)) }
 
       it "should use before_filter" do
         content = <<-EOF
@@ -29,7 +29,7 @@ module RailsBestPractices
         EOF
         runner.review('app/controllers/posts_controller.rb', content)
         runner.should have(1).errors
-        runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:3,7,11,16 - use before_filter for show,edit,update,destroy"
+        runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:2,6,10,15 - use before_filter for show,edit,update,destroy"
       end
 
       it "should not use before_filter when equal to customize count" do
@@ -51,7 +51,7 @@ module RailsBestPractices
       it "should not use before_filter" do
         content = <<-EOF
         class PostsController < ApplicationController
-          before_filter :find_post, :only => [:show, :edit, :update, :destroy]
+          before_filter :find_post, only: [:show, :edit, :update, :destroy]
 
           def update
             @post.update_attributes(params[:post])

data/spec/rails_best_practices/reviews/use_model_association_review_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 module RailsBestPractices
   module Reviews
     describe UseModelAssociationReview do
-      let(:runner) { Core::Runner.new(:reviews => UseModelAssociationReview.new) }
+      let(:runner) { Core::Runner.new(reviews: UseModelAssociationReview.new) }
 
       it "should use model association for instance variable" do
         content = <<-EOF
@@ -17,7 +17,7 @@ module RailsBestPractices
         EOF
         runner.review('app/controllers/posts_controller.rb', content)
         runner.should have(1).errors
-        runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:3 - use model association (for @post)"
+        runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:2 - use model association (for @post)"
       end
 
       it "should not use model association without association assign" do
@@ -45,7 +45,7 @@ module RailsBestPractices
         EOF
         runner.review('app/controllers/posts_controller.rb', content)
         runner.should have(1).errors
-        runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:3 - use model association (for post)"
+        runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:2 - use model association (for post)"
       end
 
       it "should not use model association" do

data/spec/rails_best_practices/reviews/use_multipart_alternative_as_content_type_of_email_review_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 module RailsBestPractices
   module Reviews
     describe UseMultipartAlternativeAsContentTypeOfEmailReview do
-      let(:runner) { Core::Runner.new(:reviews => UseMultipartAlternativeAsContentTypeOfEmailReview.new) }
+      let(:runner) { Core::Runner.new(reviews: UseMultipartAlternativeAsContentTypeOfEmailReview.new) }
 
       before(:each) { Core::Runner.stub!(:base_path).and_return(".") }
 
@@ -35,7 +35,7 @@ module RailsBestPractices
                 from          email.from
                 recipients    email.recipients
                 sent_on       Time.now
-                body          :email => email
+                body          email: email
               end
             end
             EOF
@@ -149,7 +149,7 @@ module RailsBestPractices
                 from          email.from
                 recipients    email.recipients
                 sent_on       Time.now
-                body          :email => email
+                body          email: email
               end
             end
             EOF

data/spec/rails_best_practices/reviews/use_observer_review_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 module RailsBestPractices
   module Reviews
     describe UseObserverReview do
-      let(:runner) { Core::Runner.new(:prepares => Prepares::MailerPrepare.new, :reviews => UseObserverReview.new) }
+      let(:runner) { Core::Runner.new(prepares: Prepares::MailerPrepare.new, reviews: UseObserverReview.new) }
 
       before :each do
         content =<<-EOF
@@ -29,7 +29,7 @@ module RailsBestPractices
           EOF
           runner.review('app/models/project.rb', content)
           runner.should have(1).errors
-          runner.errors[0].to_s.should == "app/models/project.rb:6 - use observer"
+          runner.errors[0].to_s.should == "app/models/project.rb:5 - use observer"
         end
 
         it "should not use observer without callback" do
@@ -65,7 +65,7 @@ module RailsBestPractices
           EOF
           runner.review('app/models/project.rb', content)
           runner.should have(1).errors
-          runner.errors[0].to_s.should == "app/models/project.rb:6 - use observer"
+          runner.errors[0].to_s.should == "app/models/project.rb:5 - use observer"
         end
 
         it "should not raise when initiate an object in callback" do
@@ -94,7 +94,7 @@ module RailsBestPractices
           EOF
           runner.review('app/models/project.rb', content)
           runner.should have(1).errors
-          runner.errors[0].to_s.should == "app/models/project.rb:6 - use observer"
+          runner.errors[0].to_s.should == "app/models/project.rb:5 - use observer"
         end
 
         it "should not use observer without callback" do
@@ -130,7 +130,7 @@ module RailsBestPractices
           EOF
           runner.review('app/models/project.rb', content)
           runner.should have(1).errors
-          runner.errors[0].to_s.should == "app/models/project.rb:6 - use observer"
+          runner.errors[0].to_s.should == "app/models/project.rb:5 - use observer"
         end
 
         it "should not raise when initiate an object in callback" do

data/spec/rails_best_practices/reviews/use_parenthesis_in_method_def_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+module RailsBestPractices
+  module Reviews
+    describe UseParenthesesInMethodDefReview do
+      let(:runner) { Core::Runner.new(reviews: UseParenthesesInMethodDefReview.new) }
+
+      it "should find missing parentheses" do
+        content = <<-EOF
+        class PostsController < ApplicationController
+          def edit foo, bar
+          end
+        end
+        EOF
+        runner.review('app/controllers/posts_controller.rb', content)
+        runner.should have(1).errors
+        runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:2 - use parentheses around parameters in method definitions"
+      end
+      it "should find parentheses with no error" do
+        content = <<-EOF
+        class PostsController < ApplicationController
+          def edit(foo, bar)
+          end
+        end
+        EOF
+        runner.review('app/controllers/posts_controller.rb', content)
+        runner.should have(0).errors
+      end
+      it "should not throw an error without parameters" do
+        content = <<-EOF
+        class PostsController < ApplicationController
+          def edit
+          end
+        end
+        EOF
+        runner.review('app/controllers/posts_controller.rb', content)
+        runner.should have(0).errors
+      end
+    end
+  end
+end

data/spec/rails_best_practices/reviews/use_query_attribute_review_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 module RailsBestPractices
   module Reviews
     describe UseQueryAttributeReview do
-      let(:runner) { Core::Runner.new(:prepares => [Prepares::ModelPrepare.new, Prepares::SchemaPrepare.new], :reviews => UseQueryAttributeReview.new) }
+      let(:runner) { Core::Runner.new(prepares: [Prepares::ModelPrepare.new, Prepares::SchemaPrepare.new], reviews: UseQueryAttributeReview.new) }
 
       before :each do
         content = <<-EOF
@@ -12,13 +12,13 @@ module RailsBestPractices
           belongs_to :location
           has_one :phone
 
-          belongs_to :category, :class_name => 'IssueCategory', :foreign_key => 'category_id'
+          belongs_to :category, class_name: 'IssueCategory', foreign_key: 'category_id'
         end
         EOF
         runner.prepare('app/models/user.rb', content)
 
         content = <<-EOF
-        ActiveRecord::Schema.define(:version => 20110216150853) do
+        ActiveRecord::Schema.define(version: 20110216150853) do
           create_table "users", force => true do |t|
             t.string :login
             t.integer :age

data/spec/rails_best_practices/reviews/use_say_with_time_in_migrations_review_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 module RailsBestPractices
   module Reviews
     describe UseSayWithTimeInMigrationsReview do
-      let(:runner) { Core::Runner.new(:reviews => UseSayWithTimeInMigrationsReview.new) }
+      let(:runner) { Core::Runner.new(reviews: UseSayWithTimeInMigrationsReview.new) }
 
       it "should use say with time in migrations" do
         content =<<-EOF
@@ -93,18 +93,18 @@ module RailsBestPractices
           end
 
           def self.up
-            add_column :people, :admin, :boolean, :default => false, :null => false
+            add_column :people, :admin, :boolean, default: false, null: false
             add_column :people, :deactivated, :boolean,
-                                :default => false, :null => false
+                                default: false, null: false
 
             key = Crypto::Key.from_file("\#{RAILS_ROOT}/rsa_key.pub")
-            person = Person.new(:email => "admin@example.com",
-                                :name => "admin",
-                                :crypted_password => key.encrypt("admin"),
-                                :description => "")
+            person = Person.new(email: "admin@example.com",
+                                name: "admin",
+                                crypted_password: key.encrypt("admin"),
+                                description: "")
             person.admin = true
             person.save!
-            Blog.create(:person_id => person.id)
+            Blog.create(person_id: person.id)
           end
 
           def self.down

data/spec/rails_best_practices/reviews/use_scope_access_review_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 module RailsBestPractices
   module Reviews
     describe UseScopeAccessReview do
-      let(:runner) { Core::Runner.new(:reviews => UseScopeAccessReview.new) }
+      let(:runner) { Core::Runner.new(reviews: UseScopeAccessReview.new) }
 
       context "if" do
         it "shoud use scope access" do
@@ -21,7 +21,7 @@ module RailsBestPractices
           EOF
           runner.review('app/controllers/posts_controller.rb', content)
           runner.should have(1).errors
-          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:6 - use scope access"
+          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:5 - use scope access"
         end
 
         it "shoud use scope access by comparing with id" do
@@ -39,7 +39,7 @@ module RailsBestPractices
           EOF
           runner.review('app/controllers/posts_controller.rb', content)
           runner.should have(1).errors
-          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:6 - use scope access"
+          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:5 - use scope access"
         end
 
         it "shoud use scope access with current_user ==" do
@@ -57,7 +57,7 @@ module RailsBestPractices
           EOF
           runner.review('app/controllers/posts_controller.rb', content)
           runner.should have(1).errors
-          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:6 - use scope access"
+          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:5 - use scope access"
         end
 
         it "shoud use scope access by current_user.id ==" do
@@ -75,7 +75,7 @@ module RailsBestPractices
           EOF
           runner.review('app/controllers/posts_controller.rb', content)
           runner.should have(1).errors
-          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:6 - use scope access"
+          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:5 - use scope access"
         end
       end
 
@@ -95,7 +95,7 @@ module RailsBestPractices
           EOF
           runner.review('app/controllers/posts_controller.rb', content)
           runner.should have(1).errors
-          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:6 - use scope access"
+          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:5 - use scope access"
         end
 
         it "shoud use scope access by comparing with id" do
@@ -113,7 +113,7 @@ module RailsBestPractices
           EOF
           runner.review('app/controllers/posts_controller.rb', content)
           runner.should have(1).errors
-          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:6 - use scope access"
+          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:5 - use scope access"
         end
 
         it "shoud use scope access with current_user ==" do
@@ -131,7 +131,7 @@ module RailsBestPractices
           EOF
           runner.review('app/controllers/posts_controller.rb', content)
           runner.should have(1).errors
-          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:6 - use scope access"
+          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:5 - use scope access"
         end
 
         it "shoud use scope access by current_user.id ==" do
@@ -149,7 +149,7 @@ module RailsBestPractices
           EOF
           runner.review('app/controllers/posts_controller.rb', content)
           runner.should have(1).errors
-          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:6 - use scope access"
+          runner.errors[0].to_s.should == "app/controllers/posts_controller.rb:5 - use scope access"
         end
 
         it "should no error in use_scope_access_review" do

data/spec/spec_helper.rb

@@ -8,7 +8,7 @@ RSpec.configure do |config|
   config.after do
     RailsBestPractices::Prepares.clear
   end
-  config.filter_run :focus => true
+  config.filter_run focus: true
   config.run_all_when_everything_filtered = true
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails_best_practices
 version: !ruby/object:Gem::Version
-  version: 1.9.1
+  version: 1.10.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-04-19 00:00:00.000000000 Z
+date: 2012-06-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sexp_processor
-  requirement: &70273211510520 !ruby/object:Gem::Requirement
+  requirement: &70312121266920 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70273211510520
+  version_requirements: *70312121266920
 - !ruby/object:Gem::Dependency
   name: progressbar
-  requirement: &70273211510000 !ruby/object:Gem::Requirement
+  requirement: &70312121266140 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70273211510000
+  version_requirements: *70312121266140
 - !ruby/object:Gem::Dependency
   name: colored
-  requirement: &70273211509320 !ruby/object:Gem::Requirement
+  requirement: &70312121264060 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70273211509320
+  version_requirements: *70312121264060
 - !ruby/object:Gem::Dependency
   name: erubis
-  requirement: &70273211508720 !ruby/object:Gem::Requirement
+  requirement: &70312121262840 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70273211508720
+  version_requirements: *70312121262840
 - !ruby/object:Gem::Dependency
   name: i18n
-  requirement: &70273211508220 !ruby/object:Gem::Requirement
+  requirement: &70312121261320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70273211508220
+  version_requirements: *70312121261320
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70273211507520 !ruby/object:Gem::Requirement
+  requirement: &70312121281720 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70273211507520
+  version_requirements: *70312121281720
 - !ruby/object:Gem::Dependency
   name: awesome_print
-  requirement: &70273211506140 !ruby/object:Gem::Requirement
+  requirement: &70312121280640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70273211506140
+  version_requirements: *70312121280640
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70273211504420 !ruby/object:Gem::Requirement
+  requirement: &70312121279160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,10 +98,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70273211504420
+  version_requirements: *70312121279160
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70273211503240 !ruby/object:Gem::Requirement
+  requirement: &70312121276940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -109,10 +109,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70273211503240
+  version_requirements: *70312121276940
 - !ruby/object:Gem::Dependency
   name: haml
-  requirement: &70273211502360 !ruby/object:Gem::Requirement
+  requirement: &70312121275740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -120,10 +120,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70273211502360
+  version_requirements: *70312121275740
 - !ruby/object:Gem::Dependency
   name: slim
-  requirement: &70273211501280 !ruby/object:Gem::Requirement
+  requirement: &70312121295400 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -131,10 +131,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70273211501280
+  version_requirements: *70312121295400
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &70273211500360 !ruby/object:Gem::Requirement
+  requirement: &70312121292680 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -142,7 +142,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70273211500360
+  version_requirements: *70312121292680
 description: a code metric tool for rails codes, written in Ruby.
 email:
 - flyerhzm@gmail.com
@@ -154,9 +154,7 @@ files:
 - .gemtest
 - .gitignore
 - .rspec
-- .rspec.example
 - .rvmrc
-- .rvmrc.example
 - .travis.yml
 - Gemfile
 - Gemfile.lock
@@ -191,6 +189,7 @@ files:
 - lib/rails_best_practices/core_ext/erubis.rb
 - lib/rails_best_practices/core_ext/sexp.rb
 - lib/rails_best_practices/lexicals.rb
+- lib/rails_best_practices/lexicals/long_line_check.rb
 - lib/rails_best_practices/lexicals/remove_tab_check.rb
 - lib/rails_best_practices/lexicals/remove_trailing_whitespace_check.rb
 - lib/rails_best_practices/prepares.rb
@@ -205,6 +204,7 @@ files:
 - lib/rails_best_practices/reviews/add_model_virtual_attribute_review.rb
 - lib/rails_best_practices/reviews/always_add_db_index_review.rb
 - lib/rails_best_practices/reviews/dry_bundler_in_capistrano_review.rb
+- lib/rails_best_practices/reviews/hash_syntax_review.rb
 - lib/rails_best_practices/reviews/isolate_seed_data_review.rb
 - lib/rails_best_practices/reviews/keep_finders_on_their_own_model_review.rb
 - lib/rails_best_practices/reviews/law_of_demeter_review.rb
@@ -232,6 +232,7 @@ files:
 - lib/rails_best_practices/reviews/use_model_association_review.rb
 - lib/rails_best_practices/reviews/use_multipart_alternative_as_content_type_of_email_review.rb
 - lib/rails_best_practices/reviews/use_observer_review.rb
+- lib/rails_best_practices/reviews/use_parentheses_in_method_def_review.rb
 - lib/rails_best_practices/reviews/use_query_attribute_review.rb
 - lib/rails_best_practices/reviews/use_say_with_time_in_migrations_review.rb
 - lib/rails_best_practices/reviews/use_scope_access_review.rb
@@ -260,6 +261,7 @@ files:
 - spec/rails_best_practices/core_ext/enumerable_spec.rb
 - spec/rails_best_practices/core_ext/erubis_spec.rb
 - spec/rails_best_practices/core_ext/sexp_spec.rb
+- spec/rails_best_practices/lexicals/long_line_check_spec.rb
 - spec/rails_best_practices/lexicals/remove_tab_check_spec.rb
 - spec/rails_best_practices/lexicals/remove_trailing_whitespace_check_spec.rb
 - spec/rails_best_practices/prepares/config_prepare_spec.rb
@@ -272,6 +274,7 @@ files:
 - spec/rails_best_practices/reviews/add_model_virtual_attribute_review_spec.rb
 - spec/rails_best_practices/reviews/always_add_db_index_review_spec.rb
 - spec/rails_best_practices/reviews/dry_bundler_in_capistrano_review_spec.rb
+- spec/rails_best_practices/reviews/hash_syntax_review_spec.rb
 - spec/rails_best_practices/reviews/isolate_seed_data_review_spec.rb
 - spec/rails_best_practices/reviews/keep_finders_on_their_own_model_review_spec.rb
 - spec/rails_best_practices/reviews/law_of_demeter_review_spec.rb
@@ -298,6 +301,7 @@ files:
 - spec/rails_best_practices/reviews/use_model_association_review_spec.rb
 - spec/rails_best_practices/reviews/use_multipart_alternative_as_content_type_of_email_review_spec.rb
 - spec/rails_best_practices/reviews/use_observer_review_spec.rb
+- spec/rails_best_practices/reviews/use_parenthesis_in_method_def_spec.rb
 - spec/rails_best_practices/reviews/use_query_attribute_review_spec.rb
 - spec/rails_best_practices/reviews/use_say_with_time_in_migrations_review_spec.rb
 - spec/rails_best_practices/reviews/use_scope_access_review_spec.rb
@@ -321,7 +325,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1930159151247489089
+      hash: -3377092891025064967
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -356,6 +360,7 @@ test_files:
 - spec/rails_best_practices/core_ext/enumerable_spec.rb
 - spec/rails_best_practices/core_ext/erubis_spec.rb
 - spec/rails_best_practices/core_ext/sexp_spec.rb
+- spec/rails_best_practices/lexicals/long_line_check_spec.rb
 - spec/rails_best_practices/lexicals/remove_tab_check_spec.rb
 - spec/rails_best_practices/lexicals/remove_trailing_whitespace_check_spec.rb
 - spec/rails_best_practices/prepares/config_prepare_spec.rb
@@ -368,6 +373,7 @@ test_files:
 - spec/rails_best_practices/reviews/add_model_virtual_attribute_review_spec.rb
 - spec/rails_best_practices/reviews/always_add_db_index_review_spec.rb
 - spec/rails_best_practices/reviews/dry_bundler_in_capistrano_review_spec.rb
+- spec/rails_best_practices/reviews/hash_syntax_review_spec.rb
 - spec/rails_best_practices/reviews/isolate_seed_data_review_spec.rb
 - spec/rails_best_practices/reviews/keep_finders_on_their_own_model_review_spec.rb
 - spec/rails_best_practices/reviews/law_of_demeter_review_spec.rb
@@ -394,6 +400,7 @@ test_files:
 - spec/rails_best_practices/reviews/use_model_association_review_spec.rb
 - spec/rails_best_practices/reviews/use_multipart_alternative_as_content_type_of_email_review_spec.rb
 - spec/rails_best_practices/reviews/use_observer_review_spec.rb
+- spec/rails_best_practices/reviews/use_parenthesis_in_method_def_spec.rb
 - spec/rails_best_practices/reviews/use_query_attribute_review_spec.rb
 - spec/rails_best_practices/reviews/use_say_with_time_in_migrations_review_spec.rb
 - spec/rails_best_practices/reviews/use_scope_access_review_spec.rb