mutant 0.6.7 → 0.7.1

data/spec/unit/mutant/runner/worker_spec.rb

@@ -0,0 +1,73 @@
+RSpec.describe Mutant::Runner::Worker do
+  setup_shared_context
+
+  let(:actor)  { actor_env.actor(:worker)         }
+  let(:parent) { actor_env.actor(:parent).sender  }
+
+  before do
+    message_sequence.add(:parent, :ready, actor.sender)
+  end
+
+  let(:attributes) do
+    {
+      config: config,
+      parent: parent,
+      id:     1
+    }
+  end
+
+  describe '.run' do
+    subject { described_class.run(attributes) }
+
+    let(:actor_names) { [:worker] }
+
+    context 'when receving :job command' do
+
+      let(:test_result) { double('Test Result') }
+
+      before do
+        expect(mutation).to receive(:kill).with(config.isolation, config.integration).and_return(test_result).ordered
+
+        message_sequence.add(:worker, :job, job)
+        message_sequence.add(:parent, :result, job_result)
+        message_sequence.add(:parent, :ready, actor.sender)
+        message_sequence.add(:worker, :stop)
+      end
+
+      let(:test)            { double('Test')                                                   }
+      let(:index)           { double('Index')                                                  }
+      let(:test_result)     { double('Test Result')                                            }
+      let(:mutation)        { double('Mutation')                                               }
+      let(:job_result)      { Mutant::Runner::JobResult.new(job: job, result: mutation_result) }
+      let(:job)             { Mutant::Runner::Job.new(index: index, mutation: mutation)        }
+
+      let(:mutation_result) do
+        Mutant::Result::Mutation.new(
+          mutation:    mutation,
+          index:       job.index,
+          test_result: test_result
+        )
+      end
+
+      it 'signals ready and status to parent' do
+        subject
+      end
+
+      it { should eql(actor.sender) }
+
+      it 'consumes all messages' do
+        expect { subject }.to change(&message_sequence.method(:consumed?)).from(false).to(true)
+      end
+    end
+
+    context 'when receiving unknown command' do
+      before do
+        message_sequence.add(:worker, :other)
+      end
+
+      it 'raises error' do
+        expect { subject }.to raise_error(Mutant::Actor::ProtocolError, 'Unknown command: :other')
+      end
+    end
+  end
+end

data/spec/unit/mutant/runner_spec.rb

@@ -1,143 +1,85 @@
-class Double
-  include Concord.new(:name, :attributes)
-
-  def self.new(name, attributes = {})
-    super
-  end
-
-  def update(_attributes)
-    self
-  end
-
-  def method_missing(name, *arguments)
-    super unless attributes.key?(name)
-    fail "Arguments provided for #{name}" if arguments.any?
-    attributes.fetch(name)
-  end
-end
-
-# FIXME: This is not even close to a mutation covering spec.
 RSpec.describe Mutant::Runner do
-  let(:object) { described_class.new(env) }
+  setup_shared_context
 
-  let(:reporter) { Mutant::Reporter::Trace.new                        }
-  let(:config)   { Mutant::Config::DEFAULT.update(reporter: reporter, isolation: Mutant::Isolation::None) }
-  let(:subjects) { [subject_a, subject_b]                             }
+  let(:integration)    { double('Integration')    }
+  let(:master_sender)  { actor_env.spawn          }
+  let(:runner_actor)   { actor_env.actor(:runner) }
 
-  let(:subject_a) { Double.new('Subject A', mutations: mutations_a, tests: subject_a_tests) }
-  let(:subject_b) { Double.new('Subject B', mutations: mutations_b) }
-
-  let(:subject_a_tests) { [test_a1, test_a2] }
-
-  let(:env) do
-    subjects = self.subjects
-    Class.new(Mutant::Env) do
-      define_method(:subjects) { subjects }
-    end.new(config)
+  before do
+    expect(integration).to receive(:setup).ordered
+    expect(Mutant::Runner::Master).to receive(:call).with(env).and_return(master_sender).ordered
   end
 
-  let(:mutations_a) { [mutation_a1, mutation_a2] }
-  let(:mutations_b) { [] }
+  describe '.call' do
+    update(:config) { { integration: integration } }
+    let(:actor_names) { [:runner, :master] }
 
-  let(:mutation_a1) { Double.new('Mutation A1') }
-  let(:mutation_a2) { Double.new('Mutation A2') }
+    subject { described_class.call(env) }
 
-  let(:test_a1) { Double.new('Test A1') }
-  let(:test_a2) { Double.new('Test A2') }
+    context 'when status done gets returned immediately' do
+      before do
+        message_sequence.add(:runner, :status, actor_env.actor(:current).sender)
+        message_sequence.add(:current, :status, status)
+        message_sequence.add(:runner, :stop, actor_env.actor(:current).sender)
+        message_sequence.add(:current, :stop)
+      end
 
-  let(:test_report_a1) { Double.new('Test Report A1') }
+      it 'returns env result' do
+        should be(status.env_result)
+      end
 
-  before do
-    allow(mutation_a1).to receive(:subject).and_return(subject_a)
-    allow(mutation_a1).to receive(:insert)
-    allow(mutation_a2).to receive(:subject).and_return(subject_a)
-    allow(mutation_a2).to receive(:insert)
-    allow(test_a1).to receive(:run).and_return(test_report_a1)
-    allow(mutation_a1).to receive(:killed_by?).with(test_report_a1).and_return(true)
-    allow(mutation_a2).to receive(:killed_by?).with(test_report_a1).and_return(true)
-  end
+      it 'logs start' do
+        expect { subject }.to change { config.reporter.start_calls }.from([]).to([env])
+      end
 
-  before do
-    time = Time.at(0)
-    allow(Time).to receive(:now).and_return(time)
-  end
+      it 'logs process' do
+        expect { subject }.to change { config.reporter.progress_calls }.from([]).to([status])
+      end
 
-  let(:expected_subject_results) do
-    [
-      Mutant::Result::Subject.new(
-        subject:          subject_a,
-        mutation_results: [
-          Mutant::Result::Mutation.new(
-            index:        0,
-            mutation:     mutation_a1,
-            runtime:      0.0,
-            test_results: [test_report_a1]
-          ),
-          Mutant::Result::Mutation.new(
-            index:        1,
-            mutation:     mutation_a2,
-            runtime:      0.0,
-            test_results: [test_report_a1]
-          )
-        ],
-        runtime:          0.0
-      ),
-      Mutant::Result::Subject.new(
-        subject:          subject_b,
-        mutation_results: [],
-        runtime:          0.0
-      )
-    ]
-  end
+      it 'logs result' do
+        expect { subject }.to change { config.reporter.report_calls }.from([]).to([status.env_result])
+      end
 
-  describe '#result' do
-    let(:expected_result) do
-      Mutant::Result::Env.new(
-        env:             env,
-        runtime:         0.0,
-        subject_results: expected_subject_results
-      )
+      it 'consumes all messages' do
+        expect { subject }.to change(&message_sequence.method(:consumed?)).from(false).to(true)
+      end
     end
 
-    context 'on error free execution' do
-      subject { object.result }
-
-      its(:env) { should be(env) }
+    context 'when status done gets returned immediately' do
+      let(:incomplete_status) { status.update(done: false) }
 
-      it 'reports result' do
-        expect { subject }.to change { config.reporter.report_calls }.from([]).to([expected_result])
+      before do
+        expect(Kernel).to receive(:sleep).with(1 / 20.0).exactly(2).times.ordered
+        message_sequence.add(:runner,  :status, actor_env.actor(:current).sender)
+        message_sequence.add(:current, :status, incomplete_status)
+        message_sequence.add(:runner,  :status, actor_env.actor(:current).sender)
+        message_sequence.add(:current, :status, incomplete_status)
+        message_sequence.add(:runner,  :status, actor_env.actor(:current).sender)
+        message_sequence.add(:current, :status, status)
+        message_sequence.add(:runner,  :stop,   actor_env.actor(:current).sender)
+        message_sequence.add(:current, :stop)
       end
-    end
 
-    context 'when isolation raises error' do
-      subject { object.result }
+      it 'returns env result' do
+        should be(status.env_result)
+      end
 
-      its(:env)             { should be(env)                       }
-      its(:subject_results) { should eql(expected_subject_results) }
+      it 'logs start' do
+        expect { subject }.to change { config.reporter.start_calls }.from([]).to([env])
+      end
 
-      it { should eql(expected_result) }
+      it 'logs result' do
+        expect { subject }.to change { config.reporter.report_calls }.from([]).to([status.env_result])
+      end
 
-      before do
-        expect(Mutant::Isolation::None).to receive(:call)
-          .twice
-          .and_raise(Mutant::Isolation::Error.new('test-exception-message'))
-
-        expect(Mutant::Result::Test).to receive(:new).with(
-          test:     test_a1,
-          mutation: mutation_a1,
-          runtime:  0.0,
-          output:   'test-exception-message',
-          passed:   false
-        ).and_return(test_report_a1)
-        expect(Mutant::Result::Test).to receive(:new).with(
-          test:     test_a1,
-          mutation: mutation_a2,
-          runtime:  0.0,
-          output:   'test-exception-message',
-          passed:   false
-        ).and_return(test_report_a1)
+      it 'logs process' do
+        expected = [incomplete_status, incomplete_status, status]
+        expect { subject }.to change { config.reporter.progress_calls }.from([]).to(expected)
       end
 
+      it 'consumes all messages' do
+        expect { subject }.to change(&message_sequence.method(:consumed?)).from(false).to(true)
+      end
     end
   end
 end

data/spec/unit/mutant/subject/method/instance_spec.rb

@@ -48,36 +48,11 @@ RSpec.describe Mutant::Subject::Method::Instance do
 
     subject { object.prepare }
 
-    context 'on non initialize methods' do
-
-      it 'undefines method on scope' do
-        expect { subject }.to change { scope.instance_methods.include?(:foo) }.from(true).to(false)
-      end
-
-      it_should_behave_like 'a command method'
-
+    it 'undefines method on scope' do
+      expect { subject }.to change { scope.instance_methods.include?(:foo) }.from(true).to(false)
     end
 
-    context 'on initialize method' do
-
-      let(:node) do
-        s(:def, :initialize, s(:args))
-      end
-
-      it 'does not write warnings' do
-        warnings = Mutant::WarningFilter.use do
-          subject
-        end
-        expect(warnings).to eql([])
-      end
-
-      it 'undefines method on scope' do
-        subject
-        expect { scope.new }.to raise_error(NoMethodError)
-      end
-
-      it_should_behave_like 'a command method'
-    end
+    it_should_behave_like 'a command method'
   end
 
   describe '#source' do
@@ -157,9 +132,21 @@ RSpec.describe Mutant::Subject::Method::Instance::Memoized do
 
     let(:expected) do
       [
-        Mutant::Mutation::Neutral.new(object, s(:begin, s(:def, :foo, s(:args)), s(:send, nil, :memoize, s(:args, s(:sym, :foo))))),
-        Mutant::Mutation::Evil.new(object, s(:begin, s(:def, :foo, s(:args), s(:send, nil, :raise)), s(:send, nil, :memoize, s(:args, s(:sym, :foo))))),
-        Mutant::Mutation::Evil.new(object, s(:begin, s(:def, :foo, s(:args), nil), s(:send, nil, :memoize, s(:args, s(:sym, :foo))))),
+        Mutant::Mutation::Neutral.new(
+          object,
+          s(:begin,
+            s(:def, :foo, s(:args)), s(:send, nil, :memoize, s(:args, s(:sym, :foo))))
+        ),
+        Mutant::Mutation::Evil.new(
+          object,
+          s(:begin,
+            s(:def, :foo, s(:args), s(:send, nil, :raise)), s(:send, nil, :memoize, s(:args, s(:sym, :foo))))
+        ),
+        Mutant::Mutation::Evil.new(
+          object,
+          s(:begin,
+            s(:def, :foo, s(:args), nil), s(:send, nil, :memoize, s(:args, s(:sym, :foo))))
+        )
       ]
     end
 

data/spec/unit/mutant/warning_filter_spec.rb

@@ -59,7 +59,7 @@ RSpec.describe Mutant::WarningFilter do
       expect(found).to be(true)
     end
 
-    it 'resets to original stderr after execution with exeception ' do
+    it 'resets to original stderr after execution with exception ' do
       original = $stderr
       begin
         object.use { fail }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.6.7
+  version: 0.7.1
 platform: ruby
 authors:
 - Markus Schirp
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-17 00:00:00.000000000 Z
+date: 2014-12-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parser
@@ -128,14 +128,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.11.0
+        version: 0.11.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.11.0
+        version: 0.11.1
 - !ruby/object:Gem::Dependency
   name: adamantium
   requirement: !ruby/object:Gem::Requirement
@@ -274,6 +274,12 @@ files:
 - config/triage.yml
 - config/yardstick.yml
 - lib/mutant.rb
+- lib/mutant/actor.rb
+- lib/mutant/actor/actor.rb
+- lib/mutant/actor/env.rb
+- lib/mutant/actor/mailbox.rb
+- lib/mutant/actor/receiver.rb
+- lib/mutant/actor/sender.rb
 - lib/mutant/ast.rb
 - lib/mutant/ast/meta.rb
 - lib/mutant/ast/named_children.rb
@@ -382,14 +388,15 @@ files:
 - lib/mutant/require_highjack.rb
 - lib/mutant/result.rb
 - lib/mutant/runner.rb
-- lib/mutant/runner/collector.rb
+- lib/mutant/runner/master.rb
+- lib/mutant/runner/scheduler.rb
+- lib/mutant/runner/worker.rb
 - lib/mutant/subject.rb
 - lib/mutant/subject/method.rb
 - lib/mutant/subject/method/instance.rb
 - lib/mutant/subject/method/singleton.rb
 - lib/mutant/test.rb
 - lib/mutant/version.rb
-- lib/mutant/warning_expectation.rb
 - lib/mutant/warning_filter.rb
 - lib/mutant/zombifier.rb
 - lib/mutant/zombifier/file.rb
@@ -460,11 +467,19 @@ files:
 - spec/spec_helper.rb
 - spec/support/compress_helper.rb
 - spec/support/corpus.rb
+- spec/support/fake_actor.rb
 - spec/support/ice_nine_config.rb
 - spec/support/mutation_verifier.rb
 - spec/support/rb_bug.rb
 - spec/support/rspec.rb
+- spec/support/shared_context.rb
 - spec/support/test_app.rb
+- spec/unit/mutant/actor/actor_spec.rb
+- spec/unit/mutant/actor/binding_spec.rb
+- spec/unit/mutant/actor/env_spec.rb
+- spec/unit/mutant/actor/message_spec.rb
+- spec/unit/mutant/actor/receiver_spec.rb
+- spec/unit/mutant/actor/sender_spec.rb
 - spec/unit/mutant/ast_spec.rb
 - spec/unit/mutant/cli_spec.rb
 - spec/unit/mutant/context/root_spec.rb
@@ -481,6 +496,7 @@ files:
 - spec/unit/mutant/integration_spec.rb
 - spec/unit/mutant/isolation_spec.rb
 - spec/unit/mutant/loader/eval_spec.rb
+- spec/unit/mutant/mailbox_spec.rb
 - spec/unit/mutant/matcher/chain_spec.rb
 - spec/unit/mutant/matcher/compiler/subject_prefix_spec.rb
 - spec/unit/mutant/matcher/compiler_spec.rb
@@ -497,19 +513,20 @@ files:
 - spec/unit/mutant/reporter/cli_spec.rb
 - spec/unit/mutant/reporter/null_spec.rb
 - spec/unit/mutant/require_highjack_spec.rb
-- spec/unit/mutant/runner/collector_spec.rb
+- spec/unit/mutant/result/env_spec.rb
+- spec/unit/mutant/result/subject_spec.rb
+- spec/unit/mutant/runner/master_spec.rb
+- spec/unit/mutant/runner/scheduler_spec.rb
+- spec/unit/mutant/runner/worker_spec.rb
 - spec/unit/mutant/runner_spec.rb
 - spec/unit/mutant/subject/context_spec.rb
 - spec/unit/mutant/subject/method/instance_spec.rb
 - spec/unit/mutant/subject/method/singleton_spec.rb
 - spec/unit/mutant/subject_spec.rb
-- spec/unit/mutant/test_spec.rb
-- spec/unit/mutant/warning_expectation_spec.rb
 - spec/unit/mutant/warning_filter_spec.rb
 - spec/unit/mutant_spec.rb
 - test_app/.rspec
 - test_app/Gemfile.devtools
-- test_app/Gemfile.rspec2
 - test_app/Gemfile.rspec3.0
 - test_app/Gemfile.rspec3.1
 - test_app/lib/test_app.rb
@@ -547,6 +564,12 @@ test_files:
 - spec/integration/mutant/rspec_spec.rb
 - spec/integration/mutant/test_mutator_handles_types_spec.rb
 - spec/integration/mutant/zombie_spec.rb
+- spec/unit/mutant/actor/actor_spec.rb
+- spec/unit/mutant/actor/binding_spec.rb
+- spec/unit/mutant/actor/env_spec.rb
+- spec/unit/mutant/actor/message_spec.rb
+- spec/unit/mutant/actor/receiver_spec.rb
+- spec/unit/mutant/actor/sender_spec.rb
 - spec/unit/mutant/ast_spec.rb
 - spec/unit/mutant/cli_spec.rb
 - spec/unit/mutant/context/root_spec.rb
@@ -563,6 +586,7 @@ test_files:
 - spec/unit/mutant/integration_spec.rb
 - spec/unit/mutant/isolation_spec.rb
 - spec/unit/mutant/loader/eval_spec.rb
+- spec/unit/mutant/mailbox_spec.rb
 - spec/unit/mutant/matcher/chain_spec.rb
 - spec/unit/mutant/matcher/compiler/subject_prefix_spec.rb
 - spec/unit/mutant/matcher/compiler_spec.rb
@@ -579,14 +603,15 @@ test_files:
 - spec/unit/mutant/reporter/cli_spec.rb
 - spec/unit/mutant/reporter/null_spec.rb
 - spec/unit/mutant/require_highjack_spec.rb
-- spec/unit/mutant/runner/collector_spec.rb
+- spec/unit/mutant/result/env_spec.rb
+- spec/unit/mutant/result/subject_spec.rb
+- spec/unit/mutant/runner/master_spec.rb
+- spec/unit/mutant/runner/scheduler_spec.rb
+- spec/unit/mutant/runner/worker_spec.rb
 - spec/unit/mutant/runner_spec.rb
 - spec/unit/mutant/subject/context_spec.rb
 - spec/unit/mutant/subject/method/instance_spec.rb
 - spec/unit/mutant/subject/method/singleton_spec.rb
 - spec/unit/mutant/subject_spec.rb
-- spec/unit/mutant/test_spec.rb
-- spec/unit/mutant/warning_expectation_spec.rb
 - spec/unit/mutant/warning_filter_spec.rb
 - spec/unit/mutant_spec.rb
-has_rdoc: