URL encode everything

lib/quick_travel/adapter.rb

@@ -9,6 +9,7 @@
 require 'quick_travel/config'
 require 'quick_travel/adapter_error'
 require 'quick_travel/init_from_hash'
+require 'uri'
 
 module QuickTravel
   class Adapter
@@ -284,7 +285,7 @@ def self.call_and_validate(http_method, path, query = {}, opts = {})
       http_params[:follow_redirects] = false
 
       begin
-        response = self.send(http_method, path, http_params)
+        response = self.send(http_method, URI.escape(path), http_params)
       rescue Errno::ECONNREFUSED
         raise ConnectionError.new('Connection refused')
       rescue SocketError

lib/quick_travel/booking.rb

@@ -4,7 +4,6 @@
 require 'quick_travel/payment'
 require 'quick_travel/payment_type'
 require 'quick_travel/price_changes'
-require 'uri'
 
 module QuickTravel
   class Booking < Adapter
@@ -13,8 +12,7 @@ def self.api_base
     end
 
     def self.find_by_reference(reference)
-      uri = URI.escape("#{api_base}/reference/#{reference}.json")
-      find_all!(uri).first
+      find_all!("#{api_base}/reference/#{reference}.json").first
     end
 
     def documents(regenerate = false)

spec/adapter_spec.rb

@@ -9,6 +9,15 @@
     allow(QuickTravel::Api).to receive(:post).and_return(response)
   end
 
+  context 'when the query needs to be url encoded' do
+    let(:url) { 'http://test.quicktravel.com.au/ blah' }
+    before do
+      QuickTravel::Adapter.post_and_validate(url, {})
+    end
+    let(:expected_url) { 'http://test.quicktravel.com.au/%20blah' }
+    specify { expect(QuickTravel::Api).to have_received(:post).with(expected_url, instance_of(Hash)) }
+  end
+
   context 'when the query contains empty arrays' do
     let(:url) { 'http://test.quicktravel.com.au' }
     let(:query) {