Merge pull request Driftt#9 from Driftt/integration-options

send integration options to drift identify

Author: mjelicic

lib/index.js

@@ -69,8 +69,9 @@ Drift.prototype.identify = function(identify) {
   if (!identify.userId()) return this.debug('user id required');
   var traits = identify.traits();
   var id = identify.userId();
+  var integrationOptions = identify.options(this.name);
   delete traits.id;
-  window.drift.identify(id, traits);
+  window.drift.identify(id, traits, integrationOptions);
   this.identified = true;
 };
 

test/index.test.js

@@ -84,6 +84,41 @@ describe('Drift', function() {
         analytics.identify('id', { email: '[email protected]' });
         analytics.called(window.drift.identify, 'id', { email: '[email protected]' });
       });
+
+      it('should send id, traits and integration options', function() {
+        analytics.identify(
+          'id',
+          { email: '[email protected]' },
+          { drift: { signedIdentity: 'signedidentityjwt' } }
+        );
+
+        analytics.called(
+          window.drift.identify,
+          'id',
+          { email: '[email protected]' },
+          { signedIdentity: 'signedidentityjwt' }
+        );
+      });
+
+      it('should send only integration options for drift', function() {
+        analytics.identify(
+          'id',
+          { email: '[email protected]' },
+          { otherintegration: { option: '123' }, drift: { option: 'test' } }
+        );
+
+        analytics.called(window.drift.identify, 'id', { email: '[email protected]' }, { option: 'test' });
+      });
+
+      it('should send empty object as integration options if they are undefined', function() {
+        analytics.identify(
+          'id',
+          { email: '[email protected]' },
+          { otherintegration: { option: '123' }, drift: undefined }
+        );
+
+        analytics.called(window.drift.identify, 'id', { email: '[email protected]' }, {});
+      });
     });
 
     describe('#track', function() {