transpec 0.2.6 → 1.0.0

data/spec/transpec/syntax/should_receive_spec.rb

@@ -14,32 +14,35 @@ module Transpec
           return ShouldReceive.new(
             node,
             ancestor_nodes,
-            source_rewriter
+            source_rewriter,
+            runtime_data
           )
         end
         fail 'No should_receive node is found!'
       end
 
-      let(:record) { should_receive_object.report.records.first }
+      let(:runtime_data) { nil }
 
-      before do
-        should_receive_object.context.stub(:non_monkey_patch_mock_available?).and_return(true)
-      end
+      let(:record) { should_receive_object.report.records.first }
 
       describe '#expectize!' do
         context 'when it is `subject.should_receive(:method)` form' do
           let(:source) do
             <<-END
-              it 'receives #foo' do
-                subject.should_receive(:foo)
+              describe 'example' do
+                it 'receives #foo' do
+                  subject.should_receive(:foo)
+                end
               end
             END
           end
 
           let(:expected_source) do
             <<-END
-              it 'receives #foo' do
-                expect(subject).to receive(:foo)
+              describe 'example' do
+                it 'receives #foo' do
+                  expect(subject).to receive(:foo)
+                end
               end
             END
           end
@@ -54,21 +57,95 @@ module Transpec
             record.original_syntax.should  == 'obj.should_receive(:message)'
             record.converted_syntax.should == 'expect(obj).to receive(:message)'
           end
+
+          context 'and #expect and #receive are not available in the context' do
+            context 'and the context is determinable statically' do
+              let(:source) do
+                <<-END
+                  describe 'example' do
+                    class TestRunner
+                      def run
+                        something = 'something'
+                        something.should_receive(:foo)
+                        something.foo
+                      end
+                    end
+
+                    it 'receives #foo' do
+                      TestRunner.new.run
+                    end
+                  end
+                END
+              end
+
+              context 'with runtime information' do
+                include_context 'dynamic analysis objects'
+
+                it 'raises InvalidContextError' do
+                  -> { should_receive_object.expectize! }.should raise_error(InvalidContextError)
+                end
+              end
+
+              context 'without runtime information' do
+                it 'raises InvalidContextError' do
+                  -> { should_receive_object.expectize! }.should raise_error(InvalidContextError)
+                end
+              end
+            end
+
+            context 'and the context is not determinable statically' do
+              let(:source) do
+                <<-END
+                  def my_eval(&block)
+                    Object.new.instance_eval(&block)
+                  end
+
+                  describe 'example' do
+                    it 'receives #foo' do
+                      my_eval do
+                        something = 'something'
+                        something.should_receive(:foo)
+                        something.foo
+                      end
+                    end
+                  end
+                END
+              end
+
+              context 'with runtime information' do
+                include_context 'dynamic analysis objects'
+
+                it 'raises InvalidContextError' do
+                  -> { should_receive_object.expectize! }.should raise_error(InvalidContextError)
+                end
+              end
+
+              context 'without runtime information' do
+                it 'does not raise InvalidContextError' do
+                  -> { should_receive_object.expectize! }.should_not raise_error
+                end
+              end
+            end
+          end
         end
 
         context 'when it is `subject.should_not_receive(:method)` form' do
           let(:source) do
             <<-END
-              it 'does not receive #foo' do
-                subject.should_not_receive(:foo)
+              describe 'example' do
+                it 'does not receive #foo' do
+                  subject.should_not_receive(:foo)
+                end
               end
             END
           end
 
           let(:expected_source) do
             <<-END
-              it 'does not receive #foo' do
-                expect(subject).not_to receive(:foo)
+              describe 'example' do
+                it 'does not receive #foo' do
+                  expect(subject).not_to receive(:foo)
+                end
               end
             END
           end
@@ -88,8 +165,10 @@ module Transpec
           context 'and "to_not" is passed as negative form' do
             let(:expected_source) do
             <<-END
-              it 'does not receive #foo' do
-                expect(subject).to_not receive(:foo)
+              describe 'example' do
+                it 'does not receive #foo' do
+                  expect(subject).to_not receive(:foo)
+                end
               end
             END
             end
@@ -139,22 +218,26 @@ module Transpec
         context 'when it is `subject.should_receive(:method).with do .. end` form' do
           let(:source) do
             <<-END
-              it 'receives #foo with 1' do
-                subject.should_receive(:foo).with do |arg|
-                  arg == 1
+              describe 'example' do
+                it 'receives #foo with 1' do
+                  subject.should_receive(:foo).with do |arg|
+                    arg == 1
+                  end
+                  subject.foo(1)
                 end
-                subject.foo(1)
               end
             END
           end
 
           let(:expected_source) do
             <<-END
-              it 'receives #foo with 1' do
-                expect(subject).to receive(:foo).with { |arg|
-                  arg == 1
-                }
-                subject.foo(1)
+              describe 'example' do
+                it 'receives #foo with 1' do
+                  expect(subject).to receive(:foo).with { |arg|
+                    arg == 1
+                  }
+                  subject.foo(1)
+                end
               end
             END
           end
@@ -169,22 +252,26 @@ module Transpec
         context 'when it is `subject.should_receive(:method).with(arg) do .. end` form' do
           let(:source) do
             <<-END
-              it 'receives #foo with 1' do
-                subject.should_receive(:foo).with(1) do |arg|
-                  do_some_substitute_implementation
+              describe 'example' do
+                it 'receives #foo with 1' do
+                  subject.should_receive(:foo).with(1) do |arg|
+                    do_some_substitute_implementation
+                  end
+                  subject.foo(1)
                 end
-                subject.foo(1)
               end
             END
           end
 
           let(:expected_source) do
             <<-END
-              it 'receives #foo with 1' do
-                expect(subject).to receive(:foo).with(1) do |arg|
-                  do_some_substitute_implementation
+              describe 'example' do
+                it 'receives #foo with 1' do
+                  expect(subject).to receive(:foo).with(1) do |arg|
+                    do_some_substitute_implementation
+                  end
+                  subject.foo(1)
                 end
-                subject.foo(1)
               end
             END
           end
@@ -226,22 +313,26 @@ module Transpec
         context 'when it is `subject.should_receive(:method) do .. end.once` form' do
           let(:source) do
             <<-END
-              it 'receives #foo with 1' do
-                subject.should_receive(:foo) do |arg|
-                  arg == 1
-                end.once
-                subject.foo(1)
+              describe 'example' do
+                it 'receives #foo with 1' do
+                  subject.should_receive(:foo) do |arg|
+                    arg == 1
+                  end.once
+                  subject.foo(1)
+                end
               end
             END
           end
 
           let(:expected_source) do
             <<-END
-              it 'receives #foo with 1' do
-                expect(subject).to receive(:foo) { |arg|
-                  arg == 1
-                }.once
-                subject.foo(1)
+              describe 'example' do
+                it 'receives #foo with 1' do
+                  expect(subject).to receive(:foo) { |arg|
+                    arg == 1
+                  }.once
+                  subject.foo(1)
+                end
               end
             END
           end
@@ -256,22 +347,26 @@ module Transpec
         context 'when it is `subject.should_receive(:method) do .. end` form' do
           let(:source) do
             <<-END
-              it 'receives #foo with 1' do
-                subject.should_receive(:foo) do |arg|
-                  expect(arg).to eq(1)
+              describe 'example' do
+                it 'receives #foo with 1' do
+                  subject.should_receive(:foo) do |arg|
+                    expect(arg).to eq(1)
+                  end
+                  subject.foo(1)
                 end
-                subject.foo(1)
               end
             END
           end
 
           let(:expected_source) do
             <<-END
-              it 'receives #foo with 1' do
-                expect(subject).to receive(:foo) do |arg|
-                  expect(arg).to eq(1)
+              describe 'example' do
+                it 'receives #foo with 1' do
+                  expect(subject).to receive(:foo) do |arg|
+                    expect(arg).to eq(1)
+                  end
+                  subject.foo(1)
                 end
-                subject.foo(1)
               end
             END
           end
@@ -285,16 +380,20 @@ module Transpec
         context 'when it is `SomeClass.any_instance.should_receive(:method)` form' do
           let(:source) do
             <<-END
-              it 'receives #foo' do
-                SomeClass.any_instance.should_receive(:foo)
+              describe 'example' do
+                it 'receives #foo' do
+                  SomeClass.any_instance.should_receive(:foo)
+                end
               end
             END
           end
 
           let(:expected_source) do
             <<-END
-              it 'receives #foo' do
-                expect_any_instance_of(SomeClass).to receive(:foo)
+              describe 'example' do
+                it 'receives #foo' do
+                  expect_any_instance_of(SomeClass).to receive(:foo)
+                end
               end
             END
           end
@@ -319,8 +418,10 @@ module Transpec
         context 'when it is `subject.should_receive(:method).any_number_of_times` form' do
           let(:source) do
             <<-END
-              it 'responds to #foo' do
-                subject.should_receive(:foo).any_number_of_times
+              describe 'example' do
+                it 'responds to #foo' do
+                  subject.should_receive(:foo).any_number_of_times
+                end
               end
             END
           end
@@ -331,8 +432,10 @@ module Transpec
         context 'when it is `subject.should_receive(:method).with(arg).any_number_of_times` form' do
           let(:source) do
             <<-END
-              it 'responds to #foo with 1' do
-                subject.should_receive(:foo).with(1).any_number_of_times
+              describe 'example' do
+                it 'responds to #foo with 1' do
+                  subject.should_receive(:foo).with(1).any_number_of_times
+                end
               end
             END
           end
@@ -343,8 +446,10 @@ module Transpec
         context 'when it is `subject.should_receive(:method).at_least(0)` form' do
           let(:source) do
             <<-END
-              it 'responds to #foo' do
-                subject.should_receive(:foo).at_least(0)
+              describe 'example' do
+                it 'responds to #foo' do
+                  subject.should_receive(:foo).at_least(0)
+                end
               end
             END
           end
@@ -355,8 +460,10 @@ module Transpec
         context 'when it is `subject.should_receive(:method).at_least(1)` form' do
           let(:source) do
             <<-END
-              it 'receives #foo at least once' do
-                subject.should_receive(:foo).with(1).at_least(1)
+              describe 'example' do
+                it 'receives #foo at least once' do
+                  subject.should_receive(:foo).with(1).at_least(1)
+                end
               end
             END
           end
@@ -367,8 +474,10 @@ module Transpec
         context 'when it is `subject.should_receive(:method)` form' do
           let(:source) do
             <<-END
-              it 'receives to #foo' do
-                subject.should_receive(:foo)
+              describe 'example' do
+                it 'receives to #foo' do
+                  subject.should_receive(:foo)
+                end
               end
             END
           end
@@ -378,61 +487,137 @@ module Transpec
       end
 
       describe '#allowize_useless_expectation!' do
-        before do
-          should_receive_object.allowize_useless_expectation!
-        end
-
         context 'when it is `subject.should_receive(:method).any_number_of_times` form' do
           let(:source) do
             <<-END
-              it 'responds to #foo' do
-                subject.should_receive(:foo).any_number_of_times
+              describe 'example' do
+                it 'responds to #foo' do
+                  subject.should_receive(:foo).any_number_of_times
+                end
               end
             END
           end
 
           let(:expected_source) do
             <<-END
-              it 'responds to #foo' do
-                allow(subject).to receive(:foo)
+              describe 'example' do
+                it 'responds to #foo' do
+                  allow(subject).to receive(:foo)
+                end
               end
             END
           end
 
           it 'converts into `allow(subject).to receive(:method)` form' do
+            should_receive_object.allowize_useless_expectation!
             rewritten_source.should == expected_source
           end
 
           it 'adds record ' +
              '"`obj.should_receive(:message).any_number_of_times` -> `allow(obj).to receive(:message)`"' do
+            should_receive_object.allowize_useless_expectation!
             record.original_syntax.should  == 'obj.should_receive(:message).any_number_of_times'
             record.converted_syntax.should == 'allow(obj).to receive(:message)'
           end
+
+          context 'and #allow and #receive are not available in the context' do
+            context 'and the context is determinable statically' do
+              let(:source) do
+                <<-END
+                  describe 'example' do
+                    class TestRunner
+                      def run
+                        'something'.should_receive(:foo).any_number_of_times
+                      end
+                    end
+
+                    it 'responds to #foo' do
+                      TestRunner.new.run
+                    end
+                  end
+                END
+              end
+
+              context 'with runtime information' do
+                include_context 'dynamic analysis objects'
+
+                it 'raises InvalidContextError' do
+                  -> { should_receive_object.allowize_useless_expectation! }
+                    .should raise_error(InvalidContextError)
+                end
+              end
+
+              context 'without runtime information' do
+                it 'raises InvalidContextError' do
+                  -> { should_receive_object.allowize_useless_expectation! }
+                    .should raise_error(InvalidContextError)
+                end
+              end
+            end
+
+            context 'and the context is not determinable statically' do
+              let(:source) do
+                <<-END
+                  def my_eval(&block)
+                    Object.new.instance_eval(&block)
+                  end
+
+                  describe 'example' do
+                    it 'responds to #foo' do
+                      my_eval { 'something'.should_receive(:foo).any_number_of_times }
+                    end
+                  end
+                END
+              end
+
+              context 'with runtime information' do
+                include_context 'dynamic analysis objects'
+
+                it 'raises InvalidContextError' do
+                  -> { should_receive_object.allowize_useless_expectation! }
+                    .should raise_error(InvalidContextError)
+                end
+              end
+
+              context 'without runtime information' do
+                it 'does not raise InvalidContextError' do
+                  -> { should_receive_object.allowize_useless_expectation! }
+                    .should_not raise_error
+                end
+              end
+            end
+          end
         end
 
         context 'when it is `SomeClass.any_instance.should_receive(:method).any_number_of_times` form' do
           let(:source) do
             <<-END
-              it 'responds to #foo' do
-                SomeClass.any_instance.should_receive(:foo).any_number_of_times
+              describe 'example' do
+                it 'responds to #foo' do
+                  SomeClass.any_instance.should_receive(:foo).any_number_of_times
+                end
               end
             END
           end
 
           let(:expected_source) do
             <<-END
-              it 'responds to #foo' do
-                allow_any_instance_of(SomeClass).to receive(:foo)
+              describe 'example' do
+                it 'responds to #foo' do
+                  allow_any_instance_of(SomeClass).to receive(:foo)
+                end
               end
             END
           end
 
           it 'converts into `allow_any_instance_of(subject).to receive(:method)` form' do
+            should_receive_object.allowize_useless_expectation!
             rewritten_source.should == expected_source
           end
 
           it 'adds record "`SomeClass.any_instance.should_receive(:message).any_number_of_times` ' +
              '-> `allow_any_instance_of(SomeClass).to receive(:message)`"' do
+            should_receive_object.allowize_useless_expectation!
             record.original_syntax.should  == 'SomeClass.any_instance.should_receive(:message).any_number_of_times'
             record.converted_syntax.should == 'allow_any_instance_of(SomeClass).to receive(:message)'
           end
@@ -441,26 +626,32 @@ module Transpec
         context 'when it is `subject.should_receive(:method).at_least(0)` form' do
           let(:source) do
             <<-END
-              it 'responds to #foo' do
-                subject.should_receive(:foo).at_least(0)
+              describe 'example' do
+                it 'responds to #foo' do
+                  subject.should_receive(:foo).at_least(0)
+                end
               end
             END
           end
 
           let(:expected_source) do
             <<-END
-              it 'responds to #foo' do
-                allow(subject).to receive(:foo)
+              describe 'example' do
+                it 'responds to #foo' do
+                  allow(subject).to receive(:foo)
+                end
               end
             END
           end
 
           it 'converts into `allow(subject).to receive(:method)` form' do
+            should_receive_object.allowize_useless_expectation!
             rewritten_source.should == expected_source
           end
 
           it 'adds record ' +
              '"`obj.should_receive(:message).at_least(0)` -> `allow(obj).to receive(:message)`"' do
+            should_receive_object.allowize_useless_expectation!
             record.original_syntax.should  == 'obj.should_receive(:message).at_least(0)'
             record.converted_syntax.should == 'allow(obj).to receive(:message)'
           end
@@ -469,26 +660,32 @@ module Transpec
         context 'when it is `SomeClass.any_instance.should_receive(:method).at_least(0)` form' do
           let(:source) do
             <<-END
-              it 'responds to #foo' do
-                SomeClass.any_instance.should_receive(:foo).at_least(0)
+              describe 'example' do
+                it 'responds to #foo' do
+                  SomeClass.any_instance.should_receive(:foo).at_least(0)
+                end
               end
             END
           end
 
           let(:expected_source) do
             <<-END
-              it 'responds to #foo' do
-                allow_any_instance_of(SomeClass).to receive(:foo)
+              describe 'example' do
+                it 'responds to #foo' do
+                  allow_any_instance_of(SomeClass).to receive(:foo)
+                end
               end
             END
           end
 
           it 'converts into `allow_any_instance_of(subject).to receive(:method)` form' do
+            should_receive_object.allowize_useless_expectation!
             rewritten_source.should == expected_source
           end
 
           it 'adds record "`SomeClass.any_instance.should_receive(:message).at_least(0)` ' +
              '-> `allow_any_instance_of(SomeClass).to receive(:message)`"' do
+            should_receive_object.allowize_useless_expectation!
             record.original_syntax.should  == 'SomeClass.any_instance.should_receive(:message).at_least(0)'
             record.converted_syntax.should == 'allow_any_instance_of(SomeClass).to receive(:message)'
           end
@@ -497,13 +694,16 @@ module Transpec
         context 'when it is `subject.should_receive(:method)` form' do
           let(:source) do
             <<-END
-              it 'receives to #foo' do
-                subject.should_receive(:foo)
+              describe 'example' do
+                it 'receives to #foo' do
+                  subject.should_receive(:foo)
+                end
               end
             END
           end
 
           it 'does nothing' do
+            should_receive_object.allowize_useless_expectation!
             rewritten_source.should == source
           end
         end
@@ -517,16 +717,20 @@ module Transpec
         context 'when it is `subject.should_receive(:method).any_number_of_times` form' do
           let(:source) do
             <<-END
-              it 'responds to #foo' do
-                subject.should_receive(:foo).any_number_of_times
+              describe 'example' do
+                it 'responds to #foo' do
+                  subject.should_receive(:foo).any_number_of_times
+                end
               end
             END
           end
 
           let(:expected_source) do
             <<-END
-              it 'responds to #foo' do
-                subject.stub(:foo)
+              describe 'example' do
+                it 'responds to #foo' do
+                  subject.stub(:foo)
+                end
               end
             END
           end
@@ -545,8 +749,10 @@ module Transpec
         context 'when it is `subject.should_receive(:method)` form' do
           let(:source) do
             <<-END
-              it 'receives to #foo' do
-                subject.should_receive(:foo)
+              describe 'example' do
+                it 'receives to #foo' do
+                  subject.should_receive(:foo)
+                end
               end
             END
           end