feat: navigation tracking support with expo router (#1270)

* feat: add screen tracker on screen change listener and tests

* feat (example): add screen change listener

Author: kholood-ea

examples/default/src/App.tsx

@@ -35,9 +35,7 @@ export const App: React.FC = () => {
   }, []);
 
   useEffect(() => {
-    const unregisterListener = Instabug.setNavigationListener(navigationRef);
-
-    return unregisterListener;
+    Instabug.setNavigationListener(navigationRef);
   }, [navigationRef]);
 
   return (

src/modules/Instabug.ts

@@ -544,7 +544,7 @@ export const onStateChange = (state?: NavigationStateV5) => {
 export const setNavigationListener = (
   navigationRef: NavigationContainerRefWithCurrent<ReactNavigation.RootParamList>,
 ) => {
-  return navigationRef.addListener('state', () => {
+  navigationRef.addListener('state', () => {
     onStateChange(navigationRef.getRootState());
   });
 };

test/modules/Instabug.spec.ts

@@ -238,39 +238,20 @@ describe('Instabug Module', () => {
   });
 
   it('setNavigationListener should call the onStateChange on a screen change', async () => {
-    const mockedState = { routes: [{ name: 'ScreenName' }], index: 0 };
-
     const mockNavigationContainerRef = {
       current: null,
       navigate: jest.fn(),
       reset: jest.fn(),
       goBack: jest.fn(),
       dispatch: jest.fn(),
-      getRootState: () => mockedState,
+      getRootState: jest.fn(),
       canGoBack: jest.fn(),
-
-      addListener: jest.fn((event, callback) => {
-        expect(event).toBe('state');
-        callback(mockedState);
-        return jest.fn();
-      }),
+      addListener: jest.fn(),
       removeListener: jest.fn(),
     } as unknown as NavigationContainerRefWithCurrent<ReactNavigation.RootParamList>;
 
-    const onStateChangeMock = jest.fn();
-
-    jest.spyOn(Instabug, 'onStateChange').mockImplementation(onStateChangeMock);
-
     Instabug.setNavigationListener(mockNavigationContainerRef);
-
     expect(mockNavigationContainerRef.addListener).toBeCalledTimes(1);
-    expect(mockNavigationContainerRef.addListener).toHaveBeenCalledWith(
-      'state',
-      expect.any(Function),
-    );
-
-    expect(onStateChangeMock).toBeCalledTimes(1);
-    expect(onStateChangeMock).toHaveBeenCalledWith(mockNavigationContainerRef.getRootState());
   });
 
   it('should call the native method init', () => {