mutant 0.6.6 → 0.6.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f3fbd46aa447dc402152bb4199a0ecfb1f1e9c8e
-  data.tar.gz: f01ecbd6a0dc0fac2c5676ab51d957b7a64744b8
+  metadata.gz: 11e3b674e08512b58bd258c3ab1434bdf684d71a
+  data.tar.gz: f1c0559335e982f5348d6fb71c4ed2f7ae01b5b4
 SHA512:
-  metadata.gz: 04d88ed0b9914df3e8709711bd47a88390196a6316b385b07ea4a311b79b547936a5c3660b1ff85740fc8162dbe9c12ed463861695e9733ffcd87fe84eaebe56
-  data.tar.gz: 9702946488ff1c82f7a8ba2de538903acdc200ac161d33f016c37548d93594134243dd590c15f0647e5fed36b54b1cef7c3980a4af688b53e90801c9250ad5af
+  metadata.gz: d7decc3ff8888ea325265bfaf8ea098ee4656a9254978c72b0561997944cdf287d003a724ff9511ec2d15355e708b070f10c7d5561a107bfdfb5eec21bcd3263
+  data.tar.gz: 9e229f14f8b9b808ab72293ac15855627cd821cdf151413aa38dfd5fd3815c64c11eb37665cca847652c98db90c51f50ec53c2afa1c6e27ae2f9bf3185fd3094

data/Changelog.md

@@ -1,3 +1,8 @@
+# v0.6.7 2014-11-17
+
+* Fix duplicate neutral emit for memoized instance method subjects
+* Fix neutral error detection edge cases
+
 # v0.6.6 2014-11-11
 
 * Fix emitter to recurse into left and right of binary nodes.

data/Gemfile.devtools

@@ -8,6 +8,7 @@ group :development do
 
   platform :rbx do
     gem 'rubysl-singleton', '~> 2.0.0'
+    gem 'rubinius-coverage', '~> 2.0.1'
   end
 end
 
@@ -38,7 +39,7 @@ group :metrics do
   gem 'flay',      '~> 2.5.0'
   gem 'flog',      '~> 4.2.1'
   gem 'reek',      '~> 1.3.7'
-  gem 'rubocop',   '~> 0.26.1'
+  gem 'rubocop',   '~> 0.26.0'
   gem 'simplecov', '~> 0.7.1'
   gem 'yardstick', '~> 0.9.9'
 

data/config/flay.yml

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

data/lib/mutant/ast.rb

@@ -44,7 +44,7 @@ module Mutant
     # @api private
     #
     def self.find_last_path(node, &predicate)
-      raise ArgumentError, 'block expected' unless block_given?
+      fail ArgumentError, 'block expected' unless block_given?
       path = []
       walk(node, [node]) do |candidate, stack|
         if predicate.call(candidate, &predicate)

data/lib/mutant/cli.rb

@@ -22,7 +22,7 @@ module Mutant
     # @api private
     #
     def self.run(arguments)
-      Env.call(call(arguments)).success? ? EXIT_SUCCESS : EXIT_FAILURE
+      Runner.call(Env.new(call(arguments))).success? ? EXIT_SUCCESS : EXIT_FAILURE
     rescue Error => exception
       $stderr.puts(exception.message)
       EXIT_FAILURE

data/lib/mutant/context/scope.rb

@@ -45,12 +45,12 @@ module Mutant
       def self.wrap(scope, node)
         name = s(:const, nil, scope.name.split(NAMESPACE_DELIMITER).last.to_sym)
         case scope
-        when ::Class
+        when Class
           s(:class, name, nil, node)
-        when ::Module
+        when Module
           s(:module, name, node)
         else
-          raise "Cannot wrap scope: #{scope.inspect}"
+          fail "Cannot wrap scope: #{scope.inspect}"
         end
       end
 
@@ -79,16 +79,6 @@ module Mutant
         name_nesting.last
       end
 
-      # Return name
-      #
-      # @return [String]
-      #
-      # @api private
-      #
-      def name
-        scope.name
-      end
-
       # Return match expressions
       #
       # @return [Enumerable<Expression>]

data/lib/mutant/env.rb

@@ -1,7 +1,7 @@
 module Mutant
   # Abstract base class for mutant environments
   class Env
-    include Adamantium::Flat, Concord::Public.new(:config, :cache), Procto.call(:run)
+    include Adamantium::Flat, Concord::Public.new(:config, :cache)
 
     # Return new env
     #
@@ -30,16 +30,6 @@ module Mutant
       initialize_mutations
     end
 
-    # Run mutant producing a report on configured env
-    #
-    # @return [Report]
-    #
-    # @api private
-    #
-    def run
-      Runner.call(self)
-    end
-
     # Print warning message
     #
     # @param [String]
@@ -96,7 +86,7 @@ module Mutant
     def scope_name(scope)
       scope.name
     rescue => exception
-      warn("While obtaining #{scope.class}#name from: #{scope.inspect} It raised an error: #{exception.inspect} fix your lib!")
+      warn("#{scope.class}#name from: #{scope.inspect} raised an error: #{exception.inspect} fix your lib to follow normal ruby semantics!")
       nil
     end
 
@@ -116,7 +106,7 @@ module Mutant
       name = scope_name(scope) or return
 
       unless name.is_a?(String)
-        warn("#{scope.class}#name from: #{scope.inspect} did not return a String or nil. Fix your lib to support normal ruby semantics!")
+        warn("#{scope.class}#name from: #{scope.inspect} returned #{name.inspect} instead String or nil. Fix your lib to follow normal ruby semantics!")
         return
       end
 

data/lib/mutant/expression/method.rb

@@ -25,7 +25,7 @@ module Mutant
         methods_matcher = MATCHERS.fetch(scope_symbol).new(env, scope)
         method = methods_matcher.methods.detect do |meth|
           meth.name.equal?(method_name)
-        end or raise NameError, "Cannot find method #{method_name}"
+        end or fail NameError, "Cannot find method #{method_name}"
         methods_matcher.matcher.build(env, scope, method)
       end
 

data/lib/mutant/isolation.rb

@@ -40,7 +40,7 @@ module Mutant
       def self.call(&block)
         reader, writer = IO.pipe
 
-        pid = fork do
+        pid = Process.fork do
           File.open('/dev/null', 'w') do |file|
             $stderr.reopen(file)
             reader.close

data/lib/mutant/loader.rb

@@ -13,14 +13,13 @@ module Mutant
       # @api private
       #
       def call
-        subject.prepare
         eval(
           source,
           TOPLEVEL_BINDING,
           subject.source_path.to_s,
           subject.source_line
         )
-        nil
+        self
       end
 
     private

data/lib/mutant/meta/example/dsl.rb

@@ -40,7 +40,7 @@ module Mutant
         # @api private
         #
         def example
-          raise 'source not defined' unless @source
+          fail 'source not defined' unless @source
           Example.new(@file, @source, @expected)
         end
 
@@ -55,7 +55,7 @@ module Mutant
         # @api private
         #
         def source(input)
-          raise 'source already defined' if @source
+          fail 'source already defined' if @source
           @source = node(input)
 
           self
@@ -72,7 +72,7 @@ module Mutant
         def mutation(input)
           node = node(input)
           if @expected.include?(node)
-            raise "Node for input: #{input.inspect} is already expected"
+            fail "Node for input: #{input.inspect} is already expected"
           end
           @expected << node
 
@@ -108,7 +108,7 @@ module Mutant
           when Parser::AST::Node
             input
           else
-            raise "Cannot coerce to node: #{source.inspect}"
+            fail "Cannot coerce to node: #{source.inspect}"
           end
         end
 

data/lib/mutant/mutation.rb

@@ -7,17 +7,6 @@ module Mutant
     CODE_DELIMITER = "\0".freeze
     CODE_RANGE     = (0..4).freeze
 
-    # Return mutated root node
-    #
-    # @return [Parser::AST::Node]
-    #
-    # @api private
-    #
-    def root
-      subject.root(node)
-    end
-    memoize :root
-
     # Insert mutated node
     #
     # FIXME: Cache subject visibility in a better way! Ideally dont mutate it
@@ -30,6 +19,7 @@ module Mutant
     #
     def insert
       subject.public?
+      subject.prepare
       Loader::Eval.call(root, subject)
       self
     end
@@ -102,6 +92,16 @@ module Mutant
     end
     memoize :sha1
 
+    # Return mutated root node
+    #
+    # @return [Parser::AST::Node]
+    #
+    # @api private
+    #
+    def root
+      subject.context.root(node)
+    end
+
     # Evil mutation that should case mutations to fail tests
     class Evil < self
 

data/lib/mutant/mutator.rb

@@ -16,7 +16,7 @@ module Mutant
     #
     def self.each(input, parent = nil, &block)
       return to_enum(__method__, input, parent) unless block_given?
-      Registry.lookup(input).new(input, parent, block)
+      REGISTRY.lookup(input).new(input, parent, block)
 
       self
     end
@@ -29,7 +29,7 @@ module Mutant
     #
     def self.handle(*types)
       types.each do |type|
-        Registry.register(type, self)
+        REGISTRY.register(type, self)
       end
     end
     private_class_method :handle

data/lib/mutant/mutator/registry.rb

@@ -1,27 +1,34 @@
 module Mutant
   class Mutator
     # Registry for mutators
-    module Registry
+    class Registry
 
-      # Raised when the type is an invalid type
-      InvalidTypeError = Class.new(TypeError)
+      # Initialize object
+      #
+      # @return [undefined]
+      #
+      # @api private
+      #
+      def initialize
+        @registry = {}
+      end
 
-      # Raised when the type is a duplicate
-      DuplicateTypeError = Class.new(ArgumentError)
+      # Raised when the type is an invalid type
+      RegistryError = Class.new(TypeError)
 
       # Register mutator class for AST node class
       #
       # @param [Symbol] type
-      # @param [Class] mutator_class
+      # @param [Class:Mutator] mutator
       #
       # @api private
       #
       # @return [self]
       #
-      def self.register(type, mutator_class)
-        assert_valid_type(type)
-        assert_unique_type(type)
-        registry[type] = mutator_class
+      def register(type, mutator)
+        fail RegistryError, "Invalid type registration: #{type}" unless AST::Types::ALL.include?(type)
+        fail RegistryError, "Duplicate type registration: #{type}" if @registry.key?(type)
+        @registry[type] = mutator
         self
       end
 
@@ -36,58 +43,16 @@ module Mutant
       #
       # @api private
       #
-      def self.lookup(node)
+      def lookup(node)
         type = node.type
-        registry.fetch(type) do
-          raise ArgumentError, "No mutator to handle: #{type.inspect}"
+        @registry.fetch(type) do
+          fail RegistryError, "No mutator to handle: #{type.inspect}"
         end
       end
 
-      # Return registry state
-      #
-      # @return [Hash]
-      #
-      # @api private
-      #
-      def self.registry
-        @registry ||= {}
-      end
-      private_class_method :registry
-
-      # Assert the node type is valid
-      #
-      # @param [Symbol] type
-      #
-      # @return [undefined]
-      #
-      # @raise [InvalidTypeError]
-      #   raised when the node type is invalid
-      #
-      # @api private
-      #
-      def self.assert_valid_type(type)
-        unless AST::Types::ALL.include?(type) || type.is_a?(Class)
-          raise InvalidTypeError, "invalid type registration: #{type}"
-        end
-      end
-      private_class_method :assert_valid_type
+    end # Registry
 
-      # Assert the node type is unique and not already registered
-      #
-      # @return [undefined]
-      #
-      # @raise [DuplicateTypeError]
-      #   raised when the node type is a duplicate
-      #
-      # @api private
-      #
-      def self.assert_unique_type(type)
-        if registry.key?(type)
-          raise DuplicateTypeError, "duplicate type registration: #{type}"
-        end
-      end
-      private_class_method :assert_unique_type
+    REGISTRY = Registry.new
 
-    end # Registry
   end # Mutator
 end # Mutant

data/lib/mutant/mutator/util/array.rb

@@ -5,8 +5,6 @@ module Mutant
       # Mutators that mutates an array of inputs
       class Array < self
 
-        handle(::Array)
-
         # Element presence mutator
         class Presence < Util
 

data/lib/mutant/mutator/util/symbol.rb

@@ -5,8 +5,6 @@ module Mutant
       # Utility symbol mutator
       class Symbol < self
 
-        handle(::Symbol)
-
         POSTFIX = '__mutant__'.freeze
 
       private

data/lib/mutant/subject.rb

@@ -88,18 +88,6 @@ module Mutant
     end
     memoize :source
 
-    # Return root AST for node
-    #
-    # @param [Parser::AST::Node] node
-    #
-    # @return [Parser::AST::Node]
-    #
-    # @api private
-    #
-    def root(node)
-      context.root(node)
-    end
-
     # Return match expression
     #
     # @return [Expression]

data/lib/mutant/subject/method/instance.rb

@@ -9,11 +9,7 @@ module Mutant
 
         # A list of methods that will warn when they are undefined
         WARN_METHODS_UNDEFINED =
-          if RUBY_ENGINE.eql?('ruby')
-            [:initialize, :__send__, :object_id].freeze
-          else
-            EMPTY_ARRAY
-          end
+          RUBY_ENGINE.eql?('ruby') ? [:initialize, :__send__, :object_id].freeze : EMPTY_ARRAY
 
         # Test if method is public
         #
@@ -85,7 +81,6 @@ module Mutant
           # @api private
           #
           def generate_mutations(emitter)
-            emitter << neutral_mutation
             Mutator.each(node) do |mutant|
               emitter << Mutation::Evil.new(self, memoizer_node(mutant))
             end

data/lib/mutant/version.rb

@@ -1,4 +1,4 @@
 module Mutant
   # The current mutant version
-  VERSION = '0.6.6'.freeze
+  VERSION = '0.6.7'.freeze
 end # Mutant

data/spec/integration/mutant/corpus_spec.rb

@@ -1,7 +1,7 @@
 RSpec.describe 'Mutant on ruby corpus' do
 
   before do
-    skip 'Corpus test is deactivated on 1.9.3' if RUBY_VERSION.eql?('1.9.3')
+    skip 'Corpus test is deactivated on < 2.1' if RUBY_VERSION < '2.1'
     skip 'Corpus test is deactivated on RBX' if RUBY_ENGINE.eql?('rbx')
   end
 

data/spec/integration/mutant/test_mutator_handles_types_spec.rb

@@ -2,7 +2,7 @@ RSpec.describe do
 
   specify 'mutant should not crash for any node parser can generate' do
     Mutant::AST::Types::ALL.each do |type|
-      Mutant::Mutator::Registry.lookup(s(type))
+      Mutant::Mutator::REGISTRY.lookup(s(type))
     end
   end
 end

data/spec/unit/mutant/cli_spec.rb

@@ -18,14 +18,15 @@ RSpec.describe Mutant::CLI do
   describe '.run' do
     subject { object.run(arguments) }
 
-    let(:arguments) { double('arguments') }
-
-    let(:report) { double('Report', success?: report_success) }
-    let(:config) { double('Config') }
+    let(:arguments) { double('arguments')                        }
+    let(:report)    { double('Report', success?: report_success) }
+    let(:config)    { double('Config')                           }
+    let(:env)       { double('env')                              }
 
     before do
       expect(Mutant::CLI).to receive(:call).with(arguments).and_return(config)
-      expect(Mutant::Env).to receive(:call).with(config).and_return(report)
+      expect(Mutant::Env).to receive(:new).with(config).and_return(env)
+      expect(Mutant::Runner).to receive(:call).with(env).and_return(report)
     end
 
     context 'when report signalls success' do

data/spec/unit/mutant/context_spec.rb

@@ -0,0 +1,56 @@
+RSpec.describe Mutant::Context::Scope do
+  describe '.wrap' do
+    subject { described_class.wrap(scope, node) }
+
+    let(:node)  { s(:str, 'test') }
+
+    context 'with Module as scope' do
+      let(:scope) { Mutant          }
+
+      let(:expected) do
+        s(:module,
+          s(:const, nil, :Mutant),
+          s(:str, 'test')
+        )
+      end
+
+      it { should eql(expected) }
+    end
+
+    context 'with Class as scope' do
+      let(:scope) { Mutant::Context }
+
+      let(:expected) do
+        s(:class,
+          s(:const, nil, :Context),
+          nil,
+          s(:str, 'test')
+        )
+      end
+
+      it { should eql(expected) }
+    end
+
+    context 'with Class as scope' do
+      let(:scope) { Mutant::Context }
+
+      let(:expected) do
+        s(:class,
+          s(:const, nil, :Context),
+          nil,
+          s(:str, 'test')
+        )
+      end
+
+      it { should eql(expected) }
+    end
+
+    context 'with anything else as scope' do
+      let(:scope) { double(name: 'Foo') }
+
+      it 'raises exception' do
+        expect { subject }.to raise_error(RuntimeError, "Cannot wrap scope: #{scope.inspect}")
+      end
+    end
+  end
+end

data/spec/unit/mutant/env_spec.rb

@@ -0,0 +1,49 @@
+RSpec.describe Mutant::Env do
+  let(:config) { Mutant::Config::DEFAULT.update(jobs: 1, reporter: Mutant::Reporter::Trace.new) }
+
+  context '.new' do
+    subject { described_class.new(config) }
+
+    context 'when Module#name calls result in exceptions' do
+      it 'warns via reporter' do
+        klass = Class.new do
+          def self.name
+            raise
+          end
+        end
+
+        expected_warnings = ["Class#name from: #{klass} raised an error: RuntimeError fix your lib to follow normal ruby semantics!"]
+
+        expect { subject }.to change { config.reporter.warn_calls }.from([]).to(expected_warnings)
+
+        # Fix Class#name so other specs do not see this one
+        class << klass
+          undef :name
+          def name
+          end
+        end
+      end
+    end
+
+    context 'when Module#name does not return a String or nil' do
+      it 'warns via reporter' do
+        klass = Class.new do
+          def self.name
+            Object
+          end
+        end
+
+        expected_warnings = ["Class#name from: #{klass.inspect} returned #{Object.inspect} instead String or nil. Fix your lib to follow normal ruby semantics!"]
+
+        expect { subject }.to change { config.reporter.warn_calls }.from([]).to(expected_warnings)
+
+        # Fix Class#name so other specs do not see this one
+        class << klass
+          undef :name
+          def name
+          end
+        end
+      end
+    end
+  end
+end

data/spec/unit/mutant/isolation_spec.rb

@@ -63,5 +63,25 @@ RSpec.describe Mutant::Isolation::Fork do
         $stderr = STDERR
       end
     end
+
+    # Spec stubbing out the fork to ensure all lines are covered
+    # with expectations
+    it 'covers all lines' do
+      reader, writer = double('reader'), double('writer')
+      expect(IO).to receive(:pipe).ordered.and_return([reader, writer])
+      pid = double('PID')
+      expect(Process).to receive(:fork).ordered.and_yield.and_return(pid)
+      file = double('file')
+      expect(File).to receive(:open).ordered.with('/dev/null', 'w').and_yield(file)
+      expect($stderr).to receive(:reopen).ordered.with(file)
+      expect(reader).to receive(:close).ordered
+      expect(writer).to receive(:write).ordered.with(Marshal.dump(:foo))
+      expect(writer).to receive(:close).ordered
+      expect(writer).to receive(:close).ordered
+      expect(reader).to receive(:read).ordered.and_return(Marshal.dump(:foo))
+      expect(Process).to receive(:waitpid).with(pid)
+
+      expect(object.call { :foo }).to be(:foo)
+    end
   end
 end

data/spec/unit/mutant/loader/eval_spec.rb

@@ -10,10 +10,6 @@ RSpec.describe Mutant::Loader::Eval, '.call' do
     double('Subject', source_path: path, source_line: line)
   end
 
-  before do
-    expect(mutation_subject).to receive(:prepare).and_return(mutation_subject)
-  end
-
   let(:source) do
     <<-RUBY
       class SomeNamespace

data/spec/unit/mutant/mutation_spec.rb

@@ -4,9 +4,9 @@ RSpec.describe Mutant::Mutation do
     SYMBOL = 'test'.freeze
   end
 
-  let(:object)           { TestMutation.new(mutation_subject, Mutant::AST::Nodes::N_NIL)    }
-  let(:mutation_subject) { double('Subject', identification: 'subject', source: 'original') }
-  let(:node)             { double('Node')                                                   }
+  let(:object)           { TestMutation.new(mutation_subject, Mutant::AST::Nodes::N_NIL)                      }
+  let(:mutation_subject) { double('Subject', identification: 'subject', context: context, source: 'original') }
+  let(:context)          { double('Context')                                                                  }
 
   describe '#code' do
     subject { object.code }
@@ -24,6 +24,21 @@ RSpec.describe Mutant::Mutation do
     it_should_behave_like 'an idempotent method'
   end
 
+  describe '#insert' do
+    subject { object.insert }
+
+    let(:wrapped_node) { double('Wrapped Node') }
+
+    before do
+      expect(mutation_subject).to receive(:public?).ordered.and_return(true)
+      expect(mutation_subject).to receive(:prepare).ordered
+      expect(context).to receive(:root).ordered.with(s(:nil)).and_return(wrapped_node)
+      expect(Mutant::Loader::Eval).to receive(:call).ordered.with(wrapped_node, mutation_subject).and_return(nil)
+    end
+
+    it_should_behave_like 'a command method'
+  end
+
   describe '#source' do
     subject { object.source }
 

data/spec/unit/mutant/mutator/registry_spec.rb

@@ -0,0 +1,57 @@
+RSpec.describe Mutant::Mutator::Registry do
+  describe '#lookup' do
+    subject { Mutant::Mutator::REGISTRY.lookup(node) }
+
+    context 'on registred node' do
+      let(:node) { s(:true) }
+
+      it { should eql(Mutant::Mutator::Node::Literal::Boolean) }
+    end
+
+    context 'on unknown node' do
+      let(:node) { s(:unknown) }
+
+      it 'raises error' do
+        expect { subject }.to raise_error(described_class::RegistryError, "No mutator to handle: :unknown")
+      end
+    end
+  end
+
+  describe '#register' do
+    let(:object) { described_class.new }
+
+    let(:mutator) { double('Mutator') }
+
+    subject { object.register(type, mutator) }
+
+    context 'when registring an invalid node type' do
+      let(:type) { :invalid }
+
+      it 'raises error' do
+        expect { subject }.to raise_error(described_class::RegistryError, 'Invalid type registration: invalid')
+      end
+    end
+
+    context 'when registring a valid node type' do
+      let(:type) { :true }
+
+      it 'allows to lookup mutator' do
+        subject
+        expect(object.lookup(s(type))).to be(mutator)
+      end
+
+      it_behaves_like 'a command method'
+    end
+
+    context 'when duplicate the registration of a valid node type' do
+      let(:type) { :true }
+
+      it 'allows to lookup mutator' do
+        object.register(type, mutator)
+        expect { subject }.to raise_error(described_class::RegistryError, 'Duplicate type registration: true')
+      end
+
+      it_behaves_like 'a command method'
+    end
+  end
+end

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

@@ -85,6 +85,34 @@ RSpec.describe Mutant::Subject::Method::Instance do
 
     it { should eql("def foo\nend") }
   end
+
+  describe '#public?' do
+    subject { object.public? }
+
+    context 'when method is public' do
+      it { should be(true) }
+    end
+
+    context 'when method is private' do
+      before do
+        scope.class_eval do
+          private :foo
+        end
+      end
+
+      it { should be(false) }
+    end
+
+    context 'when method is protected' do
+      before do
+        scope.class_eval do
+          protected :foo
+        end
+      end
+
+      it { should be(false) }
+    end
+  end
 end
 
 RSpec.describe Mutant::Subject::Method::Instance::Memoized do
@@ -124,6 +152,20 @@ RSpec.describe Mutant::Subject::Method::Instance::Memoized do
     it_should_behave_like 'a command method'
   end
 
+  describe '#mutations' do
+    subject { object.mutations }
+
+    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))))),
+      ]
+    end
+
+    it { should eql(expected) }
+  end
+
   describe '#source' do
     subject { object.source }
 

data/spec/unit/mutant/subject_spec.rb

@@ -2,7 +2,11 @@ RSpec.describe Mutant::Subject do
   let(:class_under_test) do
     Class.new(described_class) do
       def expression
-        Mutant::Expression.parse('Test')
+        Mutant::Expression.parse('SubjectA')
+      end
+
+      def match_expressions
+        [expression] << Mutant::Expression.parse('SubjectB')
       end
     end
   end
@@ -34,7 +38,7 @@ RSpec.describe Mutant::Subject do
   describe '#identification' do
     subject { object.identification }
 
-    it { should eql('Test:source_path:source_line') }
+    it { should eql('SubjectA:source_path:source_line') }
   end
 
   describe '#prepare' do
@@ -43,6 +47,56 @@ RSpec.describe Mutant::Subject do
     it_should_behave_like 'a command method'
   end
 
+  describe '#tests' do
+    let(:config)      { Mutant::Config::DEFAULT.update(integration: integration)         }
+    let(:integration) { double('Integration', all_tests: all_tests)                      }
+    let(:test_a)      { double('test', expression: Mutant::Expression.parse('SubjectA')) }
+    let(:test_b)      { double('test', expression: Mutant::Expression.parse('SubjectB')) }
+    let(:test_c)      { double('test', expression: Mutant::Expression.parse('SubjectC')) }
+
+    subject { object.tests }
+
+    context 'without available tests' do
+      let(:all_tests) { [] }
+
+      it { should eql([]) }
+
+      it_should_behave_like 'an idempotent method'
+    end
+
+    context 'without qualifying tests' do
+      let(:all_tests) { [test_c] }
+
+      it { should eql([]) }
+
+      it_should_behave_like 'an idempotent method'
+    end
+
+    context 'with qualifying tests for first match expression' do
+      let(:all_tests) { [test_a] }
+
+      it { should eql([test_a]) }
+
+      it_should_behave_like 'an idempotent method'
+    end
+
+    context 'with qualifying tests for second match expression' do
+      let(:all_tests) { [test_b] }
+
+      it { should eql([test_b]) }
+
+      it_should_behave_like 'an idempotent method'
+    end
+
+    context 'with qualifying tests for the first and second match expression' do
+      let(:all_tests) { [test_a, test_b] }
+
+      it { should eql([test_a]) }
+
+      it_should_behave_like 'an idempotent method'
+    end
+  end
+
   describe '#node' do
     subject { object.node }
 

data/spec/unit/mutant/{warning_expectation.rb → warning_expectation_spec.rb}

@@ -49,7 +49,7 @@ RSpec.describe Mutant::WarningExpectation do
         let(:expected_warnings) { [warning_a] }
 
         it 'raises an expectation error' do
-          expect { subject }.to raise_error(Mutant::WarningExpectation::ExpectationError.new(expected_warnings))
+          expect { subject }.to raise_error(Mutant::WarningExpectation::ExpectationError.new([warning_b], expected_warnings))
         end
       end
 
@@ -64,9 +64,17 @@ RSpec.describe Mutant::WarningExpectation do
         let(:actual_warnings)   { [warning_b] }
 
         it 'raises an expectation error' do
-          expect { subject }.to raise_error(Mutant::WarningExpectation::ExpectationError.new([warning_b]))
+          expect { subject }.to raise_error(Mutant::WarningExpectation::ExpectationError.new([warning_b], expected_warnings))
         end
       end
     end
   end
 end
+
+RSpec.describe Mutant::WarningExpectation::ExpectationError do
+  describe '#message' do
+    subject { described_class.new(['unexpected-a'], ['expected-b']).message }
+
+    it { should eql('Unexpected warnings: ["unexpected-a"], expected: ["expected-b"]') }
+  end
+end

data/test_app/Gemfile.devtools

@@ -8,6 +8,7 @@ group :development do
 
   platform :rbx do
     gem 'rubysl-singleton', '~> 2.0.0'
+    gem 'rubinius-coverage', '~> 2.0.1'
   end
 end
 
@@ -38,7 +39,7 @@ group :metrics do
   gem 'flay',      '~> 2.5.0'
   gem 'flog',      '~> 4.2.1'
   gem 'reek',      '~> 1.3.7'
-  gem 'rubocop',   '~> 0.26.1'
+  gem 'rubocop',   '~> 0.26.0'
   gem 'simplecov', '~> 0.7.1'
   gem 'yardstick', '~> 0.9.9'
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mutant
 version: !ruby/object:Gem::Version
-  version: 0.6.6
+  version: 0.6.7
 platform: ruby
 authors:
 - Markus Schirp
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-11 00:00:00.000000000 Z
+date: 2014-11-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parser
@@ -470,7 +470,9 @@ files:
 - spec/unit/mutant/context/root_spec.rb
 - spec/unit/mutant/context/scope/root_spec.rb
 - spec/unit/mutant/context/scope/unqualified_name_spec.rb
+- spec/unit/mutant/context_spec.rb
 - spec/unit/mutant/diff_spec.rb
+- spec/unit/mutant/env_spec.rb
 - spec/unit/mutant/expression/method_spec.rb
 - spec/unit/mutant/expression/methods_spec.rb
 - spec/unit/mutant/expression/namespace/flat_spec.rb
@@ -491,6 +493,7 @@ files:
 - spec/unit/mutant/matcher/null_spec.rb
 - spec/unit/mutant/mutation_spec.rb
 - spec/unit/mutant/mutator/node_spec.rb
+- spec/unit/mutant/mutator/registry_spec.rb
 - spec/unit/mutant/reporter/cli_spec.rb
 - spec/unit/mutant/reporter/null_spec.rb
 - spec/unit/mutant/require_highjack_spec.rb
@@ -501,7 +504,7 @@ files:
 - 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.rb
+- spec/unit/mutant/warning_expectation_spec.rb
 - spec/unit/mutant/warning_filter_spec.rb
 - spec/unit/mutant_spec.rb
 - test_app/.rspec
@@ -549,7 +552,9 @@ test_files:
 - spec/unit/mutant/context/root_spec.rb
 - spec/unit/mutant/context/scope/root_spec.rb
 - spec/unit/mutant/context/scope/unqualified_name_spec.rb
+- spec/unit/mutant/context_spec.rb
 - spec/unit/mutant/diff_spec.rb
+- spec/unit/mutant/env_spec.rb
 - spec/unit/mutant/expression/method_spec.rb
 - spec/unit/mutant/expression/methods_spec.rb
 - spec/unit/mutant/expression/namespace/flat_spec.rb
@@ -570,6 +575,7 @@ test_files:
 - spec/unit/mutant/matcher/null_spec.rb
 - spec/unit/mutant/mutation_spec.rb
 - spec/unit/mutant/mutator/node_spec.rb
+- spec/unit/mutant/mutator/registry_spec.rb
 - spec/unit/mutant/reporter/cli_spec.rb
 - spec/unit/mutant/reporter/null_spec.rb
 - spec/unit/mutant/require_highjack_spec.rb
@@ -580,7 +586,7 @@ test_files:
 - 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.rb
+- spec/unit/mutant/warning_expectation_spec.rb
 - spec/unit/mutant/warning_filter_spec.rb
 - spec/unit/mutant_spec.rb
 has_rdoc: