Use default event dispatcher in RN entry point when none provided by user

Author: Matt Carroll

packages/optimizely-sdk/lib/index.react_native.js

@@ -90,11 +90,11 @@ module.exports = {
         {
           clientEngine: enums.JAVASCRIPT_CLIENT_ENGINE,
           eventBatchSize: DEFAULT_EVENT_BATCH_SIZE,
+          eventDispatcher: defaultEventDispatcher,
           eventFlushInterval: DEFAULT_EVENT_FLUSH_INTERVAL,
         },
         config,
         {
-          eventDispatcher: config.eventDispatcher,
           // always get the OptimizelyLogger facade from logging
           logger: logger,
           errorHandler: logging.getErrorHandler(),

packages/optimizely-sdk/lib/index.react_native.tests.js

@@ -22,6 +22,7 @@ var packageJSON = require('../package.json');
 var testData = require('./tests/test_data');
 var eventProcessor = require('@optimizely/js-sdk-event-processor');
 var eventProcessorConfigValidator = require('./utils/event_processor_config_validator');
+var defaultEventDispatcher = require('./plugins/event_dispatcher/index.browser');
 
 var chai = require('chai');
 var assert = chai.assert;
@@ -131,6 +132,30 @@ describe('javascript-sdk/react-native', function() {
         assert.strictEqual(activate, 'control');
       });
 
+      describe('when no event dispatcher passed to createInstance', function() {
+        beforeEach(function() {
+          sinon.stub(defaultEventDispatcher, 'dispatchEvent', function(evt, cb) {
+            cb();
+          });
+        })
+
+        afterEach(function() {
+          defaultEventDispatcher.dispatchEvent.restore();
+        });
+
+        it('uses the default event dispatcher', function() {
+          var optlyInstance = optimizelyFactory.createInstance({
+            datafile: testData.getTestProjectConfig(),
+            errorHandler: fakeErrorHandler,
+            logger: silentLogger,
+          });
+          optlyInstance.activate('testExperiment', 'testUser');
+          return optlyInstance.close().then(function() {
+            sinon.assert.calledOnce(defaultEventDispatcher.dispatchEvent);
+          });
+        });
+      });
+
       describe('when passing in logLevel', function() {
         beforeEach(function() {
           sinon.stub(logging, 'setLogLevel');