ripper_ruby_parser 1.1.1 → 1.1.2

data/test/end_to_end/comparison_test.rb

@@ -1,7 +1,7 @@
 require File.expand_path('../test_helper.rb', File.dirname(__FILE__))
 require 'ruby_parser'
 
-describe "Using RipperRubyParser and RubyParser" do
+describe 'Using RipperRubyParser and RubyParser' do
   let :newparser do
     RipperRubyParser::Parser.new
   end
@@ -10,12 +10,12 @@ describe "Using RipperRubyParser and RubyParser" do
     RubyParser.new
   end
 
-  describe "for a simple well known program" do
+  describe 'for a simple well known program' do
     let :program do
       "puts 'Hello World'"
     end
 
-    it "gives the same result" do
+    it 'gives the same result' do
       original = oldparser.parse program
       imitation = newparser.parse program
 
@@ -23,7 +23,7 @@ describe "Using RipperRubyParser and RubyParser" do
     end
   end
 
-  describe "for a more complex program" do
+  describe 'for a more complex program' do
     let :program do
       <<-END
       module Quux
@@ -44,7 +44,7 @@ describe "Using RipperRubyParser and RubyParser" do
       END
     end
 
-    it "gives the same result" do
+    it 'gives the same result' do
       original = oldparser.parse program
       imitation = newparser.parse program
 
@@ -52,12 +52,12 @@ describe "Using RipperRubyParser and RubyParser" do
     end
   end
 
-  describe "for an example with yield from Reek" do
+  describe 'for an example with yield from Reek' do
     let :program do
       'def fred() yield(3) if block_given?; end'
     end
 
-    it "gives the same result" do
+    it 'gives the same result' do
       original = oldparser.parse program
       imitation = newparser.parse program
 
@@ -65,7 +65,7 @@ describe "Using RipperRubyParser and RubyParser" do
     end
   end
 
-  describe "for an example with floats from Reek" do
+  describe 'for an example with floats from Reek' do
     let :program do
       <<-END
         def total_envy
@@ -78,7 +78,7 @@ describe "Using RipperRubyParser and RubyParser" do
       END
     end
 
-    it "gives the same result" do
+    it 'gives the same result' do
       original = oldparser.parse program
       imitation = newparser.parse program
 
@@ -86,7 +86,7 @@ describe "Using RipperRubyParser and RubyParser" do
     end
   end
 
-  describe "for an example with operators and explicit block parameter from Reek" do
+  describe 'for an example with operators and explicit block parameter from Reek' do
     let :program do
       <<-END
         def parse(arg, argv, &error)
@@ -105,7 +105,7 @@ describe "Using RipperRubyParser and RubyParser" do
       END
     end
 
-    it "gives the same result" do
+    it 'gives the same result' do
       original = oldparser.parse program
       imitation = newparser.parse program
 
@@ -113,12 +113,12 @@ describe "Using RipperRubyParser and RubyParser" do
     end
   end
 
-  describe "for an example of a complex regular expression from Reek" do
+  describe 'for an example of a complex regular expression from Reek' do
     let :program do
       "/(\#{@types})\\s*(\\w+)\\s*\\(([^)]*)\\)/"
     end
 
-    it "gives the same result" do
+    it 'gives the same result' do
       original = oldparser.parse program
       imitation = newparser.parse program
 
@@ -126,8 +126,8 @@ describe "Using RipperRubyParser and RubyParser" do
     end
   end
 
-  describe "for an example with regular expressions with different encoding flags" do
-    it "gives the same result" do
+  describe 'for an example with regular expressions with different encoding flags' do
+    it 'gives the same result' do
       program = <<-END
         regular = /foo/
         noenc = /foo/n

data/test/end_to_end/error_conditions_test.rb

@@ -1,50 +1,50 @@
 require File.expand_path('../test_helper.rb', File.dirname(__FILE__))
 
-describe "Handling errors" do
-  describe "RipperRubyParser::Parser#parse" do
+describe 'Handling errors' do
+  describe 'RipperRubyParser::Parser#parse' do
     let :newparser do
       RipperRubyParser::Parser.new
     end
 
-    it "raises an error for an incomplete source" do
+    it 'raises an error for an incomplete source' do
       proc {
-        newparser.parse "def foo"
+        newparser.parse 'def foo'
       }.must_raise RipperRubyParser::SyntaxError
     end
 
-    it "raises an error for an invalid class name" do
+    it 'raises an error for an invalid class name' do
       proc {
-        newparser.parse("class foo; end")
+        newparser.parse('class foo; end')
       }.must_raise RipperRubyParser::SyntaxError
     end
 
-    it "raises an error aliasing $1 as foo" do
+    it 'raises an error aliasing $1 as foo' do
       proc {
-        newparser.parse "alias foo $1"
+        newparser.parse 'alias foo $1'
       }.must_raise RipperRubyParser::SyntaxError
     end
 
-    it "raises an error aliasing foo as $1" do
+    it 'raises an error aliasing foo as $1' do
       proc {
-        newparser.parse "alias $1 foo"
+        newparser.parse 'alias $1 foo'
       }.must_raise RipperRubyParser::SyntaxError
     end
 
-    it "raises an error aliasing $2 as $1" do
+    it 'raises an error aliasing $2 as $1' do
       proc {
-        newparser.parse "alias $1 $2"
+        newparser.parse 'alias $1 $2'
       }.must_raise RipperRubyParser::SyntaxError
     end
 
-    it "raises an error assigning to $1" do
+    it 'raises an error assigning to $1' do
       proc {
-        newparser.parse "$1 = foo"
+        newparser.parse '$1 = foo'
       }.must_raise RipperRubyParser::SyntaxError
     end
 
-    it "raises an error using an invalid parameter name" do
+    it 'raises an error using an invalid parameter name' do
       proc {
-        newparser.parse "def foo(BAR); end"
+        newparser.parse 'def foo(BAR); end'
       }.must_raise RipperRubyParser::SyntaxError
     end
   end

data/test/end_to_end/lib_comparison_test.rb

@@ -1,7 +1,7 @@
 require File.expand_path('../test_helper.rb', File.dirname(__FILE__))
 require 'ruby_parser'
 
-describe "Using RipperRubyParser and RubyParser" do
+describe 'Using RipperRubyParser and RubyParser' do
   let :newparser do
     RipperRubyParser::Parser.new
   end
@@ -10,13 +10,13 @@ describe "Using RipperRubyParser and RubyParser" do
     RubyParser.new
   end
 
-  Dir.glob("lib/**/*.rb").each do |file|
+  Dir.glob('lib/**/*.rb').each do |file|
     describe "for #{file}" do
       let :program do
         File.read file
       end
 
-      it "gives the same result" do
+      it 'gives the same result' do
         original = oldparser.parse program
         imitation = newparser.parse program
 

data/test/end_to_end/line_numbering_test.rb

@@ -1,7 +1,7 @@
 require File.expand_path('../test_helper.rb', File.dirname(__FILE__))
 require 'ruby_parser'
 
-describe "Using RipperRubyParser and RubyParser" do
+describe 'Using RipperRubyParser and RubyParser' do
   def to_line_numbers exp
     exp.map! do |sub_exp|
       if sub_exp.is_a? Sexp
@@ -26,7 +26,7 @@ describe "Using RipperRubyParser and RubyParser" do
     RubyParser.new
   end
 
-  describe "for a multi-line program" do
+  describe 'for a multi-line program' do
     let :program do
       <<-END
       class Foo
@@ -50,11 +50,11 @@ describe "Using RipperRubyParser and RubyParser" do
       newparser.parse program
     end
 
-    it "gives the same result" do
+    it 'gives the same result' do
       imitation.must_equal original
     end
 
-    it "gives the same result with line numbers" do
+    it 'gives the same result with line numbers' do
       formatted(to_line_numbers(imitation)).
         must_equal formatted(to_line_numbers(original))
     end

data/test/end_to_end/samples_comparison_test.rb

@@ -1,7 +1,7 @@
 require File.expand_path('../test_helper.rb', File.dirname(__FILE__))
 require 'ruby_parser'
 
-describe "Using RipperRubyParser and RubyParser" do
+describe 'Using RipperRubyParser and RubyParser' do
   let :newparser do
     RipperRubyParser::Parser.new
   end
@@ -10,7 +10,7 @@ describe "Using RipperRubyParser and RubyParser" do
     RubyParser.new
   end
 
-  Dir.glob(File.expand_path("../samples/*.rb", File.dirname(__FILE__))).each do |file|
+  Dir.glob(File.expand_path('../samples/*.rb', File.dirname(__FILE__))).each do |file|
     describe "for #{file}" do
       let :program do
         File.read file
@@ -25,11 +25,11 @@ describe "Using RipperRubyParser and RubyParser" do
         newparser.parse program
       end
 
-      it "gives the same result" do
+      it 'gives the same result' do
         formatted(imitation).must_equal formatted(original)
       end
 
-      it "gives the same result with comments" do
+      it 'gives the same result with comments' do
         formatted(to_comments(imitation)).
           must_equal formatted(to_comments(original))
       end

data/test/end_to_end/test_comparison_test.rb

@@ -1,7 +1,7 @@
 require File.expand_path('../test_helper.rb', File.dirname(__FILE__))
 require 'ruby_parser'
 
-describe "Using RipperRubyParser and RubyParser" do
+describe 'Using RipperRubyParser and RubyParser' do
   let :newparser do
     RipperRubyParser::Parser.new
   end
@@ -10,13 +10,13 @@ describe "Using RipperRubyParser and RubyParser" do
     RubyParser.new
   end
 
-  Dir.glob("test/**/*.rb").each do |file|
+  Dir.glob('test/**/*.rb').each do |file|
     describe "for #{file}" do
       let :program do
         File.read file
       end
 
-      it "gives the same result" do
+      it 'gives the same result' do
         # Clone string because ruby_parser destroys it when there's a heredoc
         # inside.
         copy = program.clone

data/test/pt_testcase/pt_test.rb

@@ -7,17 +7,17 @@ class TestParser < RipperRubyParser::Parser
   end
 end
 
-SKIPPED_TESTS = ["dstr_heredoc_windoze_sucks"].freeze
+SKIPPED_TESTS = ['dstr_heredoc_windoze_sucks'].freeze
 
 class RubyParserTestCase < ParseTreeTestCase
   def self.previous _key
-    "Ruby"
+    'Ruby'
   end
 
   def self.generate_test klass, node, data, input_name, output_name
     if data['Ruby'].is_a? Array
       klass.send :define_method, "test_#{node}" do
-        skip "Not a parser test"
+        skip 'Not a parser test'
       end
       return
     end
@@ -29,7 +29,7 @@ class RubyParserTestCase < ParseTreeTestCase
       return
     end
 
-    output_name = "ParseTree"
+    output_name = 'ParseTree'
 
     super
   end

data/test/test_helper.rb

@@ -1,6 +1,6 @@
 require 'simplecov'
 SimpleCov.start do
-  add_filter "/test/"
+  add_filter '/test/'
 end
 
 require 'minitest/autorun'

data/test/unit/commenting_ripper_parser_test.rb

@@ -8,110 +8,110 @@ describe RipperRubyParser::CommentingRipperParser do
 
   def empty_params_list
     @empty_params_list ||= begin
-                             num_params = RUBY_VERSION < "2.0.0" ? 5 : 7
+                             num_params = RUBY_VERSION < '2.0.0' ? 5 : 7
                              s(:params, *([nil] * num_params))
                            end
   end
 
-  describe "handling comments" do
-    it "produces a comment node surrounding a commented def" do
+  describe 'handling comments' do
+    it 'produces a comment node surrounding a commented def' do
       result = parse_with_builder "# Foo\ndef foo; end"
       result.must_equal s(:program,
                           s(s(:comment,
                               "# Foo\n",
                               s(:def,
-                                s(:@ident, "foo", s(2, 4)),
+                                s(:@ident, 'foo', s(2, 4)),
                                 empty_params_list,
                                 s(:bodystmt, s(s(:void_stmt)), nil, nil, nil)))))
     end
 
-    it "produces a blank comment node surrounding a def that has no comment" do
-      result = parse_with_builder "def foo; end"
+    it 'produces a blank comment node surrounding a def that has no comment' do
+      result = parse_with_builder 'def foo; end'
       result.must_equal s(:program,
                           s(s(:comment,
-                              "",
+                              '',
                               s(:def,
-                                s(:@ident, "foo", s(1, 4)),
+                                s(:@ident, 'foo', s(1, 4)),
                                 empty_params_list,
                                 s(:bodystmt, s(s(:void_stmt)), nil, nil, nil)))))
     end
 
-    it "produces a comment node surrounding a commented class" do
+    it 'produces a comment node surrounding a commented class' do
       result = parse_with_builder "# Foo\nclass Foo; end"
       result.must_equal s(:program,
                           s(s(:comment,
                               "# Foo\n",
                               s(:class,
-                                s(:const_ref, s(:@const, "Foo", s(2, 6))),
+                                s(:const_ref, s(:@const, 'Foo', s(2, 6))),
                                 nil,
                                 s(:bodystmt, s(s(:void_stmt)), nil, nil, nil)))))
     end
 
-    it "produce a blank comment node surrounding a class that has no comment" do
-      result = parse_with_builder "class Foo; end"
+    it 'produce a blank comment node surrounding a class that has no comment' do
+      result = parse_with_builder 'class Foo; end'
       result.must_equal s(:program,
                           s(s(:comment,
-                              "",
+                              '',
                               s(:class,
-                                s(:const_ref, s(:@const, "Foo", s(1, 6))),
+                                s(:const_ref, s(:@const, 'Foo', s(1, 6))),
                                 nil,
                                 s(:bodystmt, s(s(:void_stmt)), nil, nil, nil)))))
     end
 
-    it "produces a comment node surrounding a commented module" do
+    it 'produces a comment node surrounding a commented module' do
       result = parse_with_builder "# Foo\nmodule Foo; end"
       result.must_equal s(:program,
                           s(s(:comment,
                               "# Foo\n",
                               s(:module,
-                                s(:const_ref, s(:@const, "Foo", s(2, 7))),
+                                s(:const_ref, s(:@const, 'Foo', s(2, 7))),
                                 s(:bodystmt, s(s(:void_stmt)), nil, nil, nil)))))
     end
 
-    it "produces a blank comment node surrounding a module that has no comment" do
-      result = parse_with_builder "module Foo; end"
+    it 'produces a blank comment node surrounding a module that has no comment' do
+      result = parse_with_builder 'module Foo; end'
       result.must_equal s(:program,
                           s(s(:comment,
-                              "",
+                              '',
                               s(:module,
-                                s(:const_ref, s(:@const, "Foo", s(1, 7))),
+                                s(:const_ref, s(:@const, 'Foo', s(1, 7))),
                                 s(:bodystmt, s(s(:void_stmt)), nil, nil, nil)))))
     end
 
-    it "is not confused by a symbol containing a keyword" do
-      result = parse_with_builder ":class; def foo; end"
+    it 'is not confused by a symbol containing a keyword' do
+      result = parse_with_builder ':class; def foo; end'
       result.must_equal s(:program,
-                          s(s(:symbol_literal, s(:symbol, s(:@kw, "class", s(1, 1)))),
+                          s(s(:symbol_literal, s(:symbol, s(:@kw, 'class', s(1, 1)))),
                             s(:comment,
-                              "",
+                              '',
                               s(:def,
-                                s(:@ident, "foo", s(1, 12)),
+                                s(:@ident, 'foo', s(1, 12)),
                                 empty_params_list,
                                 s(:bodystmt, s(s(:void_stmt)), nil, nil, nil)))))
     end
 
-    it "is not confused by a dynamic symbol" do
+    it 'is not confused by a dynamic symbol' do
       result = parse_with_builder ":'foo'; def bar; end"
       result.must_equal s(:program,
-                          s(s(:dyna_symbol, s(s(:@tstring_content, "foo", s(1, 2)))),
+                          s(s(:dyna_symbol, s(s(:@tstring_content, 'foo', s(1, 2)))),
                             s(:comment,
-                              "",
+                              '',
                               s(:def,
-                                s(:@ident, "bar", s(1, 12)),
+                                s(:@ident, 'bar', s(1, 12)),
                                 empty_params_list,
                                 s(:bodystmt, s(s(:void_stmt)), nil, nil, nil)))))
     end
 
-    it "is not confused by a dynamic symbol containing a class definition" do
+    it 'is not confused by a dynamic symbol containing a class definition' do
       result = parse_with_builder ":\"foo\#{class Bar;end}\""
       result.must_equal s(:program,
                           s(s(:dyna_symbol,
-                              s(s(:@tstring_content, "foo", s(1, 2)),
+                              s(s(:@tstring_content, 'foo', s(1, 2)),
                                 s(:string_embexpr,
                                   s(s(:comment,
-                                      "",
+                                      '',
                                       s(:class,
-                                        s(:const_ref, s(:@const, "Bar", s(1, 13))),
+                                        s(:const_ref, s(:@const, 'Bar', s(1, 13))),
                                         nil,
                                         s(:bodystmt, s(s(:void_stmt)), nil, nil, nil)))))))))
     end

data/test/unit/parser_assignment_test.rb

@@ -1,25 +1,25 @@
 require File.expand_path('../test_helper.rb', File.dirname(__FILE__))
 
 describe RipperRubyParser::Parser do
-  describe "#parse" do
-    describe "for single assignment" do
-      it "works when assigning to a namespaced constant" do
-        "Foo::Bar = baz".
+  describe '#parse' do
+    describe 'for single assignment' do
+      it 'works when assigning to a namespaced constant' do
+        'Foo::Bar = baz'.
           must_be_parsed_as s(:cdecl,
                               s(:colon2, s(:const, :Foo), :Bar),
                               s(:call, nil, :baz))
       end
 
-      it "works when assigning to constant in the root namespace" do
-        "::Foo = bar".
+      it 'works when assigning to constant in the root namespace' do
+        '::Foo = bar'.
           must_be_parsed_as s(:cdecl,
                               s(:colon3, :Foo),
                               s(:call, nil, :bar))
       end
 
-      describe "with a right-hand splat" do
+      describe 'with a right-hand splat' do
         specify do
-          "foo = *bar".
+          'foo = *bar'.
             must_be_parsed_as s(:lasgn, :foo,
                                 s(:svalue,
                                   s(:splat,
@@ -27,7 +27,7 @@ describe RipperRubyParser::Parser do
         end
 
         specify do
-          "foo = bar, *baz".
+          'foo = bar, *baz'.
             must_be_parsed_as s(:lasgn, :foo,
                                 s(:svalue,
                                   s(:array,
@@ -37,9 +37,9 @@ describe RipperRubyParser::Parser do
         end
       end
 
-      describe "with several items on the right hand side" do
+      describe 'with several items on the right hand side' do
         specify do
-          "foo = bar, baz".
+          'foo = bar, baz'.
             must_be_parsed_as s(:lasgn, :foo,
                                 s(:svalue,
                                   s(:array,
@@ -48,9 +48,9 @@ describe RipperRubyParser::Parser do
         end
       end
 
-      describe "with an array literal on the right hand side" do
+      describe 'with an array literal on the right hand side' do
         specify do
-          "foo = [bar, baz]".
+          'foo = [bar, baz]'.
             must_be_parsed_as s(:lasgn, :foo,
                                 s(:array,
                                   s(:call, nil, :bar),
@@ -59,16 +59,16 @@ describe RipperRubyParser::Parser do
       end
     end
 
-    describe "for multiple assignment" do
+    describe 'for multiple assignment' do
       specify do
-        "foo, * = bar".
+        'foo, * = bar'.
           must_be_parsed_as s(:masgn,
                               s(:array, s(:lasgn, :foo), s(:splat)),
                               s(:to_ary, s(:call, nil, :bar)))
       end
 
       specify do
-        "(foo, *bar) = baz".
+        '(foo, *bar) = baz'.
           must_be_parsed_as s(:masgn,
                               s(:array,
                                 s(:lasgn, :foo),
@@ -77,7 +77,7 @@ describe RipperRubyParser::Parser do
       end
 
       specify do
-        "*foo, bar = baz".
+        '*foo, bar = baz'.
           must_be_parsed_as s(:masgn,
                               s(:array,
                                 s(:splat, s(:lasgn, :foo)),
@@ -86,9 +86,9 @@ describe RipperRubyParser::Parser do
       end
     end
 
-    describe "for assignment to a collection element" do
-      it "handles multiple indices" do
-        "foo[bar, baz] = qux".
+    describe 'for assignment to a collection element' do
+      it 'handles multiple indices' do
+        'foo[bar, baz] = qux'.
           must_be_parsed_as s(:attrasgn,
                               s(:call, nil, :foo),
                               :[]=,
@@ -98,10 +98,10 @@ describe RipperRubyParser::Parser do
       end
     end
 
-    describe "for operator assignment" do
-      describe "assigning to a collection element" do
-        it "handles multiple indices" do
-          "foo[bar, baz] += qux".
+    describe 'for operator assignment' do
+      describe 'assigning to a collection element' do
+        it 'handles multiple indices' do
+          'foo[bar, baz] += qux'.
             must_be_parsed_as s(:op_asgn1,
                                 s(:call, nil, :foo),
                                 s(:arglist,
@@ -111,24 +111,24 @@ describe RipperRubyParser::Parser do
                                 s(:call, nil, :qux))
         end
 
-        it "works with &&=" do
-          "foo &&= bar".
+        it 'works with &&=' do
+          'foo &&= bar'.
             must_be_parsed_as s(:op_asgn_and,
                                 s(:lvar, :foo), s(:lasgn, :foo, s(:call, nil, :bar)))
         end
       end
     end
 
-    describe "for multiple assignment" do
-      describe "with a right-hand splat" do
+    describe 'for multiple assignment' do
+      describe 'with a right-hand splat' do
         specify do
-          "foo, bar = *baz".
+          'foo, bar = *baz'.
             must_be_parsed_as s(:masgn,
                                 s(:array, s(:lasgn, :foo), s(:lasgn, :bar)),
                                 s(:splat, s(:call, nil, :baz)))
         end
         specify do
-          "foo, bar = baz, *qux".
+          'foo, bar = baz, *qux'.
             must_be_parsed_as s(:masgn,
                                 s(:array, s(:lasgn, :foo), s(:lasgn, :bar)),
                                 s(:array,