mutant 0.5.24 → 0.5.25

data/lib/mutant/matcher/chain.rb

@@ -2,7 +2,7 @@ module Mutant
   class Matcher
     # A chain of matchers
     class Chain < self
-      include Concord::Public.new(:matchers)
+      include Concord.new(:matchers)
 
       # Enumerate subjects
       #
@@ -24,22 +24,6 @@ module Mutant
         self
       end
 
-      # Build matcher chain
-      #
-      # @param [Enumerable<Matcher>] matchers
-      #
-      # @return [Matcher]
-      #
-      # @api private
-      #
-      def self.build(matchers)
-        if matchers.length.equal?(1)
-          return matchers.first
-        end
-
-        new(matchers)
-      end
-
     end # Chain
   end # Matcher
 end # Mutant

data/lib/mutant/matcher/compiler.rb

@@ -0,0 +1,108 @@
+module Mutant
+  class Matcher
+
+    # Compiler for complex matchers
+    class Compiler
+      include Concord.new(:env, :config), AST::Sexp, Procto.call(:result)
+
+      # Return generated matcher
+      #
+      # @return [Mutant::Matcher]
+      #
+      # @api private
+      #
+      def result
+        Filter.new(
+          Chain.build(config.match_expressions.map(&method(:matcher))),
+          predicate
+        )
+      end
+
+      # Subject expression prefix predicate
+      class SubjectPrefix
+        include Concord.new(:expression)
+
+        # Test if subject expression is matched by prefix
+        #
+        # @return [Boolean]
+        #
+        # @api private
+        #
+        def call(subject)
+          expression.prefix?(subject.expression)
+        end
+
+      end # SubjectPrefix
+
+    private
+
+      # Return predicate
+      #
+      # @return [#call]
+      #
+      # @api private
+      #
+      def predicate
+        if subject_selector && subject_rejector
+          Morpher::Evaluator::Predicate::Boolean::And.new([
+            subject_selector,
+            Morpher::Evaluator::Predicate::Negation.new(subject_rejector)
+          ])
+        elsif subject_selector
+          subject_selector
+        elsif subject_rejector
+          Morpher::Evaluator::Predicate::Negation.new(subject_rejector)
+        else
+          Morpher::Evaluator::Predicate::Tautology.new
+        end
+      end
+
+      # Return subject selector
+      #
+      # @return [#call]
+      #   if selector is present
+      #
+      # @return [nil]
+      #   otherwise
+      #
+      # @api private
+      #
+      def subject_selector
+        selectors = config.subject_selects.map do |attribute, value|
+          Morpher.compile(s(:eql, s(:attribute, attribute), s(:static, value)))
+        end
+
+        Morpher::Evaluator::Predicate::Boolean::Or.new(selectors) if selectors.any?
+      end
+
+      # Return subject rejector
+      #
+      # @return [#call]
+      #   if there is a subject rejector
+      #
+      # @return [nil]
+      #   otherwise
+      #
+      # @api private
+      #
+      def subject_rejector
+        rejectors = config.subject_ignores.map(&SubjectPrefix.method(:new))
+
+        Morpher::Evaluator::Predicate::Boolean::Or.new(rejectors) if rejectors.any?
+      end
+
+      # Return a matcher from expression
+      #
+      # @param [Mutant::Expression] expression
+      #
+      # @return [Matcher]
+      #
+      # @api private
+      #
+      def matcher(expression)
+        expression.matcher(env)
+      end
+
+    end # Compiler
+  end # Matcher
+end # Mutant

data/lib/mutant/matcher/config.rb

@@ -0,0 +1,28 @@
+module Mutant
+  class Matcher
+    # Match configuration
+    class Config
+      include Adamantium, Anima::Update, Anima.new(
+        :match_expressions,
+        :subject_ignores,
+        :subject_selects
+      )
+
+      DEFAULT = new(Hash[anima.attribute_names.map { |name| [name, []] }])
+
+      # Return configuration with added value
+      #
+      # @param [Symbol] attribute
+      # @param [Object] value
+      #
+      # @return [Config]
+      #
+      # @api private
+      #
+      def add(attribute, value)
+        update(attribute => public_send(attribute).dup << value)
+      end
+
+    end # Config
+  end # Matcher
+end # Mutant

data/lib/mutant/matcher/method.rb

@@ -120,7 +120,7 @@ module Mutant
       def subject
         node = matched_node
         return unless node
-        self.class::SUBJECT_CLASS.new(context, node)
+        self.class::SUBJECT_CLASS.new(env.config, context, node)
       end
       memoize :subject
 

data/lib/mutant/matcher/namespace.rb

@@ -19,8 +19,9 @@ module Mutant
       #
       def each(&block)
         return to_enum unless block_given?
-        scopes.each do |scope|
-          scope.each(&block)
+
+        env.matchable_scopes.select do |scope|
+          scope.each(&block) if match?(scope)
         end
 
         self
@@ -28,64 +29,16 @@ module Mutant
 
     private
 
-      # Return scope enumerator
-      #
-      # @return [Array<Class, Module>]
-      #
-      # @api private
-      #
-      def scopes
-        ::ObjectSpace.each_object(Module).each_with_object([]) do |scope, aggregate|
-          aggregate << Scope.new(env, scope) if match?(scope)
-        end.sort_by(&:identification)
-      end
-      memoize :scopes
-
-      # Return scope name
-      #
-      # @param [Class,Module] scope
-      #
-      # @return [String]
-      #   if scope has a name and does not raise exceptions optaining it
-      #
-      # @return [nil]
-      #   otherwise
-      #
-      # @api private
-      #
-      # rubocop:disable LineLength
-      #
-      def scope_name(scope)
-        scope.name
-      rescue => exception
-        env.warn("While optaining #{scope.class}#name from: #{scope.inspect} It raised an error: #{exception.inspect} fix your lib!")
-        nil
-      end
-
       # Test scope if name matches expresion
       #
-      # @param [Module,Class] scope
+      # @param [Module, Class] scope
       #
       # @return [Boolean]
       #
       # @api private
       #
       def match?(scope)
-        name = scope_name(scope) or return false
-
-        unless name.kind_of?(String)
-          env.warn("#{scope.class}#name from: #{scope.inspect} did not return a String or nil.  Fix your lib to support normal ruby semantics!")
-          return false
-        end
-
-        scope_expression = Expression.try_parse(name)
-
-        unless scope_expression
-          $stderr.puts("WARNING: #{name.inspect} is not an identifiable ruby class name.")
-          return false
-        end
-
-        expression.prefix?(scope_expression)
+        expression.prefix?(scope.expression)
       end
 
     end # Namespace

data/lib/mutant/matcher/null.rb

@@ -2,7 +2,7 @@ module Mutant
   class Matcher
     # A null matcher, that does not match any subjects
     class Null < self
-      include Equalizer.new
+      include Concord.new
 
       # Enumerate subjects
       #

data/lib/mutant/matcher/scope.rb

@@ -2,7 +2,7 @@ module Mutant
   class Matcher
     # Matcher for specific namespace
     class Scope < self
-      include Concord::Public.new(:env, :scope)
+      include Concord::Public.new(:env, :scope, :expression)
 
       MATCHERS = [
         Matcher::Methods::Singleton,

data/lib/mutant/mutation.rb

@@ -18,16 +18,6 @@ module Mutant
     end
     memoize :root
 
-    # Test if killer is successful
-    #
-    # @param [Killer] killer
-    #
-    # @return [Boolean]
-    #
-    # @api private
-    #
-    abstract_method :success?
-
     # Insert mutated node
     #
     # FIXME: Cache subject visibility in a better way! Ideally dont mutate it
@@ -87,14 +77,16 @@ module Mutant
       subject.source
     end
 
-    # Test if test should fail under mutation
+    # Test if mutation is killed by test report
+    #
+    # @param [Report::Test] test_report
     #
     # @return [Boolean]
     #
     # @api private
     #
-    def should_fail?
-      self.class::SHOULD_FAIL
+    def killed_by?(test_report)
+      self.class::SHOULD_PASS.equal?(test_report.passed)
     end
 
   private
@@ -110,5 +102,29 @@ module Mutant
     end
     memoize :sha1
 
+    # Evil mutation that should case mutations to fail tests
+    class Evil < self
+
+      SHOULD_PASS = false
+      SYMBOL      = 'evil'.freeze
+
+    end # Evil
+
+    # Neutral mutation that should not cause mutations to fail tests
+    class Neutral < self
+
+      SYMBOL      = 'neutral'.freeze
+      SHOULD_PASS = true
+
+    end # Neutral
+
+    # Noop mutation, special case of neutral
+    class Noop < self
+
+      SYMBOL      = 'noop'.freeze
+      SHOULD_PASS = true
+
+    end # Noop
+
   end # Mutation
 end # Mutant

data/lib/mutant/mutator/node.rb

@@ -49,22 +49,12 @@ module Mutant
       #
       alias_method :dup_node, :dup_input
 
-      # Emit children mutations
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
-      def emit_children_mutations
-        Mutator::Util::Array.each(children, self) do |children|
-          emit_type(*children)
-        end
-      end
-
       # Return ast meta description
       #
       # @return [AST::Meta]
       #
+      # @api private
+      #
       def meta
         AST::Meta.for(node)
       end

data/lib/mutant/mutator/node/named_value/variable_assignment.rb

@@ -12,7 +12,7 @@ module Mutant
             gvasgn: '$',
             cvasgn: '@@',
             ivasgn: '@',
-            lvasgn: ''
+            lvasgn: EMPTY_STRING
           }
 
           MAP = IceNine.deep_freeze(

data/lib/mutant/reporter/cli.rb

@@ -4,8 +4,6 @@ module Mutant
     class CLI < self
       include Concord.new(:output)
 
-      NL = "\n".freeze
-
       # Report progress object
       #
       # @param [Object] object

data/lib/mutant/reporter/cli/printer.rb

@@ -6,6 +6,8 @@ module Mutant
       class Printer
         include AbstractType, Delegator, Adamantium::Flat, Concord.new(:output, :object)
 
+        NL = "\n".freeze
+
         # Run printer on object to output
         #
         # @param [IO] output
@@ -41,6 +43,18 @@ module Mutant
           success? ? Color::GREEN : Color::RED
         end
 
+        # Visit a collection of objects
+        #
+        # @return [Enumerable<Object>] collection
+        #
+        # @return [undefined]
+        #
+        # @api private
+        #
+        def visit_collection(collection)
+          collection.each(&method(:visit))
+        end
+
         # Visit object
         #
         # @param [Object] object

data/lib/mutant/reporter/cli/progress.rb

@@ -1,11 +1,9 @@
 module Mutant
   class Reporter
     class CLI
-      # Abstract base class for process printers
+      # Abstract base and namespace class for process printers
       class Progress < Printer
         include AbstractType, Registry.new
-
-        delegate :running?
       end # Progress
     end # CLI
   end # Reporter

data/lib/mutant/reporter/cli/progress/config.rb

@@ -5,9 +5,7 @@ module Mutant
         # Progress printer for configuration
         class Config < self
 
-          handle(Mutant::Runner::Config)
-
-          delegate :config
+          handle(Mutant::Config)
 
           # Report configuration
           #
@@ -18,12 +16,10 @@ module Mutant
           # @api private
           #
           def run
-            if running?
-              info 'Mutant configuration:'
-              info 'Matcher:         %s',      config.matcher.inspect
-              info 'Integration:     %s',      config.integration.name
-              info 'Expect Coverage: %0.2f%%', config.expected_coverage.inspect
-            end
+            info 'Mutant configuration:'
+            info 'Matcher:            %s',      object.matcher_config.inspect
+            info 'Integration:        %s',      object.integration.name
+            info 'Expect Coverage:    %0.2f%%', object.expected_coverage.inspect
             self
           end