rails_best_practices 1.20.0 → 1.20.1

data/lib/rails_best_practices/reviews/simplify_render_in_views_review.rb

@@ -23,9 +23,9 @@ module RailsBestPractices
       # then it should be replaced by simplified syntax.
       add_callback :start_command do |node|
         if node.message.to_s == 'render'
-          hash_node =  node.arguments.all.first
-          if hash_node && hash_node.sexp_type == :bare_assoc_hash &&
-             include_partial?(hash_node) && valid_hash?(hash_node)
+          hash_node = node.arguments.all.first
+          if hash_node && hash_node.sexp_type == :bare_assoc_hash && include_partial?(hash_node) &&
+               valid_hash?(hash_node)
             add_error 'simplify render in views'
           end
         end

data/lib/rails_best_practices/reviews/use_before_filter_review.rb

@@ -12,6 +12,7 @@ module RailsBestPractices
     # Review process:
     #   check all first code line_number in method definitions (actions),
     #   if they are duplicated, then they should be moved to before_filter.
+
     class UseBeforeFilterReview < Review
       interesting_nodes :class
       interesting_files CONTROLLER_FILES
@@ -48,7 +49,7 @@ module RailsBestPractices
 
       private
 
-        # check method define node, and remember the first sentence.
+      # check method define node, and remember the first sentence.
       def remember_first_sentence(node)
         first_sentence = node.body.statements.first
         return unless first_sentence

data/lib/rails_best_practices/reviews/use_model_association_review.rb

@@ -43,8 +43,8 @@ module RailsBestPractices
 
       private
 
-        # check an attribute assignment node, if its message is xxx_id,
-        # then remember the receiver of the attribute assignment in @assignments.
+      # check an attribute assignment node, if its message is xxx_id,
+      # then remember the receiver of the attribute assignment in @assignments.
       def attribute_assignment(node)
         if node.left_value.message.is_a?(Sexp) && node.left_value.message.to_s =~ /_id$/
           receiver = node.left_value.receiver.to_s
@@ -52,9 +52,9 @@ module RailsBestPractices
         end
       end
 
-        # check a call node with message "save" or "save!",
-        # if the receiver of call node exists in @assignments,
-        # then the attribute assignment should be replaced by using model association.
+      # check a call node with message "save" or "save!",
+      # if the receiver of call node exists in @assignments,
+      # then the attribute assignment should be replaced by using model association.
       def call_assignment(node)
         if ['save', 'save!'].include? node.message.to_s
           receiver = node.receiver.to_s

data/lib/rails_best_practices/reviews/use_multipart_alternative_as_content_type_of_email_review.rb

@@ -11,6 +11,7 @@ module RailsBestPractices
     # Review process:
     #   check class node to remember the class name,
     #   and check the method definition nodes to see if the corresponding mailer views exist or not.
+
     class UseMultipartAlternativeAsContentTypeOfEmailReview < Review
       interesting_nodes :class, :def
       interesting_files MAILER_FILES
@@ -31,14 +32,14 @@ module RailsBestPractices
 
       private
 
-        # check if rails's syntax mailer views are canonical.
-        #
-        # @param [String] name method name in action_mailer
+      # check if rails's syntax mailer views are canonical.
+      #
+      # @param [String] name method name in action_mailer
       def rails_canonical_mailer_views?(name); end
 
-        # check if rails3's syntax mailer views are canonical.
-        #
-        # @param [String] name method name in action_mailer
+      # check if rails3's syntax mailer views are canonical.
+      #
+      # @param [String] name method name in action_mailer
       def rails3_canonical_mailer_views?(name)
         return true if mailer_files(name).empty?
         return true if mailer_files(name).none? { |filename| filename.index 'html' }
@@ -47,12 +48,12 @@ module RailsBestPractices
           mailer_files(name).any? { |filename| filename.index 'text' }
       end
 
-        # all mail view files for a method name.
+      # all mail view files for a method name.
       def mailer_files(name)
         Dir.entries(mailer_directory) { |filename| filename.index name.to_s }
       end
 
-        # the view directory of mailer.
+      # the view directory of mailer.
       def mailer_directory
         File.join(Core::Runner.base_path, "app/views/#{@klazz_name.to_s.underscore}")
       end

data/lib/rails_best_practices/reviews/use_observer_review.rb

@@ -49,8 +49,8 @@ module RailsBestPractices
 
       private
 
-        # check a command node, if it is a callback definition, such as after_create, before_create,
-        # then save the callback methods in @callbacks
+      # check a command node, if it is a callback definition, such as after_create, before_create,
+      # then save the callback methods in @callbacks
       def remember_callback(node)
         if node.message.to_s =~ /^after_|^before_/
           node.arguments.all.each do |argument|
@@ -60,21 +60,21 @@ module RailsBestPractices
         end
       end
 
-        # check a defn node to see if the method name exists in the @callbacks.
+      # check a defn node to see if the method name exists in the @callbacks.
       def callback_method?(node)
         @callbacks.find { |callback| callback == node.method_name.to_s }
       end
 
-        # check a def node to see if it contains a actionmailer deliver call.
-        #
-        # if the message of call node is deliver,
-        # and the receiver of the call node is with receiver node who exists in @callbacks,
-        # then the call node is actionmailer deliver call.
+      # check a def node to see if it contains a actionmailer deliver call.
+      #
+      # if the message of call node is deliver,
+      # and the receiver of the call node is with receiver node who exists in @callbacks,
+      # then the call node is actionmailer deliver call.
       def deliver_mailer?(node)
         node.grep_nodes(sexp_type: :call) do |child_node|
           if child_node.message.to_s == 'deliver'
             if child_node.receiver.sexp_type == :method_add_arg &&
-               mailers.include?(child_node.receiver[1].receiver.to_s)
+                 mailers.include?(child_node.receiver[1].receiver.to_s)
               return true
             end
           end

data/lib/rails_best_practices/reviews/use_query_attribute_review.rb

@@ -34,11 +34,12 @@ module RailsBestPractices
       #
       # then the call node can use query attribute instead.
       add_callback :start_if, :start_unless, :start_elsif, :start_ifop, :start_if_mod, :start_unless_mod do |node|
-        all_conditions = if node.conditional_statement == node.conditional_statement.all_conditions
-                           [node.conditional_statement]
-                         else
-                           node.conditional_statement.all_conditions
-        end
+        all_conditions =
+          if node.conditional_statement == node.conditional_statement.all_conditions
+            [node.conditional_statement]
+          else
+            node.conditional_statement.all_conditions
+          end
         all_conditions.each do |condition_node|
           next unless query_attribute_node = query_attribute_node(condition_node)
 
@@ -51,13 +52,13 @@ module RailsBestPractices
 
       private
 
-        # recursively check conditional statement nodes to see if there is a call node that may be
-        # possible query attribute.
+      # recursively check conditional statement nodes to see if there is a call node that may be
+      # possible query attribute.
       def query_attribute_node(conditional_statement_node)
         case conditional_statement_node.sexp_type
         when :and, :or
-          node = query_attribute_node(conditional_statement_node[1]) ||
-                 query_attribute_node(conditional_statement_node[2])
+          node =
+            query_attribute_node(conditional_statement_node[1]) || query_attribute_node(conditional_statement_node[2])
           node.file = conditional_statement_code.file
           return node
         when :not
@@ -71,17 +72,17 @@ module RailsBestPractices
         nil
       end
 
-        # check if the node may use query attribute instead.
-        #
-        # if the node contains two method calls, e.g. @user.login.nil?
-        #
-        # for the first call, the receiver should be one of the class names and
-        # the message should be one of the attribute name.
-        #
-        # for the second call, the message should be one of nil?, blank? or present? or
-        # it is compared with an empty string.
-        #
-        # the node that may use query attribute.
+      # check if the node may use query attribute instead.
+      #
+      # if the node contains two method calls, e.g. @user.login.nil?
+      #
+      # for the first call, the receiver should be one of the class names and
+      # the message should be one of the attribute name.
+      #
+      # for the second call, the message should be one of nil?, blank? or present? or
+      # it is compared with an empty string.
+      #
+      # the node that may use query attribute.
       def possible_query_attribute?(node)
         return false unless node.receiver.sexp_type == :call
 
@@ -92,7 +93,7 @@ module RailsBestPractices
           (QUERY_METHODS.include?(node.message.to_s) || compare_with_empty_string?(node))
       end
 
-        # check if the receiver is one of the models.
+      # check if the receiver is one of the models.
       def is_model?(variable_node)
         return false if variable_node.const?
 
@@ -100,18 +101,17 @@ module RailsBestPractices
         models.include?(class_name)
       end
 
-        # check if the receiver and message is one of the model's attribute.
-        # the receiver should match one of the class model name, and the message should match one of attribute name.
+      # check if the receiver and message is one of the model's attribute.
+      # the receiver should match one of the class model name, and the message should match one of attribute name.
       def model_attribute?(variable_node, message)
         class_name = variable_node.to_s.sub(/^@/, '').classify
         attribute_type = model_attributes.get_attribute_type(class_name, message)
         attribute_type && !%w[integer float].include?(attribute_type)
       end
 
-        # check if the node is with node type :binary, node message :== and node argument is empty string.
+      # check if the node is with node type :binary, node message :== and node argument is empty string.
       def compare_with_empty_string?(node)
-        node.sexp_type == :binary &&
-          ['==', '!='].include?(node.message.to_s) &&
+        node.sexp_type == :binary && ['==', '!='].include?(node.message.to_s) &&
           s(:string_literal, s(:string_content)) == node.argument
       end
     end

data/lib/rails_best_practices/reviews/use_say_with_time_in_migrations_review.rb

@@ -31,13 +31,14 @@ module RailsBestPractices
         node.body.statements.each do |child_node|
           next if child_node.grep_nodes_count(sexp_type: %i[fcall command], message: WITH_SAY_METHODS) > 0
 
-          receiver_node = if child_node.sexp_type == :method_add_block
-                            child_node[1]
-                          elsif child_node.sexp_type == :method_add_arg
-                            child_node[1]
-                          else
-                            child_node
-                         end
+          receiver_node =
+            if child_node.sexp_type == :method_add_block
+              child_node[1]
+            elsif child_node.sexp_type == :method_add_arg
+              child_node[1]
+            else
+              child_node
+            end
           if receiver_node.sexp_type == :call
             add_error('use say with time in migrations', node.file, child_node.line_number)
           end

data/lib/rails_best_practices/reviews/use_scope_access_review.rb

@@ -30,25 +30,27 @@ module RailsBestPractices
 
       private
 
-        # check a if node to see
-        #
-        # if the conditional statement is compared with current_user or current_user.id,
-        # and there is a redirect_to method call in the block body,
-        # then it should be replaced by using scope access.
+      # check a if node to see
+      #
+      # if the conditional statement is compared with current_user or current_user.id,
+      # and there is a redirect_to method call in the block body,
+      # then it should be replaced by using scope access.
       def current_user_redirect?(node)
-        all_conditions = if node.conditional_statement == node.conditional_statement.all_conditions
-                           [node.conditional_statement]
-                         else
-                           node.conditional_statement.all_conditions
-        end
-        results = all_conditions.map do |condition_node|
-          ['==', '!='].include?(condition_node.message.to_s) &&
-            (current_user?(condition_node.argument) || current_user?(condition_node.receiver))
-        end
+        all_conditions =
+          if node.conditional_statement == node.conditional_statement.all_conditions
+            [node.conditional_statement]
+          else
+            node.conditional_statement.all_conditions
+          end
+        results =
+          all_conditions.map do |condition_node|
+            ['==', '!='].include?(condition_node.message.to_s) &&
+              (current_user?(condition_node.argument) || current_user?(condition_node.receiver))
+          end
         results.any? { |result| result == true } && node.body.grep_node(message: 'redirect_to')
       end
 
-        # check a call node to see if it uses current_user, or current_user.id.
+      # check a call node to see if it uses current_user, or current_user.id.
       def current_user?(node)
         node.to_s == 'current_user' || (node.receiver.to_s == 'current_user' && node.message.to_s == 'id')
       end

data/lib/rails_best_practices/reviews/use_turbo_sprockets_rails3_review.rb

@@ -18,7 +18,8 @@ module RailsBestPractices
       # check command node to see if load 'deploy/assets'
       add_callback :start_command do |node|
         if Prepares.gems.gem_version('rails').to_i == 3
-          if !Prepares.gems.has_gem?('turbo-sprockets-rails3') && node.message.to_s == 'load' && node.arguments.to_s == 'deploy/assets'
+          if !Prepares.gems.has_gem?('turbo-sprockets-rails3') && node.message.to_s == 'load' &&
+               node.arguments.to_s == 'deploy/assets'
             add_error 'speed up assets precompile with turbo-sprockets-rails3'
           end
         end

data/lib/rails_best_practices/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RailsBestPractices
-  VERSION = '1.20.0'
+  VERSION = '1.20.1'
 end

data/rails_best_practices.gemspec

@@ -2,42 +2,41 @@
 
 require File.expand_path('lib/rails_best_practices/version', __dir__)
 
-Gem::Specification.new do |s|
-  s.name        = 'rails_best_practices'
-  s.version     = RailsBestPractices::VERSION
-  s.platform    = Gem::Platform::RUBY
-  s.authors     = ['Richard Huang']
-  s.email       = ['flyerhzm@gmail.com']
-  s.homepage    = 'http://rails-bestpractices.com'
-  s.summary     = 'a code metric tool for rails codes.'
-  s.description = 'a code metric tool for rails codes, written in Ruby.'
-  s.license     = 'MIT'
-
-  s.required_ruby_version = '>= 1.9.0'
-  s.required_rubygems_version = '>= 1.3.6'
-
-  s.add_dependency('activesupport')
-  # TODO: add a dependency for Rails >= 3 ?
-  s.add_dependency('code_analyzer', '>= 0.5.1')
-  s.add_dependency('erubis')
-  s.add_dependency('i18n')
-  s.add_dependency('json')
-  s.add_dependency('require_all', '~> 3.0')
-  s.add_dependency('ruby-progressbar')
-
-  s.add_development_dependency('awesome_print')
-  s.add_development_dependency('bundler')
-  s.add_development_dependency('haml')
-  s.add_development_dependency('rake')
-  s.add_development_dependency('rspec')
-  s.add_development_dependency('slim')
-
-  s.files         = `git ls-files`.split("\n")
-  s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  s.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
-  s.require_paths = %w[lib assets]
-
-  s.post_install_message = <<~POST_INSTALL_MESSAGE
+Gem::Specification.new do |spec|
+  spec.name        = 'rails_best_practices'
+  spec.version     = RailsBestPractices::VERSION
+  spec.platform    = Gem::Platform::RUBY
+  spec.authors     = ['Richard Huang']
+  spec.email       = ['flyerhzm@gmail.com']
+  spec.homepage    = 'http://rails-bestpractices.com'
+  spec.summary     = 'a code metric tool for rails codes.'
+  spec.description = 'a code metric tool for rails codes, written in Ruby.'
+  spec.license     = 'MIT'
+
+  spec.required_ruby_version = '>= 1.9.0'
+  spec.required_rubygems_version = '>= 1.3.6'
+
+  spec.add_dependency('activesupport')
+  spec.add_dependency('code_analyzer', '>= 0.5.2')
+  spec.add_dependency('erubis')
+  spec.add_dependency('i18n')
+  spec.add_dependency('json')
+  spec.add_dependency('require_all', '~> 3.0')
+  spec.add_dependency('ruby-progressbar')
+
+  spec.add_development_dependency('awesome_print')
+  spec.add_development_dependency('bundler')
+  spec.add_development_dependency('haml')
+  spec.add_development_dependency('rake')
+  spec.add_development_dependency('rspec')
+  spec.add_development_dependency('slim')
+
+  spec.files         = `git ls-files`.split($/)
+  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = %w[lib assets]
+
+  spec.post_install_message = <<~POST_INSTALL_MESSAGE
     #{'*' * 80}
 
       rails_best_practices is a code metric tool to check the quality of rails codes.

data/spec/fixtures/lib/rails_best_practices/plugins/reviews/not_use_rails_root_review.rb

@@ -5,8 +5,7 @@ require 'rails_best_practices/reviews/review'
 module RailsBestPractices
   module Plugins
     module Reviews
-      class NotUseRailsRootReview < RailsBestPractices::Reviews::Review
-      end
+      class NotUseRailsRootReview < RailsBestPractices::Reviews::Review; end
     end
   end
 end

data/spec/rails_best_practices/analyzer_spec.rb

@@ -5,39 +5,70 @@ require 'tmpdir'
 
 module RailsBestPractices
   describe Analyzer do
-    subject { Analyzer.new('.') }
+    subject { described_class.new('.') }
 
     describe '::new' do
-      it 'should expand a relative path to an absolute' do
+      it 'expands a relative path to an absolute' do
         expect(subject.path).to eq File.expand_path('.')
       end
     end
 
     describe 'expand_dirs_to_files' do
-      it 'should expand all files in spec directory' do
+      it 'expands all files in spec directory' do
         dir = File.dirname(__FILE__)
         expect(subject.expand_dirs_to_files(dir)).to be_include(dir + '/analyzer_spec.rb')
       end
     end
 
     describe 'file_sort' do
-      it 'should get models first, mailers, helpers and then others' do
-        files = ['app/controllers/users_controller.rb', 'app/mailers/user_mailer.rb', 'app/helpers/users_helper.rb', 'app/models/user.rb', 'app/views/users/index.html.haml', 'app/views/users/show.html.slim', 'lib/user.rb']
-        expect(subject.file_sort(files)).to eq(['app/models/user.rb', 'app/mailers/user_mailer.rb', 'app/helpers/users_helper.rb', 'app/controllers/users_controller.rb', 'app/views/users/index.html.haml', 'app/views/users/show.html.slim', 'lib/user.rb'])
+      it 'gets models first, mailers, helpers and then others' do
+        files = [
+          'app/controllers/users_controller.rb',
+          'app/mailers/user_mailer.rb',
+          'app/helpers/users_helper.rb',
+          'app/models/user.rb',
+          'app/views/users/index.html.haml',
+          'app/views/users/show.html.slim',
+          'lib/user.rb'
+        ]
+        expect(subject.file_sort(files)).to eq(
+          [
+            'app/models/user.rb',
+            'app/mailers/user_mailer.rb',
+            'app/helpers/users_helper.rb',
+            'app/controllers/users_controller.rb',
+            'app/views/users/index.html.haml',
+            'app/views/users/show.html.slim',
+            'lib/user.rb'
+          ]
+        )
       end
     end
 
     describe 'file_ignore' do
       before do
-        @all = ['app/controllers/users_controller.rb', 'app/mailers/user_mailer.rb', 'app/models/user.rb', 'app/views/users/index.html.haml', 'app/views/users/show.html.slim', 'lib/user.rb']
-        @filtered = ['app/controllers/users_controller.rb', 'app/mailers/user_mailer.rb', 'app/models/user.rb', 'app/views/users/index.html.haml', 'app/views/users/show.html.slim']
-      end
-
-      it 'should ignore lib' do
+        @all = [
+          'app/controllers/users_controller.rb',
+          'app/mailers/user_mailer.rb',
+          'app/models/user.rb',
+          'app/views/users/index.html.haml',
+          'app/views/users/show.html.slim',
+          'lib/user.rb'
+        ]
+        @filtered = [
+          'app/controllers/users_controller.rb',
+          'app/mailers/user_mailer.rb',
+          'app/models/user.rb',
+          'app/views/users/index.html.haml',
+          'app/views/users/show.html.slim'
+        ]
+      end
+
+      it 'ignores lib' do
         expect(subject.file_ignore(@all, 'lib/')).to eq(@filtered)
       end
 
-      it 'should ignore regexp patterns' do
+      it 'ignores regexp patterns' do
         expect(subject.file_ignore(@all, /lib/)).to eq(@filtered)
       end
     end
@@ -88,7 +119,7 @@ module RailsBestPractices
     end
 
     describe 'output_terminal_errors' do
-      it 'should output errors in terminal' do
+      it 'outputs errors in terminal' do
         check1 = Reviews::LawOfDemeterReview.new
         check2 = Reviews::UseQueryAttributeReview.new
         runner = Core::Runner.new(reviews: [check1, check2])
@@ -102,12 +133,14 @@ module RailsBestPractices
         subject.output_terminal_errors
         result = $stdout.string
         $stdout = $origin_stdout
-        expect(result).to eq([
-          "\e[31mapp/models/user.rb:10 - law of demeter\e[0m",
-          "\e[31mapp/models/post.rb:100 - use query attribute\e[0m",
-          "\e[32m\nPlease go to https://rails-bestpractices.com to see more useful Rails Best Practices.\e[0m",
-          "\e[31m\nFound 2 warnings.\e[0m"
-        ].join("\n") + "\n")
+        expect(result).to eq(
+          [
+            "\e[31mapp/models/user.rb:10 - law of demeter\e[0m",
+            "\e[31mapp/models/post.rb:100 - use query attribute\e[0m",
+            "\e[32m\nPlease go to https://rails-bestpractices.com to see more useful Rails Best Practices.\e[0m",
+            "\e[31m\nFound 2 warnings.\e[0m"
+          ].join("\n") + "\n"
+        )
       end
     end
 
@@ -115,9 +148,7 @@ module RailsBestPractices
       let(:output_file) { 'rails_best_practices_output.json' }
 
       subject do
-        described_class.new('.',
-                            'format' => 'json',
-                            'output-file' => output_file)
+        described_class.new('.', 'format' => 'json', 'output-file' => output_file)
       end
 
       let(:check1) { Reviews::LawOfDemeterReview.new }
@@ -142,54 +173,54 @@ module RailsBestPractices
     end
 
     describe 'parse_files' do
-      it 'should not filter out all files when the path contains "vendor"' do
-        Dir.mktmpdir { |random_dir|
+      it 'does not filter out all files when the path contains "vendor"' do
+        Dir.mktmpdir do |random_dir|
           Dir.mkdir(File.join(random_dir, 'vendor'))
           Dir.mkdir(File.join(random_dir, 'vendor', 'my_project'))
           File.open(File.join(random_dir, 'vendor', 'my_project', 'my_file.rb'), 'w') { |file| file << 'woot' }
-          analyzer = Analyzer.new(File.join(random_dir, 'vendor', 'my_project'))
+          analyzer = described_class.new(File.join(random_dir, 'vendor', 'my_project'))
           expect(analyzer.parse_files).to be_include File.join(random_dir, 'vendor', 'my_project', 'my_file.rb')
-        }
+        end
       end
 
-      it 'should not filter out all files when the path contains "spec"' do
-        Dir.mktmpdir { |random_dir|
+      it 'does not filter out all files when the path contains "spec"' do
+        Dir.mktmpdir do |random_dir|
           Dir.mkdir(File.join(random_dir, 'spec'))
           Dir.mkdir(File.join(random_dir, 'spec', 'my_project'))
           File.open(File.join(random_dir, 'spec', 'my_project', 'my_file.rb'), 'w') { |file| file << 'woot' }
-          analyzer = Analyzer.new(File.join(random_dir, 'spec', 'my_project'))
+          analyzer = described_class.new(File.join(random_dir, 'spec', 'my_project'))
           expect(analyzer.parse_files).to be_include File.join(random_dir, 'spec', 'my_project', 'my_file.rb')
-        }
+        end
       end
 
-      it 'should not filter out all files when the path contains "test"' do
-        Dir.mktmpdir { |random_dir|
+      it 'does not filter out all files when the path contains "test"' do
+        Dir.mktmpdir do |random_dir|
           Dir.mkdir(File.join(random_dir, 'test'))
           Dir.mkdir(File.join(random_dir, 'test', 'my_project'))
           File.open(File.join(random_dir, 'test', 'my_project', 'my_file.rb'), 'w') { |file| file << 'woot' }
-          analyzer = Analyzer.new(File.join(random_dir, 'test', 'my_project'))
+          analyzer = described_class.new(File.join(random_dir, 'test', 'my_project'))
           expect(analyzer.parse_files).to be_include File.join(random_dir, 'test', 'my_project', 'my_file.rb')
-        }
+        end
       end
 
-      it 'should not filter out all files when the path contains "features"' do
-        Dir.mktmpdir { |random_dir|
+      it 'does not filter out all files when the path contains "features"' do
+        Dir.mktmpdir do |random_dir|
           Dir.mkdir(File.join(random_dir, 'test'))
           Dir.mkdir(File.join(random_dir, 'test', 'my_project'))
           File.open(File.join(random_dir, 'test', 'my_project', 'my_file.rb'), 'w') { |file| file << 'woot' }
-          analyzer = Analyzer.new(File.join(random_dir, 'test', 'my_project'))
+          analyzer = described_class.new(File.join(random_dir, 'test', 'my_project'))
           expect(analyzer.parse_files).to be_include File.join(random_dir, 'test', 'my_project', 'my_file.rb')
-        }
+        end
       end
 
-      it 'should not filter out all files when the path contains "tmp"' do
-        Dir.mktmpdir { |random_dir|
+      it 'does not filter out all files when the path contains "tmp"' do
+        Dir.mktmpdir do |random_dir|
           Dir.mkdir(File.join(random_dir, 'tmp'))
           Dir.mkdir(File.join(random_dir, 'tmp', 'my_project'))
           File.open(File.join(random_dir, 'tmp', 'my_project', 'my_file.rb'), 'w') { |file| file << 'woot' }
-          analyzer = Analyzer.new(File.join(random_dir, 'tmp', 'my_project'))
+          analyzer = described_class.new(File.join(random_dir, 'tmp', 'my_project'))
           expect(analyzer.parse_files).to be_include File.join(random_dir, 'tmp', 'my_project', 'my_file.rb')
-        }
+        end
       end
     end
   end