mutant 0.8.0 → 0.8.1

data/lib/mutant/ast/sexp.rb

@@ -12,7 +12,6 @@ module Mutant
       # @return [Parser::AST::Node]
       #
       # @api private
-      #
       def s(type, *children)
         Parser::AST::Node.new(type, children)
       end
@@ -24,7 +23,6 @@ module Mutant
       # @return [Parser::AST::Node]
       #
       # @api private
-      #
       def n_not(node)
         s(:send, node, :!)
       end

data/lib/mutant/cache.rb

@@ -8,19 +8,17 @@ module Mutant
     # @return [undefined]
     #
     # @api private
-    #
     def initialize
       @cache = {}
     end
 
-    # Return node for file
+    # Root node parsed from file
     #
     # @param [#to_s] path
     #
     # @return [AST::Node]
     #
     # @api private
-    #
     def parse(path)
       @cache.fetch(path) do
         @cache[path] = Parser::CurrentRuby.parse(File.read(path))

data/lib/mutant/cli.rb

@@ -18,7 +18,6 @@ module Mutant
     #   the exit status
     #
     # @api private
-    #
     def self.run(arguments)
       Runner.call(Env::Bootstrap.call(call(arguments))).success? ? EXIT_SUCCESS : EXIT_FAILURE
     rescue Error => exception
@@ -33,19 +32,17 @@ module Mutant
     # @return [undefined]
     #
     # @api private
-    #
     def initialize(arguments)
       @config = Config::DEFAULT
 
       parse(arguments)
     end
 
-    # Return config
+    # Config parsed from CLI
     #
     # @return [Config]
     #
     # @api private
-    #
     attr_reader :config
 
   private
@@ -61,7 +58,6 @@ module Mutant
     # @return [undefined]
     #
     # @api private
-    #
     def parse(arguments)
       opts = OptionParser.new do |builder|
         builder.banner = 'usage: mutant [options] MATCH_EXPRESSION ...'
@@ -82,12 +78,11 @@ module Mutant
     # @return [undefined]
     #
     # @api private
-    #
     def parse_match_expressions(expressions)
       fail Error, 'No expressions given' if expressions.empty?
 
-      expressions.map(&Expression.method(:parse)).each do |expression|
-        add_matcher(:match_expressions, expression)
+      expressions.each do |expression|
+        add_matcher(:match_expressions, config.expression_parser.(expression))
       end
     end
 
@@ -97,10 +92,9 @@ module Mutant
     #
     # @return [undefined]
     #
-    # @api private
-    #
     # rubocop:disable MethodLength
     #
+    # @api private
     def add_environment_options(opts)
       opts.separator('Environment:')
       opts.on('--zombie', 'Run mutant zombified') do
@@ -124,7 +118,6 @@ module Mutant
     # @return [undefined]
     #
     # @api private
-    #
     def setup_integration(name)
       update(integration: Integration.setup(name))
     rescue LoadError
@@ -138,7 +131,6 @@ module Mutant
     # @return [undefined]
     #
     # @api private
-    #
     def add_mutation_options(opts)
       opts.separator(nil)
       opts.separator('Options:')
@@ -159,10 +151,12 @@ module Mutant
     # @return [undefined]
     #
     # @api private
-    #
     def add_filter_options(opts)
-      opts.on('--ignore-subject PATTERN', 'Ignore subjects that match PATTERN') do |pattern|
-        add_matcher(:subject_ignores, Expression.parse(pattern))
+      opts.on('--ignore-subject EXPRESSION', 'Ignore subjects that match EXPRESSION as prefix') do |pattern|
+        add_matcher(:ignore_expressions, config.expression_parser.(pattern))
+      end
+      opts.on('--since REVISION', 'Only select subjects touched since REVISION') do |revision|
+        add_matcher(:subject_filters, Repository::SubjectFilter.new(Repository::Diff.from_head(revision)))
       end
     end
 
@@ -173,7 +167,6 @@ module Mutant
     # @return [undefined]
     #
     # @api private
-    #
     def add_debug_options(opts)
       opts.on('--fail-fast', 'Fail fast') do
         update(fail_fast: true)
@@ -198,7 +191,6 @@ module Mutant
     # @return [undefined]
     #
     # @api private
-    #
     def update(attributes)
       @config = @config.update(attributes)
     end
@@ -214,7 +206,6 @@ module Mutant
     # @return [undefined]
     #
     # @api private
-    #
     def add(attribute, value)
       update(attribute => config.public_send(attribute).dup << value)
     end
@@ -230,7 +221,6 @@ module Mutant
     # @return [undefined]
     #
     # @api private
-    #
     def add_matcher(attribute, value)
       update(matcher: config.matcher.add(attribute, value))
     end

data/lib/mutant/color.rb

@@ -10,7 +10,6 @@ module Mutant
     # @return [String]
     #
     # @api private
-    #
     def format(text)
       "\e[#{@code}m#{text}\e[0m"
     end
@@ -25,7 +24,6 @@ module Mutant
       #   the argument string
       #
       # @api private
-      #
       def format(text)
         text
       end
@@ -37,7 +35,6 @@ module Mutant
       # @return [undefined]
       #
       # @api private
-      #
       def initialize
       end
 

data/lib/mutant/config.rb

@@ -1,5 +1,8 @@
 module Mutant
-  # The configuration of a mutator run
+  # Standalone configuration of a mutant execution.
+  #
+  # Does not reference any "external" volatile state. The configuration applied
+  # to current environment is being represented by the Mutant::Env object.
   class Config
     include Adamantium::Flat, Anima::Update, Anima.new(
       :debug,
@@ -12,7 +15,8 @@ module Mutant
       :fail_fast,
       :jobs,
       :zombie,
-      :expected_coverage
+      :expected_coverage,
+      :expression_parser
     )
 
     %i[fail_fast zombie debug].each do |name|

data/lib/mutant/context.rb

@@ -3,22 +3,20 @@ module Mutant
   class Context
     include Adamantium::Flat, AbstractType, Concord::Public.new(:source_path)
 
-    # Return root ast node
+    # Root ast node
     #
     # @param [Parser::AST::Node] node
     #
     # @return [Parser::AST::Node]
     #
     # @api private
-    #
     abstract_method :root
 
-    # Return identification
+    # Identification string
     #
     # @return [String]
     #
     # @api private
-    #
     abstract_method :identification
 
   end # Context

data/lib/mutant/context/scope.rb

@@ -7,24 +7,22 @@ module Mutant
 
       NAMESPACE_DELIMITER = '::'.freeze
 
-      # Return AST wrapping mutated node
+      # Return root node for mutation
       #
       # @return [Parser::AST::Node]
       #
       # @api private
-      #
       def root(node)
         nesting.reverse.reduce(node) do |current, scope|
           self.class.wrap(scope, current)
         end
       end
 
-      # Return identification
+      # Identification string
       #
       # @return [String]
       #
       # @api private
-      #
       def identification
         scope.name
       end
@@ -41,7 +39,6 @@ module Mutant
       #   if scope is of kind module
       #
       # @api private
-      #
       def self.wrap(scope, node)
         name = s(:const, nil, scope.name.split(NAMESPACE_DELIMITER).last.to_sym)
         case scope
@@ -54,12 +51,11 @@ module Mutant
         end
       end
 
-      # Return nesting
+      # Nesting of scope
       #
       # @return [Enumerable<Class,Module>]
       #
       # @api private
-      #
       def nesting
         const = ::Object
         name_nesting.each_with_object([]) do |name, nesting|
@@ -69,45 +65,43 @@ module Mutant
       end
       memoize :nesting
 
-      # Return unqualified name of scope
+      # Unqualified name of scope
       #
       # @return [String]
       #
       # @api private
-      #
       def unqualified_name
         name_nesting.last
       end
 
-      # Return match expressions
+      # Match expressions for scope
       #
       # @return [Enumerable<Expression>]
       #
       # @api private
-      #
       def match_expressions
         name_nesting.each_index.reverse_each.map do |index|
-          Expression.parse("#{name_nesting.take(index.succ).join(NAMESPACE_DELIMITER)}*")
+          Expression::Namespace::Recursive.new(
+            scope_name: name_nesting.take(index.succ).join(NAMESPACE_DELIMITER)
+          )
         end
       end
       memoize :match_expressions
 
-      # Return scope wrapped by context
+      # Scope wrapped by context
       #
       # @return [::Module|::Class]
       #
       # @api private
-      #
       attr_reader :scope
 
     private
 
-      # Return nesting of names of scope
+      # Nesting of names in scope
       #
       # @return [Array<String>]
       #
       # @api private
-      #
       def name_nesting
         scope.name.split(NAMESPACE_DELIMITER)
       end

data/lib/mutant/delegator.rb

@@ -11,7 +11,6 @@ module Mutant
       # @return [undefined]
       #
       # @api private
-      #
       def delegate(*names)
         names.each(&method(:define_delegator))
       end
@@ -23,7 +22,6 @@ module Mutant
       # @return [undefined]
       #
       # @api private
-      #
       def define_delegator(name)
         fail "method #{name} already defined" if instance_methods.include?(name)
         define_method(name) do
@@ -41,7 +39,6 @@ module Mutant
     # @return [undefined]
     #
     # @api private
-    #
     def self.included(host)
       super
 

data/lib/mutant/diff.rb

@@ -7,7 +7,7 @@ module Mutant
     DELETION = '-'.freeze
     NEWLINE  = "\n".freeze
 
-    # Return source diff
+    # Unified source diff between old and new
     #
     # @return [String]
     #   if there is exactly one diff
@@ -16,7 +16,6 @@ module Mutant
     #   otherwise
     #
     # @api private
-    #
     def diff
       return if diffs.empty?
 
@@ -26,7 +25,7 @@ module Mutant
     end
     memoize :diff
 
-    # Return colorized source diff
+    # Colorized unified source diff between old and new
     #
     # @return [String]
     #   if there is a diff
@@ -35,14 +34,13 @@ module Mutant
     #   otherwise
     #
     # @api private
-    #
     def colorized_diff
       return unless diff
       diff.lines.map(&self.class.method(:colorize_line)).join
     end
     memoize :colorized_diff
 
-    # Return new object
+    # Build new object from source strings
     #
     # @param [String] old
     # @param [String] new
@@ -50,7 +48,6 @@ module Mutant
     # @return [Diff]
     #
     # @api private
-    #
     def self.build(old, new)
       new(lines(old), lines(new))
     end
@@ -62,7 +59,6 @@ module Mutant
     # @return [Array<String>]
     #
     # @api private
-    #
     def self.lines(source)
       source.lines.map(&:chomp)
     end
@@ -70,34 +66,31 @@ module Mutant
 
   private
 
-    # Return diffs
+    # Diffs between old and new
     #
     # @return [Array<Array>]
     #
     # @api private
-    #
     def diffs
       ::Diff::LCS.diff(old, new)
     end
 
-    # Return hunks
+    # Raw diff-lcs hunks
     #
     # @return [Array<Diff::LCS::Hunk>]
     #
     # @api private
-    #
     def hunks
       diffs.map do |diff|
         ::Diff::LCS::Hunk.new(old, new, diff, max_length, 0)
       end
     end
 
-    # Return minimized hunks
+    # Minimized hunks
     #
     # @return [Array<Diff::LCS::Hunk>]
     #
     # @api private
-    #
     def minimized_hunks
       head, *tail = hunks
 
@@ -111,24 +104,22 @@ module Mutant
       end
     end
 
-    # Return max length
+    # Max length of source line in new and old
     #
     # @return [Fixnum]
     #
     # @api private
-    #
     def max_length
       [old, new].map(&:length).max
     end
 
-    # Return colorized diff line
+    # Colorized a unified diff line
     #
     # @param [String] line
     #
     # @return [String]
     #
     # @api private
-    #
     def self.colorize_line(line)
       case line[0]
       when ADDITION