Add user identity tracking for Heap

Author: carlosramireziii

README.md

@@ -627,6 +627,10 @@ config.middleware.use(Rack::Tracker) do
 end
 ```
 
+Other options:
+
+* `user_id` - This value will be used as the user identity
+
 ### Custom Handlers
 
 Tough we give you handlers for a few tracking services right out of the box, you might

lib/rack/tracker/heap/heap.rb

@@ -1,2 +1,3 @@
 class Rack::Tracker::Heap < Rack::Tracker::Handler
+  self.allowed_tracker_options = [:user_id]
 end

lib/rack/tracker/heap/template/heap.erb

@@ -1,4 +1,7 @@
 <script>
   window.heap=window.heap||[],heap.load=function(e,t){window.heap.appid=e,window.heap.config=t=t||{};var r=t.forceSSL||"https:"===document.location.protocol,a=document.createElement("script");a.type="text/javascript",a.async=!0,a.src=(r?"https:":"http:")+"//cdn.heapanalytics.com/js/heap-"+e+".js";var n=document.getElementsByTagName("script")[0];n.parentNode.insertBefore(a,n);for(var o=function(e){return function(){heap.push([e].concat(Array.prototype.slice.call(arguments,0)))}},p=["addEventProperties","addUserProperties","clearEventProperties","identify","resetIdentity","removeEventProperty","setEventProperties","track","unsetEventProperty"],c=0;c<p.length;c++)heap[p[c]]=o(p[c])};
   heap.load("<%= options[:env_id] %>");
+  <% if tracker_options[:user_id].present? %>
+  heap.identify("<%= tracker_options[:user_id] %>");
+  <% end %>
 </script>

spec/handler/heap_spec.rb

@@ -7,4 +7,30 @@ def env
     expect(described_class.position).to eq(:head)
     expect(described_class.new(env).position).to eq(:head)
   end
+
+  describe 'with user_id tracking' do
+    subject { described_class.new(env, { user_id: user_id }).render }
+
+    let(:user_id) { '123' }
+
+    it 'will include identify call with user_id value' do
+      expect(subject).to match(%r{heap.identify\("123"\);})
+    end
+
+    context 'when user_id value is a proc' do
+      let(:user_id) { proc { '123' } }
+
+      it 'will include identify call with the user_id called value' do
+        expect(subject).to match(%r{heap.identify\("123"\);})
+      end
+    end
+
+    context 'when user_id value is blank' do
+      let(:user_id) { '' }
+
+      it 'will not include identify call' do
+        expect(subject).not_to match(%r{heap.identify})
+      end
+    end
+  end
 end