0
@@ -299,7 +299,7 @@
0
       http.use_ssl = true if self.instance_of? URI::HTTPS
0
 
0
       puts "Requesting #{self}"  if Buildr.application.options.trace
0
-      request = Net::HTTP::Get.new(path.empty? ? '/' : path, headers)
0
+      request = Net::HTTP::Get.new(request_uri.empty? ? '/' : request_uri, headers)
0
       request.basic_auth self.user, self.password if self.user
0
       http.request request do |response|
0
         case response

0
@@ -221,7 +221,9 @@
0
     @proxy = 'http://john:smith@myproxy:8080'
0
     @domain = 'domain'
0
     @host_domain = "host.#{@domain}"
0
-    @uri = URI("http://#{@host_domain}")
0
+    @path = "/foo/bar/baz"
0
+    @query = "?query"
0
+    @uri = URI("http://#{@host_domain}#{@path}#{@query}")
0
     @no_proxy_args = [@host_domain, 80]
0
     @proxy_args = @no_proxy_args + ['myproxy', 8080, 'john', 'smith']
0
     @http = mock('http')
0
@@ -297,6 +299,14 @@
0
     request.should_receive(:basic_auth).with('john', 'secret')
0
     URI("http://john:secret@#{@host_domain}").read
0
   end
0
+
0
+  it 'should include the query part when performing HTTP GET' do
0
+    # should this test be generalized or shared with any other URI subtypes?
0
+    Net::HTTP.stub!(:new).and_return(@http)
0
+    Net::HTTP::Get.should_receive(:new).with(/#{Regexp.escape(@query)}$/, nil)
0
+    @uri.read
0
+  end
0
+
0
 end
0
 
0