Merge pull request #442 from rspec/fix-jruby-ripper

Restore original JRuby ripper suppression

Author: JonRowe

lib/rspec/support/ruby_features.rb

@@ -113,8 +113,7 @@ def supports_taint?
         # or cannot parse source including `:if`.
         # Ripper on JRuby 9.x.x.x < 9.1.17.0 can't handle keyword arguments
         # Neither can JRuby 9.2, e.g. < 9.2.1.0
-        ripper_requirements.push(!Ruby.jruby_version.between?('9.0.0.0.rc1', '9.1.16.0'))
-        ripper_requirements.push(!Ruby.jruby_version.between?('9.1.999.0', '9.1.999.0'))
+        ripper_requirements.push(!Ruby.jruby_version.between?('9.0.0.0.rc1', '9.2.0.0'))
       end
 
       if ripper_requirements.all?

spec/rspec/support/ruby_features_spec.rb

@@ -165,6 +165,9 @@ def ripper_can_parse_source_referencing_keyword_arguments?
         end
 
         it 'returns whether Ripper is correctly implemented in the current environment' do
+          if RSpec::Support::Ruby.jruby? && RSpec::Support::Ruby.jruby_version < '9.2.1.0'
+            pending "Ripper is not supported on JRuby 9.1.17.0 despite this tests claims"
+          end
           expect(RubyFeatures.ripper_supported?).to eq(ripper_is_implemented? && ripper_works_correctly?)
         end