regexp_parser 1.5.0 → 1.8.0

data/spec/lexer/refcalls_spec.rb

@@ -1,54 +1,55 @@
 require 'spec_helper'
 
 RSpec.describe('RefCall lexing') do
-  tests = {
-    # Traditional numerical group back-reference
-    '(abc)\1'           => [3, :backref, :number,                '\1',         5,  7, 0, 0, 0],
-
-    # Group back-references, named, numbered, and relative
-    '(?<X>abc)\k<X>'    => [3, :backref, :name_ref,              '\k<X>',      9, 14, 0, 0, 0],
-    "(?<X>abc)\\k'X'"   => [3, :backref, :name_ref,              "\\k'X'",     9, 14, 0, 0, 0],
-
-    '(abc)\k<1>'        => [3, :backref, :number_ref,            '\k<1>',      5, 10, 0, 0, 0],
-    "(abc)\\k'1'"       => [3, :backref, :number_ref,            "\\k'1'",     5, 10, 0, 0, 0],
-
-    '(abc)\k<-1>'       => [3, :backref, :number_rel_ref,        '\k<-1>',     5, 11, 0, 0, 0],
-    "(abc)\\k'-1'"      => [3, :backref, :number_rel_ref,        "\\k'-1'",    5, 11, 0, 0, 0],
-
-    # Sub-expression invocation, named, numbered, and relative
-    '(?<X>abc)\g<X>'    => [3, :backref, :name_call,             '\g<X>',      9, 14, 0, 0, 0],
-    "(?<X>abc)\\g'X'"   => [3, :backref, :name_call,             "\\g'X'",     9, 14, 0, 0, 0],
-
-    '(abc)\g<1>'        => [3, :backref, :number_call,           '\g<1>',      5, 10, 0, 0, 0],
-    "(abc)\\g'1'"       => [3, :backref, :number_call,           "\\g'1'",     5, 10, 0, 0, 0],
-
-    '(abc)\g<-1>'       => [3, :backref, :number_rel_call,       '\g<-1>',     5, 11, 0, 0, 0],
-    "(abc)\\g'-1'"      => [3, :backref, :number_rel_call,       "\\g'-1'",    5, 11, 0, 0, 0],
-
-    '(abc)\g<+1>'       => [3, :backref, :number_rel_call,       '\g<+1>',     5, 11, 0, 0, 0],
-    "(abc)\\g'+1'"      => [3, :backref, :number_rel_call,       "\\g'+1'",    5, 11, 0, 0, 0],
-
-    # Group back-references, with nesting level
-    '(?<X>abc)\k<X-0>'  => [3, :backref, :name_recursion_ref,    '\k<X-0>',    9, 16, 0, 0, 0],
-    "(?<X>abc)\\k'X-0'" => [3, :backref, :name_recursion_ref,    "\\k'X-0'",   9, 16, 0, 0, 0],
-
-    '(abc)\k<1-0>'      => [3, :backref, :number_recursion_ref,  '\k<1-0>',    5, 12, 0, 0, 0],
-    "(abc)\\k'1-0'"     => [3, :backref, :number_recursion_ref,  "\\k'1-0'",   5, 12, 0, 0, 0],
-  }
-
-  tests.each_with_index do |(pattern, (index, type, token, text, ts, te, level, set_level, conditional_level)), count|
-    specify("lexer_#{type}_#{token}_#{count}") do
-      tokens = RL.lex(pattern, 'ruby/1.9')
-      struct = tokens.at(index)
-
-      expect(struct.type).to eq type
-      expect(struct.token).to eq token
-      expect(struct.text).to eq text
-      expect(struct.ts).to eq ts
-      expect(struct.te).to eq te
-      expect(struct.level).to eq level
-      expect(struct.set_level).to eq set_level
-      expect(struct.conditional_level).to eq conditional_level
-    end
-  end
+  # Traditional numerical group back-reference
+  include_examples 'lex', '(abc)\1',
+    3 => [:backref, :number,                '\1',         5,  7, 0, 0, 0]
+
+  # Group back-references, named, numbered, and relative
+  include_examples 'lex', '(?<X>abc)\k<X>',
+    3 => [:backref, :name_ref,              '\k<X>',      9, 14, 0, 0, 0]
+  include_examples 'lex', "(?<X>abc)\\k'X'",
+    3 => [:backref, :name_ref,              "\\k'X'",     9, 14, 0, 0, 0]
+
+  include_examples 'lex', '(abc)\k<1>',
+    3 => [:backref, :number_ref,            '\k<1>',      5, 10, 0, 0, 0]
+  include_examples 'lex', "(abc)\\k'1'",
+    3 => [:backref, :number_ref,            "\\k'1'",     5, 10, 0, 0, 0]
+
+  include_examples 'lex', '(abc)\k<-1>',
+    3 => [:backref, :number_rel_ref,        '\k<-1>',     5, 11, 0, 0, 0]
+  include_examples 'lex', "(abc)\\k'-1'",
+    3 => [:backref, :number_rel_ref,        "\\k'-1'",    5, 11, 0, 0, 0]
+
+  # Sub-expression invocation, named, numbered, and relative
+  include_examples 'lex', '(?<X>abc)\g<X>',
+    3 => [:backref, :name_call,             '\g<X>',      9, 14, 0, 0, 0]
+  include_examples 'lex', "(?<X>abc)\\g'X'",
+    3 => [:backref, :name_call,             "\\g'X'",     9, 14, 0, 0, 0]
+
+  include_examples 'lex', '(abc)\g<1>',
+    3 => [:backref, :number_call,           '\g<1>',      5, 10, 0, 0, 0]
+  include_examples 'lex', "(abc)\\g'1'",
+    3 => [:backref, :number_call,           "\\g'1'",     5, 10, 0, 0, 0]
+
+  include_examples 'lex', '(abc)\g<-1>',
+    3 => [:backref, :number_rel_call,       '\g<-1>',     5, 11, 0, 0, 0]
+  include_examples 'lex', "(abc)\\g'-1'",
+    3 => [:backref, :number_rel_call,       "\\g'-1'",    5, 11, 0, 0, 0]
+
+  include_examples 'lex', '(abc)\g<+1>',
+    3 => [:backref, :number_rel_call,       '\g<+1>',     5, 11, 0, 0, 0]
+  include_examples 'lex', "(abc)\\g'+1'",
+    3 => [:backref, :number_rel_call,       "\\g'+1'",    5, 11, 0, 0, 0]
+
+  # Group back-references, with nesting level
+  include_examples 'lex', '(?<X>abc)\k<X-0>',
+    3 => [:backref, :name_recursion_ref,    '\k<X-0>',    9, 16, 0, 0, 0]
+  include_examples 'lex', "(?<X>abc)\\k'X-0'",
+    3 => [:backref, :name_recursion_ref,    "\\k'X-0'",   9, 16, 0, 0, 0]
+
+  include_examples 'lex', '(abc)\k<1-0>',
+    3 => [:backref, :number_recursion_ref,  '\k<1-0>',    5, 12, 0, 0, 0]
+  include_examples 'lex', "(abc)\\k'1-0'",
+    3 => [:backref, :number_recursion_ref,  "\\k'1-0'",   5, 12, 0, 0, 0]
 end

data/spec/parser/all_spec.rb

@@ -2,7 +2,11 @@ require 'spec_helper'
 
 RSpec.describe(Regexp::Parser) do
   specify('parse returns a root expression') do
-    expect(RP.parse('abc')).to be_instance_of(Regexp::Expression::Root)
+    expect(RP.parse('abc')).to be_instance_of(Root)
+  end
+
+  specify('parse can be called with block') do
+    expect(RP.parse('abc') { |root| root.class }).to eq Root
   end
 
   specify('parse root contains expressions') do
@@ -10,6 +14,14 @@ RSpec.describe(Regexp::Parser) do
     expect(root.expressions).to all(be_a Regexp::Expression::Base)
   end
 
+  specify('parse root options mi') do
+    root = RP.parse(/[abc]/mi, 'ruby/1.8')
+
+    expect(root.m?).to be true
+    expect(root.i?).to be true
+    expect(root.x?).to be false
+  end
+
   specify('parse node types') do
     root = RP.parse('^(one){2,3}([^d\\]efm-qz\\,\\-]*)(ghi)+$')
 

data/spec/parser/anchors_spec.rb

@@ -1,31 +1,17 @@
 require 'spec_helper'
 
 RSpec.describe('Anchor parsing') do
-  tests = {
-    '^a'      => [0, :anchor,   :bol,                 Anchor::BOL],
-    'a$'      => [1, :anchor,   :eol,                 Anchor::EOL],
+  include_examples 'parse', /^a/,   0 =>  [:anchor,   :bol,               Anchor::BOL]
+  include_examples 'parse', /a$/,   1 =>  [:anchor,   :eol,               Anchor::EOL]
 
-    '\Aa'     => [0, :anchor,   :bos,                 Anchor::BOS],
-    'a\z'     => [1, :anchor,   :eos,                 Anchor::EOS],
-    'a\Z'     => [1, :anchor,   :eos_ob_eol,          Anchor::EOSobEOL],
+  include_examples 'parse', /\Aa/,  0 =>  [:anchor,   :bos,               Anchor::BOS]
+  include_examples 'parse', /a\z/,  1 =>  [:anchor,   :eos,               Anchor::EOS]
+  include_examples 'parse', /a\Z/,  1 =>  [:anchor,   :eos_ob_eol,        Anchor::EOSobEOL]
 
-    'a\b'     => [1, :anchor,   :word_boundary,       Anchor::WordBoundary],
-    'a\B'     => [1, :anchor,   :nonword_boundary,    Anchor::NonWordBoundary],
+  include_examples 'parse', /a\b/,  1 =>  [:anchor,   :word_boundary,     Anchor::WordBoundary]
+  include_examples 'parse', /a\B/,  1 =>  [:anchor,   :nonword_boundary,  Anchor::NonWordBoundary]
 
-    'a\G'     => [1, :anchor,   :match_start,         Anchor::MatchStart],
+  include_examples 'parse', /a\G/,  1 =>  [:anchor,   :match_start,       Anchor::MatchStart]
 
-    "\\\\Aa"  => [0, :escape,   :backslash,           EscapeSequence::Literal],
-  }
-
-  tests.each_with_index do |(pattern, (index, type, token, klass)), count|
-    specify("parse_anchor_#{token}_#{count}") do
-      root = RP.parse(pattern, 'ruby/1.9')
-      exp = root.expressions.at(index)
-
-      expect(exp).to be_a(klass)
-
-      expect(exp.type).to eq type
-      expect(exp.token).to eq token
-    end
-  end
+  include_examples 'parse', /\\A/,  0 =>  [:escape,   :backslash,         EscapeSequence::Literal]
 end

data/spec/parser/conditionals_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 RSpec.describe('Conditional parsing') do
   specify('parse conditional') do
-    regexp = Regexp.new('(?<A>a)(?(<A>)T|F)/')
+    regexp = /(?<A>a)(?(<A>)T|F)/
 
     root = RP.parse(regexp, 'ruby/2.0')
     exp = root[1]
@@ -16,7 +16,7 @@ RSpec.describe('Conditional parsing') do
   end
 
   specify('parse conditional condition') do
-    regexp = Regexp.new('(?<A>a)(?(<A>)T|F)/')
+    regexp = /(?<A>a)(?(<A>)T|F)/
 
     root = RP.parse(regexp, 'ruby/2.0')
     exp = root[1].condition
@@ -31,7 +31,7 @@ RSpec.describe('Conditional parsing') do
   end
 
   specify('parse conditional condition with number ref') do
-    regexp = Regexp.new('(a)(?(1)T|F)/')
+    regexp = /(a)(?(1)T|F)/
 
     root = RP.parse(regexp, 'ruby/2.0')
     exp = root[1].condition
@@ -46,7 +46,7 @@ RSpec.describe('Conditional parsing') do
   end
 
   specify('parse conditional nested groups') do
-    regexp = Regexp.new('((a)|(b)|((?(2)(c(d|e)+)?|(?(3)f|(?(4)(g|(h)(i)))))))/')
+    regexp = /((a)|(b)|((?(2)(c(d|e)+)?|(?(3)f|(?(4)(g|(h)(i)))))))/
 
     root = RP.parse(regexp, 'ruby/2.0')
 
@@ -80,7 +80,7 @@ RSpec.describe('Conditional parsing') do
   end
 
   specify('parse conditional nested') do
-    regexp = Regexp.new('(a(b(c(d)(e))))(?(1)(?(2)d|(?(3)e|f))|(?(4)(?(5)g|h)))/')
+    regexp = /(a(b(c(d)(e))))(?(1)(?(2)d|(?(3)e|f))|(?(4)(?(5)g|h)))/
 
     root = RP.parse(regexp, 'ruby/2.0')
 
@@ -102,7 +102,7 @@ RSpec.describe('Conditional parsing') do
   end
 
   specify('parse conditional nested alternation') do
-    regexp = Regexp.new('(a)(?(1)(b|c|d)|(e|f|g))(h)(?(2)(i|j|k)|(l|m|n))|o|p/')
+    regexp = /(a)(?(1)(b|c|d)|(e|f|g))(h)(?(2)(i|j|k)|(l|m|n))|o|p/
 
     root = RP.parse(regexp, 'ruby/2.0')
 
@@ -125,7 +125,7 @@ RSpec.describe('Conditional parsing') do
   end
 
   specify('parse conditional extra separator') do
-    regexp = Regexp.new('(?<A>a)(?(<A>)T|)/')
+    regexp = /(?<A>a)(?(<A>)T|)/
 
     root = RP.parse(regexp, 'ruby/2.0')
     branches = root[1].branches
@@ -146,7 +146,7 @@ RSpec.describe('Conditional parsing') do
   end
 
   specify('parse conditional quantified') do
-    regexp = Regexp.new('(foo)(?(1)\d|(\w)){42}/')
+    regexp = /(foo)(?(1)\d|(\w)){42}/
 
     root = RP.parse(regexp, 'ruby/2.0')
     conditional = root[1]
@@ -158,7 +158,7 @@ RSpec.describe('Conditional parsing') do
   end
 
   specify('parse conditional branch content quantified') do
-    regexp = Regexp.new('(foo)(?(1)\d{23}|(\w){42})/')
+    regexp = /(foo)(?(1)\d{23}|(\w){42})/
 
     root = RP.parse(regexp, 'ruby/2.0')
     conditional = root[1]

data/spec/parser/errors_spec.rb

@@ -1,51 +1,30 @@
 require 'spec_helper'
 
 RSpec.describe('Parsing errors') do
-  let(:rp) { Regexp::Parser.new }
-  before { rp.parse(/foo/) }
+  let(:parser) { Regexp::Parser.new }
+  before { parser.parse(/foo/) } # initializes ivars
 
-  specify('parser unknown token type') do
-    expect { rp.send(:parse_token, Regexp::Token.new(:foo, :bar)) }
+  it('raises UnknownTokenTypeError for unknown token types') do
+    expect { parser.send(:parse_token, Regexp::Token.new(:foo, :bar)) }
       .to raise_error(Regexp::Parser::UnknownTokenTypeError)
   end
 
-  specify('parser unknown set token') do
-    expect { rp.send(:parse_token, Regexp::Token.new(:set, :foo)) }
-      .to raise_error(Regexp::Parser::UnknownTokenError)
-  end
-
-  specify('parser unknown meta token') do
-    expect { rp.send(:parse_token, Regexp::Token.new(:meta, :foo)) }
-      .to raise_error(Regexp::Parser::UnknownTokenError)
-  end
-
-  specify('parser unknown character type token') do
-    expect { rp.send(:parse_token, Regexp::Token.new(:type, :foo)) }
-      .to raise_error(Regexp::Parser::UnknownTokenError)
-  end
-
-  specify('parser unknown unicode property token') do
-    expect { rp.send(:parse_token, Regexp::Token.new(:property, :foo)) }
-      .to raise_error(Regexp::Parser::UnknownTokenError)
-  end
-
-  specify('parser unknown unicode nonproperty token') do
-    expect { rp.send(:parse_token, Regexp::Token.new(:nonproperty, :foo)) }
-      .to raise_error(Regexp::Parser::UnknownTokenError)
-  end
-
-  specify('parser unknown anchor token') do
-    expect { rp.send(:parse_token, Regexp::Token.new(:anchor, :foo)) }
-      .to raise_error(Regexp::Parser::UnknownTokenError)
-  end
-
-  specify('parser unknown quantifier token') do
-    expect { rp.send(:parse_token, Regexp::Token.new(:quantifier, :foo)) }
-      .to raise_error(Regexp::Parser::UnknownTokenError)
-  end
-
-  specify('parser unknown group open token') do
-    expect { rp.send(:parse_token, Regexp::Token.new(:group, :foo)) }
-      .to raise_error(Regexp::Parser::UnknownTokenError)
-  end
+  RSpec.shared_examples 'UnknownTokenError' do |type, token|
+    it "raises for unkown tokens of type #{type}" do
+      expect { parser.send(:parse_token, Regexp::Token.new(type, :foo)) }
+        .to raise_error(Regexp::Parser::UnknownTokenError)
+    end
+  end
+
+  include_examples 'UnknownTokenError', :anchor
+  include_examples 'UnknownTokenError', :backref
+  include_examples 'UnknownTokenError', :conditional
+  include_examples 'UnknownTokenError', :free_space
+  include_examples 'UnknownTokenError', :group
+  include_examples 'UnknownTokenError', :meta
+  include_examples 'UnknownTokenError', :nonproperty
+  include_examples 'UnknownTokenError', :property
+  include_examples 'UnknownTokenError', :quantifier
+  include_examples 'UnknownTokenError', :set
+  include_examples 'UnknownTokenError', :type
 end

data/spec/parser/escapes_spec.rb

@@ -1,49 +1,35 @@
 require 'spec_helper'
 
-RSpec.describe('Escape parsing') do
-  tests = {
-    /a\ac/          => [1, :escape, :bell,              EscapeSequence::Bell],
-    /a\ec/          => [1, :escape, :escape,            EscapeSequence::AsciiEscape],
-    /a\fc/          => [1, :escape, :form_feed,         EscapeSequence::FormFeed],
-    /a\nc/          => [1, :escape, :newline,           EscapeSequence::Newline],
-    /a\rc/          => [1, :escape, :carriage,          EscapeSequence::Return],
-    /a\tc/          => [1, :escape, :tab,               EscapeSequence::Tab],
-    /a\vc/          => [1, :escape, :vertical_tab,      EscapeSequence::VerticalTab],
-
-    # meta character escapes
-    /a\.c/          => [1, :escape, :dot,               EscapeSequence::Literal],
-    /a\?c/          => [1, :escape, :zero_or_one,       EscapeSequence::Literal],
-    /a\*c/          => [1, :escape, :zero_or_more,      EscapeSequence::Literal],
-    /a\+c/          => [1, :escape, :one_or_more,       EscapeSequence::Literal],
-    /a\|c/          => [1, :escape, :alternation,       EscapeSequence::Literal],
-    /a\(c/          => [1, :escape, :group_open,        EscapeSequence::Literal],
-    /a\)c/          => [1, :escape, :group_close,       EscapeSequence::Literal],
-    /a\{c/          => [1, :escape, :interval_open,     EscapeSequence::Literal],
-    /a\}c/          => [1, :escape, :interval_close,    EscapeSequence::Literal],
-
-    # unicode escapes
-    /a\u0640/       => [1, :escape, :codepoint,         EscapeSequence::Codepoint],
-    /a\u{41 1F60D}/ => [1, :escape, :codepoint_list,    EscapeSequence::CodepointList],
-    /a\u{10FFFF}/   => [1, :escape, :codepoint_list,    EscapeSequence::CodepointList],
-
-     # hex escapes
-    /a\xFF/n        => [1, :escape, :hex,               EscapeSequence::Hex],
-
-    # octal escapes
-    /a\177/n        => [1, :escape, :octal,             EscapeSequence::Octal],
-  }
-
-  tests.each_with_index do |(pattern, (index, type, token, klass)), count|
-    specify("parse_escape_#{token}_#{count = (count + 1)}") do
-      root = RP.parse(pattern, 'ruby/1.9')
-      exp = root.expressions.at(index)
-
-      expect(exp).to be_a(klass)
-
-      expect(exp.type).to eq type
-      expect(exp.token).to eq token
-    end
-  end
+RSpec.describe('EscapeSequence parsing') do
+  include_examples 'parse', /a\ac/,          1 => [:escape, :bell,              EscapeSequence::Bell]
+  include_examples 'parse', /a\ec/,          1 => [:escape, :escape,            EscapeSequence::AsciiEscape]
+  include_examples 'parse', /a\fc/,          1 => [:escape, :form_feed,         EscapeSequence::FormFeed]
+  include_examples 'parse', /a\nc/,          1 => [:escape, :newline,           EscapeSequence::Newline]
+  include_examples 'parse', /a\rc/,          1 => [:escape, :carriage,          EscapeSequence::Return]
+  include_examples 'parse', /a\tc/,          1 => [:escape, :tab,               EscapeSequence::Tab]
+  include_examples 'parse', /a\vc/,          1 => [:escape, :vertical_tab,      EscapeSequence::VerticalTab]
+
+  # meta character escapes
+  include_examples 'parse', /a\.c/,          1 => [:escape, :dot,               EscapeSequence::Literal]
+  include_examples 'parse', /a\?c/,          1 => [:escape, :zero_or_one,       EscapeSequence::Literal]
+  include_examples 'parse', /a\*c/,          1 => [:escape, :zero_or_more,      EscapeSequence::Literal]
+  include_examples 'parse', /a\+c/,          1 => [:escape, :one_or_more,       EscapeSequence::Literal]
+  include_examples 'parse', /a\|c/,          1 => [:escape, :alternation,       EscapeSequence::Literal]
+  include_examples 'parse', /a\(c/,          1 => [:escape, :group_open,        EscapeSequence::Literal]
+  include_examples 'parse', /a\)c/,          1 => [:escape, :group_close,       EscapeSequence::Literal]
+  include_examples 'parse', /a\{c/,          1 => [:escape, :interval_open,     EscapeSequence::Literal]
+  include_examples 'parse', /a\}c/,          1 => [:escape, :interval_close,    EscapeSequence::Literal]
+
+  # unicode escapes
+  include_examples 'parse', /a\u0640/,       1 => [:escape, :codepoint,         EscapeSequence::Codepoint]
+  include_examples 'parse', /a\u{41 1F60D}/, 1 => [:escape, :codepoint_list,    EscapeSequence::CodepointList]
+  include_examples 'parse', /a\u{10FFFF}/,   1 => [:escape, :codepoint_list,    EscapeSequence::CodepointList]
+
+    # hex escapes
+  include_examples 'parse', /a\xFF/n,        1 => [:escape, :hex,               EscapeSequence::Hex]
+
+  # octal escapes
+  include_examples 'parse', /a\177/n,        1 => [:escape, :octal,             EscapeSequence::Octal]
 
   specify('parse chars and codepoints') do
     root = RP.parse(/\n\?\101\x42\u0043\u{44 45}/)
@@ -65,6 +51,9 @@ RSpec.describe('Escape parsing') do
 
     expect(root[5].chars).to eq %w[D E]
     expect(root[5].codepoints).to eq [68, 69]
+
+    expect { root[5].char }.to raise_error(/#chars/)
+    expect { root[5].codepoint }.to raise_error(/#codepoints/)
   end
 
   specify('parse escape control sequence lower') do

data/spec/parser/free_space_spec.rb

@@ -24,13 +24,34 @@ RSpec.describe('FreeSpace parsing') do
     expect(root.first.text).to eq 'a b c d'
   end
 
+  specify('parse single-line free space comments without spaces') do
+    regexp = /a#b/x
+
+    root = RP.parse(regexp)
+    expect(root.length).to eq 2
+
+    expect(root[0]).to be_instance_of(Literal)
+    expect(root[1]).to be_instance_of(Comment)
+  end
+
+  specify('parse single-line free space comments with spaces') do
+    regexp = /a # b/x
+
+    root = RP.parse(regexp)
+    expect(root.length).to eq 3
+
+    expect(root[0]).to be_instance_of(Literal)
+    expect(root[1]).to be_instance_of(WhiteSpace)
+    expect(root[2]).to be_instance_of(Comment)
+  end
+
   specify('parse free space comments') do
     regexp = /
       a   ?     # One letter
       b {2,5}   # Another one
       [c-g]  +  # A set
       (h|i|j) | # A group
-      klm *
+      klm#nospace before or after comment hash
       nop +
     /x
 
@@ -51,11 +72,11 @@ RSpec.describe('FreeSpace parsing') do
 
     alt_2 = alt.alternatives.last
     expect(alt_2).to be_instance_of(Alternative)
-    expect(alt_2.length).to eq 7
+    expect(alt_2.length).to eq 8
 
-    [0, 2, 4, 6].each { |i| expect(alt_2[i].class).to eq WhiteSpace }
+    [0, 2, 5, 7].each { |i| expect(alt_2[i].class).to eq WhiteSpace }
 
-    expect(alt_2[1]).to be_instance_of(Comment)
+    [1, 4].each { |i| expect(alt_2[i]).to be_instance_of(Comment) }
   end
 
   specify('parse free space nested comments') do