mutant 0.8.10 → 0.8.11

data/spec/unit/mutant/meta/example/verification_spec.rb

@@ -0,0 +1,120 @@
+RSpec.describe Mutant::Meta::Example::Verification do
+  let(:object) { described_class.new(example, mutations) }
+
+  let(:example) do
+    Mutant::Meta::Example.new(
+      file:      'foo.rb',
+      node:      s(:true),
+      node_type: :true,
+      expected:  expected_nodes
+    )
+  end
+
+  let(:mutations) do
+    generated_nodes.map do |node|
+      Mutant::Mutation::Evil.new(example, node)
+    end
+  end
+
+  let(:generated_nodes) { [] }
+  let(:expected_nodes)  { [] }
+
+  describe '#success?' do
+    subject { object.success? }
+
+    context 'when generated nodes equal expected nodes' do
+      it { should be(true) }
+    end
+
+    context 'when expected node is missing' do
+      let(:expected_nodes) { [s(:false)] }
+
+      it { should be(false) }
+    end
+
+    context 'when there is extra generated node' do
+      let(:generated_nodes) { [s(:false)] }
+
+      it { should be(false) }
+    end
+
+    context 'when there is no diff to original source' do
+      let(:expected_nodes)  { [s(:true)] }
+      let(:generated_nodes) { [s(:true)] }
+
+      it { should be(false) }
+    end
+  end
+
+  describe '#error_report' do
+    subject { object.error_report }
+
+    context 'on success' do
+      specify do
+        expect { subject }.to raise_error(
+          RuntimeError,
+          'no error report on successful validation'
+        )
+      end
+    end
+
+    context 'when expected node is missing' do
+      let(:expected_nodes) { [s(:false), s(:nil)] }
+
+      specify do
+        should eql(strip_indent(<<-'REPORT'))
+          ---
+          file: foo.rb
+          original_ast: s(:true)
+          original_source: 'true'
+          missing:
+          - node: s(:false)
+            source: 'false'
+          - node: s(:nil)
+            source: nil
+          unexpected: []
+          no_diff: []
+        REPORT
+      end
+    end
+
+    context 'when there is extra generated node' do
+      let(:generated_nodes) { [s(:false), s(:nil)] }
+
+      specify do
+        should eql(strip_indent(<<-'REPORT'))
+          ---
+          file: foo.rb
+          original_ast: s(:true)
+          original_source: 'true'
+          missing: []
+          unexpected:
+          - node: s(:false)
+            source: 'false'
+          - node: s(:nil)
+            source: nil
+          no_diff: []
+        REPORT
+      end
+    end
+
+    context 'when there is no diff to original source' do
+      let(:expected_nodes)  { [s(:true)] }
+      let(:generated_nodes) { [s(:true)] }
+
+      specify do
+        should eql(strip_indent(<<-'REPORT'))
+          ---
+          file: foo.rb
+          original_ast: s(:true)
+          original_source: 'true'
+          missing: []
+          unexpected: []
+          no_diff:
+          - node: s(:true)
+            source: 'true'
+        REPORT
+      end
+    end
+  end
+end

data/spec/unit/mutant/meta/example_spec.rb

@@ -0,0 +1,32 @@
+RSpec.describe Mutant::Meta::Example do
+  let(:object) do
+    described_class.new(
+      file:      file,
+      node:      node,
+      node_type: node.type,
+      expected:  mutation_nodes
+    )
+  end
+
+  let(:file)           { 'foo/bar.rb'         }
+  let(:node)           { s(:true)             }
+  let(:mutation_nodes) { [s(:nil), s(:false)] }
+
+  let(:mutations) do
+    mutation_nodes.map do |node|
+      Mutant::Mutation::Evil.new(object, node)
+    end
+  end
+
+  describe '#source' do
+    subject { object.source }
+
+    it { should eql('true') }
+  end
+
+  describe '#verification' do
+    subject { object.verification }
+
+    it { should eql(Mutant::Meta::Example::Verification.new(object, mutations)) }
+  end
+end

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

@@ -1,7 +1,9 @@
-RSpec.describe Mutant::Mutator::Node do
-  Mutant::Meta::Example::ALL.each do |example|
-    context "on #{example.node.type.inspect}" do
-      it 'generates the correct mutations' do
+Mutant::Meta::Example::ALL.each.group_by(&:node_type).each do |type, examples|
+  RSpec.describe Mutant::Mutator::REGISTRY.lookup(type) do
+    toplevel_nodes = examples.map { |example| example.node.type }.uniq
+
+    it "generates the correct mutations on #{toplevel_nodes} toplevel examples" do
+      examples.each do |example|
         verification = example.verification
         unless verification.success?
           fail verification.error_report
@@ -10,3 +12,34 @@ RSpec.describe Mutant::Mutator::Node do
     end
   end
 end
+
+RSpec.describe Mutant::Mutator::Node do
+  describe 'internal DSL' do
+    let(:klass) do
+      Class.new(described_class) do
+        children(:left, :right)
+
+        def dispatch
+          left
+          emit_left(s(:nil))
+          emit_right_mutations do |node|
+            node.eql?(s(:nil))
+          end
+        end
+      end
+    end
+
+    def apply
+      klass.call(s(:and, s(:true), s(:true)))
+    end
+
+    specify do
+      expect(apply).to eql(
+        [
+          s(:and, s(:nil), s(:true)),
+          s(:and, s(:true), s(:nil))
+        ].to_set
+      )
+    end
+  end
+end

data/spec/unit/mutant/mutator_spec.rb

@@ -0,0 +1,21 @@
+RSpec.describe Mutant::Mutator do
+  describe '.handle' do
+    let(:object) { described_class }
+
+    subject do
+      Class.new(described_class) do
+        const_set(:REGISTRY, Mutant::Registry.new)
+
+        handle :send
+
+        def dispatch
+          emit(parent)
+        end
+      end
+    end
+
+    it 'should register mutator' do
+      expect(subject.mutate(s(:send), s(:parent))).to eql([s(:parent)].to_set)
+    end
+  end
+end

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

@@ -68,13 +68,13 @@ RSpec.describe Mutant::Parallel::Master do
     def stop?
       @stop
     end
-  end
+  end # FakeSink
 
   # Needed because of rubies undefined-ivar-read-is-nil stuff
   describe 'object initialization' do
-    let(:object) { described_class.send(:new, config, actor_env.mailbox(:master)) }
+    let(:object) { described_class.__send__(:new, config, actor_env.mailbox(:master)) }
 
-    it 'initializes falsy ivars'do
+    it 'initializes falsy ivars' do
       expect(object.instance_variable_get(:@stop)).to be(false)
     end
   end
@@ -104,13 +104,13 @@ RSpec.describe Mutant::Parallel::Master do
 
         sink.stop
 
-        message_sequence.add(:master,   :ready,  worker_a)
+        message_sequence.add(:master,   :ready, worker_a)
         message_sequence.add(:worker_a, :stop)
 
-        message_sequence.add(:master,   :ready,  worker_b)
+        message_sequence.add(:master,   :ready, worker_b)
         message_sequence.add(:worker_b, :stop)
 
-        message_sequence.add(:master,   :stop,   parent)
+        message_sequence.add(:master,   :stop, parent)
         message_sequence.add(:parent,   :stop)
       end
 
@@ -124,10 +124,10 @@ RSpec.describe Mutant::Parallel::Master do
         before do
           sink.stop
 
-          message_sequence.add(:master,   :ready,  worker_a)
+          message_sequence.add(:master,   :ready, worker_a)
           message_sequence.add(:worker_a, :stop)
 
-          message_sequence.add(:master,   :stop,   parent)
+          message_sequence.add(:master,   :stop, parent)
           message_sequence.add(:parent,   :stop)
         end
 
@@ -154,7 +154,7 @@ RSpec.describe Mutant::Parallel::Master do
           message_sequence.add(:master,   :ready,  worker_a)
           message_sequence.add(:worker_a, :stop)
 
-          message_sequence.add(:master,   :stop,   parent)
+          message_sequence.add(:master,   :stop, parent)
           message_sequence.add(:parent,   :stop)
         end
 
@@ -182,7 +182,7 @@ RSpec.describe Mutant::Parallel::Master do
           message_sequence.add(:master,   :ready,  worker_a)
           message_sequence.add(:worker_a, :stop)
 
-          message_sequence.add(:master,   :stop,   parent)
+          message_sequence.add(:master,   :stop, parent)
           message_sequence.add(:parent,   :stop)
         end
 
@@ -244,7 +244,7 @@ RSpec.describe Mutant::Parallel::Master do
           message_sequence.add(:parent,   :status, expected_status) do |message|
             expect(message.payload.active_jobs.frozen?).to be(true)
           end
-          message_sequence.add(:master,   :stop,   parent)
+          message_sequence.add(:master,   :stop, parent)
           message_sequence.add(:parent,   :stop)
         end
 
@@ -275,7 +275,7 @@ RSpec.describe Mutant::Parallel::Master do
           message_sequence.add(:master,   :ready,  worker_a)
           message_sequence.add(:worker_a, :stop)
 
-          message_sequence.add(:master,   :stop,   parent)
+          message_sequence.add(:master,   :stop, parent)
           message_sequence.add(:parent,   :stop)
         end
 
@@ -302,7 +302,7 @@ RSpec.describe Mutant::Parallel::Master do
           message_sequence.add(:master,   :ready,  worker_a)
           message_sequence.add(:worker_a, :stop)
 
-          message_sequence.add(:master,   :stop,   parent)
+          message_sequence.add(:master,   :stop, parent)
           message_sequence.add(:parent,   :stop)
         end
 

data/spec/unit/mutant/registry_spec.rb

@@ -0,0 +1,47 @@
+RSpec.describe Mutant::Registry do
+  let(:lookup)             { object.lookup(type)           }
+  let(:object)             { described_class.new           }
+  let(:mutator)            { class_double(Mutant::Mutator) }
+  let(:node)               { s(:true)                      }
+  let(:expected_arguments) { [node, nil]                   }
+
+  def register_mutator
+    object.register(type, mutator)
+  end
+
+  context 'on registered type' do
+    subject { register_mutator }
+
+    let(:type) { :true }
+
+    before { subject }
+
+    it 'returns registered mutator' do
+      expect(lookup).to be(mutator)
+    end
+
+    it_behaves_like 'a command method'
+
+    context 'when registered twice' do
+      it 'fails upon registration' do
+        expect { register_mutator }.to raise_error(described_class::RegistryError, 'Duplicate type registration: :true')
+      end
+    end
+  end
+
+  context 'on unknown type' do
+    let(:type) { :unknown }
+
+    it 'raises error' do
+      expect { lookup }.to raise_error(described_class::RegistryError, 'No entry for: :unknown')
+    end
+  end
+
+  context 'when registering an invalid node type' do
+    let(:type) { :invalid }
+
+    it 'raises error' do
+      expect { register_mutator }.to raise_error(described_class::RegistryError, 'Invalid type registration: :invalid')
+    end
+  end
+end

data/spec/unit/mutant/reporter/cli/printer/config_spec.rb

@@ -9,7 +9,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::Config do
         Mutant configuration:
         Matcher:         #<Mutant::Matcher::Config empty>
         Integration:     Mutant::Integration::Null
-        Expect Coverage: 100.00%
         Jobs:            1
         Includes:        []
         Requires:        []
@@ -17,13 +16,10 @@ RSpec.describe Mutant::Reporter::CLI::Printer::Config do
     end
 
     context 'with non default coverage expectation' do
-      with(:config) { { expected_coverage: 0.1r } }
-
       it_reports(<<-REPORT)
         Mutant configuration:
         Matcher:         #<Mutant::Matcher::Config empty>
         Integration:     Mutant::Integration::Null
-        Expect Coverage: 10.00%
         Jobs:            1
         Includes:        []
         Requires:        []

data/spec/unit/mutant/reporter/cli/printer/env_progress_spec.rb

@@ -1,8 +1,6 @@
 RSpec.describe Mutant::Reporter::CLI::Printer::EnvProgress do
   setup_shared_context
 
-  with(:config) { { expected_coverage: 0.1r } }
-
   let(:reportable) { env_result }
 
   describe '.call' do
@@ -13,7 +11,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::EnvProgress do
         Mutant configuration:
         Matcher:         #<Mutant::Matcher::Config empty>
         Integration:     Mutant::Integration::Null
-        Expect Coverage: 10.00%
         Jobs:            1
         Includes:        []
         Requires:        []
@@ -27,7 +24,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::EnvProgress do
         Overhead:        Inf%
         Mutations/s:     0.00
         Coverage:        100.00%
-        Expected:        10.00%
       STR
     end
 
@@ -36,7 +32,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::EnvProgress do
         Mutant configuration:
         Matcher:         #<Mutant::Matcher::Config empty>
         Integration:     Mutant::Integration::Null
-        Expect Coverage: 10.00%
         Jobs:            1
         Includes:        []
         Requires:        []
@@ -50,7 +45,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::EnvProgress do
         Overhead:        100.00%
         Mutations/s:     0.50
         Coverage:        100.00%
-        Expected:        10.00%
       STR
     end
 
@@ -61,7 +55,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::EnvProgress do
         Mutant configuration:
         Matcher:         #<Mutant::Matcher::Config empty>
         Integration:     Mutant::Integration::Null
-        Expect Coverage: 10.00%
         Jobs:            1
         Includes:        []
         Requires:        []
@@ -75,7 +68,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::EnvProgress do
         Overhead:        100.00%
         Mutations/s:     0.50
         Coverage:        50.00%
-        Expected:        10.00%
       STR
     end
   end

data/spec/unit/mutant/reporter/cli/printer/env_result_spec.rb

@@ -17,7 +17,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::EnvResult do
       Mutant configuration:
       Matcher:         #<Mutant::Matcher::Config empty>
       Integration:     Mutant::Integration::Null
-      Expect Coverage: 100.00%
       Jobs:            1
       Includes:        []
       Requires:        []
@@ -31,7 +30,6 @@ RSpec.describe Mutant::Reporter::CLI::Printer::EnvResult do
       Overhead:        100.00%
       Mutations/s:     0.50
       Coverage:        50.00%
-      Expected:        100.00%
     STR
   end
 end