Merge pull request #63 from lostisland/disable-yaml-ruby-2.5

Disable yaml tests in Ruby 2.5+ because of issues with Psych 3.0

.travis.yml

@@ -13,8 +13,6 @@ matrix:
   allow_failures:
     - rvm: rbx-2
     - rvm: jruby
-    - rvm: 2.5
-    - rvm: 2.6
 
 notifications:
   emails: false

test/agent_test.rb

@@ -169,6 +169,8 @@ def test_does_not_encode_non_json_content_types
     end
 
     def test_handle_yaml_dump_and_load
+      return unless supports_yaml?
+
       require 'yaml'
       res = Agent.new 'http://example.com', :a => 1
       YAML.load(YAML.dump(res))

test/helper.rb

@@ -4,4 +4,17 @@
 class Sawyer::TestCase < Minitest::Test
   def default_test
   end
+
+  def supports_yaml?
+    return true if ruby_version < yaml_disabled_version
+    ENV["SAWYER_YAML_ENABLED"] == "1"
+  end
+
+  def ruby_version
+    Gem::Version.new(RUBY_VERSION)
+  end
+
+  def yaml_disabled_version
+    Gem::Version.new("2.5.0")
+  end
 end

test/resource_test.rb

@@ -167,6 +167,8 @@ def test_simple_rel_parsing
     end
 
     def test_handle_yaml_dump
+      return unless supports_yaml?
+
       require 'yaml'
       res = Resource.new @agent, :a => 1
       YAML.dump(res)