[FSSDK-10439] client test: track method test update

Author: junaed-optimizely

src/client.spec.ts

@@ -333,12 +333,6 @@ describe('ReactSDKClient', () => {
           instance._client = null;
         });
 
-        it('cannot use pre-set or override user for track', () => {
-          const mockFn = mockInnerClient.track as jest.Mock;
-          instance.track('evt1');
-          expect(mockFn).toHaveBeenCalledTimes(0);
-        });
-
         it('cannot use pre-set or override user for setForcedVariation', () => {
           const mockFn = mockInnerClient.setForcedVariation as jest.Mock;
           mockFn.mockReturnValue(true);
@@ -382,30 +376,6 @@ describe('ReactSDKClient', () => {
         });
       });
 
-      it('can use pre-set and override user for track', () => {
-        const mockFn = mockInnerClient.track as jest.Mock;
-        instance.track('evt1');
-        expect(mockFn).toHaveBeenCalledTimes(1);
-        expect(mockFn).toHaveBeenCalledWith('evt1', 'user1', { foo: 'bar' }, undefined);
-        mockFn.mockReset();
-
-        instance.track('evt1', 'user2', { bar: 'baz' });
-        expect(mockFn).toHaveBeenCalledTimes(1);
-        expect(mockFn).toHaveBeenCalledWith('evt1', 'user2', { bar: 'baz' }, undefined);
-        mockFn.mockReset();
-
-        // Use pre-set user with event tags
-        instance.track('evt1', { tagKey: 'tagVal' });
-        expect(mockFn).toHaveBeenCalledTimes(1);
-        expect(mockFn).toHaveBeenCalledWith('evt1', 'user1', { foo: 'bar' }, { tagKey: 'tagVal' });
-        mockFn.mockReset();
-
-        // Use overrides with event tags
-        instance.track('evt1', 'user3', { bla: 'bla' }, { tagKey: 'tagVal' });
-        expect(mockFn).toHaveBeenCalledTimes(1);
-        expect(mockFn).toHaveBeenCalledWith('evt1', 'user3', { bla: 'bla' }, { tagKey: 'tagVal' });
-      });
-
       it('can use pre-set and override user for setForcedVariation', () => {
         const mockFn = mockInnerClient.setForcedVariation as jest.Mock;
         mockFn.mockReturnValue(true);
@@ -1331,6 +1301,44 @@ describe('ReactSDKClient', () => {
     });
   });
 
+  describe('track', () => {
+    beforeEach(async () => {
+      await setUpUserContext();
+    });
+
+    it('cannot use pre-set or override user for track', () => {
+      // @ts-ignore
+      instance._client = null;
+      const mockFn = mockInnerClient.track as jest.Mock;
+      instance.track('evt1');
+      expect(mockFn).toHaveBeenCalledTimes(0);
+    });
+
+    it('can use pre-set and override user for track', () => {
+      const mockFn = mockInnerClient.track as jest.Mock;
+      instance.track('evt1');
+      expect(mockFn).toHaveBeenCalledTimes(1);
+      expect(mockFn).toHaveBeenCalledWith('evt1', 'user1', { foo: 'bar' }, undefined);
+      mockFn.mockReset();
+
+      instance.track('evt1', 'user2', { bar: 'baz' });
+      expect(mockFn).toHaveBeenCalledTimes(1);
+      expect(mockFn).toHaveBeenCalledWith('evt1', 'user2', { bar: 'baz' }, undefined);
+      mockFn.mockReset();
+
+      // Use pre-set user with event tags
+      instance.track('evt1', { tagKey: 'tagVal' });
+      expect(mockFn).toHaveBeenCalledTimes(1);
+      expect(mockFn).toHaveBeenCalledWith('evt1', 'user1', { foo: 'bar' }, { tagKey: 'tagVal' });
+      mockFn.mockReset();
+
+      // Use overrides with event tags
+      instance.track('evt1', 'user3', { bla: 'bla' }, { tagKey: 'tagVal' });
+      expect(mockFn).toHaveBeenCalledTimes(1);
+      expect(mockFn).toHaveBeenCalledWith('evt1', 'user3', { bla: 'bla' }, { tagKey: 'tagVal' });
+    });
+  });
+
   describe('fetchQualifedSegments', () => {
     beforeEach(() => {
       instance = createInstance(config);