mutest 0.0.9 → 0.0.10

data/spec/support/xspec.rb

@@ -31,11 +31,11 @@ module XSpec
     end
 
     def exception(_, exception)
-      fail exception
+      raise exception
     end
 
     def yields(observation, yields)
-      block = observation.block or fail 'No block passed where expected'
+      block = observation.block or raise 'No block passed where expected'
 
       validate_block_arity(observation, yields)
 
@@ -48,12 +48,13 @@ module XSpec
 
       # block allows anything we can skip the check
       return if observed.equal?(-1)
-      fail 'Optargs currently not supported' if observed < -1
+      raise 'Optargs currently not supported' if observed < -1
+
       block_arity_mismatch(observation, expected, observed) unless expected.equal?(observed)
     end
 
     def block_arity_mismatch(observation, expected, observed)
-      fail "block arity mismatch, expected #{expected} observed #{observed}\nobservation:\n#{observation.inspect}"
+      raise "block arity mismatch, expected #{expected} observed #{observed}\nobservation:\n#{observation.inspect}"
     end
 
     alias_method :yields_return, :yields
@@ -67,23 +68,23 @@ module XSpec
 
     def self.assert_valid_events(event_list)
       event_list.map(&:first).each do |event|
-        fail "Invalid event: #{event}" unless VALID_EVENTS.include?(event)
+        raise "Invalid event: #{event}" unless VALID_EVENTS.include?(event)
       end
     end
     private_class_method :assert_valid_events
 
     def self.assert_not_empty(event_list)
-      fail 'no events' if event_list.empty?
+      raise 'no events' if event_list.empty?
     end
     private_class_method :assert_not_empty
 
     def self.assert_total(event_list)
       return unless event_list[0..-2].map(&:first).any?(&TERMINATE_EVENTS.method(:include?))
 
-      fail "Reaction not total: #{event_list}"
+      raise "Reaction not total: #{event_list}"
     end
     private_class_method :assert_total
-  end # MessageReaction
+  end
 
   class MessageExpectation
     include Anima.new(:receiver, :selector, :arguments, :reaction)
@@ -118,30 +119,30 @@ module XSpec
       end
 
       def error(message)
-        fail "#{message},\n observation:\n #{observation.inspect}\n expectation:\n #{expectation.inspect}"
+        raise "#{message},\n observation:\n #{observation.inspect}\n expectation:\n #{expectation.inspect}"
       end
 
       def trigger_exception
         exception = expectation.exception
-        fail exception if exception
+        raise exception if exception
       end
-    end # Verifier
-  end # MessageExpectation
+    end
+  end
 
   class MessageObservation
     include Anima.new(:receiver, :selector, :arguments, :block)
-  end # MessageObservation
+  end
 
   class ExpectationVerifier
     include Concord.new(:expectations)
 
     def call(observation)
-      expectation = expectations.shift or fail "No expected message but observed #{observation}"
+      expectation = expectations.shift or raise "No expected message but observed #{observation}"
       expectation.call(observation)
     end
 
     def assert_done
-      expectations.empty? or fail "unconsumed expectations:\n#{expectations.map(&:inspect).join}"
+      expectations.empty? or raise "unconsumed expectations:\n#{expectations.map(&:inspect).join}"
     end
 
     # rubocop:disable MethodLength
@@ -171,5 +172,5 @@ module XSpec
 
       verifier.assert_done
     end
-  end # ExpectationVerifier
-end # XSpec
+  end
+end

data/spec/unit/mutest/actor/binding_spec.rb

@@ -18,6 +18,7 @@ RSpec.describe Mutest::Actor::Binding do
 
     context 'when return type equals request type' do
       let(:response_type) { type }
+
       it { is_expected.to be(payload) }
     end
 

data/spec/unit/mutest/ast/meta/optarg_spec.rb

@@ -15,6 +15,7 @@ RSpec.describe Mutest::AST::Meta::Optarg do
 
     context 'when name is prefixed with an underscore' do
       let(:name) { :_foo }
+
       it { is_expected.to be false }
     end
   end

data/spec/unit/mutest/ast/meta/send/proc_predicate_spec.rb

@@ -2,7 +2,7 @@ RSpec.describe Mutest::AST::Meta::Send, '#proc?' do
   subject { described_class.new(node).proc? }
 
   shared_context 'proc send' do |source|
-    let(:node) { Parser::CurrentRuby.parse(source).children.first }
+    let(:node) { Unparser.parse(source).children.first }
   end
 
   shared_examples 'proc definition' do |*args|

data/spec/unit/mutest/ast/meta/send/receiver_possible_top_level_const_predicate_spec.rb

@@ -2,7 +2,7 @@ RSpec.describe Mutest::AST::Meta::Send, '#receiver_possible_top_level_const?' do
   subject { described_class.new(node).receiver_possible_top_level_const? }
 
   def parse(source)
-    Parser::CurrentRuby.parse(source)
+    Unparser.parse(source)
   end
 
   context 'when implicit top level const' do

data/spec/unit/mutest/ast/meta/send_spec.rb

@@ -7,35 +7,31 @@ RSpec.describe Mutest::AST::Meta::Send do
 
   let(:method_call)            { parse('foo.bar(baz)')             }
   let(:attribute_read)         { parse('foo.bar')                  }
-  let(:index_assignment)       { parse('foo[0] = bar')             }
   let(:attribute_assignment)   { parse('foo.bar = baz')            }
   let(:binary_method_operator) { parse('foo == bar')               }
   let(:method_method_call)     { parse('foo.method(:bar)')         }
   let(:public_method_call)     { parse("foo.public_method('bar')") }
 
   class Expectation
-    include Adamantium,
-            Anima.new(
-              :name,
-              :assignment,
-              :attribute_assignment,
-              :index_assignment,
-              :binary_method_operator,
-              :method_object_selector
-            )
+    include Anima.new(
+      :name,
+      :attribute_assignment,
+      :binary_method_operator,
+      :method_object_selector
+    )
+    include Adamantium
 
     ALL = [
-      [:method_call,            false, false, false, false, false],
-      [:attribute_read,         false, false, false, false, false],
-      [:index_assignment,       true,  false, true,  false, false],
-      [:attribute_assignment,   true,  true,  false, false, false],
-      [:binary_method_operator, false, false, false, true,  false],
-      [:method_method_call,     false, false, false, false, true],
-      [:public_method_call,     false, false, false, false, true]
+      [:method_call,            false, false, false],
+      [:attribute_read,         false, false, false],
+      [:attribute_assignment,   true,  false, false],
+      [:binary_method_operator, false, true,  false],
+      [:method_method_call,     false, false,  true],
+      [:public_method_call,     false, false,  true]
     ].map do |values|
       new(Hash[anima.attribute_names.zip(values)])
     end.freeze
-  end # Expectation
+  end
 
   # Rspec should have a build in for this kind of "n-dimensional assertion with context"
   (Expectation.anima.attribute_names - %i[name]).each do |name|

data/spec/unit/mutest/ast/regexp_spec.rb

@@ -20,8 +20,8 @@ module RegexpSpec
       NONE = Object.new.freeze
 
       include Equalizer.new(:token, :text, :mode, :min, :max)
-    end # Quantifier
-  end # Expression
+    end
+  end
 
   RSpec.shared_context 'regexp transformation' do
     let(:parsed)     { Mutest::AST::Regexp.parse(regexp)      }
@@ -66,6 +66,7 @@ module RegexpSpec
     RSpec.describe Mutest::AST::Regexp::Transformer.lookup(type) do
       context "when mapping #{regexp.inspect}" do
         let(:regexp) { regexp }
+
         let(:expected, &block)
 
         include_context 'regexp transformation'
@@ -74,7 +75,7 @@ module RegexpSpec
       end
     end
   end
-end # RegexpSpec
+end
 
 RegexpSpec.expect_mapping(/A/, :regexp_root_expression) do
   s(:regexp_root_expression,

data/spec/unit/mutest/ast_spec.rb

@@ -28,7 +28,7 @@ RSpec.describe Mutest::AST do
     end
 
     context 'on non Parser::AST::Node child' do
-      let(:block)   { ->(node) { fail if node.equal?(child_a) } }
+      let(:block)   { ->(node) { raise if node.equal?(child_a) } }
       let(:child_a) { AST::Node.new(:foo) }
 
       it 'does not yield that node' do

data/spec/unit/mutest/cli_spec.rb

@@ -107,21 +107,21 @@ RSpec.describe Mutest::CLI do
       it_behaves_like 'a cli parser'
 
       let(:expected_message) do
-        strip_indent(<<-MESSAGE)
-usage: mutest [options] MATCH_EXPRESSION ...
-Environment:
-        --zombie                     Run mutest zombified
-    -I, --include DIRECTORY          Add DIRECTORY to $LOAD_PATH
-    -r, --require NAME               Require file with NAME
-    -j, --jobs NUMBER                Number of kill jobs. Defaults to number of processors.
-
-Options:
-        --use INTEGRATION            Use INTEGRATION to kill mutations
-        --ignore-subject EXPRESSION  Ignore subjects that match EXPRESSION as prefix
-        --since REVISION             Only select subjects touched since REVISION
-        --fail-fast                  Fail fast
-        --version                    Print mutests version
-    -h, --help                       Show this message
+        <<~MESSAGE
+          usage: mutest [options] MATCH_EXPRESSION ...
+          Environment:
+                  --zombie                     Run mutest zombified
+              -I, --include DIRECTORY          Add DIRECTORY to $LOAD_PATH
+              -r, --require NAME               Require file with NAME
+              -j, --jobs NUMBER                Number of kill jobs. Defaults to number of processors.
+
+          Options:
+                  --use INTEGRATION            Use INTEGRATION to kill mutations
+                  --ignore-subject EXPRESSION  Ignore subjects that match EXPRESSION as prefix
+                  --since REVISION             Only select subjects touched since REVISION
+                  --fail-fast                  Fail fast
+                  --version                    Print mutests version
+              -h, --help                       Show this message
         MESSAGE
       end
     end

data/spec/unit/mutest/env/bootstrap_spec.rb

@@ -84,7 +84,7 @@ RSpec.describe Mutest::Env::Bootstrap do
       let(:invalid_class) do
         Class.new do
           def self.name
-            fail
+            raise
           end
         end
       end

data/spec/unit/mutest/integration_spec.rb

@@ -7,6 +7,7 @@ RSpec.describe Mutest::Integration do
 
   describe '#setup' do
     subject { object.setup }
+
     it_behaves_like 'a command method'
   end
 

data/spec/unit/mutest/isolation/none_spec.rb

@@ -7,7 +7,7 @@ RSpec.describe Mutest::Isolation::None do
     end
 
     it 'wraps *all* exceptions' do
-      expect { object.call { fail 'foo' } }.to raise_error(
+      expect { object.call { raise 'foo' } }.to raise_error(
         Mutest::Isolation::Error,
         'foo'
       )

data/spec/unit/mutest/matcher/config_spec.rb

@@ -10,6 +10,7 @@ RSpec.describe Mutest::Matcher::Config do
 
     context 'with one expression' do
       let(:object) { described_class::DEFAULT.add(:match_expressions, parse_expression('Foo')) }
+
       it { is_expected.to eql('#<Mutest::Matcher::Config match_expressions: [Foo]>') }
     end
 

data/spec/unit/mutest/mutator/node_spec.rb

@@ -5,7 +5,7 @@ Mutest::Meta::Example::ALL.each.group_by(&:node_type).each do |type, examples|
     it "generates the correct mutations on #{toplevel_nodes} toplevel examples" do
       examples.each do |example|
         verification = example.verification
-        fail verification.error_report unless verification.success?
+        raise verification.error_report unless verification.success?
       end
     end
   end

data/spec/unit/mutest/parallel/master_spec.rb

@@ -67,7 +67,7 @@ RSpec.describe Mutest::Parallel::Master do
     def stop?
       @stop
     end
-  end # FakeSink
+  end
 
   # Needed because of rubies undefined-ivar-read-is-nil stuff
   describe 'object initialization' do
@@ -79,6 +79,8 @@ RSpec.describe Mutest::Parallel::Master do
   end
 
   describe '.call' do
+    subject { described_class.call(config) }
+
     before do
       expect(Mutest::Parallel::Worker).to receive(:run).with(
         mailbox:   actor_env.mailbox(:worker_a),
@@ -87,8 +89,6 @@ RSpec.describe Mutest::Parallel::Master do
       ).and_return(worker_a)
     end
 
-    subject { described_class.call(config) }
-
     context 'with multiple workers configured' do
       let(:config)           { super().with(jobs: 2) }
       let(:expected_results) { []                    }

data/spec/unit/mutest/reporter/cli/printer_spec.rb

@@ -67,6 +67,7 @@ RSpec.describe Mutest::Reporter::CLI::Printer do
 
       context 'on failure' do
         let(:success?) { false }
+
         it_reports Mutest::Color::RED.format('foo bar') << "\n"
       end
     end
@@ -155,6 +156,7 @@ RSpec.describe Mutest::Reporter::CLI::Printer do
 
     context 'when output is NOT a tty?' do
       let(:tty?) { false }
+
       it_reports "foo\n"
     end
   end

data/spec/unit/mutest/reporter/cli/tput_spec.rb

@@ -38,9 +38,11 @@ RSpec.describe Mutest::Reporter::CLI::Tput do
     context 'when ed fails' do
       let(:tput_ed?) { false }
       let(:tput_cd?) { true }
+
       before do
         expect_command('tput cd', '[cd]', tput_cd?)
       end
+
       its(:prepare) { is_expected.to eql('[reset][sc]') }
       its(:restore) { is_expected.to eql('[rc][cd]')    }
     end

data/spec/unit/mutest/reporter/cli_spec.rb

@@ -82,11 +82,13 @@ RSpec.describe Mutest::Reporter::CLI do
 
     context 'when on CI' do
       let(:ci?) { true }
+
       it { is_expected.to eql(described_class.new(output, progressive_format)) }
     end
 
     context 'when output is not a tty?' do
       let(:tty?) { false }
+
       it { is_expected.to eql(described_class.new(output, progressive_format)) }
     end
 

data/spec/unit/mutest/repository/diff_spec.rb

@@ -31,7 +31,7 @@ describe Mutest::Repository::Diff do
         expect(config.kernel).to receive(:system)
           .ordered
           .with(
-            *%W[git ls-files --error-unmatch -- #{path}],
+            *%W[git ls-files --error-unmatch -- #{path}], # rubocop:disable Lint/UnneededSplatExpansion
             out: File::NULL,
             err: File::NULL
           ).and_return(git_ls_success?)

data/spec/unit/mutest/runner_spec.rb

@@ -39,7 +39,7 @@ RSpec.describe Mutest::Runner do
       Mutest::Parallel::Config.new(
         env:       actor_env,
         jobs:      1,
-        processor: ->(_object) { fail },
+        processor: ->(_object) { raise },
         sink:      Mutest::Runner::Sink.new(env),
         source:    Mutest::Parallel::Source::Array.new(env.mutations)
       )

data/spec/unit/mutest/subject_spec.rb

@@ -17,7 +17,7 @@ RSpec.describe Mutest::Subject do
   let(:object) { class_under_test.new(context, node) }
 
   let(:node) do
-    Parser::CurrentRuby.parse(<<-RUBY)
+    Unparser.parse(<<-RUBY)
       def foo
       end
     RUBY

data/spec/unit/mutest/warning_filter_spec.rb

@@ -54,8 +54,8 @@ RSpec.describe Mutest::WarningFilter do
     it 'resets to original stderr after execution with exception ' do
       original = $stderr
       begin
-        object.use { fail }
-      rescue
+        object.use { raise }
+      rescue StandardError
         :make_rubo_cop_happy
       end
       expect($stderr).to be(original)
@@ -64,7 +64,6 @@ RSpec.describe Mutest::WarningFilter do
     it 'returns warnings generated within block' do
       warnings =
         object.use do
-          # rubocop:disable Security/Eval
           eval(<<-RUBY)
             Class.new do
               def foo

data/spec/unit/mutest/zombifier_spec.rb

@@ -78,7 +78,7 @@ RSpec.describe Mutest::Zombifier do
   let(:file_entries) do
     {
       'a/project.rb' => { file: true, contents: 'module Project; end' },
-      'b/bar.rb' =>     { file: true, contents: 'module Bar; end'     }
+      'b/bar.rb'     => { file: true, contents: 'module Bar; end'     }
     }
   end
 

data/test_app/Gemfile.rspec3.8

@@ -0,0 +1,7 @@
+source 'https://rubygems.org'
+
+gem 'rspec',      '~> 3.8.0'
+gem 'rspec-core', '~> 3.8.0'
+gem 'mutest',       path: '../'
+gem 'mutest-rspec', path: '../'
+gem 'adamantium'