transpec 0.2.6 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2d1e226cde065f5be1ba8ed1a21d0cf0a8721791
-  data.tar.gz: 6e53c222eb7070fe7a958a24615c6fb10071709c
+  metadata.gz: e89b721e834bdcbbc84616c93aeb74dcdcd69d29
+  data.tar.gz: 5678c00ce5a1bcc36e55cdcc41233fc0bd24d075
 SHA512:
-  metadata.gz: 65dad2b78acf24ce17d78d677b99daddc806ef5bc72f5a6ea7f0f9418b40828d0b709bbe313c711a6ce85b0769ff2577097582ae04207e2426c1c7cef4aea99c
-  data.tar.gz: f02830fbb953af904c67c6a5bdec5cddc20fb57fcc8d536cdde5cdf8da16842e4b6fbf125ae0d489349b49cf2f70eb0723713a8ae36a628f82c6f576fb63b34f
+  metadata.gz: b71de56cdff03743ced725d67478c80be77a2772421dd73c1bdc8d8d4a67fc17f6b1907f819b421feb40de91960afa64cf71ab5a9e5dbe3193924fd49ba8289c
+  data.tar.gz: 39ca211fe213c2d6858ac2fbd669953d363de542443ceb85c4c0f7d8dc578cde12be976e94eb0a578d718e29cfeb65e29bba1516060f1f1c57279c2563e3f95f

data/.gitignore

@@ -12,6 +12,7 @@ lib/bundler/man
 pkg
 rdoc
 spec/reports
+spec/cache
 test/tmp
 test/version_tmp
 tmp

data/.rubocop.yml

@@ -50,4 +50,4 @@ IndentationWidth:
 
 # TODO: Shorten to 100.
 ClassLength:
-  Max: 186
+  Max: 194

data/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## Master
 
+* Now Transpec does dynamic code analysis!
+* Support conversion of `have(n).items` matcher ([#5](https://github.com/yujinakayama/transpec/issues/5))
+* Add `-s/--skip-dynamic-analysis` option that allows to skip dynamic analysis and convert with only static analysis
+* Add `-c/--rspec-command` option that allows to specify command to run RSpec that is used for dynamic analysis
+* Check contexts correctly with runtime information
+* Detect same name but non-RSpec methods with runtime information ([#4](https://github.com/yujinakayama/transpec/issues/4))
+* Consider runtime type information when converting `=~` to `match_array`
+* Rename `-d/--disable` option to `-k/--keep` and change its syntax types
+* Rename `--commit-message` option to `--generate-commit-message`
+
 ## v0.2.6
 
 * Fix a bug where `Node#each_descendent_node` enumerates only within depth 2

data/README.md

@@ -2,7 +2,7 @@
 
 # Transpec
 
-**Transpec** automatically converts your specs into latest [RSpec](http://rspec.info/) syntax with static analysis.
+**Transpec** automatically converts your specs into latest [RSpec](http://rspec.info/) syntax with static and dynamic code analysis.
 
 This aims to facilitate smooth transition to RSpec 3.
 
@@ -91,9 +91,9 @@ describe Account do
 end
 ```
 
-### Real Examples
+### Actual Examples
 
-You can see real conversion examples below:
+You can see actual conversion examples below:
 
 * https://github.com/yujinakayama/guard/commit/transpec-demo
 * https://github.com/yujinakayama/mail/commit/transpec-demo
@@ -113,27 +113,32 @@ Before converting your specs:
 * Run `rspec` and check if all the specs pass.
 * Ensure the Git repository is clean. (You don't want to mix up your changes and Transpec's changes, right?)
 
-Then, run `transpec` (using `--commit-message` is recommended) in the project root directory:
+Then, run `transpec` (using `--generate-commit-message` is recommended) in the project root directory:
 
 ```bash
 $ cd some-project
-$ transpec --commit-message
-Processing spec/spec_helper.rb
-Processing spec/spec_spec.rb
-Processing spec/support/file_helper.rb
-Processing spec/support/shared_context.rb
-Processing spec/transpec/ast/scanner_spec.rb
-Processing spec/transpec/ast/scope_stack_spec.rb
+$ transpec --generate-commit-message
+Copying project for dynamic analysis...
+Running dynamic analysis with command "bundle exec rspec"...
+...............................................................................
+...................
+
+Finished in 13.07 seconds
+100 examples, 0 failures
+
+Converting spec/spec_helper.rb
+Converting spec/support/cache_helper.rb
+Converting spec/support/file_helper.rb
+Converting spec/support/shared_context.rb
+Converting spec/transpec/ast/node_spec.rb
 ```
 
-This will convert and overwrite all spec files in the `spec` directory.
+This will run your specs, convert them, and overwrite all spec files in the `spec` directory.
 
 After the conversion, run `rspec` again and check whether all pass:
 
 ```bash
 $ bundle exec rspec
-# ...
-843 examples, 0 failures
 ```
 
 If all pass, commit the changes with auto-generated message:
@@ -162,7 +167,23 @@ Processing spec/spec_spec.rb
 Processing spec/support/file_helper.rb
 ```
 
-### `-m/--commit-message`
+### `-s/--skip-dynamic-analysis`
+
+Skip dynamic analysis and convert with only static analysis. Note that specifying this option decreases the conversion accuracy.
+
+### `-c/--rspec-command`
+
+Specify command to run RSpec that is used for dynamic analysis.
+
+Transpec needs to run your specs in copied project directory for dynamic analysis.
+If your project requires some special setup or commands to run specs, use this option.
+`bundle exec rspec` is used by default.
+
+```bash
+$ transpec --rspec-command "./some_special_setup.sh && bundle exec rspec"
+```
+
+### `-m/--generate-commit-message`
 
 Generate commit message that describes conversion summary.
 Currently only Git is supported.
@@ -173,22 +194,23 @@ When you commit, you need to run the following command to use the generated mess
 $ git commit -eF .git/COMMIT_EDITMSG
 ```
 
-### `-d/--disable`
+### `-k/--keep`
 
-Disable specific conversions.
+Keep specific syntaxes by disabling conversions.
 
 ```bash
-$ transpec --disable expect_to_receive,allow_to_receive
+$ transpec --keep should_receive,stub
 ```
 
-#### Available conversion types
+#### Available syntax types
 
-Conversion Type     | Target Syntax                    | Converted Syntax
---------------------|----------------------------------|----------------------------
-`expect_to_matcher` | `obj.should matcher`             | `expect(obj).to matcher`
-`expect_to_receive` | `obj.should_receive`             | `expect(obj).to receive`
-`allow_to_receive`  | `obj.stub`                       | `allow(obj).to receive`
-`deprecated`        | `obj.stub!`, `mock('foo')`, etc. | `obj.stub`, `double('foo')`
+Type             | Target Syntax                    | Converted Syntax
+-----------------|----------------------------------|----------------------------
+`should`         | `obj.should matcher`             | `expect(obj).to matcher`
+`should_receive` | `obj.should_receive`             | `expect(obj).to receive`
+`stub`           | `obj.stub`                       | `allow(obj).to receive`
+`have_items`     | `expect(obj).to have(x).items`   | `expect(obj.size).to eq(x)`
+`deprecated`     | `obj.stub!`, `mock('foo')`, etc. | `obj.stub`, `double('foo')`
 
 ### `-n/--negative-form`
 
@@ -242,7 +264,7 @@ describe 'converted spec with -p/--no-parentheses-matcher-arg option' do
 end
 ```
 
-## Troubleshooting
+## Inconvertible Specs
 
 You might see the following warning while conversion:
 
@@ -270,11 +292,11 @@ end
 
 ### Reason
 
-* `should` is defined on `Kernel` (included by `Object`), so you can use `should` almost everywhere.
-* `expect` is defined on `RSpec::Matchers` (included by `RSpec::Core::ExampleGroup`), so you can use `expect` only where `self` is an instance of `RSpec::Core::ExampleGroup`.
+* `should` is defined on `Kernel` module that is included by `Object` class, so you can use `should` almost everywhere.
+* `expect` is defined on `RSpec::Matchers` module that is included by `RSpec::Core::ExampleGroup` class, so you can use `expect` only where `self` is an instance of `RSpec::Core::ExampleGroup` (i.e. in `it` blocks, `:each` hook blocks or included methods).
 
 With the above example, in the context of `1.should == 1`, the `self` is an instance of `MyAwesomeTestRunner`.
-So Transpec tracks contexts and skips conversion if the target syntax cannot be converted in a case like this.
+Transpec tracks contexts and skips conversion if the target syntax cannot be converted in a case like this.
 
 ### Solution
 
@@ -295,8 +317,8 @@ expect(obj).not_to matcher
 expect(obj).to_not matcher # with `--negative-form to_not`
 ```
 
-* Disabled by: `--disable expect_to_matcher`
-* Related Information: [Myron Marston » RSpec's New Expectation Syntax](http://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax)
+* Disabled by: `--keep should`
+* See also: [Myron Marston » RSpec's New Expectation Syntax](http://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax)
 
 ### Operator matchers
 
@@ -316,7 +338,7 @@ expect('string').to match(/^str/)
 expect([1, 2, 3]).to match_array([2, 1, 3])
 ```
 
-* Related Information: [Myron Marston » RSpec's New Expectation Syntax](http://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax#almost_all_matchers_are_supported)
+* See also: [(Almost) All Matchers Are Supported - RSpec's New Expectation Syntax](http://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax#almost_all_matchers_are_supported)
 
 ### `be_close` matcher
 
@@ -328,10 +350,43 @@ expect([1, 2, 3]).to match_array([2, 1, 3])
 (1.0 / 3.0).should be_within(0.001).of(0.333)
 ```
 
-* Disabled by: `--disable deprecated`
-* Related Information: [New be within matcher and RSpec.deprecate fix · rspec/rspec-expectations](https://github.com/rspec/rspec-expectations/pull/32)
+* Disabled by: `--keep deprecated`
+* See also: [New be within matcher and RSpec.deprecate fix · rspec/rspec-expectations](https://github.com/rspec/rspec-expectations/pull/32)
+
+### `have(n).items` matcher
+
+```ruby
+# Targets
+expect(collection).to have(3).items
+expect(collection).to have_exactly(3).items
+expect(collection).to have_at_least(3).items
+expect(collection).to have_at_most(3).items
+collection.should have(3).items
+
+expect(team).to have(3).players
+
+# Assume #players is a private method.
+expect(team).to have(3).players
+
+# Converted
+expect(collection.size).to eq(3)
+expect(collection.size).to be >= 3
+expect(collection.size).to be <= 3
+collection.size.should == 3  # with `--keep should`
+
+expect(team.players.size).to eq(3)
+
+# have(n).items matcher invokes #players even if it's a private method.
+expect(team.send(:players).size).to eq(3)
+```
+
+You have the option to continue using `have(n).items` matcher with [rspec-collection_matchers](https://github.com/rspec/rspec-collection_matchers) that is an external gem extracted from `rspec-expectations`.
+If you choose so, disable this conversion with `--keep have_items`.
+
+* Disabled by: `--keep have_items`
+* See also: [Expectations: have(x).items matchers will be moved into an external gem - The Plan for RSpec 3](http://myronmars.to/n/dev-blog/2013/07/the-plan-for-rspec-3#expectations__matchers_will_be_moved_into_an_external_gem)
 
-### Expectations on Proc
+### Expectations on Block
 
 ```ruby
 # Targets
@@ -343,8 +398,8 @@ proc { do_something }.should raise_error
 expect { do_something }.to raise_error
 ```
 
-* Disabled by: `--disable expect_to_matcher`
-* Related Information: [Myron Marston » RSpec's New Expectation Syntax](http://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax#unification_of_block_vs_value_syntaxes)
+* Disabled by: `--keep should`
+* See also: [Unification of Block vs. Value Syntaxes - RSpec's New Expectation Syntax](http://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax#unification_of_block_vs_value_syntaxes)
 
 ### Negative error expectations with specific error
 
@@ -357,11 +412,11 @@ lambda { do_something }.should_not raise_error(SomeErrorClass)
 
 # Converted
 expect { do_something }.not_to raise_error
-lambda { do_something }.should_not raise_error # with `--disable expect_to_matcher`
+lambda { do_something }.should_not raise_error # with `--keep should`
 ```
 
-* Disabled by: `--disable deprecated`
-* Related Information: [Consider deprecating `expect { }.not_to raise_error(SpecificErrorClass)` · rspec/rspec-expectations](https://github.com/rspec/rspec-expectations/issues/231)
+* Disabled by: `--keep deprecated`
+* See also: [Consider deprecating `expect { }.not_to raise_error(SpecificErrorClass)` · rspec/rspec-expectations](https://github.com/rspec/rspec-expectations/issues/231)
 
 ### Message expectations
 
@@ -375,8 +430,8 @@ expect(obj).to receive(:foo)
 expect_any_instance_of(SomeClass).to receive(:foo)
 ```
 
-* Disabled by: `--disable expect_to_receive`
-* Related Information: [RSpec's new message expectation syntax - Tea is awesome.](http://teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/)
+* Disabled by: `--keep should_receive`
+* See also: [RSpec's new message expectation syntax - Tea is awesome.](http://teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/)
 
 ### Message expectations that are actually method stubs
 
@@ -390,14 +445,14 @@ SomeClass.any_instance.should_receive(:foo).at_least(0)
 
 # Converted
 allow(obj).to receive(:foo)
-obj.stub(:foo) # with `--disable allow_to_receive`
+obj.stub(:foo) # with `--keep stub`
 
 allow_any_instance_of(SomeClass).to receive(:foo)
-SomeClass.any_instance.stub(:foo) # with `--disable allow_to_receive`
+SomeClass.any_instance.stub(:foo) # with `--keep stub`
 ```
 
-* Disabled by: `--disable deprecated`
-* Related Information: [Don't allow at_least(0) · rspec/rspec-mocks](https://github.com/rspec/rspec-mocks/issues/133)
+* Disabled by: `--keep deprecated`
+* See also: [Don't allow at_least(0) · rspec/rspec-mocks](https://github.com/rspec/rspec-mocks/issues/133)
 
 ### Method stubs
 
@@ -422,8 +477,8 @@ allow(obj).to receive(:bar).and_return(2)
 allow_any_instance_of(SomeClass).to receive(:foo)
 ```
 
-* Disabled by: `--disable allow_to_receive`
-* Related Information: [RSpec's new message expectation syntax - Tea is awesome.](http://teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/)
+* Disabled by: `--keep stub`
+* See also: [RSpec's new message expectation syntax - Tea is awesome.](http://teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/)
 
 ### Deprecated method stub aliases
 
@@ -433,12 +488,12 @@ obj.stub!(:foo)
 obj.unstub!(:foo)
 
 # Converted
-obj.stub(:foo) # with `--disable allow_to_receive`
+obj.stub(:foo) # with `--keep stub`
 obj.unstub(:foo)
 ```
 
-* Disabled by: `--disable deprecated`
-* Related Information: [Consider deprecating and/or removing #stub! and #unstub! at some point · rspec/rspec-mocks](https://github.com/rspec/rspec-mocks/issues/122)
+* Disabled by: `--keep deprecated`
+* See also: [Consider deprecating and/or removing #stub! and #unstub! at some point · rspec/rspec-mocks](https://github.com/rspec/rspec-mocks/issues/122)
 
 ### Method stubs with deprecated specification of number of times
 
@@ -449,11 +504,11 @@ obj.stub(:foo).at_least(0)
 
 # Converted
 allow(obj).to receive(:foo)
-obj.stub(:foo) # with `--disable allow_to_receive`
+obj.stub(:foo) # with `--keep stub`
 ```
 
-* Disabled by: `--disable deprecated`
-* Related Information: [Don't allow at_least(0) · rspec/rspec-mocks](https://github.com/rspec/rspec-mocks/issues/133)
+* Disabled by: `--keep deprecated`
+* See also: [Don't allow at_least(0) · rspec/rspec-mocks](https://github.com/rspec/rspec-mocks/issues/133)
 
 ### Deprecated test double aliases
 
@@ -466,8 +521,8 @@ mock('something')
 double('something')
 ```
 
-* Disabled by: `--disable deprecated`
-* Related Information: [Deprecate "stub" for doubles · rspec/rspec-mocks](https://github.com/rspec/rspec-mocks/issues/214)
+* Disabled by: `--keep deprecated`
+* See also: [Deprecate "stub" for doubles · rspec/rspec-mocks](https://github.com/rspec/rspec-mocks/issues/214)
 
 ## Compatibility