mutant 0.8.8 → 0.8.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d83617d43f73b49c391d1b7e8f3317a666297c83
-  data.tar.gz: 186b8ce58089718ba3cafa1c08f609b89941541e
+  metadata.gz: 6629a951cb8a81c688c0adc0acac1f3ff41f13b6
+  data.tar.gz: b8e775063ca733fe989083a38c31ef680135cbc1
 SHA512:
-  metadata.gz: 3d4ee1e94a96ba909cf327bde6afdca8ff5e50f80603ca8835115fdba2abbfa05ad89897366908f8b64bca2e9302605e7accddf1167260c187cf4e40417d0df5
-  data.tar.gz: 7b599e22fa7ddd03e9f9ac91af9ce0e730b155c8df2b5ab1b5a67fc8cd62f8bf8972ca8df64b45fb5114a556e7416f3c86baf666ea62f5de5201e8b068406afc
+  metadata.gz: f11084993d2875f56e6fd86174eaae8a1b8681feee2f81d4442914fc63ab9a7e8a9be395145b55b424519f5c7975f114bf5adf0c7e01ce1dca5b2aa091fd0364
+  data.tar.gz: 44e651cc4312db4388d975f1955791966055c67fc4a648a297e34535d9f47cea0588e94921098416d2a0636d0d4d26e017a75377960beeb28dcfa8ad555032c0

data/Changelog.md

@@ -1,5 +1,14 @@
+# v0.8.9 2016-01-05
+
+* Add mutation from `Hash#[]` to `Hash#key?` (#511)
+* Add mutation from `Hash#fetch` to `Hash#key?` (#511)
+* Add mutation from `#at` to `#key?` (#511)
+* Add mutation from `Hash#values_at` to `Hash#fetch_values` (#510)
+
 # v0.8.8 2015-11-15
 
+* Drop support for rspec-3.2
+* Remove CI specific job number default
 * Add support for rspec-3.3.4
 * Add mutations/s performance metric to report
 

data/README.md

@@ -286,7 +286,7 @@ people do talks about it at conferences, user groups or other chances. Thanks fo
 As I (the author @mbj) am not too happy with some of the facts being presented about
 mutant the last month.
 
-So if you plan to do a presentation: I offer to review your slides / talk - for free off course.
+So if you plan to do a presentation: I offer to review your slides / talk - for free of course.
 My intention is NOT to change your bias pro / against this tool. Just to help to fix
 invalid statements about the tool.
 

data/Rakefile

@@ -6,17 +6,21 @@ Rake.application.load_imports
 
 task('metrics:mutant').clear
 namespace :metrics do
-  task :mutant => :coverage do
-    success = Kernel.system(*%w[
+  task mutant: :coverage do
+    arguments = %w[
       bundle exec mutant
-      --zombie
-      --use rspec
+      --ignore-subject Mutant::Meta*
       --include lib
-      --require mutant
       --since HEAD~1
-      --
-      Mutant*
-    ]) or fail 'Mutant task is not successful'
+      --require mutant
+      --use rspec
+      --zombie
+    ]
+    arguments.concat(%W[--jobs 4]) if ENV.key?('CIRCLE_CI')
+
+    arguments.concat(%w[-- Mutant*])
+
+    success = Kernel.system(*arguments) or fail 'Mutant task is not successful'
   end
 end
 

data/bin/mutant

@@ -15,7 +15,11 @@ namespace =
       load_path:        $LOAD_PATH,
       kernel:           Kernel,
       pathname:         Pathname,
-      require_highjack: Mutant::RequireHighjack.method(:call).to_proc.curry.call(Kernel),
+      require_highjack: Mutant::RequireHighjack
+        .method(:call)
+        .to_proc
+        .curry
+        .call(Kernel),
       root_require:     'mutant',
       includes: %w[
         mutant

data/config/flay.yml

@@ -1,3 +1,3 @@
 ---
 threshold: 18
-total_score: 1232
+total_score: 1173

data/lib/mutant.rb

@@ -23,6 +23,8 @@ require 'unparser'
 Thread.abort_on_exception = true
 
 # Library namespace
+#
+# @api private
 module Mutant
   EMPTY_STRING   = ''.freeze
   EMPTY_ARRAY    = [].freeze
@@ -32,8 +34,6 @@ module Mutant
   # Test if CI is detected via environment
   #
   # @return [Boolean]
-  #
-  # @api private
   def self.ci?
     ENV.key?('CI')
   end
@@ -60,7 +60,7 @@ require 'mutant/actor/receiver'
 require 'mutant/actor/sender'
 require 'mutant/actor/mailbox'
 require 'mutant/actor/env'
-require 'mutant/cache'
+require 'mutant/parser'
 require 'mutant/delegator'
 require 'mutant/isolation'
 require 'mutant/parallel'
@@ -69,8 +69,8 @@ require 'mutant/parallel/worker'
 require 'mutant/parallel/source'
 require 'mutant/warning_filter'
 require 'mutant/require_highjack'
-require 'mutant/mutator'
 require 'mutant/mutation'
+require 'mutant/mutator'
 require 'mutant/mutator/registry'
 require 'mutant/mutator/util'
 require 'mutant/mutator/util/array'
@@ -168,7 +168,6 @@ require 'mutant/runner/sink'
 require 'mutant/result'
 require 'mutant/reporter'
 require 'mutant/reporter/null'
-require 'mutant/reporter/trace'
 require 'mutant/reporter/cli'
 require 'mutant/reporter/cli/printer'
 require 'mutant/reporter/cli/printer/config'
@@ -189,26 +188,28 @@ require 'mutant/zombifier'
 module Mutant
   # Reopen class to initialize constant to avoid dep circle
   class Config
-    CI_DEFAULT_PROCESSOR_COUNT = 4
-
     DEFAULT = new(
       debug:             false,
-      fail_fast:         false,
-      integration:       Integration::Null,
-      matcher:           Matcher::Config::DEFAULT,
-      includes:          EMPTY_ARRAY,
-      requires:          EMPTY_ARRAY,
-      isolation:         Mutant::Isolation::Fork,
-      reporter:          Reporter::CLI.build($stdout),
-      zombie:            false,
-      jobs:              Mutant.ci? ? CI_DEFAULT_PROCESSOR_COUNT : ::Parallel.processor_count,
       expected_coverage: Rational(1),
       expression_parser: Expression::Parser.new([
         Expression::Method,
         Expression::Methods,
         Expression::Namespace::Exact,
         Expression::Namespace::Recursive
-      ])
+      ]),
+      fail_fast:         false,
+      includes:          EMPTY_ARRAY,
+      integration:       Integration::Null,
+      isolation:         Mutant::Isolation::Fork,
+      jobs:              ::Parallel.processor_count,
+      kernel:            Kernel,
+      load_path:         $LOAD_PATH,
+      matcher:           Matcher::Config::DEFAULT,
+      open3:             Open3,
+      pathname:          Pathname,
+      requires:          EMPTY_ARRAY,
+      reporter:          Reporter::CLI.build($stdout),
+      zombie:            false
     )
   end # Config
 end # Mutant

data/lib/mutant/actor.rb

@@ -13,8 +13,6 @@ module Mutant
       # Object inspection
       #
       # @return [String]
-      #
-      # @api private
       def inspect
         INSPECT
       end
@@ -30,15 +28,13 @@ module Mutant
       # @param [Object] payload
       #
       # @return [Message]
-      #
-      # @api private
       def self.new(_type, _payload = Undefined)
         super
       end
 
     end # Message
 
-    # Binding to othersactors sender for simple RPC
+    # Binding to other actors sender for simple RPC
     class Binding
       include Concord.new(:mailbox, :other)
 
@@ -47,8 +43,6 @@ module Mutant
       # @param [Symbol] type
       #
       # @return [Object]
-      #
-      # @api private
       def call(type)
         other.call(Message.new(type, mailbox.sender))
         message = mailbox.receiver.call

data/lib/mutant/actor/env.rb

@@ -7,8 +7,6 @@ module Mutant
       # Spawn a new actor executing block
       #
       # @return [Actor::Sender]
-      #
-      # @api private
       def spawn
         mailbox = new_mailbox
 
@@ -22,8 +20,6 @@ module Mutant
       # New unbound mailbox
       #
       # @return [Mailbox]
-      #
-      # @api private
       def new_mailbox
         Mailbox.new
       end

data/lib/mutant/actor/mailbox.rb

@@ -7,8 +7,6 @@ module Mutant
       # New mailbox
       #
       # @return [Mailbox]
-      #
-      # @api private
       def self.new
         mutex              = Mutex.new
         condition_variable = ConditionVariable.new
@@ -25,8 +23,6 @@ module Mutant
       # @param [Actor::Sender] other
       #
       # @return [Binding]
-      #
-      # @api private
       def bind(other)
         Binding.new(self, other)
       end

data/lib/mutant/actor/receiver.rb

@@ -7,8 +7,6 @@ module Mutant
       # Receives a message, blocking
       #
       # @return [Object]
-      #
-      # @api private
       def call
         2.times do
           message = try_blocking_receive
@@ -26,8 +24,6 @@ module Mutant
       #
       # @return [Object]
       #   if there is a message
-      #
-      # @api private
       def try_blocking_receive
         mutex.synchronize do
           if messages.empty?

data/lib/mutant/actor/sender.rb

@@ -10,8 +10,6 @@ module Mutant
       # @param [Object] message
       #
       # @return [self]
-      #
-      # @api private
       def call(message)
         mutex.synchronize do
           messages << message

data/lib/mutant/ast.rb

@@ -14,8 +14,6 @@ module Mutant
     #
     # @return [nil]
     #   otherwise
-    #
-    # @api private
     def self.find_last_path(node, &predicate)
       fail ArgumentError, 'block expected' unless block_given?
       path = []
@@ -36,11 +34,9 @@ module Mutant
     #   all nodes visited recursively including root
     #
     # @return [undefined]
-    #
-    # @api private
     def self.walk(node, stack, &block)
       block.call(node, stack)
-      node.children.grep(Parser::AST::Node) do |child|
+      node.children.grep(::Parser::AST::Node) do |child|
         stack.push(child)
         walk(child, stack, &block)
         stack.pop

data/lib/mutant/ast/meta/const.rb

@@ -12,8 +12,6 @@ module Mutant
         # Test if AST node is possibly a top level constant
         #
         # @return [Boolean]
-        #
-        # @api private
         def possible_top_level?
           base.nil? || n_cbase?(base)
         end

data/lib/mutant/ast/meta/optarg.rb

@@ -14,8 +14,6 @@ module Mutant
         # Test if optarg definition intends to be used
         #
         # @return [Boolean]
-        #
-        # @api private
         def used?
           !name.to_s.start_with?(UNDERSCORE)
         end

data/lib/mutant/ast/meta/send.rb

@@ -15,15 +15,11 @@ module Mutant
         # Arguments of mutated node
         #
         # @return [Enumerable<Parser::AST::Node>]
-        #
-        # @api private
         alias_method :arguments, :remaining_children
 
         # Test if AST node is a valid assignment target
         #
         # @return [Boolean]
-        #
-        # @api private
         def assignment?
           index_assignment? || attribute_assignment?
         end
@@ -31,8 +27,6 @@ module Mutant
         # Test if AST node is an attribute assignment?
         #
         # @return [Boolean]
-        #
-        # @api private
         def attribute_assignment?
           !Types::METHOD_OPERATORS.include?(selector) &&
           selector.to_s.end_with?(ATTRIBUTE_ASSIGNMENT_SELECTOR_SUFFIX)
@@ -41,8 +35,6 @@ module Mutant
         # Test if AST node is an index assign
         #
         # @return [Boolean]
-        #
-        # @api private
         def index_assignment?
           selector.equal?(INDEX_ASSIGNMENT_SELECTOR)
         end
@@ -50,8 +42,6 @@ module Mutant
         # Test for binary operator implemented as method
         #
         # @return [Boolean]
-        #
-        # @api private
         def binary_method_operator?
           Types::BINARY_METHOD_OPERATORS.include?(selector)
         end
@@ -59,8 +49,6 @@ module Mutant
         # Test if receiver is possibly a top level constant
         #
         # @return [Boolean]
-        #
-        # @api private
         def receiver_possible_top_level_const?
           return false unless receiver && n_const?(receiver)
 

data/lib/mutant/ast/named_children.rb

@@ -9,8 +9,6 @@ module Mutant
       # @param [Class, Module] host
       #
       # @return [undefined]
-      #
-      # @api private
       def self.included(host)
         host.class_eval do
           include InstanceMethods
@@ -26,8 +24,6 @@ module Mutant
         # Mutated nodes children
         #
         # @return [Array<Parser::AST::Node]
-        #
-        # @api private
         def children
           node.children
         end
@@ -45,8 +41,6 @@ module Mutant
         # @param [Fixnum] index
         #
         # @return [undefined]
-        #
-        # @api private
         def define_named_child(name, index)
           define_method(name) do
             children.at(index)
@@ -58,8 +52,6 @@ module Mutant
         # @param [Array<Symbol>] names
         #
         # @return [undefined]
-        #
-        # @api private
         def define_remaining_children(names)
           define_method(:remaining_children_with_index) do
             children.each_with_index.drop(names.length)
@@ -77,8 +69,6 @@ module Mutant
         # Create name helpers
         #
         # @return [undefined]
-        #
-        # @api private
         def children(*names)
           names.each_with_index do |name, index|
             define_named_child(name, index)

data/lib/mutant/ast/sexp.rb

@@ -10,10 +10,8 @@ module Mutant
       # @param [Symbol] type
       #
       # @return [Parser::AST::Node]
-      #
-      # @api private
       def s(type, *children)
-        Parser::AST::Node.new(type, children)
+        ::Parser::AST::Node.new(type, children)
       end
 
       # Build a negated boolean node
@@ -21,8 +19,6 @@ module Mutant
       # @param [Parser::AST::Node] node
       #
       # @return [Parser::AST::Node]
-      #
-      # @api private
       def n_not(node)
         s(:send, node, :!)
       end

data/lib/mutant/cli.rb

@@ -1,37 +1,31 @@
 module Mutant
 
   # Commandline parser
+  #
+  # rubocop:disable ClassLength
   class CLI
     include Adamantium::Flat, Equalizer.new(:config), Procto.call(:config)
 
     # Error failed when CLI argv is invalid
     Error = Class.new(RuntimeError)
 
-    EXIT_FAILURE = 1
-    EXIT_SUCCESS = 0
-
     # Run cli with arguments
     #
     # @param [Array<String>] arguments
     #
-    # @return [Fixnum]
-    #   the exit status
-    #
-    # @api private
+    # @return [Boolean]
     def self.run(arguments)
-      Runner.call(Env::Bootstrap.call(call(arguments))).success? ? EXIT_SUCCESS : EXIT_FAILURE
+      Runner.call(Env::Bootstrap.call(call(arguments))).success?
     rescue Error => exception
       $stderr.puts(exception.message)
-      EXIT_FAILURE
+      false
     end
 
-    # Initialize objecct
+    # Initialize object
     #
     # @param [Array<String>]
     #
     # @return [undefined]
-    #
-    # @api private
     def initialize(arguments)
       @config = Config::DEFAULT
 
@@ -41,8 +35,6 @@ module Mutant
     # Config parsed from CLI
     #
     # @return [Config]
-    #
-    # @api private
     attr_reader :config
 
   private
@@ -56,12 +48,10 @@ module Mutant
     #   An error occurred while parsing the options.
     #
     # @return [undefined]
-    #
-    # @api private
     def parse(arguments)
       opts = OptionParser.new do |builder|
         builder.banner = 'usage: mutant [options] MATCH_EXPRESSION ...'
-        %w[add_environment_options add_mutation_options add_filter_options add_debug_options].each do |name|
+        %i[add_environment_options add_mutation_options add_filter_options add_debug_options].each do |name|
           __send__(name, builder)
         end
       end
@@ -76,8 +66,6 @@ module Mutant
     # @param [Array<String>] expressions
     #
     # @return [undefined]
-    #
-    # @api private
     def parse_match_expressions(expressions)
       fail Error, 'No expressions given' if expressions.empty?
 
@@ -93,8 +81,6 @@ module Mutant
     # @return [undefined]
     #
     # rubocop:disable MethodLength
-    #
-    # @api private
     def add_environment_options(opts)
       opts.separator('Environment:')
       opts.on('--zombie', 'Run mutant zombified') do
@@ -116,8 +102,6 @@ module Mutant
     # @param [String] name
     #
     # @return [undefined]
-    #
-    # @api private
     def setup_integration(name)
       with(integration: Integration.setup(name))
     rescue LoadError
@@ -129,8 +113,6 @@ module Mutant
     # @param [OptionParser] opts
     #
     # @return [undefined]
-    #
-    # @api private
     def add_mutation_options(opts)
       opts.separator(nil)
       opts.separator('Options:')
@@ -149,14 +131,21 @@ module Mutant
     # @param [OptionParser] opts
     #
     # @return [undefined]
-    #
-    # @api private
     def add_filter_options(opts)
       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)))
+        add_matcher(
+          :subject_filters,
+          Repository::SubjectFilter.new(
+            Repository::Diff.new(
+              config: config,
+              from:   Repository::Diff::HEAD,
+              to:     revision
+            )
+          )
+        )
       end
     end
 
@@ -165,22 +154,20 @@ module Mutant
     # @param [OptionParser] opts
     #
     # @return [undefined]
-    #
-    # @api private
     def add_debug_options(opts)
       opts.on('--fail-fast', 'Fail fast') do
         with(fail_fast: true)
       end
       opts.on('--version', 'Print mutants version') do
         puts("mutant-#{VERSION}")
-        Kernel.exit(EXIT_SUCCESS)
+        config.kernel.exit
       end
       opts.on('-d', '--debug', 'Enable debugging output') do
         with(debug: true)
       end
       opts.on_tail('-h', '--help', 'Show this message') do
         puts(opts.to_s)
-        Kernel.exit(EXIT_SUCCESS)
+        config.kernel.exit
       end
     end
 
@@ -189,8 +176,6 @@ module Mutant
     # @param [Hash<Symbol, Object>] attributes
     #
     # @return [undefined]
-    #
-    # @api private
     def with(attributes)
       @config = config.with(attributes)
     end
@@ -204,8 +189,6 @@ module Mutant
     #   the value to add
     #
     # @return [undefined]
-    #
-    # @api private
     def add(attribute, value)
       with(attribute => config.public_send(attribute) + [value])
     end
@@ -219,8 +202,6 @@ module Mutant
     #   the value to add
     #
     # @return [undefined]
-    #
-    # @api private
     def add_matcher(attribute, value)
       with(matcher: config.matcher.add(attribute, value))
     end