r2m 0.2.2 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 503c7c4d167a93d9ab4e6cdc6ca79954e6069e0fd19057d299f8ea7246af1f4d
-  data.tar.gz: c902939ee55bc510b05dd89f4518654b38299dce0a26c0f8e565a3faab69acb9
+  metadata.gz: 39cf102836307d757425f6c28d74674bb77001f61f7506ef1babf56b531f0405
+  data.tar.gz: a481c0b6dd240ea5f908d6481f8cdef1b61fa748bdf83d3c526890b0be9be894
 SHA512:
-  metadata.gz: '048f7dbdf9e279ab9bba516b66a57b5fce3faf108d9e4b0eef140c6a7131a2ba961fae58181c693055ec4cffd09acb3751da4c2dffe9527e66748572c67cf2aa'
-  data.tar.gz: 4ed428ef6032d856d2f0db3b95673d67b5473c08a8ffed490922f703854f21a86bc10d254f1bdc4930c81a2f93a653787d1d26c35e870339f6144fd205b58ba2
+  metadata.gz: 26b7a0da0e111484d6fcd55954c9d8efbccfc223279a220abe81b0438225fc868335051e2d77e78542ad8126d75115712a67cf156c927ed9f9402e284c889c84
+  data.tar.gz: 3db7a67f1bbfef5c88e1a95e942b0b93e37230c9417334a78c56d3fdaacf063c234db93389b17b59feb28ac275c344bc788ca80c7989ee883f091903635e62fd

data/Gemfile

@@ -5,3 +5,7 @@ gemspec
 
 gem "rake", "~> 12.0"
 gem "minitest", "~> 5.0"
+
+gem 'rspec'
+gem 'minitest-around'
+gem 'minitest-spec-rails'

data/exe/rspec2minitest

@@ -6,14 +6,20 @@ cp -a $tests_path test/
 
 tests_path=test/$(basename $tests_path)
 
+# require helpers
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/system_helper/application_system_test_case/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/rails_helper/test_helper/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/spec_helper/test_helper/g' {} \;
+
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/context/describe/g' {} \;
+
+# mocks
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/stub(/stubs(/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/double/stub/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/should_receive/expects/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/and_return/returns/g' {} \;
+
+# matchers
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/.to be ==/.must_equal/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/.to be_empty/.must_be_empty/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/.to_not be_empty/.wont_be_empty/g' {} \;
@@ -24,21 +30,29 @@ find $tests_path -name '*_spec.rb' -exec sed -i '' 's/.to_not be_nil/.wont_be_ni
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.not_to be_an_\(.*\)/.wont_be :\1\?/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.to be_a(/.must_be_kind_of(/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.to be_an(/.must_be_kind_of(/g' {} \;
+
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.to be_an_\(.*\)/.must_be :\1\?/g' {} \;
+find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.to be_\(.*\)/.must_be :\1\?/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.to_not be_\(.*\)/.wont_be :\1\?/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.not_to be_\(.*\)/.wont_be :\1\?/g' {} \;
-find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.to be_\(.*\)/.must_be :\1\?/g' {} \;
+
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/.to be_nil/.must_be_nil/g' {} \;
+
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.to eq/.must_equal/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.not_to eq/.wont_equal/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.to_not eq/.wont_equal/g' {} \;
+
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.not_to include/.wont_include/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.to_not include/.wont_include/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.to include/.must_include/g' {} \;
+
+find $tests_path -name '*_spec.rb' -exec sed -i '' 's/.to match/.must_match/g' {} \;
+
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/\.to be_kind_of/.must_be_kind_of/g' {} \;
+
+# requests
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/expect(response).to have_http_status/assert_response/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/.to redirect_to/.must_redirect_to/g' {} \;
-find $tests_path -name '*_spec.rb' -exec sed -i '' 's/.to match/.must_match/g' {} \;
 
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/expect(page)\.to have_content/assert_text/g' {} \;
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/expect(page)\.to have_no_content/assert_no_text/g' {} \;
@@ -46,19 +60,18 @@ find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/expect(
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/expect(page)\.to have_selector/assert_selector/g' {} \;
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/expect(page)\.not_to have_selector/assert_no_selector/g' {} \;
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/expect(page)\.to_not have_selector/assert_no_selector/g' {} \;
+find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/expect(page)\.to have_checked_field/assert_checked_field/g' {} \;
+find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/expect(page)\.not_to have_checked_field/assert_no_checked_field/g' {} \;
+find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/expect(page)\.to_not have_checked_field/assert_no_checked_field/g' {} \;
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/\.to have_content/.must_have_content/g' {} \;
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/\.not_to have_content/.wont_have_content/g' {} \;
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/\.to have_no_content/.wont_have_content/g' {} \;
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/\.to have_field/.must_have_field/g' {} \;
-find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/expect(page)\.to have_checked_field/assert_checked_field/g' {} \;
-find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/expect(page)\.not_to have_checked_field/assert_no_checked_field/g' {} \;
-find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/expect(page)\.to_not have_checked_field/assert_no_checked_field/g' {} \;
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/\.to have_link/.must_have_link/g' {} \;
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/\.to have_no_link/.wont_have_link/g' {} \;
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/\.to have_button/.must_have_button/g' {} \;
 find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/\.to have_no_button/.wont_have_button/g' {} \;
 
-find $tests_path -name '*_spec.rb' -exec sed -i '' 's/.to match/.must_match/g' {} \;
 
 find $tests_path -name '*_spec.rb' -exec sed -i '' "s/expect[ {]/skip('FIXME: NEED TO MIGRATE MANUALLY FROM RSPEC')
 expect /g" {} \;
@@ -66,21 +79,22 @@ expect /g" {} \;
 
 find $tests_path -path '*/helpers/*' -name '*_spec.rb' -exec sed -i '' 's/helper\.//g' {} \;
 
-find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/RSpec\.describe '"'"'\(.*\)'"'"' do.*/Class.new(ApplicationSystemTestCase) do #\1/g' {} \;
-find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/, screenshot: true//g' {} \;
-
 find $tests_path -path '*/controllers/*' -name '*_spec.rb' -exec sed -i '' 's/RSpec\.describe \(.*Controller\).*/class \1Test < ActionController::TestCase/g' {} \;
 find $tests_path -path '*/mailers/*'  -name '*_spec.rb' -exec sed -i '' 's/RSpec\.describe \(.*Mailer\).*/class \1Test < ActionMailer::TestCase/g' {} \;
 find $tests_path -path '*/helpers/*' -name '*_spec.rb' -exec sed -i '' 's/RSpec\.describe \(.*Helper\).*/class \1Test < ActionView::TestCase/g' {} \;
-
 find $tests_path -path '*/requests/*' -name '*_spec.rb' -exec sed -i '' 's/RSpec\.describe '"'"'\(.*\)'"'"' do.*/class \1Test < ActionDispatch::IntegrationTest/g' {} \;
 
+find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/RSpec\.describe '"'"'\(.*\)'"'"' do.*/Class.new(ApplicationSystemTestCase) do #\1/g' {} \;
+
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/RSpec\.describe '"'"'\(.*\)'"'"' do/class \1Test < ActiveSupport::TestCase/g' {} \;
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/RSpec\.describe \(.*\) do/class \1Test < ActiveSupport::TestCase/g' {} \;
 
 
 find $tests_path -name '*_spec.rb' -exec sed -i '' 's/around do/warn "FIX: AROUND"; around do/g' {} \;
 
+# other project specific
+#find $tests_path -path '*/system/*' -name '*_spec.rb' -exec sed -i '' 's/, screenshot: true//g' {} \;
+
 
 rubocop --safe-auto-correct --disable-uncorrectable $tests_path
 

data/lib/r2m/processor.rb

@@ -11,7 +11,14 @@ module R2M
 
     def process(file)
       #   it 'converts it"s to test method with # support' do # => def test_converts_it_s_to_test_method_with_support
+      convert_require_helpers(file)
+      convert_declarations(file)
+      convert_declarations(file)
+      convert_context_to_describe(file)
       convert_it_to_methods(file)
+      convert_helpers_suites(file)
+      convert_mather_be_empty(file)
+      convert_simple_matcher(file)
     end
 
     # Finds +it+ cases and converts to test methods declarations
@@ -26,6 +33,22 @@ module R2M
       @command.gsub_file(file, /it '_?(.*)' do/, 'def test_\1')
     end
 
+    # Finds +it+ cases and converts to test methods declarations
+    #
+    #   context 'with context' do # => describe 'with context' do
+    def convert_context_to_describe(file)
+      @command.gsub_file(file, /\bcontext( '.*?' do\b)/, 'describe\1')
+    end
+
+    # Finds +it+ cases and converts to test methods declarations
+    #
+    #   context 'with context' do # => describe 'with context' do
+    def convert_require_helpers(file)
+      @command.gsub_file(file, /\bsystem_helper\b/, 'application_system_test_case')
+      @command.gsub_file(file, /\b(rails_helper|spec_helper)\b/, 'test_helper')
+      @command.gsub_file(file, /require (['"]?)rspec\1/, "require 'minitest/autorun'")
+    end
+
     # Finds +be_empty+ RSpec matchers and converts to minitest matchers
     #
     #   expect(target).to be_empty # => expect(target).must_be_empty
@@ -35,5 +58,97 @@ module R2M
       @command.gsub_file(file, /\.to be_empty/, '.must_be_empty')
       @command.gsub_file(file, /\.(to_not|not_to) be_empty/, '.wont_be_empty')
     end
+
+    # Finds +equal+ RSpec matchers and converts to minitest matchers
+    #
+    #   expect(target).to eq expect # => expect(target).must_equal expect
+    #   expect(target).not_to eq expect # => expect(target).wont_equal expect
+    def convert_simple_matcher(file)
+      @command.gsub_file(file, /\.(to_not|not_to|to)\s+(eq|include|match|be_kind_of)\b/) do |match|
+        match
+          .gsub(/\s+/, ' ')
+          .gsub(/\.\w+ /, '.to_not ' => '.wont_', '.not_to ' => '.wont_', '.to ' => '.must_')
+          .gsub(/(?<=_)eq/, 'equal')
+      end
+    end
+
+    def convert_helpers_suites(file)
+      @command.gsub_file(file, /\bhelper\./, '') if located_in_helpers?(file)
+    end
+
+    def convert_declarations(file)
+      @command.gsub_file(file, /RSpec\.describe (['"])(.*?)\1 do\b/) do |match|
+        title = match[/RSpec\.describe (['"])(.*?)\1 do/, 2]
+
+        camelCasedTitle = title.split('::').map do |title_part|
+          title_part
+            .split
+            .reject(&:empty?)
+            .map { |part| part[0].upcase + part[1..-1] }.join
+        end.join('::')
+
+        "RSpec.describe '#{camelCasedTitle}' do"
+      end
+
+      @command.gsub_file(
+        file,
+        /RSpec\.describe ['"]?(.+?Controller)['"]? do\b/,
+        'class \1Test < ActionController::TestCase'
+      )
+      @command.gsub_file(
+        file,
+        /RSpec\.describe ['"]?(.+?Mailer)['"]? do\b/,
+        'class \1Test < ActionMailer::TestCase'
+      )
+
+      @command.gsub_file(
+        file,
+        /RSpec\.describe ['"]?(.+?Helper)['"]? do\b/,
+        'class \1Test < ActionView::TestCase'
+      )
+
+      if located_in_requests?(file)
+        @command.gsub_file(
+          file,
+          /RSpec\.describe ['"]?(.+?)['"]? do\b/,
+          'class \1Test < ActionDispatch::IntegrationTest'
+        )
+      end
+
+      if located_in?(file, :systems)
+        @command.gsub_file(
+          file,
+          /RSpec\.describe ['"]?(.+?)['"]? do\b/,
+          'class \1Test < ApplicationSystemTestCase'
+        )
+      end
+
+      @command.gsub_file(
+        file,
+        /RSpec\.describe ['"]?(.+?)['"]? do\b/,
+        'class \1Test < ActiveSupport::TestCase'
+      )
+    end
+
+    MANUAL_AROUND_PROCESS = "skip 'TODO: Remove this skip when `around` will be migrated manually by replacing `run` with `call`'"
+    def convert_around(file)
+      @command.gsub_file(file, /^\s*?\baround\b.+?\bdo\b.*?$/) do |match|
+        "#{MANUAL_AROUND_PROCESS}\n#{match}"
+      end
+    end
+
+    private
+
+    def located_in_helpers?(file)
+      located_in?(file, :helpers)
+    end
+
+    def located_in_requests?(file)
+      located_in?(file, :requests)
+    end
+
+    def located_in?(file, sub_folder)
+      file =~ /(test|spec)\/#{sub_folder}\//
+    end
   end
 end

data/lib/r2m/version.rb

@@ -1,3 +1,3 @@
 module R2M
-  VERSION = "0.2.2"
+  VERSION = "0.2.3"
 end

data/r2m.gemspec

@@ -1,31 +1,33 @@
+# frozen_string_literal: true
+
 require_relative 'lib/r2m/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "r2m"
+  spec.name          = 'r2m'
   spec.version       = R2M::VERSION
-  spec.authors       = ["Paul Keen"]
-  spec.email         = ["info@jetthoughts.com"]
+  spec.authors       = ['Paul Keen']
+  spec.email         = ['info@jetthoughts.com']
 
-  spec.summary       = "Converts RSpec files to minitest"
-  spec.description   = "A command-line tool for converting RSpec files to minitest."
-  spec.homepage      = "https://github.com/jetthoughts/r2m"
-  spec.license       = "MIT"
-  spec.required_ruby_version = Gem::Requirement.new(">= 2.3.0")
+  spec.summary       = 'Converts RSpec files to minitest'
+  spec.description   = 'A command-line tool for converting RSpec files to minitest.'
+  spec.homepage      = 'https://github.com/jetthoughts/r2m'
+  spec.license       = 'MIT'
+  spec.required_ruby_version = Gem::Requirement.new('>= 2.3.0')
 
   # spec.metadata["allowed_push_host"] = "TODO: Set to 'http://mygemserver.com'"
 
-  spec.metadata["homepage_uri"] = spec.homepage
-  spec.metadata["source_code_uri"] = "https://github.com/jetthoughts/r2m"
-  spec.metadata["changelog_uri"] = "https://github.com/jetthoughts/r2m/README.md"
+  spec.metadata['homepage_uri'] = spec.homepage
+  spec.metadata['source_code_uri'] = 'https://github.com/jetthoughts/r2m'
+  spec.metadata['changelog_uri'] = 'https://github.com/jetthoughts/r2m/README.md'
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
-  spec.files         = Dir.chdir(File.expand_path('..', __FILE__)) do
+  spec.files = Dir.chdir(File.expand_path(__dir__)) do
     `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test)/}) }
   end
-  spec.bindir        = "exe"
+  spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
 
-  spec.add_development_dependency 'thor'
+  spec.add_dependency 'thor'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: r2m
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.2.3
 platform: ruby
 authors:
 - Paul Keen
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-05-01 00:00:00.000000000 Z
+date: 2020-05-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor
@@ -17,7 +17,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  type: :development
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements: