only work with stringified keys

Stringify all the incomming has keys from the track methods and only work with string keys internally.
The reason is that hashes coming from the Rails session will always have string keys and this way it's consistent everwhere internally.

Author: jhilden

lib/rack/tracker/facebook/facebook.rb

@@ -12,7 +12,7 @@ def render
   end
 
   def self.track(name, *event)
-    { name.to_s => [event.last.merge(class_name: 'Event')] }
+    { name.to_s => [event.last.deep_stringify_keys.merge('class_name' => 'Event')] }
   end
 
 end

lib/rack/tracker/go_squared/go_squared.rb

@@ -32,6 +32,6 @@ def visitor_info
   end
 
   def self.track(name, *event)
-    { name.to_s => [event.last.merge(class_name: event.first.to_s.classify)] }
+    { name.to_s => [event.last.deep_stringify_keys.merge('class_name' => event.first.to_s.classify)] }
   end
 end

lib/rack/tracker/google_analytics/google_analytics.rb

@@ -50,6 +50,6 @@ def ecommerce_events
   end
 
   def self.track(name, *event)
-    { name.to_s => [event.last.merge(class_name: event.first.to_s.capitalize)] }
+    { name.to_s => [event.last.deep_stringify_keys.merge('class_name' => event.first.to_s.capitalize)] }
   end
 end

lib/rack/tracker/handler.rb

@@ -13,7 +13,7 @@ def initialize(env, options = {})
 
   def events
     events = env.fetch('tracker', {})[self.class.to_s.demodulize.underscore] || []
-    events.map{ |ev| "#{self.class}::#{ev[:class_name]}".constantize.new(ev.except(:class_name)) }
+    events.map{ |ev| "#{self.class}::#{ev['class_name']}".constantize.new(ev.except('class_name')) }
   end
 
   def render

spec/handler/facebook_spec.rb

@@ -37,10 +37,10 @@ def env
         'facebook' =>
           [
             {
-              id: '123456789',
-              value: '23',
-              currency: 'EUR',
-              class_name: 'Event'
+              'id' => '123456789',
+              'value' => '23',
+              'currency' => 'EUR',
+              'class_name' => 'Event'
             }
           ]
         }

spec/handler/go_squared_spec.rb

@@ -14,7 +14,7 @@ def env
       def env
         {'tracker' => {
           'go_squared' => [
-            { class_name: 'VisitorName', name: 'John Doe' }
+            { 'class_name' => 'VisitorName', 'name' => 'John Doe' }
           ]
         }}
       end
@@ -30,7 +30,7 @@ def env
       def env
         {'tracker' => {
           'go_squared' => [
-            { class_name: 'VisitorInfo', age: 35, favorite_food: 'pizza' }
+            { 'class_name' => 'VisitorInfo', 'age' => 35, 'favorite_food' => 'pizza' }
           ]
         }}
       end

spec/handler/google_analytics_spec.rb

@@ -76,7 +76,7 @@ def env
       def env
         {'tracker' => {
           'google_analytics' => [
-            { class_name: 'Send', category: "Users", action: "Login", label: "Standard" }
+            { 'class_name' => 'Send', 'category' => 'Users', 'action' => 'Login', 'label' => 'Standard' }
           ]
         }}
       end
@@ -90,7 +90,7 @@ def env
     describe "with a event value" do
       def env
         {'tracker' => { 'google_analytics' => [
-          { class_name: 'Send', category: "Users", action: "Login", label: "Standard", value: "5" }
+          { 'class_name' => 'Send', category: "Users", action: "Login", label: "Standard", value: "5" }
         ]}}
       end
 
@@ -106,8 +106,8 @@ def env
       def env
         {'tracker' => {
           'google_analytics' => [
-            { class_name: 'Ecommerce', type: 'addItem', id: '1234', name: 'Fluffy Pink Bunnies', sku: 'DD23444', category: 'Party Toys', price: '11.99', quantity: '1' },
-            { class_name: 'Ecommerce', type: 'addTransaction', id: '1234', affiliation: 'Acme Clothing', revenue: 11.99, shipping: '5', tax: '1.29', currency: 'EUR' }
+            { 'class_name' => 'Ecommerce', 'type' => 'addItem', 'id' => '1234', 'name' => 'Fluffy Pink Bunnies', 'sku' => 'DD23444', 'category' => 'Party Toys', 'price' => '11.99', 'quantity' => '1' },
+            { 'class_name' => 'Ecommerce', 'type' => 'addTransaction', 'id' => '1234', 'affiliation' => 'Acme Clothing', 'revenue' => 11.99, 'shipping' => '5', 'tax' => '1.29', 'currency' => 'EUR' }
           ]
         }}
       end

spec/tracker/controller_spec.rb

@@ -22,11 +22,11 @@ def index
 
 RSpec.describe Rack::Tracker::Controller do
   describe '#tracker' do
-    let(:send)       { { class_name: 'Send', category: 'foo' } }
-    let(:trx)        { { class_name: 'Ecommerce', type: 'addTransaction', some: 'thing' } }
-    let(:item_foo)   { { class_name: 'Ecommerce', type: 'addItem', name: 'foo' } }
-    let(:item_bar)   { { class_name: 'Ecommerce', type: 'addItem', name: 'bar' } }
-    let(:fb_event)   { { class_name: 'Event', id: '1', value: 1, currency: 'USD' } }
+    let(:send)       { { 'category' => 'foo', 'class_name' => 'Send' } }
+    let(:trx)        { { 'type' => 'addTransaction', 'some' => 'thing', 'class_name' => 'Ecommerce' } }
+    let(:item_foo)   { { 'type' => 'addItem', 'name' => 'foo', 'class_name' => 'Ecommerce' } }
+    let(:item_bar)   { { 'type' => 'addItem', 'name' => 'bar', 'class_name' => 'Ecommerce' } }
+    let(:fb_event)   { { 'id' => '1', 'value' => 1, 'currency' => 'USD', 'class_name' => 'Event' } }
     let(:controller) { TestController.new({}) }
 
     context 'controller' do