rubocop 0.22.0 → 0.23.0

data/spec/rubocop/cop/style/word_array_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-describe Rubocop::Cop::Style::WordArray, :config do
+describe RuboCop::Cop::Style::WordArray, :config do
   subject(:cop) { described_class.new(config) }
   let(:cop_config) { { 'MinSize' => 0 } }
 

data/spec/rubocop/cop/team_spec.rb

@@ -2,10 +2,10 @@
 
 require 'spec_helper'
 
-describe Rubocop::Cop::Team do
+describe RuboCop::Cop::Team do
   subject(:team) { described_class.new(cop_classes, config, options) }
-  let(:cop_classes) { Rubocop::Cop::Cop.non_rails }
-  let(:config) { Rubocop::ConfigLoader.default_configuration }
+  let(:cop_classes) { RuboCop::Cop::Cop.non_rails }
+  let(:config) { RuboCop::ConfigLoader.default_configuration }
   let(:options) { nil }
 
   describe '#autocorrect?' do
@@ -41,7 +41,7 @@ describe Rubocop::Cop::Team do
 
     let(:file_path) { '/tmp/example.rb' }
     let(:offenses) do
-      team.inspect_file(Rubocop::SourceParser.parse_file(file_path))
+      team.inspect_file(RuboCop::SourceParser.parse_file(file_path))
     end
 
     before do
@@ -53,7 +53,7 @@ describe Rubocop::Cop::Team do
 
     it 'returns offenses' do
       expect(offenses).not_to be_empty
-      expect(offenses.all? { |o| o.is_a?(Rubocop::Cop::Offense) }).to be_true
+      expect(offenses.all? { |o| o.is_a?(RuboCop::Cop::Offense) }).to be_true
     end
 
     context 'when Parser reports non-fatal warning for the file' do
@@ -68,11 +68,11 @@ describe Rubocop::Cop::Team do
       let(:cop_names) { offenses.map(&:cop_name) }
 
       it 'returns Parser warning offenses' do
-        expect(cop_names).to include('AmbiguousOperator')
+        expect(cop_names).to include('Lint/AmbiguousOperator')
       end
 
       it 'returns offenses from cops' do
-        expect(cop_names).to include('LineLength')
+        expect(cop_names).to include('Style/LineLength')
       end
     end
 
@@ -87,7 +87,7 @@ describe Rubocop::Cop::Team do
       end
 
       it 'does autocorrection' do
-        team.inspect_file(Rubocop::SourceParser.parse_file(file_path))
+        team.inspect_file(RuboCop::SourceParser.parse_file(file_path))
         corrected_source = File.read(file_path)
         expect(corrected_source).to eq([
           '# encoding: utf-8',
@@ -97,7 +97,7 @@ describe Rubocop::Cop::Team do
       end
 
       it 'still returns offenses' do
-        expect(offenses.first.cop_name).to eq('StringLiterals')
+        expect(offenses.first.cop_name).to eq('Style/StringLiterals')
       end
     end
   end
@@ -107,25 +107,25 @@ describe Rubocop::Cop::Team do
 
     it 'returns cop instances' do
       expect(cops).not_to be_empty
-      expect(cops.all? { |c| c.is_a?(Rubocop::Cop::Cop) }).to be_true
+      expect(cops.all? { |c| c.is_a?(RuboCop::Cop::Cop) }).to be_true
     end
 
     context 'when only some cop classes are passed to .new' do
       let(:cop_classes) do
-        [Rubocop::Cop::Lint::Void, Rubocop::Cop::Style::LineLength]
+        [RuboCop::Cop::Lint::Void, RuboCop::Cop::Style::LineLength]
       end
 
       it 'returns only intances of the classes' do
         expect(cops.size).to eq(2)
         cops.sort! { |a, b| a.name <=> b.name }
-        expect(cops[0].name).to eq('LineLength')
-        expect(cops[1].name).to eq('Void')
+        expect(cops[0].name).to eq('Lint/Void')
+        expect(cops[1].name).to eq('Style/LineLength')
       end
     end
 
     context 'when some classes are disabled with config' do
       before do
-        %w(Void LineLength).each do |cop_name|
+        %w(Lint/Void Style/LineLength).each do |cop_name|
           config.for_cop(cop_name)['Enabled'] = false
         end
       end
@@ -134,38 +134,38 @@ describe Rubocop::Cop::Team do
 
       it 'does not return intances of the classes' do
         expect(cops).not_to be_empty
-        expect(cop_names).not_to include('Void')
-        expect(cop_names).not_to include('LineLength')
+        expect(cop_names).not_to include('Style/Void')
+        expect(cop_names).not_to include('Style/LineLength')
       end
     end
   end
 
   describe '#forces' do
     subject(:forces) { team.forces }
-    let(:cop_classes) { Rubocop::Cop::Cop.non_rails }
+    let(:cop_classes) { RuboCop::Cop::Cop.non_rails }
 
     it 'returns force instances' do
       expect(forces).not_to be_empty
 
       forces.each do |force|
-        expect(force).to be_a(Rubocop::Cop::Force)
+        expect(force).to be_a(RuboCop::Cop::Force)
       end
     end
 
     context 'when a cop joined a force' do
-      let(:cop_classes) { [Rubocop::Cop::Lint::UselessAssignment] }
+      let(:cop_classes) { [RuboCop::Cop::Lint::UselessAssignment] }
 
       it 'returns the force' do
         expect(forces.size).to eq(1)
-        expect(forces.first).to be_a(Rubocop::Cop::VariableForce)
+        expect(forces.first).to be_a(RuboCop::Cop::VariableForce)
       end
     end
 
     context 'when multiple cops joined a same force' do
       let(:cop_classes) do
         [
-          Rubocop::Cop::Lint::UselessAssignment,
-          Rubocop::Cop::Lint::ShadowingOuterLocalVariable
+          RuboCop::Cop::Lint::UselessAssignment,
+          RuboCop::Cop::Lint::ShadowingOuterLocalVariable
         ]
       end
 
@@ -175,7 +175,7 @@ describe Rubocop::Cop::Team do
     end
 
     context 'when no cops joined force' do
-      let(:cop_classes) { [Rubocop::Cop::Style::For] }
+      let(:cop_classes) { [RuboCop::Cop::Style::For] }
 
       it 'returns nothing' do
         expect(forces).to be_empty

data/spec/rubocop/cop/util_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-describe Rubocop::Cop::Util do
+describe RuboCop::Cop::Util do
   describe '#line_range' do
     include ASTHelper
 

data/spec/rubocop/cop/variable_force/assignment_spec.rb

@@ -2,12 +2,12 @@
 
 require 'spec_helper'
 
-describe Rubocop::Cop::VariableForce::Assignment do
+describe RuboCop::Cop::VariableForce::Assignment do
   include ASTHelper
   include AST::Sexp
 
   let(:ast) do
-    processed_source = Rubocop::SourceParser.parse(source)
+    processed_source = RuboCop::SourceParser.parse(source)
     processed_source.ast
   end
 
@@ -42,9 +42,9 @@ describe Rubocop::Cop::VariableForce::Assignment do
   end
 
   let(:name) { lvasgn_node.children.first }
-  let(:scope) { Rubocop::Cop::VariableForce::Scope.new(def_node) }
+  let(:scope) { RuboCop::Cop::VariableForce::Scope.new(def_node) }
   let(:variable) do
-    Rubocop::Cop::VariableForce::Variable.new(name, lvasgn_node, scope)
+    RuboCop::Cop::VariableForce::Variable.new(name, lvasgn_node, scope)
   end
   let(:assignment) { described_class.new(lvasgn_node, variable) }
 

data/spec/rubocop/cop/variable_force/locatable_spec.rb

@@ -2,12 +2,12 @@
 
 require 'spec_helper'
 
-describe Rubocop::Cop::VariableForce::Locatable do
+describe RuboCop::Cop::VariableForce::Locatable do
   include ASTHelper
   include AST::Sexp
 
   class LocatableObject
-    include Rubocop::Cop::VariableForce::Locatable
+    include RuboCop::Cop::VariableForce::Locatable
 
     attr_reader :node, :scope
 
@@ -18,7 +18,7 @@ describe Rubocop::Cop::VariableForce::Locatable do
   end
 
   let(:ast) do
-    processed_source = Rubocop::SourceParser.parse(source)
+    processed_source = RuboCop::SourceParser.parse(source)
     processed_source.ast
   end
 
@@ -38,7 +38,7 @@ describe Rubocop::Cop::VariableForce::Locatable do
     found_node
   end
 
-  let(:scope) { Rubocop::Cop::VariableForce::Scope.new(def_node) }
+  let(:scope) { RuboCop::Cop::VariableForce::Scope.new(def_node) }
   let(:assignment) { LocatableObject.new(lvasgn_node, scope) }
 
   describe '#ancestor_nodes_in_scope' do

data/spec/rubocop/cop/variable_force/scope_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-describe Rubocop::Cop::VariableForce::Scope do
+describe RuboCop::Cop::VariableForce::Scope do
   include ASTHelper
   include AST::Sexp
 
@@ -23,8 +23,8 @@ describe Rubocop::Cop::VariableForce::Scope do
   end
 
   let(:ast) do
-    ast = Rubocop::SourceParser.parse(source).ast
-    Rubocop::Cop::VariableForce.wrap_with_top_level_node(ast)
+    ast = RuboCop::SourceParser.parse(source).ast
+    RuboCop::Cop::VariableForce.wrap_with_top_level_node(ast)
   end
 
   let(:scope_node_type) { :def }

data/spec/rubocop/cop/variable_force/variable_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-describe Rubocop::Cop::VariableForce::Variable do
+describe RuboCop::Cop::VariableForce::Variable do
   include AST::Sexp
 
   describe '.new' do
@@ -10,7 +10,7 @@ describe Rubocop::Cop::VariableForce::Variable do
       it 'raises error' do
         name = :foo
         declaration_node = s(:def)
-        scope = Rubocop::Cop::VariableForce::Scope.new(s(:class))
+        scope = RuboCop::Cop::VariableForce::Scope.new(s(:class))
         expect { described_class.new(name, declaration_node, scope) }
           .to raise_error(ArgumentError)
       end

data/spec/rubocop/cop/variable_force/variable_table_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-describe Rubocop::Cop::VariableForce::VariableTable do
+describe RuboCop::Cop::VariableForce::VariableTable do
   include AST::Sexp
 
   subject(:variable_table) { described_class.new }

data/spec/rubocop/cop/variable_force_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-describe Rubocop::Cop::VariableForce do
+describe RuboCop::Cop::VariableForce do
   include AST::Sexp
 
   subject(:force) { described_class.new([]) }

data/spec/rubocop/file_inspector_spec.rb

@@ -2,13 +2,13 @@
 
 require 'spec_helper'
 
-module Rubocop
+module RuboCop
   class FileInspector
     attr_writer :errors # Needed only for testing.
   end
 end
 
-describe Rubocop::FileInspector do
+describe RuboCop::FileInspector do
   subject(:inspector) { described_class.new(options) }
   let(:options) { {} }
   let(:offenses) { [] }
@@ -56,7 +56,7 @@ describe Rubocop::FileInspector do
 
     context 'if there is an offense in an inspected file' do
       let(:offenses) do
-        [Rubocop::Cop::Offense.new(:convention,
+        [RuboCop::Cop::Offense.new(:convention,
                                    Struct.new(:line, :column,
                                               :source_line).new(1, 0, ''),
                                    'Use alias_method instead of alias.',

data/spec/rubocop/formatter/base_formatter_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-module Rubocop
+module RuboCop
   module Formatter
     describe BaseFormatter do
       include FileHelper
@@ -179,7 +179,7 @@ module Rubocop
               else
                 fail
               end
-              expect(offenses.all? { |o| o.is_a?(Rubocop::Cop::Offense) })
+              expect(offenses.all? { |o| o.is_a?(RuboCop::Cop::Offense) })
                 .to be_true
             end
             run

data/spec/rubocop/formatter/clang_style_formatter_spec.rb

@@ -3,7 +3,7 @@
 require 'spec_helper'
 require 'stringio'
 
-module Rubocop
+module RuboCop
   module Formatter
     describe ClangStyleFormatter do
       subject(:formatter) { described_class.new(output) }

data/spec/rubocop/formatter/colorizable_spec.rb

@@ -4,7 +4,7 @@ require 'spec_helper'
 require 'rubocop/formatter/colorizable'
 require 'stringio'
 
-module Rubocop
+module RuboCop
   module Formatter
     describe Colorizable do
       let(:formatter_class) do

data/spec/rubocop/formatter/disabled_config_formatter_spec.rb

@@ -4,7 +4,7 @@ require 'spec_helper'
 require 'stringio'
 require 'ostruct'
 
-module Rubocop
+module RuboCop
   module Formatter
     describe DisabledConfigFormatter do
       subject(:formatter) { described_class.new(output) }
@@ -16,8 +16,8 @@ module Rubocop
         o
       end
       let(:offenses) do
-        [Rubocop::Cop::Offense.new(:convention, location, 'message', 'Cop1'),
-         Rubocop::Cop::Offense.new(:convention, location, 'message', 'Cop2')]
+        [RuboCop::Cop::Offense.new(:convention, location, 'message', 'Cop1'),
+         RuboCop::Cop::Offense.new(:convention, location, 'message', 'Cop2')]
       end
       let(:location) { OpenStruct.new(line: 1, column: 5) }
       before { $stdout = StringIO.new }

data/spec/rubocop/formatter/disabled_lines_formatter_spec.rb

@@ -4,7 +4,7 @@ require 'spec_helper'
 require 'stringio'
 require 'tempfile'
 
-module Rubocop
+module RuboCop
   module Formatter
     describe DisabledLinesFormatter do
       subject(:formatter) { described_class.new(output) }

data/spec/rubocop/formatter/emacs_style_formatter_spec.rb

@@ -3,7 +3,7 @@
 require 'spec_helper'
 require 'stringio'
 
-module Rubocop
+module RuboCop
   module Formatter
     describe EmacsStyleFormatter do
       subject(:formatter) { described_class.new(output) }

data/spec/rubocop/formatter/file_list_formatter_spec.rb

@@ -3,7 +3,7 @@
 require 'spec_helper'
 require 'stringio'
 
-module Rubocop
+module RuboCop
   module Formatter
     describe FileListFormatter do
       subject(:formatter) { described_class.new(output) }

data/spec/rubocop/formatter/formatter_set_spec.rb

@@ -3,7 +3,7 @@
 require 'spec_helper'
 require 'tempfile'
 
-module Rubocop
+module RuboCop
   module Formatter
     describe FormatterSet do
       subject(:formatter_set) { described_class.new }
@@ -109,20 +109,20 @@ module Rubocop
         end
 
         it 'returns constant represented by the passed string' do
-          expect(custom_formatter_class('Rubocop')).to eq(Rubocop)
+          expect(custom_formatter_class('RuboCop')).to eq(RuboCop)
         end
 
         it 'can handle namespaced constant name' do
-          expect(custom_formatter_class('Rubocop::CLI')).to eq(Rubocop::CLI)
+          expect(custom_formatter_class('RuboCop::CLI')).to eq(RuboCop::CLI)
         end
 
         it 'can handle top level namespaced constant name' do
-          expect(custom_formatter_class('::Rubocop::CLI')).to eq(Rubocop::CLI)
+          expect(custom_formatter_class('::RuboCop::CLI')).to eq(RuboCop::CLI)
         end
 
         context 'when non-existent constant name is passed' do
           it 'raises error' do
-            expect { custom_formatter_class('Rubocop::NonExistentClass') }
+            expect { custom_formatter_class('RuboCop::NonExistentClass') }
               .to raise_error(NameError)
           end
         end

data/spec/rubocop/formatter/fuubar_style_formatter_spec.rb

@@ -3,7 +3,7 @@
 require 'spec_helper'
 require 'stringio'
 
-module Rubocop
+module RuboCop
   describe Formatter::FuubarStyleFormatter do
     subject(:formatter) { described_class.new(output) }
     let(:output) { StringIO.new }

data/spec/rubocop/formatter/json_formatter_spec.rb

@@ -3,7 +3,7 @@
 require 'spec_helper'
 require 'stringio'
 
-module Rubocop
+module RuboCop
   describe Formatter::JSONFormatter do
     subject(:formatter) { described_class.new(output) }
     let(:output) { StringIO.new }