chore: address pr comments

src/store/annotations/__mocks__/annotations.ts

@@ -0,0 +1,3 @@
+export default jest.fn(() => ({
+    getActiveAnnotationsId: jest.fn(),
+}));

src/store/common/__mocks__/options.ts

@@ -0,0 +1,3 @@
+export default jest.fn(() => ({
+    getFileVersionId: jest.fn(),
+}));

src/store/eventing/__tests__/active-test.ts

@@ -1,17 +1,18 @@
-import { handleActiveAnnotationEvents } from '../active';
-import { AppState } from '../../types';
 import eventManager from '../../../common/EventManager';
+import { AppState } from '../../types';
 import { getActiveAnnotationId } from '../../annotations';
+import { getFileVersionId } from '../../options';
+import { handleActiveAnnotationEvents } from '../active';
 
 jest.mock('../../../common/EventManager');
-jest.mock('../../annotations', () => ({
-    getActiveAnnotationId: jest.fn(),
-}));
-jest.mock('../../options', () => ({
-    getFileVersionId: jest.fn().mockReturnValue('456'),
-}));
+jest.mock('../../annotations');
+jest.mock('../../options');
 
 describe('store/eventing/active', () => {
+    beforeEach(() => {
+        (getFileVersionId as jest.Mock).mockReturnValue('456');
+    });
+
     test('should not emit event if prev and next ids are the same', () => {
         const state = {} as AppState;
         (getActiveAnnotationId as jest.Mock).mockImplementationOnce(() => '123').mockImplementationOnce(() => '123');

src/store/eventing/active.ts

@@ -1,8 +1,8 @@
 import eventManager from '../../common/EventManager';
-import { getActiveAnnotationId } from '../annotations';
-import { getFileVersionId } from '../options';
 import { AppState } from '../types';
 import { Event } from '../../@types';
+import { getActiveAnnotationId } from '../annotations';
+import { getFileVersionId } from '../options';
 
 const handleActiveAnnotationEvents = (prevState: AppState, nextState: AppState): void => {
     const prevActiveAnnotationId = getActiveAnnotationId(prevState);