[DRE-commits] [ruby-fakeweb] 01/06: added minitest5/ruby 22 patches from Upstream

Jonas Genannt genannt at moszumanska.debian.org
Wed Aug 19 16:19:06 UTC 2015


This is an automated email from the git hooks/post-receive script.

genannt pushed a commit to branch master
in repository ruby-fakeweb.

commit 708f7d5198b4dc2f532854f4d0a045915d23f0e9
Author: Jonas Genannt <jonas at brachium-system.net>
Date:   Wed Aug 19 18:12:46 2015 +0200

    added minitest5/ruby 22 patches from Upstream
---
 debian/patches/minitest5.patch | 485 +++++++++++++++++++++++++++++++++++++++++
 debian/patches/ruby22.patch    |  25 +++
 2 files changed, 510 insertions(+)

diff --git a/debian/patches/minitest5.patch b/debian/patches/minitest5.patch
new file mode 100644
index 0000000..a1f6303
--- /dev/null
+++ b/debian/patches/minitest5.patch
@@ -0,0 +1,485 @@
+From ab67aae1c92784e048f0c7f8eb5e44e3cac555ae Mon Sep 17 00:00:00 2001
+From: Josef Stribny <jstribny at redhat.com>
+Date: Thu, 12 Jun 2014 14:47:05 +0200
+Subject: [PATCH] Use Minitest 5
+Bug: https://patch-diff.githubusercontent.com/raw/chrisk/fakeweb/pull/48.patch
+
+---
+ Gemfile                               |  6 ------
+ fakeweb.gemspec                       |  1 +
+ test/test_allow_net_connect.rb        | 30 +++++++++++++++---------------
+ test/test_deprecations.rb             |  2 +-
+ test/test_fake_authentication.rb      |  2 +-
+ test/test_fake_web.rb                 | 12 ++++++------
+ test/test_fake_web_open_uri.rb        |  2 +-
+ test/test_helper.rb                   |  8 ++++----
+ test/test_last_request.rb             |  2 +-
+ test/test_missing_open_uri.rb         |  2 +-
+ test/test_missing_pathname.rb         |  2 +-
+ test/test_other_net_http_libraries.rb |  2 +-
+ test/test_precedence.rb               |  2 +-
+ test/test_query_string.rb             |  2 +-
+ test/test_regexes.rb                  |  4 ++--
+ test/test_registering_with_io.rb      |  2 +-
+ test/test_response_headers.rb         |  2 +-
+ test/test_timeouts.rb                 |  2 +-
+ test/test_trailing_slashes.rb         |  6 +++---
+ test/test_utility.rb                  |  2 +-
+ 20 files changed, 44 insertions(+), 49 deletions(-)
+
+diff --git a/Gemfile b/Gemfile
+index 3fa5c9f..bc516c8 100644
+--- a/Gemfile
++++ b/Gemfile
+@@ -6,11 +6,5 @@ platform :rbx do
+   stdlibs = %w(benchmark cgi coverage delegate erb find logger net-http open-uri
+                optparse ostruct prettyprint singleton tempfile tmpdir yaml)
+   stdlibs.each { |lib| gem "rubysl-#{lib}", "~> 2.0" }
+-  # rubysl-test-unit 2.0.2's gemspec relaxed its dependency on minitest to allow
+-  # any version (previously, it specified "~> 4.7"). Minitest 5 doesn't have a
+-  # Test::Unit compatibility layer like 4.x, so it doesn't work with Test::Unit
+-  # at all (see e.g. https://github.com/seattlerb/minitest/issues/358). So, I'm
+-  # holding this one back until we find out what's going on.
+-  gem "rubysl-test-unit", "2.0.1"
+   gem "psych", "~> 2.0"
+ end
+diff --git a/fakeweb.gemspec b/fakeweb.gemspec
+index dfa56f9..a3bf9a8 100644
+--- a/fakeweb.gemspec
++++ b/fakeweb.gemspec
+@@ -45,6 +45,7 @@ Gem::Specification.new do |s|
+     # Otherwise, prefer up-to-date dev tools
+     s.add_development_dependency "mocha", ["~> 0.14"] + broken_mocha_spec
+     s.add_development_dependency "rake",  ["~> 10.0"]
++    s.add_development_dependency "minitest", ["~> 5.0"]
+ 
+     # ZenTest (autotest) wants at least RubyGems 1.8, which is 1.8.7+
+     # only, as is RDoc, the main dependency of sdoc.
+diff --git a/test/test_allow_net_connect.rb b/test/test_allow_net_connect.rb
+index 25f4d3d..b907f9b 100644
+--- a/test/test_allow_net_connect.rb
++++ b/test/test_allow_net_connect.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestFakeWebAllowNetConnect < Test::Unit::TestCase
++class TestFakeWebAllowNetConnect < Minitest::Test
+   def test_unregistered_requests_are_passed_through_when_allow_net_connect_is_true
+     FakeWeb.allow_net_connect = true
+     setup_expectations_for_real_apple_hot_news_request
+@@ -9,7 +9,7 @@ def test_unregistered_requests_are_passed_through_when_allow_net_connect_is_true
+ 
+   def test_raises_for_unregistered_requests_when_allow_net_connect_is_false
+     FakeWeb.allow_net_connect = false
+-    assert_raise FakeWeb::NetConnectNotAllowedError do
++    assert_raises FakeWeb::NetConnectNotAllowedError do
+       Net::HTTP.get(URI.parse("http://example.com/"))
+     end
+   end
+@@ -40,21 +40,21 @@ def test_unregistered_requests_are_passed_through_when_allow_net_connect_is_a_ma
+ 
+   def test_raises_for_unregistered_requests_when_allow_net_connect_is_a_different_string
+     FakeWeb.allow_net_connect = "http://example.com"
+-    assert_raise FakeWeb::NetConnectNotAllowedError do
++    assert_raises FakeWeb::NetConnectNotAllowedError do
+       Net::HTTP.get(URI.parse("http://example.com/path"))
+     end
+   end
+ 
+   def test_raises_for_unregistered_requests_when_allow_net_connect_is_a_different_uri
+     FakeWeb.allow_net_connect = URI.parse("http://example.com")
+-    assert_raise FakeWeb::NetConnectNotAllowedError do
++    assert_raises FakeWeb::NetConnectNotAllowedError do
+       Net::HTTP.get(URI.parse("http://example.com/path"))
+     end
+   end
+ 
+   def test_raises_for_unregistered_requests_when_allow_net_connect_is_a_non_matching_regexp
+     FakeWeb.allow_net_connect = %r[example\.net]
+-    assert_raise FakeWeb::NetConnectNotAllowedError do
++    assert_raises FakeWeb::NetConnectNotAllowedError do
+       Net::HTTP.get(URI.parse("http://example.com"))
+     end
+   end
+@@ -62,7 +62,7 @@ def test_raises_for_unregistered_requests_when_allow_net_connect_is_a_non_matchi
+   def test_changing_allow_net_connect_from_string_to_false_corretly_removes_whitelist
+     FakeWeb.allow_net_connect = "http://example.com"
+     FakeWeb.allow_net_connect = false
+-    assert_raise FakeWeb::NetConnectNotAllowedError do
++    assert_raises FakeWeb::NetConnectNotAllowedError do
+       Net::HTTP.get(URI.parse("http://example.com"))
+     end
+   end
+@@ -70,19 +70,19 @@ def test_changing_allow_net_connect_from_string_to_false_corretly_removes_whitel
+   def test_changing_allow_net_connect_from_true_to_string_corretly_limits_connections
+     FakeWeb.allow_net_connect = true
+     FakeWeb.allow_net_connect = "http://example.com"
+-    assert_raise FakeWeb::NetConnectNotAllowedError do
++    assert_raises FakeWeb::NetConnectNotAllowedError do
+       Net::HTTP.get(URI.parse("http://example.net"))
+     end
+   end
+ 
+   def test_exception_message_includes_unregistered_request_method_and_uri_but_no_default_port
+     FakeWeb.allow_net_connect = false
+-    exception = assert_raise FakeWeb::NetConnectNotAllowedError do
++    exception = assert_raises FakeWeb::NetConnectNotAllowedError do
+       Net::HTTP.get(URI.parse("http://example.com/"))
+     end
+     assert exception.message.include?("GET http://example.com/")
+ 
+-    exception = assert_raise FakeWeb::NetConnectNotAllowedError do
++    exception = assert_raises FakeWeb::NetConnectNotAllowedError do
+       http = Net::HTTP.new("example.com", 443)
+       http.use_ssl = true
+       http.get("/")
+@@ -92,12 +92,12 @@ def test_exception_message_includes_unregistered_request_method_and_uri_but_no_d
+ 
+   def test_exception_message_includes_unregistered_request_port_when_not_default
+     FakeWeb.allow_net_connect = false
+-    exception = assert_raise FakeWeb::NetConnectNotAllowedError do
++    exception = assert_raises FakeWeb::NetConnectNotAllowedError do
+       Net::HTTP.start("example.com", 8000) { |http| http.get("/") }
+     end
+     assert exception.message.include?("GET http://example.com:8000/")
+ 
+-    exception = assert_raise FakeWeb::NetConnectNotAllowedError do
++    exception = assert_raises FakeWeb::NetConnectNotAllowedError do
+       http = Net::HTTP.new("example.com", 4433)
+       http.use_ssl = true
+       http.get("/")
+@@ -107,12 +107,12 @@ def test_exception_message_includes_unregistered_request_port_when_not_default
+ 
+   def test_exception_message_includes_unregistered_request_port_when_not_default_with_path
+     FakeWeb.allow_net_connect = false
+-    exception = assert_raise FakeWeb::NetConnectNotAllowedError do
++    exception = assert_raises FakeWeb::NetConnectNotAllowedError do
+       Net::HTTP.start("example.com", 8000) { |http| http.get("/test") }
+     end
+     assert exception.message.include?("GET http://example.com:8000/test")
+ 
+-    exception = assert_raise FakeWeb::NetConnectNotAllowedError do
++    exception = assert_raises FakeWeb::NetConnectNotAllowedError do
+       http = Net::HTTP.new("example.com", 4433)
+       http.use_ssl = true
+       http.get("/test")
+@@ -132,7 +132,7 @@ def test_question_mark_method_returns_false_after_setting_allow_net_connect_to_f
+ 
+   def test_question_mark_method_raises_with_no_argument_when_allow_net_connect_is_a_whitelist
+     FakeWeb.allow_net_connect = "http://example.com"
+-    exception = assert_raise ArgumentError do
++    exception = assert_raises ArgumentError do
+       FakeWeb.allow_net_connect?
+     end
+     assert_equal "You must supply a URI to test", exception.message
+@@ -156,7 +156,7 @@ def test_question_mark_method_returns_false_when_argument_does_not_match_allow_n
+ end
+ 
+ 
+-class TestFakeWebAllowNetConnectWithCleanState < Test::Unit::TestCase
++class TestFakeWebAllowNetConnectWithCleanState < Minitest::Test
+   # Our test_helper.rb sets allow_net_connect = false in an inherited #setup
+   # method. Disable that here to test the default setting.
+   def setup; end
+diff --git a/test/test_deprecations.rb b/test/test_deprecations.rb
+index e5b8953..d54b1b3 100644
+--- a/test/test_deprecations.rb
++++ b/test/test_deprecations.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestDeprecations < Test::Unit::TestCase
++class TestDeprecations < Minitest::Test
+ 
+   def test_register_uri_without_method_argument_prints_deprecation_warning
+     warning = capture_stderr do
+diff --git a/test/test_fake_authentication.rb b/test/test_fake_authentication.rb
+index cff2764..d717be1 100644
+--- a/test/test_fake_authentication.rb
++++ b/test/test_fake_authentication.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestFakeAuthentication < Test::Unit::TestCase
++class TestFakeAuthentication < Minitest::Test
+ 
+   def test_register_uri_with_authentication
+     FakeWeb.register_uri(:get, 'http://user:pass@mock/test_example.txt', :body => "example")
+diff --git a/test/test_fake_web.rb b/test/test_fake_web.rb
+index f344612..3c426cc 100644
+--- a/test/test_fake_web.rb
++++ b/test/test_fake_web.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestFakeWeb < Test::Unit::TestCase
++class TestFakeWeb < Minitest::Test
+ 
+   def test_register_uri
+     FakeWeb.register_uri(:get, 'http://mock/test_example.txt', :body => "example")
+@@ -116,7 +116,7 @@ def test_clean_registry_affects_net_http_requests
+     response = Net::HTTP.start("example.com") { |query| query.get("/") }
+     assert_equal "registered", response.body
+     FakeWeb.clean_registry
+-    assert_raise FakeWeb::NetConnectNotAllowedError do
++    assert_raises FakeWeb::NetConnectNotAllowedError do
+       Net::HTTP.start("example.com") { |query| query.get("/") }
+     end
+   end
+@@ -508,7 +508,7 @@ def test_mock_rotate_responses
+   def test_mock_request_using_response_with_transfer_encoding_header_has_valid_transfer_encoding_header
+     FakeWeb.register_uri(:get, 'http://www.google.com/', :response => fixture_path("google_response_with_transfer_encoding"))
+     response = Net::HTTP.start('www.google.com') { |query| query.get('/') }
+-    assert_not_nil response['transfer-encoding']
++    assert !response['transfer-encoding'].nil?
+     assert response['transfer-encoding'] == 'chunked'
+   end
+ 
+@@ -522,7 +522,7 @@ def test_mock_request_using_response_without_transfer_encoding_header_does_not_h
+   def test_mock_request_using_response_from_curl_has_original_transfer_encoding_header
+     FakeWeb.register_uri(:get, 'http://www.google.com/', :response => fixture_path("google_response_from_curl"))
+     response = Net::HTTP.start('www.google.com') { |query| query.get('/') }
+-    assert_not_nil response['transfer-encoding']
++    assert !response['transfer-encoding'].nil?
+     assert response['transfer-encoding'] == 'chunked'
+   end
+ 
+@@ -542,7 +542,7 @@ def test_registering_with_string_containing_null_byte
+     # The string should be treated as a response body, instead, and an
+     # EOFError is raised when the byte is encountered.
+     FakeWeb.register_uri(:get, "http://example.com", :response => "test\0test")
+-    assert_raise EOFError do
++    assert_raises EOFError do
+       Net::HTTP.get(URI.parse("http://example.com"))
+     end
+ 
+@@ -555,7 +555,7 @@ def test_registering_with_string_that_is_a_directory_name
+     # Similar to above, but for Errno::EISDIR being raised since File.exists?
+     # returns true for directories
+     FakeWeb.register_uri(:get, "http://example.com", :response => File.dirname(__FILE__))
+-    assert_raise EOFError do
++    assert_raises EOFError do
+       Net::HTTP.get(URI.parse("http://example.com"))
+     end
+ 
+diff --git a/test/test_fake_web_open_uri.rb b/test/test_fake_web_open_uri.rb
+index c47c03f..9cd625a 100644
+--- a/test/test_fake_web_open_uri.rb
++++ b/test/test_fake_web_open_uri.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestFakeWebOpenURI < Test::Unit::TestCase
++class TestFakeWebOpenURI < Minitest::Test
+ 
+   def test_content_for_registered_uri
+     FakeWeb.register_uri(:get, 'http://mock/test_example.txt', :body => fixture_path("test_example.txt"))
+diff --git a/test/test_helper.rb b/test/test_helper.rb
+index e9a36af..0572c40 100644
+--- a/test/test_helper.rb
++++ b/test/test_helper.rb
+@@ -4,7 +4,7 @@
+ 
+ require 'helpers/start_simplecov'
+ 
+-require 'test/unit'
++require 'minitest/autorun'
+ require 'open-uri'
+ require 'pathname'
+ require 'fake_web'
+@@ -20,7 +20,7 @@
+ end
+ 
+ # Give all tests a common setup and teardown that prevents shared state
+-class Test::Unit::TestCase
++class Minitest::Test
+   alias setup_without_fakeweb setup
+   def setup
+     FakeWeb.clean_registry
+@@ -36,7 +36,7 @@ def teardown
+ 
+ 
+ module FakeWebTestHelper
+-  BUILTIN_ASSERTIONS = Test::Unit::TestCase.instance_methods.select { |m| m.to_s =~ /^assert/ }.map { |m| m.to_sym }
++  BUILTIN_ASSERTIONS = Minitest::Test.instance_methods.select { |m| m.to_s =~ /^assert/ }.map { |m| m.to_sym }
+ 
+   # Backport assert_empty for Ruby 1.8 (it comes from MiniTest)
+   if !BUILTIN_ASSERTIONS.include?(:assert_empty)
+@@ -178,4 +178,4 @@ def setup_expectations_for_real_apple_hot_news_request(options = {})
+ 
+ end
+ 
+-Test::Unit::TestCase.send(:include, FakeWebTestHelper)
++Minitest::Test.send(:include, FakeWebTestHelper)
+diff --git a/test/test_last_request.rb b/test/test_last_request.rb
+index 7868c83..88356ff 100644
+--- a/test/test_last_request.rb
++++ b/test/test_last_request.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestLastRequest < Test::Unit::TestCase
++class TestLastRequest < Minitest::Test
+ 
+   def test_last_request_returns_correct_net_http_request_class
+     FakeWeb.register_uri(:get, "http://example.com", :status => [200, "OK"])
+diff --git a/test/test_missing_open_uri.rb b/test/test_missing_open_uri.rb
+index 029ba1c..93f33e4 100644
+--- a/test/test_missing_open_uri.rb
++++ b/test/test_missing_open_uri.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestMissingOpenURI < Test::Unit::TestCase
++class TestMissingOpenURI < Minitest::Test
+ 
+   def setup
+     super
+diff --git a/test/test_missing_pathname.rb b/test/test_missing_pathname.rb
+index 5b60200..73cec52 100644
+--- a/test/test_missing_pathname.rb
++++ b/test/test_missing_pathname.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestMissingPathname < Test::Unit::TestCase
++class TestMissingPathname < Minitest::Test
+ 
+   def setup
+     super
+diff --git a/test/test_other_net_http_libraries.rb b/test/test_other_net_http_libraries.rb
+index 1201d7d..3a7821f 100644
+--- a/test/test_other_net_http_libraries.rb
++++ b/test/test_other_net_http_libraries.rb
+@@ -3,7 +3,7 @@
+ # Each of these tests shells out to `ruby -rfakeweb ...` so
+ # we can see how FakeWeb behaves with different combinations of
+ # other gems loaded without polluting the rest of our tests.
+-class TestOtherNetHttpLibraries < Test::Unit::TestCase
++class TestOtherNetHttpLibraries < Minitest::Test
+ 
+   def capture_output_from_requiring(libs, additional_code = "")
+     requires = libs.map { |lib| "require '#{lib}'" }.join("; ")
+diff --git a/test/test_precedence.rb b/test/test_precedence.rb
+index 388b9f8..c9f4702 100644
+--- a/test/test_precedence.rb
++++ b/test/test_precedence.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestPrecedence < Test::Unit::TestCase
++class TestPrecedence < Minitest::Test
+ 
+   def test_matching_get_strings_have_precedence_over_matching_get_regexes
+     FakeWeb.register_uri(:get, "http://example.com/test", :body => "string")
+diff --git a/test/test_query_string.rb b/test/test_query_string.rb
+index 11a2118..7c6d710 100644
+--- a/test/test_query_string.rb
++++ b/test/test_query_string.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestFakeWebQueryString < Test::Unit::TestCase
++class TestFakeWebQueryString < Minitest::Test
+ 
+   def test_register_uri_string_with_query_params
+     FakeWeb.register_uri(:get, 'http://example.com/?a=1&b=1', :body => 'foo')
+diff --git a/test/test_regexes.rb b/test/test_regexes.rb
+index e2eba1d..558b5ff 100644
+--- a/test/test_regexes.rb
++++ b/test/test_regexes.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestRegexes < Test::Unit::TestCase
++class TestRegexes < Minitest::Test
+ 
+   def test_registered_uri_with_pattern
+     FakeWeb.register_uri(:get, %r|http://example.com/test_example/\d+|, :body => "example")
+@@ -39,7 +39,7 @@ def test_request_with_authentication_and_pattern_handles_case_insensitivity
+   def test_requesting_a_uri_that_matches_two_registered_regexes_raises_an_error
+     FakeWeb.register_uri(:get, %r|http://example\.com/|, :body => "first")
+     FakeWeb.register_uri(:get, %r|http://example\.com/a|, :body => "second")
+-    assert_raise FakeWeb::MultipleMatchingURIsError do
++    assert_raises FakeWeb::MultipleMatchingURIsError do
+       Net::HTTP.start("example.com") { |query| query.get('/a') }
+     end
+   end
+diff --git a/test/test_registering_with_io.rb b/test/test_registering_with_io.rb
+index 2f1d4b7..beed905 100644
+--- a/test/test_registering_with_io.rb
++++ b/test/test_registering_with_io.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestRegisteringWithIO < Test::Unit::TestCase
++class TestRegisteringWithIO < Minitest::Test
+ 
+   def test_registering_a_file_handle_without_transfer_encoding
+     file = File.new(fixture_path("google_response_without_transfer_encoding"))
+diff --git a/test/test_response_headers.rb b/test/test_response_headers.rb
+index 45f3f5a..96d19d4 100644
+--- a/test/test_response_headers.rb
++++ b/test/test_response_headers.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestResponseHeaders < Test::Unit::TestCase
++class TestResponseHeaders < Minitest::Test
+   def test_content_type_when_registering_with_string_and_content_type_header_as_symbol_option
+     FakeWeb.register_uri(:get, "http://example.com/users.json", :body => '[{"username": "chrisk"}]', :content_type => "application/json")
+     response = Net::HTTP.start("example.com") { |query| query.get("/users.json") }
+diff --git a/test/test_timeouts.rb b/test/test_timeouts.rb
+index d168ec3..fdda2c8 100644
+--- a/test/test_timeouts.rb
++++ b/test/test_timeouts.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestTimeouts < Test::Unit::TestCase
++class TestTimeouts < Minitest::Test
+   def test_sending_timeout_accessors_after_starting_session
+     # this tests an HTTPS request because #ssl_timeout= raises otherwise
+     FakeWeb.register_uri(:get, "https://example.com", :status => [200, "OK"])
+diff --git a/test/test_trailing_slashes.rb b/test/test_trailing_slashes.rb
+index 7e687e6..225be3d 100644
+--- a/test/test_trailing_slashes.rb
++++ b/test/test_trailing_slashes.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestFakeWebTrailingSlashes < Test::Unit::TestCase
++class TestFakeWebTrailingSlashes < Minitest::Test
+ 
+   def test_registering_root_without_slash_and_ask_predicate_method_with_slash
+     FakeWeb.register_uri(:get, "http://www.example.com", :body => "root")
+@@ -32,7 +32,7 @@ def test_registering_path_without_slash_and_ask_predicate_method_with_slash
+   def test_registering_path_without_slash_and_request_with_slash
+     FakeWeb.allow_net_connect = false
+     FakeWeb.register_uri(:get, "http://www.example.com/users", :body => "User list")
+-    assert_raise FakeWeb::NetConnectNotAllowedError do
++    assert_raises FakeWeb::NetConnectNotAllowedError do
+       Net::HTTP.start("www.example.com") { |query| query.get('/users/') }
+     end
+   end
+@@ -45,7 +45,7 @@ def test_registering_path_with_slash_and_ask_predicate_method_without_slash
+   def test_registering_path_with_slash_and_request_without_slash
+     FakeWeb.allow_net_connect = false
+     FakeWeb.register_uri(:get, "http://www.example.com/users/", :body => "User list")
+-    assert_raise FakeWeb::NetConnectNotAllowedError do
++    assert_raises FakeWeb::NetConnectNotAllowedError do
+       Net::HTTP.start("www.example.com") { |query| query.get('/users') }
+     end
+   end
+diff --git a/test/test_utility.rb b/test/test_utility.rb
+index 891de87..f82bfc8 100644
+--- a/test/test_utility.rb
++++ b/test/test_utility.rb
+@@ -1,6 +1,6 @@
+ require 'test_helper'
+ 
+-class TestUtility < Test::Unit::TestCase
++class TestUtility < Minitest::Test
+ 
+   def test_decode_userinfo_from_header_handles_basic_auth
+     authorization_header = "Basic dXNlcm5hbWU6c2VjcmV0"
diff --git a/debian/patches/ruby22.patch b/debian/patches/ruby22.patch
new file mode 100644
index 0000000..0394115
--- /dev/null
+++ b/debian/patches/ruby22.patch
@@ -0,0 +1,25 @@
+From 1647ca210691429a75eb159814f42e68c3c1ce4c Mon Sep 17 00:00:00 2001
+From: Josef Stribny <jstribny at redhat.com>
+Date: Thu, 19 Mar 2015 09:32:50 +0100
+Subject: [PATCH] Fix tests by expecting TypeError for Ruby 2.2
+Bug: https://patch-diff.githubusercontent.com/raw/chrisk/fakeweb/pull/53.patch
+
+---
+ test/test_fake_web.rb | 4 +++-
+ 1 file changed, 3 insertions(+), 1 deletion(-)
+
+diff --git a/test/test_fake_web.rb b/test/test_fake_web.rb
+index f344612..682c43d 100644
+--- a/test/test_fake_web.rb
++++ b/test/test_fake_web.rb
+@@ -35,7 +35,9 @@ def test_response_for_with_wrong_number_of_arguments
+   end
+ 
+   def test_register_uri_without_domain_name
+-    assert_raises URI::InvalidURIError do
++    # Ruby 2.1 and lower raises URI::InvalidURIError
++    # Ruby 2.2 and higher raises TypeError
++    assert_raises URI::InvalidURIError, TypeError do
+       FakeWeb.register_uri(:get, 'test_example2.txt', fixture_path("test_example.txt"))
+     end
+   end

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-ruby-extras/ruby-fakeweb.git



More information about the Pkg-ruby-extras-commits mailing list