transpec 1.11.1 → 1.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: be81d8c6008e09a8dd02f276b7dd3b01155b28be
-  data.tar.gz: 06c16bc90a1ba3851bc4004f29aadb1daf736dd4
+  metadata.gz: 9028442bbb2a56a7ad1ef1a0205980d1f460ca95
+  data.tar.gz: 136e4e0450ee5747ac17922275b1b1dad1b1991a
 SHA512:
-  metadata.gz: f7cd465d319c14f360688e20e34c3cf8ce66b7f209d7cbd2f9b84397761d9ec44f39f0f78c19fa474af8509f0d894c54154f982dbc677d7ab6be1be5ec6c4478
-  data.tar.gz: 212dcc0282e32fc1a64a1887d3159635434cdfb48567a34a077c9c6712f578f277a4ab229e6c0a9e5ac569dbf51f7d308603d96ec31123d607bb18e0bc38ba71
+  metadata.gz: 401af30bd4fdfb1144dd5d1d9577b12e2d043c9800b0b0a23edc973c7be6af3da89b426e900b4d31f5ea827e594a4203243c9437b9194715453386b9a49b9688
+  data.tar.gz: 3722e2241be35ef86e5b2df35e777322c63759a994c84fb4066813650521c7d05cf27526005d2320ba341d9447798a183215cbd112c863e42ce952f7349faa3e

data/.gitignore

@@ -16,3 +16,4 @@ test/tmp
 test/version_tmp
 tmp
 /vendor
+.cache

data/CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## Development
 
+## v1.12.0
+
+* Support conversion of hook scope names ([#53](https://github.com/yujinakayama/transpec/issues/53))
+
 ## v1.11.1
 
 * Add missing description of `example_group` for `-v/--convert` option in the `-h/--help` text

data/lib/transpec/cli.rb

@@ -134,7 +134,7 @@ module Transpec
       puts 'A commit message that describes the conversion summary was generated to'.color(:cyan)
       puts '.git/COMMIT_EDITMSG. To use the message, type the following command for'.color(:cyan)
       puts 'the next commit:'.color(:cyan)
-      puts '    git commit -eF .git/COMMIT_EDITMSG'
+      puts '    git commit -aeF .git/COMMIT_EDITMSG'
     end
 
     def display_final_guide

data/lib/transpec/configuration.rb

@@ -19,6 +19,7 @@ module Transpec
       [:add_receiver_arg_to_any_instance_implementation_block, true],
       [:convert_stub_with_hash_to_allow_to_receive_and_return, false],
       [:convert_example_group,                                 false],
+      [:convert_hook_scope,                                    false],
       [:forced,                                                false],
       [:skip_dynamic_analysis,                                 false]
     ].freeze

data/lib/transpec/converter.rb

@@ -214,6 +214,11 @@ module Transpec
       have.convert_to_standard_expectation!(configuration.parenthesize_matcher_arg)
     end
 
+    def process_hook(hook)
+      return if !configuration.convert_hook_scope? || !rspec_version.hook_scope_alias_available?
+      hook.convert_scope_name!
+    end
+
     def process_messaging_host(messaging_host)
       process_useless_and_return(messaging_host)
       process_any_instance_block(messaging_host)

data/lib/transpec/option_parser.rb

@@ -21,8 +21,9 @@ module Transpec
     }
 
     CONFIG_ATTRS_FOR_CONVERT_TYPES = {
-      stub_with_hash: :convert_stub_with_hash_to_allow_to_receive_and_return=,
-       example_group: :convert_example_group=
+       example_group: :convert_example_group=,
+          hook_scope: :convert_hook_scope=,
+      stub_with_hash: :convert_stub_with_hash_to_allow_to_receive_and_return=
     }
 
     VALID_BOOLEAN_MATCHER_TYPES = %w(truthy,falsey truthy,falsy true,false)
@@ -149,6 +150,7 @@ module Transpec
           'Enable specific conversions that are disabled by default.',
           'Conversion Types:',
           '  *example_group* (`describe` to `RSpec.describe`)',
+          '  *hook_scope* (`before(:all)` to `before(:context)`)',
           '  *stub_with_hash* (`obj.stub(:msg => val)` to',
           '                  `allow(obj).to receive(:msg).and_return(val)`)',
           'These conversions are disabled by default.'

data/lib/transpec/rspec_version.rb

@@ -50,6 +50,7 @@ module Transpec
     define_feature :receive_message_chain,          '3.0.0.beta2'
     define_feature :non_should_matcher_protocol,    '3.0.0.beta2'
     define_feature :non_monkey_patch_example_group, '3.0.0.beta2'
+    define_feature :hook_scope_alias,               '3.0.0.beta2'
 
     RSPEC_2_99 = new('2.99.0.beta1')
     RSPEC_3_0  = new('3.0.0.beta1')

data/lib/transpec/static_context_inspector.rb

@@ -129,7 +129,7 @@ module Transpec
 
       if arg_node && [:sym, :str].include?(arg_node.type)
         hook_arg = arg_node.children.first.to_sym
-        return :all_before_after if hook_arg == :all
+        return :all_before_after if [:all, :context].include?(hook_arg)
       end
 
       :each_before_after

data/lib/transpec/syntax/hook.rb

@@ -0,0 +1,45 @@
+# coding: utf-8
+
+require 'transpec/syntax'
+require 'transpec/syntax/mixin/send'
+require 'transpec/rspec_dsl'
+
+module Transpec
+  class Syntax
+    class Hook < Syntax
+      include Mixin::Send, RSpecDSL
+
+      SCOPE_ALIASES = {
+        each: :example,
+         all: :context
+      }
+
+      def dynamic_analysis_target?
+        super && HOOK_METHODS.include?(method_name)
+      end
+
+      def convert_scope_name!
+        return if !scope_name || !replacement_scope_name
+        replace(arg_range, replacement_scope_name.inspect)
+        register_record
+      end
+
+      private
+
+      def scope_name
+        return nil unless arg_node
+        arg_node.children.first
+      end
+
+      def replacement_scope_name
+        SCOPE_ALIASES[scope_name]
+      end
+
+      def register_record
+        original_syntax = "#{method_name}(#{scope_name.inspect}) { }"
+        converted_syntax = "#{method_name}(#{replacement_scope_name.inspect}) { }"
+        report.records << Record.new(original_syntax, converted_syntax)
+      end
+    end
+  end
+end

data/lib/transpec/version.rb

@@ -4,8 +4,8 @@ module Transpec
   # http://semver.org/
   module Version
     MAJOR = 1
-    MINOR = 11
-    PATCH = 1
+    MINOR = 12
+    PATCH = 0
 
     def self.to_s
       [MAJOR, MINOR, PATCH].join('.')

data/spec/support/cache_helper.rb

@@ -48,8 +48,8 @@ module CacheHelper
   def cache_dir
     @cache_dir ||= begin
       project_root = File.expand_path(File.join(File.dirname(__FILE__), '..', '..'))
-      ruby_version = "#{RUBY_ENGINE}#{RUBY_VERSION}"
-      cache_dir = File.join(project_root, 'tmp', 'spec_cache', ruby_version)
+      ruby_version = [RUBY_ENGINE, RUBY_VERSION].join('-')
+      cache_dir = File.join(project_root, '.cache', 'spec', ruby_version)
 
       unless Dir.exist?(cache_dir)
         require 'fileutils'

data/spec/transpec/configuration_spec.rb

@@ -21,6 +21,7 @@ module Transpec
         [:add_receiver_arg_to_any_instance_implementation_block?, true],
         [:convert_stub_with_hash_to_allow_to_receive_and_return?, false],
         [:convert_example_group?,                                 false],
+        [:convert_hook_scope?,                                    false],
         [:forced?,                                                false],
         [:skip_dynamic_analysis?,                                 false],
         [:negative_form_of_to,                                    'not_to'],

data/spec/transpec/converter_spec.rb

@@ -830,6 +830,45 @@ module Transpec
       end
     end
 
+    describe '#process_hook' do
+      let(:hook_object) { double('hook_object').as_null_object }
+
+      context 'when Configuration#convert_hook_scope? is true' do
+        before { configuration.convert_hook_scope = true }
+
+        context 'when RSpecVersion#hook_scope_alias_available? returns true' do
+          before { rspec_version.stub(:hook_scope_alias_available?).and_return(true) }
+
+          it 'invokes Hook#convert_scope_name!' do
+            hook_object.should_receive(:convert_scope_name!)
+            converter.process_hook(hook_object)
+          end
+        end
+
+        context 'when RSpecVersion#hook_scope_alias_available? returns false' do
+          before { rspec_version.stub(:hook_scope_alias_available?).and_return(false) }
+
+          it 'does nothing' do
+            hook_object.should_not_receive(:convert_scope_name!)
+            converter.process_hook(hook_object)
+          end
+        end
+      end
+
+      context 'when Configuration#convert_hook_scope? is false' do
+        before { configuration.convert_hook_scope = false }
+
+        context 'when RSpecVersion#hook_scope_alias_available? returns true' do
+          before { rspec_version.stub(:hook_scope_alias_available?).and_return(true) }
+
+          it 'does nothing' do
+            hook_object.should_not_receive(:convert_scope_name!)
+            converter.process_hook(hook_object)
+          end
+        end
+      end
+    end
+
     describe '#process_raise_error' do
       let(:raise_error_object) { double('raise_error_object').as_null_object }
 

data/spec/transpec/option_parser_spec.rb

@@ -103,8 +103,9 @@ module Transpec
 
       describe '-v/--convert option' do
         [
-          ['stub_with_hash', :convert_stub_with_hash_to_allow_to_receive_and_return?],
-          ['example_group',  :convert_example_group?]
+          ['example_group',  :convert_example_group?],
+          ['hook_scope',     :convert_hook_scope?],
+          ['stub_with_hash', :convert_stub_with_hash_to_allow_to_receive_and_return?]
         ].each do |cli_type, config_attr|
           context "when #{cli_type.inspect} is specified" do
             let(:args) { ['--convert', cli_type] }
@@ -256,20 +257,29 @@ module Transpec
         end
       end
 
-      it 'describes all conversion types for -k/--keep option' do
-        option_sections = help_text.lines.slice_before(/^\s*-/)
+      def description_for_option(option)
+        description_lines = parser.send(:descriptions)[option]
+        description_lines.map { |line| parser.send(:highlight_text, line) }
+      end
 
-        keep_section = option_sections.find do |lines|
-          lines.first =~ /^\s*-k/
-        end
+      def conversion_types_for_option(option)
+        section = description_for_option(option)
 
-        conversion_types = keep_section.reduce([]) do |types, line|
-          match = line.match(/^[ ]{37}([a-z_]+)/)
-          next types unless match
+        section.each_with_object([]) do |line, types|
+          match = line.match(/^[ ]{2}([a-z_]+)/)
+          next unless match
           types << match.captures.first
         end
+      end
+
+      it 'describes all conversion types for -k/--keep option' do
+        conversion_types = conversion_types_for_option('-k')
+        conversion_types.should =~ OptionParser::CONFIG_ATTRS_FOR_KEEP_TYPES.keys.map(&:to_s)
+      end
 
-        conversion_types.should =~ OptionParser.available_conversion_types.map(&:to_s)
+      it 'describes all conversion types for -v/--convert option' do
+        conversion_types = conversion_types_for_option('-v')
+        conversion_types.should =~ OptionParser::CONFIG_ATTRS_FOR_CONVERT_TYPES.keys.map(&:to_s)
       end
     end
   end

data/spec/transpec/rspec_version_spec.rb

@@ -71,7 +71,8 @@ module Transpec
     [
       :receive_message_chain_available?,
       :non_should_matcher_protocol_available?,
-      :non_monkey_patch_example_group_available?
+      :non_monkey_patch_example_group_available?,
+      :hook_scope_alias_available?
     ].each do |method|
       include_examples 'version comparisons', method, [
         ['2.14.0',       false],

data/spec/transpec/syntax/hook_spec.rb

@@ -0,0 +1,215 @@
+# coding: utf-8
+
+require 'spec_helper'
+require 'transpec/syntax/hook'
+
+module Transpec
+  class Syntax
+    describe Hook do
+      include_context 'parsed objects'
+      include_context 'syntax object', Hook, :hook_object
+
+      let(:record) { hook_object.report.records.last }
+
+      describe '#convert_scope_name!' do
+        before do
+          hook_object.convert_scope_name!
+        end
+
+        RSpecDSL::HOOK_METHODS.each do |hook_method|
+          context "with expression `#{hook_method}(:each) { }`" do
+            let(:source) do
+              <<-END
+                describe 'example' do
+                  #{hook_method}(:each) do
+                    do_something
+                  end
+                end
+              END
+            end
+
+            let(:expected_source) do
+              <<-END
+                describe 'example' do
+                  #{hook_method}(:example) do
+                    do_something
+                  end
+                end
+              END
+            end
+
+            it "converts to `#{hook_method}(:example) { }` form" do
+              rewritten_source.should == expected_source
+            end
+
+            it "adds record `#{hook_method}(:each) { }` -> `#{hook_method}(:example) { }`" do
+              record.original_syntax.should  == "#{hook_method}(:each) { }"
+              record.converted_syntax.should == "#{hook_method}(:example) { }"
+            end
+          end
+        end
+
+        [:before, :after].each do |hook_method|
+          context "with expression `#{hook_method}(:all) { }`" do
+            let(:source) do
+              <<-END
+                describe 'example' do
+                  #{hook_method}(:all) do
+                    do_something
+                  end
+                end
+              END
+            end
+
+            let(:expected_source) do
+              <<-END
+                describe 'example' do
+                  #{hook_method}(:context) do
+                    do_something
+                  end
+                end
+              END
+            end
+
+            it "converts to `#{hook_method}(:context) { }` form" do
+              rewritten_source.should == expected_source
+            end
+
+            it "adds record `#{hook_method}(:all) { }` -> `#{hook_method}(:context) { }`" do
+              record.original_syntax.should  == "#{hook_method}(:all) { }"
+              record.converted_syntax.should == "#{hook_method}(:context) { }"
+            end
+          end
+
+          context "with expression `RSpec.configure { |c| c.#{hook_method}(:each) { } }`" do
+            let(:source) do
+              <<-END
+                RSpec.configure do |config|
+                  config.#{hook_method}(:each) do
+                    do_something
+                  end
+                end
+              END
+            end
+
+            let(:expected_source) do
+              <<-END
+                RSpec.configure do |config|
+                  config.#{hook_method}(:example) do
+                    do_something
+                  end
+                end
+              END
+            end
+
+            it "converts to `RSpec.configure { |c| c.#{hook_method}(:example) { } }` form" do
+              rewritten_source.should == expected_source
+            end
+
+            it "adds record `#{hook_method}(:each) { }` -> `#{hook_method}(:example) { }`" do
+              record.original_syntax.should  == "#{hook_method}(:each) { }"
+              record.converted_syntax.should == "#{hook_method}(:example) { }"
+            end
+          end
+
+          context "with expression `RSpec.configure { |c| c.#{hook_method}(:suite) { } }`" do
+            let(:source) do
+              <<-END
+                RSpec.configure do |config|
+                  config.#{hook_method}(:suite) do
+                    do_something
+                  end
+                end
+              END
+            end
+
+            it 'does nothing' do
+              rewritten_source.should == source
+            end
+
+            it 'does not add record' do
+              record.should be_nil
+            end
+          end
+        end
+
+        context 'with expression `before { }`' do
+          let(:source) do
+            <<-END
+              describe 'example' do
+                before do
+                  do_something
+                end
+              end
+            END
+          end
+
+          it 'does nothing' do
+            rewritten_source.should == source
+          end
+
+          it 'does not add record' do
+            record.should be_nil
+          end
+        end
+
+        context 'with expression `before(variable) { }`' do
+          let(:source) do
+            <<-END
+              scope = :each
+
+              describe 'example' do
+                before(scope) do
+                  do_something
+                end
+              end
+            END
+          end
+
+          it 'does nothing' do
+            rewritten_source.should == source
+          end
+
+          it 'does not add record' do
+            record.should be_nil
+          end
+        end
+
+        context 'with expression `before(:each, :type => :model) { }`' do
+          let(:source) do
+            <<-END
+              scope = :each
+
+              describe 'example' do
+                before(:each, :type => :model) do
+                  do_something
+                end
+              end
+            END
+          end
+
+          let(:expected_source) do
+            <<-END
+              scope = :each
+
+              describe 'example' do
+                before(:example, :type => :model) do
+                  do_something
+                end
+              end
+            END
+          end
+
+          it 'converts to `before(:example, :type => :model) { }` form' do
+            rewritten_source.should == expected_source
+          end
+
+          it 'adds record `before(:each) { }` -> `before(:example) { }`' do
+            record.original_syntax.should  == 'before(:each) { }'
+            record.converted_syntax.should == 'before(:example) { }'
+          end
+        end
+      end
+    end
+  end
+end

data/tasks/demo.rake

@@ -1,13 +1,13 @@
 # coding: utf-8
 
-require_relative 'lib/transpec_demo'
+require_relative 'lib/demo'
 
 namespace :demo do
   # rubocop:disable LineLength
   demos = [
-    TranspecDemo.new('git@github.com:yujinakayama/twitter.git', 'transpec-test-rspec-2-99'),
-    TranspecDemo.new('git@github.com:yujinakayama/guard.git', 'transpec-test-rspec-2-99', %w(--without development)),
-    TranspecDemo.new('git@github.com:yujinakayama/mail.git', 'transpec-test-rspec-2-99')
+    Demo.new('git@github.com:yujinakayama/twitter.git', 'transpec-test-rspec-2-99'),
+    Demo.new('git@github.com:yujinakayama/guard.git', 'transpec-test-rspec-2-99', %w(--without development)),
+    Demo.new('git@github.com:yujinakayama/mail.git', 'transpec-test-rspec-2-99')
   ]
   # rubocop:enable LineLength
 

data/tasks/fixtures/guard/{COMMIT_EDITMSG → 2.99.0/COMMIT_EDITMSG}

@@ -1,6 +1,6 @@
 Convert specs to RSpec 2.99.0.beta2 syntax with Transpec
 
-This conversion is done by Transpec 1.10.4 with the following command:
+This conversion is done by Transpec 1.11.1 with the following command:
     transpec --force
 
 * 396 conversions

data/tasks/fixtures/guard/3.0.0/COMMIT_EDITMSG

@@ -0,0 +1,32 @@
+Convert specs to RSpec 3.0.0.beta2 syntax with Transpec
+
+This conversion is done by Transpec 1.11.1 with the following command:
+    transpec --force --convert example_group,hook_scope
+
+* 31 conversions
+    from: describe 'something' { }
+      to: RSpec.describe 'something' { }
+
+* 16 conversions
+    from: obj.stub(:message => value)
+      to: allow(obj).to receive_messages(:message => value)
+
+* 13 conversions
+    from: before(:all) { }
+      to: before(:context) { }
+
+* 10 conversions
+    from: before(:each) { }
+      to: before(:example) { }
+
+* 8 conversions
+    from: after(:all) { }
+      to: after(:context) { }
+
+* 4 conversions
+    from: shared_examples_for 'something' { }
+      to: RSpec.shared_examples_for 'something' { }
+
+* 1 conversion
+    from: after(:each) { }
+      to: after(:example) { }

data/tasks/fixtures/mail/{COMMIT_EDITMSG → 2.99.0/COMMIT_EDITMSG}

@@ -1,6 +1,6 @@
 Convert specs to RSpec 2.99.0.beta2 syntax with Transpec
 
-This conversion is done by Transpec 1.10.4 with the following command:
+This conversion is done by Transpec 1.11.1 with the following command:
     transpec --force
 
 * 2161 conversions

data/tasks/fixtures/mail/3.0.0/COMMIT_EDITMSG

@@ -0,0 +1,28 @@
+Convert specs to RSpec 3.0.0.beta2 syntax with Transpec
+
+This conversion is done by Transpec 1.11.1 with the following command:
+    transpec --force --convert example_group,hook_scope
+
+* 82 conversions
+    from: describe 'something' { }
+      to: RSpec.describe 'something' { }
+
+* 39 conversions
+    from: before(:each) { }
+      to: before(:example) { }
+
+* 4 conversions
+    from: after(:each) { }
+      to: after(:example) { }
+
+* 1 conversion
+    from: after(:all) { }
+      to: after(:context) { }
+
+* 1 conversion
+    from: before(:all) { }
+      to: before(:context) { }
+
+* 1 conversion
+    from: obj.stub(:message => value)
+      to: allow(obj).to receive_messages(:message => value)

data/tasks/fixtures/twitter/{COMMIT_EDITMSG → 2.99.0/COMMIT_EDITMSG}

@@ -1,6 +1,6 @@
 Convert specs to RSpec 2.99.0.beta2 syntax with Transpec
 
-This conversion is done by Transpec 1.10.4 with the following command:
+This conversion is done by Transpec 1.11.1 with the following command:
     transpec --force
 
 * 899 conversions

data/tasks/fixtures/twitter/3.0.0/COMMIT_EDITMSG

@@ -0,0 +1,16 @@
+Convert specs to RSpec 3.0.0.beta2 syntax with Transpec
+
+This conversion is done by Transpec 1.11.1 with the following command:
+    transpec --force --convert example_group,hook_scope
+
+* 54 conversions
+    from: describe 'something' { }
+      to: RSpec.describe 'something' { }
+
+* 1 conversion
+    from: after(:all) { }
+      to: after(:context) { }
+
+* 1 conversion
+    from: before(:all) { }
+      to: before(:context) { }

data/tasks/lib/demo.rb

@@ -0,0 +1,46 @@
+# coding: utf-8
+
+require_relative 'project'
+
+class Demo < Project
+  DEMO_BRANCH = 'transpec-demo'
+
+  def self.base_dir_path
+    File.join('tmp', 'demo')
+  end
+
+  def run
+    puts " Publishing conversion example of #{name} project ".center(80, '=')
+
+    setup
+
+    in_project_dir do
+      transpec '--force', '--convert', 'stub_with_hash'
+      sh 'bundle exec rspec'
+      sh "git checkout --quiet -b #{DEMO_BRANCH}"
+      sh 'git commit -aF .git/COMMIT_EDITMSG'
+      sh "git push --force origin #{DEMO_BRANCH}"
+    end
+  end
+
+  private
+
+  def transpec(*args)
+    sh File.join(Transpec.root, 'bin', 'transpec'), *args
+  end
+
+  def setup_from_remote
+    FileUtils.rm_rf(project_dir) if Dir.exist?(project_dir)
+    super
+  end
+
+  def shallow_clone?
+    false
+  end
+
+  def git_local_branch_exist?(branch)
+    in_project_dir do
+      system('git', 'show-ref', '--verify', '--quiet', "refs/heads/#{branch}")
+    end
+  end
+end

data/tasks/lib/project.rb

@@ -0,0 +1,129 @@
+# coding: utf-8
+
+require 'transpec'
+require 'fileutils'
+
+class Project
+  include FileUtils # This is Rake's one.
+
+  BUNDLER_RETRY_COUNT = 3
+
+  attr_reader :url, :ref, :bundler_args
+
+  def self.base_dir
+    @base_dir ||= begin
+      path = base_dir_path
+      FileUtils.mkdir_p(path) unless Dir.exist?(path)
+      path
+    end
+  end
+
+  def self.base_dir_path
+    File.join(Transpec.root, 'tmp', 'test')
+  end
+
+  def initialize(url, ref = nil, bundler_args = [])
+    @url = url
+    @ref = ref
+    @bundler_args = bundler_args
+  end
+
+  def name
+    @name ||= File.basename(url, '.git')
+  end
+
+  def project_dir
+    @project_dir ||= File.join(self.class.base_dir, name)
+  end
+
+  def in_project_dir
+    Dir.chdir(project_dir) do
+      with_clean_bundler_env do
+        yield
+      end
+    end
+  end
+
+  def git(*args)
+    in_project_dir do
+      sh 'git', *args
+    end
+  end
+
+  private
+
+  def setup
+    if url.start_with?('/')
+      setup_from_local
+    else
+      setup_from_remote
+    end
+  end
+
+  def setup_from_local
+    FileUtils.rm_rf(project_dir) if Dir.exist?(project_dir)
+    Dir.mkdir(project_dir)
+
+    Dir.chdir(url) do
+      Dir.new('.').each do |entry|
+        next if ['.', '..', 'tmp'].include?(entry)
+        FileUtils.cp_r(entry, project_dir)
+      end
+    end
+
+    bundle_install
+  end
+
+  def setup_from_remote
+    if Dir.exist?(project_dir)
+      git 'checkout', '.'
+      git 'checkout', ref
+    else
+      git_clone
+      bundle_install
+    end
+  end
+
+  def git_clone
+    Dir.chdir(self.class.base_dir) do
+      # Disabling checkout here to suppress "detached HEAD" warning.
+      command = %w(git clone --no-checkout)
+      command.concat(%w(--depth 1)) if shallow_clone?
+      command.concat(['--branch', ref, url])
+      sh command.join(' ')
+    end
+
+    git 'checkout', '--quiet', ref
+  end
+
+  def shallow_clone?
+    true
+  end
+
+  def bundle_install
+    in_project_dir do
+      sh 'bundle', 'install', '--retry', BUNDLER_RETRY_COUNT.to_s, *bundler_args
+    end
+  end
+
+  def with_clean_bundler_env
+    if defined?(Bundler)
+      Bundler.with_clean_env do
+        # Bundler.with_clean_env cleans environment variables
+        # which are set after bundler is loaded.
+        prepare_env
+        yield
+      end
+    else
+      prepare_env
+      yield
+    end
+  end
+
+  def prepare_env
+    # Disable Coveralls.
+    ENV['CI'] = ENV['JENKINS_URL'] = ENV['COVERALLS_RUN_LOCALLY'] = nil
+
+    ENV['TRANSPEC_TEST'] = 'true'
+  end
+end

data/tasks/lib/test.rb

@@ -0,0 +1,84 @@
+# coding: utf-8
+
+require 'rspec/expectations'
+require_relative 'project'
+
+class Test < Project
+  include RSpec::Matchers
+
+  attr_reader :simple
+  alias_method :simple?, :simple
+
+  def initialize(url, ref = nil, bundler_args = [], simple = false)
+    super(url, ref, bundler_args)
+    @simple = simple
+  end
+
+  def run
+    puts " Testing transpec on #{name} project ".center(80, '=')
+
+    setup
+
+    in_project_dir do
+      transpec '--force'
+      sh 'bundle exec rspec'
+      return if simple?
+      compare_summary!('2.99.0')
+
+      add_rspec_3_to_gemfile
+      sh 'bundle update'
+
+      transpec '--force', '--convert', 'example_group,hook_scope'
+      sh 'bundle exec rspec'
+      compare_summary!('3.0.0')
+    end
+  end
+
+  private
+
+  def transpec(*args)
+    sh File.join(Transpec.root, 'bin', 'transpec'), *args
+  end
+
+  def add_rspec_3_to_gemfile
+    gemfile = File.read('Gemfile')
+
+    pattern = /\bgem\s+['"]rspec.+/
+    rspec_3_specification = "gem 'rspec', '~> 3.0.0.beta1'"
+
+    if gemfile.match(pattern)
+      gemfile.sub!(pattern, rspec_3_specification)
+    else
+      gemfile << rspec_3_specification
+    end
+
+    File.write('Gemfile', gemfile)
+  end
+
+  def compare_summary!(key)
+    fixture_path = commit_message_fixture_path(key)
+
+    if File.exist?(fixture_path)
+      summary = summary_in_commit_message(File.read(commit_message_path))
+      expected_summary = summary_in_commit_message(File.read(fixture_path))
+      expect(summary).to eq(expected_summary)
+    else
+      warn "#{fixture_path} does not exist. Copying from #{commit_message_path}."
+      FileUtils.mkdir_p(File.dirname(fixture_path))
+      FileUtils.cp(commit_message_path, fixture_path)
+    end
+  end
+
+  def summary_in_commit_message(message)
+    message.lines.to_a[5..-1].join("\n")
+  end
+
+  def commit_message_path
+    File.join(project_dir, '.git', 'COMMIT_EDITMSG')
+  end
+
+  def commit_message_fixture_path(key)
+    path = File.join(Transpec.root, 'tasks', 'fixtures', name, key, 'COMMIT_EDITMSG')
+    File.expand_path(path)
+  end
+end

data/tasks/test.rake

@@ -1,6 +1,6 @@
 # coding: utf-8
 
-require_relative 'lib/transpec_test'
+require_relative 'lib/test'
 
 namespace :test do
   # On Travis CI, reuse system gems to speed up build.
@@ -12,14 +12,14 @@ namespace :test do
 
   # rubocop:disable LineLength
   tests = [
-    TranspecTest.new(File.expand_path('.'), nil, ['--quiet']),
-    TranspecTest.new('https://github.com/yujinakayama/twitter.git', 'transpec-test-rspec-2-99', bundler_args),
-    TranspecTest.new('https://github.com/yujinakayama/mail.git', 'transpec-test-rspec-2-99', bundler_args)
+    Test.new(Transpec.root, nil, ['--quiet'], true),
+    Test.new('https://github.com/yujinakayama/twitter.git', 'transpec-test-rspec-2-99', bundler_args),
+    Test.new('https://github.com/yujinakayama/mail.git', 'transpec-test-rspec-2-99', bundler_args)
   ]
 
   # Sometimes Guard fails with JRuby randomly.
   unless RUBY_ENGINE == 'jruby'
-    tests << TranspecTest.new('https://github.com/yujinakayama/guard.git', 'transpec-test-rspec-2-99', bundler_args + %w(--without development))
+    tests << Test.new('https://github.com/yujinakayama/guard.git', 'transpec-test-rspec-2-99', bundler_args + %w(--without development))
   end
   # rubocop:enable LineLength
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: transpec
 version: !ruby/object:Gem::Version
-  version: 1.11.1
+  version: 1.12.0
 platform: ruby
 authors:
 - Yuji Nakayama
@@ -284,6 +284,7 @@ files:
 - lib/transpec/syntax/have/dynamic_analysis.rb
 - lib/transpec/syntax/have/have_record.rb
 - lib/transpec/syntax/have/source_builder.rb
+- lib/transpec/syntax/hook.rb
 - lib/transpec/syntax/its.rb
 - lib/transpec/syntax/matcher_definition.rb
 - lib/transpec/syntax/method_stub.rb
@@ -347,6 +348,7 @@ files:
 - spec/transpec/syntax/example_spec.rb
 - spec/transpec/syntax/expect_spec.rb
 - spec/transpec/syntax/have_spec.rb
+- spec/transpec/syntax/hook_spec.rb
 - spec/transpec/syntax/its_spec.rb
 - spec/transpec/syntax/matcher_definition_spec.rb
 - spec/transpec/syntax/method_stub_spec.rb
@@ -362,11 +364,15 @@ files:
 - spec/transpec_spec.rb
 - tasks/ci/spec.rake
 - tasks/demo.rake
-- tasks/fixtures/guard/COMMIT_EDITMSG
-- tasks/fixtures/mail/COMMIT_EDITMSG
-- tasks/fixtures/twitter/COMMIT_EDITMSG
-- tasks/lib/transpec_demo.rb
-- tasks/lib/transpec_test.rb
+- tasks/fixtures/guard/2.99.0/COMMIT_EDITMSG
+- tasks/fixtures/guard/3.0.0/COMMIT_EDITMSG
+- tasks/fixtures/mail/2.99.0/COMMIT_EDITMSG
+- tasks/fixtures/mail/3.0.0/COMMIT_EDITMSG
+- tasks/fixtures/twitter/2.99.0/COMMIT_EDITMSG
+- tasks/fixtures/twitter/3.0.0/COMMIT_EDITMSG
+- tasks/lib/demo.rb
+- tasks/lib/project.rb
+- tasks/lib/test.rb
 - tasks/readme.rake
 - tasks/test.rake
 - transpec.gemspec
@@ -428,6 +434,7 @@ test_files:
 - spec/transpec/syntax/example_spec.rb
 - spec/transpec/syntax/expect_spec.rb
 - spec/transpec/syntax/have_spec.rb
+- spec/transpec/syntax/hook_spec.rb
 - spec/transpec/syntax/its_spec.rb
 - spec/transpec/syntax/matcher_definition_spec.rb
 - spec/transpec/syntax/method_stub_spec.rb

data/tasks/lib/transpec_demo.rb

@@ -1,55 +0,0 @@
-# coding: utf-8
-
-require_relative 'transpec_test'
-
-class TranspecDemo < TranspecTest
-  DEMO_BRANCH = 'transpec-demo'
-
-  def self.base_dir_path
-    @base_dir_path = File.join('tmp', 'demos')
-  end
-
-  def run
-    require 'transpec'
-
-    puts " Publishing conversion example of #{name} project ".center(80, '=')
-
-    prepare_project
-    run_demo
-  end
-
-  private
-
-  def prepare_with_git_repo
-    super
-    git_branch_delete(DEMO_BRANCH) if git_local_branch_exist?(DEMO_BRANCH)
-  end
-
-  def shallow_clone?
-    false
-  end
-
-  def run_demo(transpec_args = [])
-    in_project_dir do
-      with_clean_bundler_env do
-        sh File.join(Transpec.root, 'bin', 'transpec'), '--force', '--convert', 'stub_with_hash'
-        sh 'bundle exec rspec'
-        sh "git checkout --quiet -b #{DEMO_BRANCH}"
-        sh 'git commit --all --file .git/COMMIT_EDITMSG'
-        sh "git push --force origin #{DEMO_BRANCH}"
-      end
-    end
-  end
-
-  def git_local_branch_exist?(branch_name)
-    in_project_dir do
-      system('git', 'show-ref', '--verify', '--quiet', "refs/heads/#{branch_name}")
-    end
-  end
-
-  def git_branch_delete(branch_name)
-    in_project_dir do
-      sh "git branch -D #{branch_name}"
-    end
-  end
-end

data/tasks/lib/transpec_test.rb

@@ -1,181 +0,0 @@
-# coding: utf-8
-
-class TranspecTest # rubocop:disable ClassLength
-  include FileUtils # This is Rake's one.
-
-  BUNDLER_RETRY_COUNT = 3
-
-  attr_reader :url, :ref, :bundler_args
-
-  def self.base_dir
-    @base_dir ||= begin
-      unless Dir.exist?(base_dir_path)
-        require 'fileutils'
-        FileUtils.mkdir_p(base_dir_path)
-      end
-
-      base_dir_path
-    end
-  end
-
-  def self.base_dir_path
-    @base_dir_path = File.join('tmp', 'tests')
-  end
-
-  def initialize(url, ref = nil, bundler_args = [])
-    @url = url
-    @ref = ref
-    @bundler_args = bundler_args
-  end
-
-  def name
-    @name ||= File.basename(url, '.git')
-  end
-
-  def project_dir
-    @project_dir ||= File.join(self.class.base_dir, name)
-  end
-
-  def run
-    require 'transpec'
-    puts " Testing transpec on #{name} project ".center(80, '=')
-    prepare_project
-    run_test(%w(--force))
-  end
-
-  private
-
-  def prepare_project
-    if url.start_with?('/')
-      prepare_with_local_dir
-    else
-      prepare_with_git_repo
-    end
-  end
-
-  def prepare_with_local_dir
-    if Dir.exist?(project_dir)
-      require 'fileutils'
-      FileUtils.rm_rf(project_dir)
-    end
-
-    Dir.mkdir(project_dir)
-
-    Dir.chdir(url) do
-      Dir.new('.').each do |entry|
-        next if ['.', '..', 'tmp'].include?(entry)
-        FileUtils.cp_r(entry, project_dir)
-      end
-
-      spec_cache_dir = File.join('tmp', 'spec_cache')
-
-      if Dir.exist?(spec_cache_dir)
-        Dir.mkdir(File.join(project_dir, 'tmp'))
-        FileUtils.cp_r(spec_cache_dir, File.join(project_dir, spec_cache_dir))
-      end
-    end
-
-    bundle_install
-  end
-
-  def prepare_with_git_repo
-    if Dir.exist?(project_dir)
-      git_checkout(ref) unless current_ref == ref
-      git_checkout('.')
-    else
-      git_clone
-      bundle_install
-    end
-  end
-
-  def run_test(transpec_args = [])
-    in_project_dir do
-      with_clean_bundler_env do
-        sh File.join(Transpec.root, 'bin', 'transpec'), *transpec_args
-        sh 'bundle exec rspec'
-        compare_summary!
-      end
-    end
-  end
-
-  def compare_summary!
-    unless File.exist?(commit_message_fixture_path)
-      warn "#{commit_message_fixture_path} does not exist. Skipping summary comparison."
-      return
-    end
-
-    require 'rspec/expectations'
-    extend RSpec::Matchers
-    summary = File.read(File.join('.git', 'COMMIT_EDITMSG')).lines.to_a[5..-1]
-    expected_summary = File.read(commit_message_fixture_path).lines.to_a[5..-1]
-    expect(summary).to eq(expected_summary)
-  end
-
-  def commit_message_fixture_path
-    path = File.join(File.dirname(__FILE__), '..', 'fixtures', name, 'COMMIT_EDITMSG')
-    File.expand_path(path)
-  end
-
-  def in_project_dir(&block)
-    Dir.chdir(project_dir, &block)
-  end
-
-  def current_ref
-    in_project_dir do
-      `git describe --all`.chomp.sub(/\Aheads\//, '')
-    end
-  end
-
-  def git_checkout(*args)
-    in_project_dir do
-      sh 'git', 'checkout', *args
-    end
-  end
-
-  def git_clone
-    Dir.chdir(self.class.base_dir) do
-      # Disabling checkout here to suppress "detached HEAD" warning.
-      command = %w(git clone --no-checkout)
-      command.concat(%w(--depth 1)) if shallow_clone?
-      command.concat(['--branch', ref, url])
-      sh command.join(' ')
-    end
-
-    in_project_dir do
-      sh "git checkout --quiet #{ref}"
-    end
-  end
-
-  def shallow_clone?
-    true
-  end
-
-  def bundle_install
-    in_project_dir do
-      with_clean_bundler_env do
-        sh 'bundle', 'install', '--retry', BUNDLER_RETRY_COUNT.to_s, *bundler_args
-      end
-    end
-  end
-
-  def with_clean_bundler_env
-    if defined?(Bundler)
-      Bundler.with_clean_env do
-        # Bundler.with_clean_env cleans environment variables
-        # which are set after bundler is loaded.
-        prepare_env
-        yield
-      end
-    else
-      prepare_env
-      yield
-    end
-  end
-
-  def prepare_env
-    # Disable Coveralls.
-    ENV['CI'] = ENV['JENKINS_URL'] = ENV['COVERALLS_RUN_LOCALLY'] = nil
-
-    ENV['TRANSPEC_TEST'] = 'true'
-  end
-end