Test refactors

Author: EmilianoSanchez

src/__tests__/SplitFactory.test.tsx

@@ -31,7 +31,7 @@ describe('SplitFactory', () => {
           expect(isTimedout).toBe(false);
           expect(isDestroyed).toBe(false);
           expect(lastUpdate).toBe(0);
-          expect((factory as SplitIO.ISDK).settings.version).toContain('react-');
+          expect((factory as SplitIO.IBrowserSDK).settings.version).toContain('react-');
           return null;
         }}
       </SplitFactory>
@@ -55,7 +55,7 @@ describe('SplitFactory', () => {
           expect(isTimedout).toBe(false);
           expect(isDestroyed).toBe(false);
           expect(lastUpdate).toBe((outerFactory.client() as IClientWithContext).__getStatus().lastUpdate);
-          expect((factory as SplitIO.ISDK).settings.version).toBe(outerFactory.settings.version);
+          expect((factory as SplitIO.IBrowserSDK).settings.version).toBe(outerFactory.settings.version);
           return null;
         }}
       </SplitFactory>
@@ -239,7 +239,7 @@ describe('SplitFactory', () => {
       <SplitFactory config={sdkBrowser} >
         {({ factory }) => {
           expect(__factories.size).toBe(1);
-          destroyMainClientSpy = jest.spyOn((factory as SplitIO.ISDK).client(), 'destroy');
+          destroyMainClientSpy = jest.spyOn((factory as SplitIO.IBrowserSDK).client(), 'destroy');
           return (
             <SplitClient splitKey='other_key' >
               {({ client }) => {
@@ -267,7 +267,7 @@ describe('SplitFactory', () => {
         {({ factory }) => {
           // if factory is provided as a prop, `factories` cache is not modified
           expect(__factories.size).toBe(0);
-          destroyMainClientSpy = jest.spyOn((factory as SplitIO.ISDK).client(), 'destroy');
+          destroyMainClientSpy = jest.spyOn((factory as SplitIO.IBrowserSDK).client(), 'destroy');
           return (
             <SplitClient splitKey='other_key' >
               {({ client }) => {

src/__tests__/SplitFactoryProvider.test.tsx

@@ -14,7 +14,7 @@ const logSpy = jest.spyOn(console, 'log');
 import { ISplitFactoryChildProps } from '../types';
 import { SplitFactoryProvider } from '../SplitFactoryProvider';
 import { SplitClient } from '../SplitClient';
-import { SplitContext } from '../SplitContext';
+import { INITIAL_CONTEXT, SplitContext } from '../SplitContext';
 import { __factories, IClientWithContext } from '../utils';
 import { WARN_SF_CONFIG_AND_FACTORY } from '../constants';
 
@@ -55,7 +55,7 @@ describe('SplitFactoryProvider', () => {
           expect(isTimedout).toBe(false);
           expect(isDestroyed).toBe(false);
           expect(lastUpdate).toBe((outerFactory.client() as IClientWithContext).__getStatus().lastUpdate);
-          expect((factory as SplitIO.ISDK).settings.version).toBe(outerFactory.settings.version);
+          expect((factory as SplitIO.IBrowserSDK).settings.version).toBe(outerFactory.settings.version);
           return null;
         }}
       </SplitFactoryProvider>
@@ -301,11 +301,7 @@ describe('SplitFactoryProvider', () => {
       return (
         <SplitContext.Consumer>
           {(value) => {
-            expect(value.factory).toBe(null);
-            expect(value.client).toBe(null);
-            expect(value.isReady).toBe(false);
-            expect(value.isTimedout).toBe(false);
-            expect(value.lastUpdate).toBe(0);
+            expect(value).toEqual(INITIAL_CONTEXT);
             done();
             return null;
           }}
@@ -338,7 +334,7 @@ describe('SplitFactoryProvider', () => {
 
   test('cleans up on update and unmount if config prop is provided.', () => {
     let renderTimes = 0;
-    const createdFactories = new Set<SplitIO.ISDK>();
+    const createdFactories = new Set<SplitIO.IBrowserSDK>();
     const clientDestroySpies: jest.SpyInstance[] = [];
     const outerFactory = SplitSdk(sdkBrowser);