transpec 1.10.4 → 1.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 10bda7931ceab8672bcf49f68c925b59448543b4
-  data.tar.gz: acf1950a6c9b917d2af8fc16ea292c7ec91aa2b8
+  metadata.gz: 2a58d3e3885da1a031fbcaded738c876e1a69ae7
+  data.tar.gz: 408e608704f0245c39dbb42ea308c909174b13f9
 SHA512:
-  metadata.gz: 02b7167fc857ccb4956f439dd793eeae452d2e71e0666be7c6419f571e6b4facb026cd5a649f41fb07b1bf87ac493177deed91e1ee51945442f191d097689c93
-  data.tar.gz: 1ce039438135ccbccdeecc92b73812650d86a811671a9d7e2ec67f1907912c04b59efe753f2394aa434911e25d6f9bde82c9614aa40d5da3376b5f5cab45087f
+  metadata.gz: 8950975c78264dfc28fccb498c9e669c3a69216f32f3d669148646d2d69858c0fd41919985a240c2660aba96f0686cdf05adf07bcfa469994117cf2329d674a5
+  data.tar.gz: a33005b1b484f2e8dab7b9c18d07da52838c3db3876d2f8b2100974a553dd287dff8f36355392e7242da5ba5fa0ddc6471fe6dd9bb48cf2c17e42ab5dc75cfee

data/.rubocop.yml

@@ -1,8 +1,8 @@
 
 AllCops:
-  Includes:
+  Include:
     - tasks/*.rake
-  Excludes:
+  Exclude:
     - tmp/*
     - vendor/*
 
@@ -54,4 +54,4 @@ BracesAroundHashParameters:
 
 # TODO: Shorten to 100.
 ClassLength:
-  Max: 140
+  Max: 137

data/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 ## Development
 
+## v1.11.0
+
+* Support conversion to non-monkey-patch example groups ([#22](https://github.com/yujinakayama/transpec/issues/22))
+* Support conversion of `unstub` ([#49](https://github.com/yujinakayama/transpec/issues/49))
+* Modify `yield_receiver_to_any_instance_implementation_blocks` configuration in `RSpec.configure` only when necessary ([#56](https://github.com/yujinakayama/transpec/issues/56))
+* Modify only the main `RSpec.configure` when there are multiple ones ([#56](https://github.com/yujinakayama/transpec/issues/56))
+* Add `-v/--convert` option that allows to enable conversions that are disabled by default
+* Deprecate `-t/--convert-stub-with-hash` option
+
 ## v1.10.4
 
 * Avoid confusing methods defined with `#let` or `#subject` by user with RSpec built-in methods ([#55](https://github.com/yujinakayama/transpec/issues/55))

data/README.md

@@ -159,7 +159,9 @@ And you are done!
 
 ## Advanced Usage
 
-You can pass `transpec` arbitrary paths to be converted:
+### Convert only specific specs
+
+You can pass `transpec` arbitrary paths to convert:
 
 ```bash
 # You always need to be in the project root directory
@@ -178,9 +180,21 @@ $ transpec spec/baz_spec.rb
 Note that the current working directory always needs to be the project root directory,
 so that Transpec can copy the project in dynamic analysis.
 
+### Enable/disable specific conversions
+
+You can disable specific conversions that are enabled by default with `-k/--keep` option,
+and enable conversions that are disabled by default with the `-v/--convert` option.
+
+```bash
+$ transpec --keep its --convert example_group
+```
+
+See [`-k/--keep`](#-k--keep) and [`-v/--convert`](#-v--convert) for more details.
+
 ## Upgrade Process to RSpec 3 beta
 
-If you are going to use Transpec in the upgrade process to RSpec 3 beta, read the article by [Myron Marston](https://github.com/myronmarston):
+If you are going to use Transpec in the upgrade process to RSpec 3 beta,
+read this article by [Myron Marston](https://github.com/myronmarston), who is the lead maintainer of RSpec:
 
 * [Myron Marston » RSpec 2.99 and 3.0 betas have been released!](http://myronmars.to/n/dev-blog/2013/11/rspec-2-99-and-3-0-betas-have-been-released)
 
@@ -200,10 +214,6 @@ Copying project for dynamic analysis...
 Running dynamic analysis with command "bundle exec rspec"...
 ```
 
-### `-s/--skip-dynamic-analysis`
-
-Skip dynamic analysis and convert with only static analysis. Note that specifying this option decreases the conversion accuracy especially in the conversion of [`have(n).items` matcher](#havenitems-matcher).
-
 ### `-c/--rspec-command`
 
 Specify a command to run your specs which is used for dynamic analysis.
@@ -230,7 +240,7 @@ Keep specific syntaxes by disabling conversions.
 $ transpec --keep should_receive,stub
 ```
 
-#### Available syntax types
+#### Conversions enabled by default
 
 Note that some syntaxes are available only if your project's RSpec is specific version or later.
 If they are unavailable, conversions for such syntaxes will be disabled automatically.
@@ -248,6 +258,38 @@ Type             | Target Syntax                  | Converted Syntax
 
 See [Supported Conversions](#supported-conversions) for more details.
 
+
+### `-v/--convert`
+
+Enable specific conversions that are disabled by default.
+
+```bash
+$ transpec --convert example_group
+```
+
+#### Conversions disabled by default
+
+Most of these target syntaxes are _not_ deprecated in both RSpec 2 and 3,
+but the new syntaxes provide more modern and clear ways.
+
+Type             | Target Syntax                  | Converted Syntax
+-----------------|--------------------------------|----------------------------------------------------
+`example_group`  | `describe 'something' { }`     | `RSpec.describe 'something' { }`
+`stub_with_hash` | `obj.stub(:message => value)`  | `allow(obj).to receive(:message).and_return(value)`
+
+Note: Specifying `stub_with_hash` enables conversion of `obj.stub(:message => value)`
+to `allow(obj).to receive(:message).and_return(value)`
+when `allow(obj).to receive_messages(:message => value)` is unavailable (prior to RSpec 3.0),
+and it will be converted to multiple statements if the hash includes multiple pairs.
+If your project's RSpec is 3.0 or later, it will be converted to `receive_messages(:message => value)`
+regardless of this option.
+
+See [Supported Conversions - Method stubs with a hash argument](#method-stubs-with-a-hash-argument) for more details.
+
+### `-s/--skip-dynamic-analysis`
+
+Skip dynamic analysis and convert with only static analysis. Note that specifying this option decreases the conversion accuracy especially in the conversion of [`have(n).items` matcher](#havenitems-matcher).
+
 ### `-n/--negative-form`
 
 Specify a negative form of `to` which is used in the `expect` syntax.
@@ -281,14 +323,6 @@ Note that this is not same as `--keep deprecated` since this configures `yield_r
 
 See [Supported Conversions - `any_instance` implementation blocks](#any_instance-implementation-blocks) for more details.
 
-### `-t/--convert-stub-with-hash`
-
-Enable conversion of `obj.stub(:message => value)` to `allow(obj).to receive(:message).and_return(value)` when `receive_messages(:message => value)` is unavailable (prior to RSpec 3.0).
-It will be converted to multiple statements if the hash includes multiple pairs.
-This conversion is disabled by default.
-
-See [Supported Conversions - Method stubs with a hash argument](#method-stubs-with-a-hash-argument) for more details.
-
 ### `-p/--no-parentheses-matcher-arg`
 
 Suppress parenthesizing arguments of matchers when converting
@@ -442,6 +476,7 @@ The one-liner (implicit receiver) `should`:
 * [Pending examples](#pending-examples)
 * [Current example object](#current-example-object)
 * [Custom matcher DSL](#custom-matcher-dsl)
+* [Example Groups](#example-groups)
 
 ### Standard expectations
 
@@ -809,6 +844,9 @@ obj.stub!(:message)
 obj.stub_chain(:foo, :bar, :baz)
 
 Klass.any_instance.stub(:message)
+
+obj.unstub(:message)
+obj.unstub!(:message)
 ```
 
 Will be converted to:
@@ -821,9 +859,9 @@ allow(obj).to receive(:message)
 allow(obj).to receive_message_chain(:foo, :bar, :baz)
 
 allow_any_instance_of(Klass).to receive(:message)
-```
 
-There's no replacement for `unstub` in the `expect` syntax. See [this discussion](https://github.com/rspec/rspec-mocks/issues/153#issuecomment-12208638) for more details.
+allow(obj).to receive(:message).and_call_original
+```
 
 * This conversion can be disabled by: `--keep stub`
 * Deprecation: deprecated since RSpec 3.0
@@ -882,7 +920,7 @@ Will be converted to:
 
 ```ruby
 obj.stub(:message) # with `--keep stub`
-obj.unstub(:message)
+obj.unstub(:message) # with `--keep stub`
 ```
 
 * This conversion can be disabled by: `--keep deprecated`
@@ -1191,6 +1229,46 @@ end
 * Deprecation: deprecated since RSpec 3.0
 * See also: [Expectations: Matcher protocol and custom matcher API changes - The Plan for RSpec 3](http://myronmars.to/n/dev-blog/2013/07/the-plan-for-rspec-3#expectations_matcher_protocol_and_custom_matcher_api_changes)
 
+### Example Groups
+
+**This conversion is disabled by default and available only if your project's RSpec is `3.0.0.beta2` or later.**
+
+Targets:
+
+```ruby
+RSpec.configure do |rspec|
+end
+
+describe 'top-level example group' do
+  describe 'nested example group' do
+  end
+end
+
+shared_examples 'shared examples' do
+end
+```
+
+Will be converted to:
+
+```ruby
+RSpec.configure do |rspec|
+  rspec.expose_dsl_globally = false
+end
+
+RSpec.describe 'top-level example group' do
+  describe 'nested example group' do
+  end
+end
+
+RSpec.shared_examples 'shared examples' do
+end
+```
+
+* This conversion can be enabled by: `--convert example_group`
+* Deprecation: Not deprecated
+* See also: [Zero Monkey Patching Mode! - The Plan for RSpec 3](http://myronmars.to/n/dev-blog/2013/07/the-plan-for-rspec-3#zero_monkey_patching_mode)
+
+
 ## Compatibility
 
 Tested on MRI 1.9, 2.0, 2.1 and JRuby in 1.9 mode.

data/README.md.erb

@@ -65,7 +65,7 @@ END
 Transpec would convert it to the following form:
 
 ```ruby
-<%= Transpec::Converter.new.convert(example) -%>
+<%= convert(example) -%>
 ```
 
 ### Actual examples
@@ -133,7 +133,9 @@ And you are done!
 
 ## Advanced Usage
 
-You can pass `transpec` arbitrary paths to be converted:
+### Convert only specific specs
+
+You can pass `transpec` arbitrary paths to convert:
 
 ```bash
 # You always need to be in the project root directory
@@ -152,9 +154,21 @@ $ transpec spec/baz_spec.rb
 Note that the current working directory always needs to be the project root directory,
 so that Transpec can copy the project in dynamic analysis.
 
+### Enable/disable specific conversions
+
+You can disable specific conversions that are enabled by default with `-k/--keep` option,
+and enable conversions that are disabled by default with the `-v/--convert` option.
+
+```bash
+$ transpec --keep its --convert example_group
+```
+
+See [`-k/--keep`](#-k--keep) and [`-v/--convert`](#-v--convert) for more details.
+
 ## Upgrade Process to RSpec 3 beta
 
-If you are going to use Transpec in the upgrade process to RSpec 3 beta, read the article by [Myron Marston](https://github.com/myronmarston):
+If you are going to use Transpec in the upgrade process to RSpec 3 beta,
+read this article by [Myron Marston](https://github.com/myronmarston), who is the lead maintainer of RSpec:
 
 * [Myron Marston » RSpec 2.99 and 3.0 betas have been released!](http://myronmars.to/n/dev-blog/2013/11/rspec-2-99-and-3-0-betas-have-been-released)
 
@@ -174,10 +188,6 @@ Copying project for dynamic analysis...
 Running dynamic analysis with command "bundle exec rspec"...
 ```
 
-### `-s/--skip-dynamic-analysis`
-
-Skip dynamic analysis and convert with only static analysis. Note that specifying this option decreases the conversion accuracy especially in the conversion of [`have(n).items` matcher](#havenitems-matcher).
-
 ### `-c/--rspec-command`
 
 Specify a command to run your specs which is used for dynamic analysis.
@@ -204,7 +214,7 @@ Keep specific syntaxes by disabling conversions.
 $ transpec --keep should_receive,stub
 ```
 
-#### Available syntax types
+#### Conversions enabled by default
 
 Note that some syntaxes are available only if your project's RSpec is specific version or later.
 If they are unavailable, conversions for such syntaxes will be disabled automatically.
@@ -223,22 +233,50 @@ conversion_type_table = <<END
 `deprecated`     | All other deprecated syntaxes  | Latest syntaxes
 END
 
-types_in_readme = conversion_type_table.lines.map do |line|
-  first_column = line.split('|').first
-  first_column.gsub(/[^\w]/, '').to_sym
-end.sort
+validate_syntax_type_table(conversion_type_table, Transpec::OptionParser::CONFIG_ATTRS_FOR_KEEP_TYPES.keys)
+conversion_type_table
+-%>
 
-types_in_code = Transpec::OptionParser.available_conversion_types.sort
+See [Supported Conversions](#supported-conversions) for more details.
 
-unless types_in_readme == types_in_code
-  types_missing_description = types_in_code - types_in_readme
-  fail "No descriptions for syntax types #{types_missing_description}"
-end
 
+### `-v/--convert`
+
+Enable specific conversions that are disabled by default.
+
+```bash
+$ transpec --convert example_group
+```
+
+#### Conversions disabled by default
+
+Most of these target syntaxes are _not_ deprecated in both RSpec 2 and 3,
+but the new syntaxes provide more modern and clear ways.
+
+Type             | Target Syntax                  | Converted Syntax
+-----------------|--------------------------------|----------------------------------------------------
+<%=
+conversion_type_table = <<END
+`example_group`  | `describe 'something' { }`     | `RSpec.describe 'something' { }`
+`stub_with_hash` | `obj.stub(:message => value)`  | `allow(obj).to receive(:message).and_return(value)`
+END
+
+validate_syntax_type_table(conversion_type_table, Transpec::OptionParser::CONFIG_ATTRS_FOR_CONVERT_TYPES.keys)
 conversion_type_table
 -%>
 
-See [Supported Conversions](#supported-conversions) for more details.
+Note: Specifying `stub_with_hash` enables conversion of `obj.stub(:message => value)`
+to `allow(obj).to receive(:message).and_return(value)`
+when `allow(obj).to receive_messages(:message => value)` is unavailable (prior to RSpec 3.0),
+and it will be converted to multiple statements if the hash includes multiple pairs.
+If your project's RSpec is 3.0 or later, it will be converted to `receive_messages(:message => value)`
+regardless of this option.
+
+See [Supported Conversions - Method stubs with a hash argument](#method-stubs-with-a-hash-argument) for more details.
+
+### `-s/--skip-dynamic-analysis`
+
+Skip dynamic analysis and convert with only static analysis. Note that specifying this option decreases the conversion accuracy especially in the conversion of [`have(n).items` matcher](#havenitems-matcher).
 
 ### `-n/--negative-form`
 
@@ -273,14 +311,6 @@ Note that this is not same as `--keep deprecated` since this configures `yield_r
 
 See [Supported Conversions - `any_instance` implementation blocks](#any_instance-implementation-blocks) for more details.
 
-### `-t/--convert-stub-with-hash`
-
-Enable conversion of `obj.stub(:message => value)` to `allow(obj).to receive(:message).and_return(value)` when `receive_messages(:message => value)` is unavailable (prior to RSpec 3.0).
-It will be converted to multiple statements if the hash includes multiple pairs.
-This conversion is disabled by default.
-
-See [Supported Conversions - Method stubs with a hash argument](#method-stubs-with-a-hash-argument) for more details.
-
 ### `-p/--no-parentheses-matcher-arg`
 
 Suppress parenthesizing arguments of matchers when converting
@@ -291,7 +321,7 @@ when parentheses are necessary to keep the meaning of the expression.
 
 ```ruby
 <%=
-parenthesizing_example = <<END
+example = <<END
 describe 'original spec' do
   it 'is an example' do
     1.should == 1
@@ -304,18 +334,12 @@ end
 END
 -%>
 
-<%=
-Transpec::Converter.new.convert(parenthesizing_example).gsub(
-  'original spec',
-  'converted spec'
-)
--%>
+<%= convert(example).gsub('original spec', 'converted spec') -%>
 
 <%=
 configuration = Transpec::Configuration.new
 configuration.parenthesize_matcher_arg = false
-converter = Transpec::Converter.new(configuration)
-converter.convert(parenthesizing_example)
+convert(example, configuration)
   .gsub(
     'original spec',
     'converted spec with -p/--no-parentheses-matcher-arg option'
@@ -664,7 +688,7 @@ Targets:
 
 ```ruby
 <%=
-its_target = <<END
+example = <<END
 describe 'example' do
   subject { { foo: 1, bar: 2 } }
   its(:size) { should == 2 }
@@ -678,7 +702,7 @@ END
 Will be converted to:
 
 ```ruby
-<%= Transpec::Converter.new.convert(its_target) -%>
+<%= convert(example) -%>
 ```
 
 There's an option to continue using `its` with [rspec-its](https://github.com/rspec/rspec-its) which is a gem extracted from `rspec-core`.
@@ -772,6 +796,9 @@ obj.stub!(:message)
 obj.stub_chain(:foo, :bar, :baz)
 
 Klass.any_instance.stub(:message)
+
+obj.unstub(:message)
+obj.unstub!(:message)
 ```
 
 Will be converted to:
@@ -784,9 +811,9 @@ allow(obj).to receive(:message)
 allow(obj).to receive_message_chain(:foo, :bar, :baz)
 
 allow_any_instance_of(Klass).to receive(:message)
-```
 
-There's no replacement for `unstub` in the `expect` syntax. See [this discussion](https://github.com/rspec/rspec-mocks/issues/153#issuecomment-12208638) for more details.
+allow(obj).to receive(:message).and_call_original
+```
 
 * This conversion can be disabled by: `--keep stub`
 * Deprecation: deprecated since RSpec 3.0
@@ -845,7 +872,7 @@ Will be converted to:
 
 ```ruby
 obj.stub(:message) # with `--keep stub`
-obj.unstub(:message)
+obj.unstub(:message) # with `--keep stub`
 ```
 
 * This conversion can be disabled by: `--keep deprecated`
@@ -878,7 +905,7 @@ Targets:
 
 ```ruby
 <%=
-useless_and_return_target = <<END
+example = <<END
 expect(obj).to receive(:message).and_return { 1 }
 allow(obj).to receive(:message).and_return { 1 }
 
@@ -891,7 +918,7 @@ END
 Will be converted to:
 
 ```ruby
-<%= Transpec::Converter.new.convert(useless_and_return_target) -%>
+<%= convert(example) -%>
 ```
 
 * This conversion can be disabled by: `--keep deprecated`
@@ -906,7 +933,7 @@ Targets:
 
 ```ruby
 <%=
-any_instance_block_target = <<END
+example = <<END
 RSpec.configure do |rspec|
 end
 
@@ -927,7 +954,7 @@ Will be converted to:
 ```ruby
 <%=
 rspec_version = Transpec::RSpecVersion.yielding_receiver_to_any_instance_implementation_block_available_version
-Transpec::Converter.new(nil, rspec_version).convert(any_instance_block_target)
+convert(example, nil, rspec_version)
 -%>
 
 # With `--no-yield-any-instance`
@@ -935,7 +962,7 @@ Transpec::Converter.new(nil, rspec_version).convert(any_instance_block_target)
 configuration = Transpec::Configuration.new
 configuration.add_receiver_arg_to_any_instance_implementation_block = false
 rspec_version = Transpec::RSpecVersion.yielding_receiver_to_any_instance_implementation_block_available_version
-Transpec::Converter.new(configuration, rspec_version).convert(any_instance_block_target)
+convert(example, configuration, rspec_version)
 -%>
 ```
 
@@ -989,7 +1016,7 @@ Targets:
 
 ```ruby
 <%=
-pending_example_target = <<END
+example = <<END
 describe 'example' do
   it 'is skipped', :pending => true do
     do_something_possibly_fail # This won't be run
@@ -1019,7 +1046,7 @@ Will be converted to:
 ```ruby
 <%=
 rspec_version = Transpec::RSpecVersion.skip_available_version
-Transpec::Converter.new(nil, rspec_version).convert(pending_example_target)
+convert(example, nil, rspec_version)
   .gsub('pending', 'pending # #pending with block is no longer supported')
 -%>
 ```
@@ -1048,7 +1075,7 @@ Targets:
 
 ```ruby
 <%=
-example_target = <<END
+example = <<END
 module ScreenshotHelper
   def save_failure_screenshot
     return unless example.exception
@@ -1071,7 +1098,7 @@ Will be converted to:
 ```ruby
 <%=
 rspec_version = Transpec::RSpecVersion.yielded_example_available_version
-Transpec::Converter.new(nil, rspec_version).convert(example_target)
+convert(example, nil, rspec_version)
 -%>
 ```
 
@@ -1103,7 +1130,7 @@ Targets:
 
 ```ruby
 <%=
-custom_matcher_dsl_target = <<END
+example = <<END
 RSpec::Matchers.define :be_awesome do
   match_for_should { }
   match_for_should_not { }
@@ -1119,7 +1146,7 @@ Will be converted to:
 ```ruby
 <%=
 rspec_version = Transpec::RSpecVersion.non_should_matcher_protocol_available_version
-Transpec::Converter.new(nil, rspec_version).convert(custom_matcher_dsl_target)
+convert(example, nil, rspec_version)
 -%>
 ```
 
@@ -1127,6 +1154,45 @@ Transpec::Converter.new(nil, rspec_version).convert(custom_matcher_dsl_target)
 * Deprecation: deprecated since RSpec 3.0
 * See also: [Expectations: Matcher protocol and custom matcher API changes - The Plan for RSpec 3](http://myronmars.to/n/dev-blog/2013/07/the-plan-for-rspec-3#expectations_matcher_protocol_and_custom_matcher_api_changes)
 
+### Example Groups
+
+**This conversion is disabled by default and available only if your project's RSpec is `<%= Transpec::RSpecVersion.non_monkey_patch_example_group_available_version %>` or later.**
+
+Targets:
+
+```ruby
+<%=
+example = <<END
+RSpec.configure do |rspec|
+end
+
+describe 'top-level example group' do
+  describe 'nested example group' do
+  end
+end
+
+shared_examples 'shared examples' do
+end
+END
+-%>
+```
+
+Will be converted to:
+
+```ruby
+<%=
+configuration = Transpec::Configuration.new
+configuration.convert_example_group = true
+rspec_version = Transpec::RSpecVersion.non_monkey_patch_example_group_available_version
+convert(example, configuration, rspec_version)
+-%>
+```
+
+* This conversion can be enabled by: `--convert example_group`
+* Deprecation: Not deprecated
+* See also: [Zero Monkey Patching Mode! - The Plan for RSpec 3](http://myronmars.to/n/dev-blog/2013/07/the-plan-for-rspec-3#zero_monkey_patching_mode)
+
+
 ## Compatibility
 
 Tested on MRI 1.9, 2.0, 2.1 and JRuby in 1.9 mode.

data/lib/transpec/base_rewriter.rb

@@ -1,32 +1,42 @@
 # coding: utf-8
 
-require 'transpec/ast/builder'
-require 'transpec/parser'
+require 'transpec/processed_source'
 
 module Transpec
   class BaseRewriter
-    def rewrite_file!(file_path)
-      source = File.read(file_path)
-      rewritten_source = rewrite(source, file_path)
-      return if source == rewritten_source
-      File.write(file_path, rewritten_source)
+    def rewrite_file!(arg)
+      processed_source = case arg
+                         when String          then ProcessedSource.parse_file(arg)
+                         when ProcessedSource then arg
+                         else fail "Invalid argument: #{arg}"
+                         end
+
+      fail 'ProcessedSource must be derived from a file' unless processed_source.path
+
+      rewritten_source = rewrite(processed_source)
+      return if processed_source.to_s == rewritten_source
+      File.write(processed_source.path, rewritten_source)
     end
 
-    def rewrite(source, name = '(string)')
-      source_buffer = create_source_buffer(source, name)
-      ast = parse(source_buffer)
+    def rewrite_source(source, path = nil)
+      processed_source = ProcessedSource.parse(source, path)
+      rewrite(processed_source)
+    end
+
+    def rewrite(processed_source)
+      fail processed_source.syntax_error if processed_source.syntax_error
 
-      source_rewriter = Parser::Source::Rewriter.new(source_buffer)
-      failed_overlapping_rewrite = false
+      source_rewriter = Parser::Source::Rewriter.new(processed_source.buffer)
+      incomplete = false
       source_rewriter.diagnostics.consumer = proc do
-        failed_overlapping_rewrite = true
+        incomplete = true
         fail OverlappedRewriteError
       end
 
-      process(ast, source_rewriter)
+      process(processed_source.ast, source_rewriter)
 
       rewritten_source = source_rewriter.process
-      rewritten_source = rewrite(rewritten_source, name) if failed_overlapping_rewrite
+      rewritten_source = rewrite_source(rewritten_source, processed_source.path) if incomplete
 
       rewritten_source
     end
@@ -37,19 +47,6 @@ module Transpec
       fail NotImplementedError
     end
 
-    def create_source_buffer(source, name)
-      source_buffer = Parser::Source::Buffer.new(name)
-      source_buffer.source = source
-      source_buffer
-    end
-
-    def parse(source_buffer)
-      builder = AST::Builder.new
-      parser = Parser::CurrentRuby.new(builder)
-      ast = parser.parse(source_buffer)
-      ast
-    end
-
     class OverlappedRewriteError < StandardError; end
   end
 end