mutant 0.6.7 → 0.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 11e3b674e08512b58bd258c3ab1434bdf684d71a
-  data.tar.gz: f1c0559335e982f5348d6fb71c4ed2f7ae01b5b4
+  metadata.gz: 607569850361ac4088176c89f75ac7f7d732d8f0
+  data.tar.gz: 3247e38171198bf00c53341447881cb76fa44a3f
 SHA512:
-  metadata.gz: d7decc3ff8888ea325265bfaf8ea098ee4656a9254978c72b0561997944cdf287d003a724ff9511ec2d15355e708b070f10c7d5561a107bfdfb5eec21bcd3263
-  data.tar.gz: 9e229f14f8b9b808ab72293ac15855627cd821cdf151413aa38dfd5fd3815c64c11eb37665cca847652c98db90c51f50ec53c2afa1c6e27ae2f9bf3185fd3094
+  metadata.gz: 888a9f29aa202635ece333e17c514ca52c7a28660e0ab8cd7f4397f4f108be3a8d91477812ad48ae970f2808b1516de084f44ff02f743b21174ad2504fb4cdae
+  data.tar.gz: 5e3faa4096046383aa51274b6f201fb3f1eb416d7f8ca8049505fcf823fd994f051c34dd159256a6926e56401f5270d859b0bad0f13b645101442bfff5e80ef6

data/Changelog.md

@@ -1,3 +1,13 @@
+# v0.7.1 2014-12-04
+
+* Fix invalid dependencies on rspec for mutant-rspec
+
+# v0.7.0 2014-12-04
+
+* Use homegrown actor based parallelization
+* Fix redundant spec execution in rspec integration
+* Add mutation from #send to #__send__ (the canonical form).
+
 # v0.6.7 2014-11-17
 
 * Fix duplicate neutral emit for memoized instance method subjects

data/README.md

@@ -97,7 +97,7 @@ This can have the following unintended effects:
   counts the mutation as killed.
 
 * MRI crashes with a segfault and gets stuck when handling the segfault.
-  Depending on the number of active kill jobs mutant might appear to continue normally untill
+  Depending on the number of active kill jobs mutant might appear to continue normally until
   all workers are stuck into this state when it begins to hang.
   Currently mutant must assume that your test suite simply not terminated yet as from the outside
   (parent process) the difference between a long running test and a stuck MRI is not observable.

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
 threshold: 18
-total_score: 1132
+total_score: 1141

data/config/reek.yml

@@ -12,7 +12,6 @@ ControlParameter:
   enabled: true
   exclude:
   - Mutant::Expression#match_length
-  - Mutant::Reporter::CLI::Printer::SubjectProgress#print_mutation_result
 DataClump:
   enabled: true
   exclude: []
@@ -26,24 +25,18 @@ DuplicateMethodCall:
 FeatureEnvy:
   enabled: true
   exclude:
-  # Nature of OptionParser :(
-  - Mutant::CLI#add_environment_options
   - Mutant::Env#scope_name
   - Mutant::Diff#minimized_hunks
-  - Mutant::Integration::Rspec#run
-  - Mutant::Integration::Rspec::Rspec2#full_description
-  - Mutant::Integration::Rspec::Rspec3#full_description
+  - Mutant::Integration::Rspec#parse_example
   - Mutant::Matcher::Method::Instance#match?
   - Mutant::Matcher::Method::Singleton#receiver?
-  - Mutant::Mutation::Evil#success?
-  - Mutant::Mutation::Neutral#success?
   - Mutant::Mutator::Node#children_indices
-  # False positive, its a utility
-  - Mutant::Meta::Example::Verification#format_mutation
+  - Mutant::Meta::Example::Verification#format_mutation # False positive, its a utility
   - Mutant::Reporter::CLI#subject_results
-  - Mutant::Runner#run_mutation_test
-  - Mutant::Runner#kill_mutation
   - Mutant::Runner#finish
+  - Mutant::Runner::Master#stop_worker
+  - Mutant::Runner::Worker#run_mutation
+  - Mutant::Runner::Worker#handle
 IrresponsibleModule:
   enabled: true
   exclude: []
@@ -51,8 +44,6 @@ LongParameterList:
   enabled: true
   exclude:
   - Mutant::Matcher::Method::Instance#self.build?
-  - Mutant::Runner#finish # API client of parallel, one gets _ignored.
-  - Mutant::Runner#self.run
   max_params: 2
 LongYieldList:
   enabled: true
@@ -63,7 +54,6 @@ NestedIterators:
   exclude:
   - Mutant#self.singleton_subclass_instance
   - Mutant::CLI#parse
-  - Mutant::Integration::Rspec#run
   - Mutant::Isolation::Fork#self.call
   - Mutant::Mutator::Util::Array::Element#dispatch
   - Mutant::Mutator::Node::Resbody#mutate_captures
@@ -80,41 +70,31 @@ RepeatedConditional:
   enabled: true
   exclude:
   - Mutant::Mutator
-  - Mutant::Reporter::CLI
   - Mutant::Meta::Example::DSL
+  - Mutant::Runner::Master
   max_ifs: 1
 TooManyInstanceVariables:
   enabled: true
   exclude:
   - Mutant::Mutator # 4 vars
-  - Mutant::Runner  # 4 vars
+  - Mutant::Runner::Master # 4 vars
+  - Mutant::Runner::Scheduler # 4 vars
   max_instance_variables: 3
 TooManyMethods:
   enabled: true
   exclude:
   - Mutant::CLI
-  - Mutant::Subject
   - Mutant::Mutator::Node
-  - Mutant::Reporter::CLI
-  - Mutant::Runner
   - Mutant::Meta::Example::Verification
   max_methods: 10
 TooManyStatements:
   enabled: true
   exclude:
-  - Mutant#self.singleton_subclass_instance
-  - Mutant::Integration::Rspec#run
   - Mutant::Isolation::Fork#self.call
-  - Mutant::Reporter::CLI#colorized_diff
   - Mutant::Reporter::CLI::Printer::EnvProgress#run
   - Mutant::Reporter::CLI::Printer::Config#run
-  - Mutant::RequireHighjack#infect
-  - Mutant::Rspec::Killer#run
-  - Mutant::Runner#visit_collection
   - Mutant::Runner#initialize
-  - Mutant::Runner::Mutation#run
   - Mutant::Zombifier::File#self.find
-  - Mutant::CLI#add_debug_options
   - Mutant::CLI#add_environment_options
   max_statements: 7
 UncommunicativeMethodName:
@@ -129,9 +109,7 @@ UncommunicativeMethodName:
   accept: []
 UncommunicativeModuleName:
   enabled: true
-  exclude:
-  - Rspec2
-  - Rspec3
+  exclude: []
   reject:
   - !ruby/regexp /^.$/
   - !ruby/regexp /[0-9]$/
@@ -160,14 +138,7 @@ UtilityFunction:
   exclude:
   - Mutant::AST::Sexp#s
   - Mutant::CLI#reporter
-  - Mutant::Integration::Rspec#configuration
-  - Mutant::Integration::Rspec#options
-  - Mutant::Integration::Rspec::Rspec2#full_description
-  - Mutant::Integration::Rspec::Rspec2#new_reporter
-  - Mutant::Integration::Rspec::Rspec3#full_description
-  - Mutant::Meta::Example::Verification#format_mutation
-  - Mutant::Mutation::Evil#success?
-  - Mutant::Mutation::Neutral#success?
+  - Mutant::Integration::Rspec#parse_example
+  - Mutant::Meta::Example::Verification#format_mutation # False positive, its a utility
   - Mutant::Reporter::CLI::Format::Progressive#new_buffer
-  - Mutant::Runner#run_mutation_test
   max_helper_calls: 0

data/config/rubocop.yml

@@ -3,7 +3,7 @@ inherit_from: ../.rubocop.yml
 # General note about rubocop.
 # It does NOT allow to silence a specific rule violation.
 # For that reason I sometimes have to disable a whole cop where
-# I just tried to whitelist a specific occurence.
+# I just tried to whitelist a specific occurrence.
 
 
 AllCops:

data/lib/mutant.rb

@@ -100,10 +100,15 @@ require 'mutant/ast/nodes'
 require 'mutant/ast/named_children'
 require 'mutant/ast/node_predicates'
 require 'mutant/ast/meta'
+require 'mutant/actor'
+require 'mutant/actor/receiver'
+require 'mutant/actor/sender'
+require 'mutant/actor/mailbox'
+require 'mutant/actor/actor'
+require 'mutant/actor/env'
 require 'mutant/cache'
 require 'mutant/delegator'
 require 'mutant/warning_filter'
-require 'mutant/warning_expectation'
 require 'mutant/require_highjack'
 require 'mutant/isolation'
 require 'mutant/mutator'
@@ -196,7 +201,9 @@ require 'mutant/cli'
 require 'mutant/color'
 require 'mutant/diff'
 require 'mutant/runner'
-require 'mutant/runner/collector'
+require 'mutant/runner/scheduler'
+require 'mutant/runner/master'
+require 'mutant/runner/worker'
 require 'mutant/result'
 require 'mutant/reporter'
 require 'mutant/reporter/null'
@@ -224,6 +231,7 @@ module Mutant
       reporter:          Reporter::CLI.build($stdout),
       zombie:            false,
       jobs:              Mutant.ci? ? CI_DEFAULT_PROCESSOR_COUNT : Parallel.processor_count,
+      actor_env:         Mutant::Actor::Env.new(Thread),
       expected_coverage: 100.0
     )
   end # Config

data/lib/mutant/actor.rb

@@ -0,0 +1,64 @@
+module Mutant
+  # A minimal actor implementation
+  module Actor
+
+    # Error raised when actor signalling protocol is violated
+    class ProtocolError < RuntimeError
+    end # ProtocolError
+
+    # Undefined message payload
+    Undefined = Class.new do
+      INSPECT = 'Mutant::Actor::Undefined'.freeze
+
+      # Return object inspection
+      #
+      # @return [String]
+      #
+      # @api private
+      #
+      def inspect
+        INSPECT
+      end
+    end.new
+
+    # Message being exchanged between actors
+    class Message
+      include Concord::Public.new(:type, :payload)
+
+      # Return new message
+      #
+      # @param [Symbol] type
+      # @param [Object] payload
+      #
+      # @return [Message]
+      #
+      # @api private
+      #
+      def self.new(_type, _payload = Undefined)
+        super
+      end
+
+    end # Message
+
+    # Bindin to others actors sender for simple RPC
+    class Binding
+      include Concord.new(:actor, :other)
+
+      # Send message and wait for reply
+      #
+      # @param [Symbol] type
+      #
+      # @return [Object]
+      #
+      # @api private
+      #
+      def call(type)
+        other.call(Message.new(type, actor.sender))
+        message = actor.receiver.call
+        fail ProtocolError, "Expected #{type} but got #{message.type}" unless type.equal?(message.type)
+        message.payload
+      end
+
+    end # Binding
+  end # Actor
+end # Mutant

data/lib/mutant/actor/actor.rb

@@ -0,0 +1,50 @@
+module Mutant
+  module Actor
+    # Actor object available to acting threads
+    class Actor
+      include Concord.new(:thread, :mailbox)
+
+      # Initialize object
+      #
+      # @return [undefined]
+      #
+      # @api private
+      #
+      def initialize(*)
+        super
+        @sender = mailbox.sender(thread)
+      end
+
+      # Return sender to this actor
+      #
+      # @return [Sender]
+      #
+      # @api private
+      #
+      attr_reader :sender
+
+      # Return receiver for messages to this actor
+      #
+      # @return [Receiver]
+      #
+      # @api private
+      #
+      def receiver
+        mailbox.receiver
+      end
+
+      # Return binding for RPC to other actors
+      #
+      # @param [Actor::Sender] other
+      #
+      # @return [Binding]
+      #
+      # @api private
+      #
+      def bind(other)
+        Binding.new(self, other)
+      end
+
+    end # Actor
+  end # Actor
+end # Mutant

data/lib/mutant/actor/env.rb

@@ -0,0 +1,35 @@
+module Mutant
+  module Actor
+    # Actor root environment
+    class Env
+      include Concord.new(:thread_root)
+
+      # Spawn a new actor executing block
+      #
+      # @return [Actor::Sender]
+      #
+      # @api private
+      #
+      def spawn
+        mailbox = Mailbox.new
+
+        thread = thread_root.new do
+          yield mailbox.actor(thread_root.current)
+        end
+
+        mailbox.sender(thread)
+      end
+
+      # Return an private actor for current thread
+      #
+      # @return [Actor::Private]
+      #
+      # @api private
+      #
+      def current
+        Mailbox.new.actor(thread_root.current)
+      end
+
+    end # Env
+  end # Actor
+end # Mutant

data/lib/mutant/actor/mailbox.rb

@@ -0,0 +1,53 @@
+module Mutant
+  module Actor
+    # Unbound mailbox
+    class Mailbox
+
+      # Initialize new unbound mailbox
+      #
+      # @return [undefined]
+      #
+      # @api private
+      #
+      def initialize
+        @mutex    = Mutex.new
+        @messages = []
+        @receiver = Receiver.new(@mutex, @messages)
+        freeze
+      end
+
+      # Return receiver
+      #
+      # @return [Receiver]
+      #
+      # @api private
+      #
+      attr_reader :receiver
+
+      # Return actor that is able to read mailbox
+      #
+      # @param [Thread] thread
+      #
+      # @return [Actor]
+      #
+      # @api private
+      #
+      def actor(thread)
+        Actor.new(thread, self)
+      end
+
+      # Return sender to mailbox
+      #
+      # @param [Thread] thread
+      #
+      # @return [Sender]
+      #
+      # @api private
+      #
+      def sender(thread)
+        Sender.new(thread, @mutex, @messages)
+      end
+
+    end # Mailbox
+  end # Actor
+end # Mutant

data/lib/mutant/actor/receiver.rb

@@ -0,0 +1,48 @@
+module Mutant
+  module Actor
+    # Receiver side of an actor
+    class Receiver
+      include Concord.new(:mutex, :mailbox)
+
+      # Receives a message, blocking
+      #
+      # @return [Object]
+      #
+      # @api private
+      #
+      def call
+        2.times do
+          message = try_blocking_receive
+          return message unless message.equal?(Undefined)
+        end
+        fail ProtocolError
+      end
+
+    private
+
+      # Try a blocking receive
+      #
+      # @return [Undefined]
+      #   if there is no message yet
+      #
+      # @return [Object]
+      #   if there is a message
+      #
+      # @api private
+      #
+      def try_blocking_receive
+        mutex.lock
+        if mailbox.empty?
+          mutex.unlock
+          Thread.stop
+          Undefined
+        else
+          mailbox.shift.tap do
+            mutex.unlock
+          end
+        end
+      end
+
+    end # Receiver
+  end # Actor
+end # Mutant