kissmetrics 1.0.1 → 1.0.2

data/VERSION

@@ -1 +1 @@
-1.0.1
+1.0.2

data/lib/kissmetrics/http_client.rb

@@ -61,7 +61,7 @@ module Kissmetrics
 
       def to_s
         @hash.collect do |key, value|
-          "#{key}=#{CGI.escape(value.to_s)}"
+          "#{CGI.escape(key.to_s)}=#{CGI.escape(value.to_s)}"
         end.join('&')
       end
     end

data/spec/kissmetrics/http_client_spec.rb

@@ -67,17 +67,28 @@ describe Kissmetrics::HttpClient, "setting properties" do
 
   it "sets properties when unidentified" do
     subject.set('identity', {
-      'Button color'     => 'Blue',
-      'Background color' => 'Gray'
+      'Button'     => 'Blue',
+      'Background' => 'Gray'
     })
 
     WebMock.should have_requested(:get, "https://trk.kissmetrics.com/s").with({
       :query => {
-        :_k                => 'my-api-key',
-        :_p                => 'identity',
-        'Button color'     => 'Blue',
-        'Background color' => 'Gray'
+        :_k          => 'my-api-key',
+        :_p          => 'identity',
+        'Button'     => 'Blue',
+        'Background' => 'Gray'
       }
     })
   end
+
+  it "escapes keys and values" do
+    http = stub('http', :use_ssl= => true, :get => true)
+    Net::HTTP.stub(:new => http)
+
+    http.should_receive(:get).with(%r{Some\+property=Some\+value})
+
+    subject.set('identity', {
+      'Some property' => 'Some value'
+    })
+  end
 end

metadata

@@ -2,7 +2,7 @@
 name: kissmetrics
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors: 
 - Jason Morrison
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-06-28 00:00:00 -04:00
+date: 2011-06-30 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -116,7 +116,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: -4086314605262870467
+      hash: 2269321523201679432
       segments: 
       - 0
       version: "0"