revert: test changes

Author: mikechu-optimizely

jest.config.js

@@ -2,7 +2,7 @@ module.exports = {
   "transform": {
     "^.+\\.(ts|tsx|js|jsx)$": "ts-jest",
   },
-  "testRegex": "(/tests/.*|(\\.|/)(test|spec|tests))\\.(tsx?|js)$",
+  "testRegex": "(/tests/.*|(\\.|/)(test|spec))\\.tsx?$",
   moduleNameMapper: {
     // Force module uuid to resolve with the CJS entry point, because Jest does not support package.json.exports. See https://github.com/uuidjs/uuid/issues/451
     "uuid": require.resolve('uuid'),

lib/index.browser.tests.js

@@ -816,8 +816,7 @@ describe('javascript-sdk (Browser)', function() {
         sinon.assert.called(fakeEventManager.sendEvent);
       });
 
-      // TODO: unskip and correct failing test. Need this reference implementation for other work [FSSDK-10711]
-      it.skip('should augment odp events with user agent data if userAgentParser is provided', async () => {
+      it('should augment odp events with user agent data if userAgentParser is provided', async () => {
         const userAgentParser = {
           parseUserAgentInfo() {
             return {
@@ -1059,8 +1058,7 @@ describe('javascript-sdk (Browser)', function() {
         assert(client.odpManager.eventManager.batchSize, 1);
       });
 
-      // TODO: unskip and correct failing test. Need this reference implementation for other work [FSSDK-10711]
-      it.skip('should send an odp event to the browser endpoint', async () => {
+      it('should send an odp event to the browser endpoint', async () => {
         const odpConfig = new OdpConfig();
 
         const apiManager = new BrowserOdpEventApiManager(mockRequestHandler, logger);
@@ -1117,8 +1115,7 @@ describe('javascript-sdk (Browser)', function() {
         sinon.assert.notCalled(logger.error);
       });
 
-      // TODO: unskip and correct failing test. Need this reference implementation for other work [FSSDK-10711]
-      it.skip('should send odp client_initialized on client instantiation', async () => {
+      it('should send odp client_initialized on client instantiation', async () => {
         const odpConfig = new OdpConfig('key', 'host', 'pixel', []);
         const apiManager = new BrowserOdpEventApiManager(mockRequestHandler, logger);
         sinon.spy(apiManager, 'sendEvents');

lib/optimizely/index.tests.js

@@ -4622,8 +4622,7 @@ describe('lib/optimizely', function() {
         assert.deepEqual(user2.getUserId(), userId2);
       });
 
-      // TODO: unskip and correct failing test. Need this reference implementation for other work [FSSDK-10711]
-      it.skip('should call the error handler for invalid user ID and return null', function() {
+      it('should call the error handler for invalid user ID and return null', function() {
         assert.isNull(optlyInstance.createUserContext(1));
         sinon.assert.calledOnce(errorHandler.handleError);
         var errorMessage = errorHandler.handleError.lastCall.args[0].message;
@@ -4633,8 +4632,7 @@ describe('lib/optimizely', function() {
         assert.strictEqual(logMessage, sprintf(ERROR_MESSAGES.INVALID_INPUT_FORMAT, 'OPTIMIZELY', 'user_id'));
       });
 
-      // TODO: unskip and correct failing test. Need this reference implementation for other work [FSSDK-10711]
-      it.skip('should call the error handler for invalid attributes and return null', function() {
+      it('should call the error handler for invalid attributes and return null', function() {
         assert.isNull(optlyInstance.createUserContext('user1', 'invalid_attributes'));
         sinon.assert.calledOnce(errorHandler.handleError);
         var errorMessage = errorHandler.handleError.lastCall.args[0].message;