fix(atlas-service): dont signin twice in tests COMPASS-8837 (#6624)

* dont call signin twice

* only one signin

* no only

* wait so that sign-in is triggered

* ensure that stub is called and only once

Author: mabaasit

packages/atlas-service/src/store/atlas-signin-reducer.spec.ts

@@ -159,28 +159,31 @@ describe('atlasSignInReducer', function () {
     });
 
     it('should cancel sign in if sign in is in progress', async function () {
-      const mockAtlasService = {
-        isAuthenticated: sandbox
-          .stub()
-          .callsFake(({ signal }: { signal: AbortSignal }) => {
-            return new Promise((resolve, reject) => {
-              signal.addEventListener('abort', () => {
-                reject(signal.reason);
-              });
+      const isAuthenticatedStub = sandbox
+        .stub()
+        .callsFake(({ signal }: { signal: AbortSignal }) => {
+          return new Promise((resolve, reject) => {
+            signal.addEventListener('abort', () => {
+              reject(signal.reason);
             });
-          }),
+          });
+        });
+      const mockAtlasService = {
+        isAuthenticated: isAuthenticatedStub,
       };
       const store = configureStore({
         atlasAuthService: mockAtlasService as any,
       });
 
       void store.dispatch(performSignInAttempt()).catch(() => {});
 
-      await Promise.all([
-        store.dispatch(signIn()),
-        store.dispatch(cancelSignIn()),
-      ]);
+      // Give it some time for start the sign in attempt. It will be waiting
+      // at isAuthenticated, which never resolves.
+      await new Promise((resolve) => setTimeout(resolve, 100));
+      store.dispatch(cancelSignIn());
       expect(store.getState()).to.have.nested.property('state', 'canceled');
+
+      expect(isAuthenticatedStub).to.have.been.calledOnce;
     });
   });