Fix a bug where trailing slashes were considered significant for requ…

…ests to the root of a domain

lib/fake_web/registry.rb

@@ -56,14 +56,16 @@ def response_for(method, uri, &block)
     private
 
     def normalize_uri(uri)
-      case uri
-      when URI then uri
-      else
-        uri = 'http://' + uri unless uri.match('^https?://')
-        parsed_uri = URI.parse(uri)
-        parsed_uri.query = sort_query_params(parsed_uri.query)
-        parsed_uri
-      end
+      normalized_uri =
+        case uri
+        when URI then uri
+        else
+          uri = 'http://' + uri unless uri.match('^https?://')
+          parsed_uri = URI.parse(uri)
+          parsed_uri.query = sort_query_params(parsed_uri.query)
+          parsed_uri
+        end
+      normalized_uri.normalize
     end
 
     def sort_query_params(query)

test/test_trailing_slashes.rb

@@ -0,0 +1,62 @@
+require File.join(File.dirname(__FILE__), "test_helper")
+
+class TestFakeWebTrailingSlashes < Test::Unit::TestCase
+
+  def setup
+    FakeWeb.clean_registry
+    @original_allow_net_connect = FakeWeb.allow_net_connect?
+  end
+
+  def teardown
+    FakeWeb.allow_net_connect = @old_allow_net_conncet
+  end
+
+  def test_registering_root_without_slash_and_ask_predicate_method_with_slash
+    FakeWeb.register_uri(:get, "http://www.google.com", :string => "Google")
+    assert FakeWeb.registered_uri?(:get, "http://www.google.com/")
+  end
+
+  def test_registering_root_without_slash_and_request
+    FakeWeb.register_uri(:get, "http://www.google.com", :string => "Google")
+    response = Net::HTTP.start("www.google.com") { |query| query.get('/') }
+    assert_equal "Google", response.body
+  end
+
+  def test_registering_root_with_slash_and_ask_predicate_method_without_slash
+    FakeWeb.register_uri(:get, "http://www.google.com/", :string => "Google")
+    assert FakeWeb.registered_uri?(:get, "http://www.google.com")
+  end
+
+  def test_registering_root_with_slash_and_request
+    FakeWeb.register_uri(:get, "http://www.google.com/", :string => "Google")
+    response = Net::HTTP.start("www.google.com") { |query| query.get('/') }
+    assert_equal "Google", response.body
+  end
+
+  def test_registering_path_without_slash_and_ask_predicate_method_with_slash
+    FakeWeb.register_uri(:get, "http://www.example.com/users", :string => "User list")
+    assert !FakeWeb.registered_uri?(:get, "http://www.example.com/users/")
+  end
+
+  def test_registering_path_without_slash_and_request_with_slash
+    FakeWeb.allow_net_connect = false
+    FakeWeb.register_uri(:get, "http://www.example.com/users", :string => "User list")
+    assert_raise FakeWeb::NetConnectNotAllowedError do
+      response = Net::HTTP.start("www.example.com") { |query| query.get('/users/') }
+    end
+  end
+
+  def test_registering_path_with_slash_and_ask_predicate_method_without_slash
+    FakeWeb.register_uri(:get, "http://www.example.com/users/", :string => "User list")
+    assert !FakeWeb.registered_uri?(:get, "http://www.example.com/users")
+  end
+
+  def test_registering_path_with_slash_and_request_without_slash
+    FakeWeb.allow_net_connect = false
+    FakeWeb.register_uri(:get, "http://www.example.com/users/", :string => "User list")
+    assert_raise FakeWeb::NetConnectNotAllowedError do
+      response = Net::HTTP.start("www.example.com") { |query| query.get('/users') }
+    end
+  end
+
+end