Fix tests with excon >= 0.63 (vcr#809)

* Fix tests with excon >= 0.63 In this version an 'Accept' header has been added in excon [1] and needs to be added to the default headers to merge too. This should fix vcr#801 and with the version check being backwards compatible [1] https://github.com/excon/excon/blame/master/lib/excon/constants.rb#L141 * Update spec/support/http_library_adapters.rb Prefer start_with() over to_f() for version comparison Co-Authored-By: Kurtis Rainbolt-Greene <[email protected]> * Use patch by @terceiro Co-Authored-By: Olle Jonsson <[email protected]> Co-authored-by: Kurtis Rainbolt-Greene <[email protected]> Co-authored-by: Olle Jonsson <[email protected]>
z1lk · Mar 30, 2020 · 6cd7072 · 6cd7072
1 parent ed3cacf
commit 6cd7072
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/spec/support/http_library_adapters.rb b/spec/support/http_library_adapters.rb
@@ -227,7 +227,11 @@ def make_http_request(method, url, body = nil, headers = {})
   end
 
   def normalize_request_headers(headers)
-    headers.merge('User-Agent' => [Excon::USER_AGENT])
+    if Gem::Version.new(Excon::VERSION) < Gem::Version.new("0.63")
+      headers.merge('User-Agent' => [Excon::USER_AGENT])
+    else
+      headers.merge('User-Agent' => [Excon::USER_AGENT], 'Accept' => ['*/*'])
+    end
   end
 end