Fix test issues

Author: sam-gc

packages-exp/auth-exp/src/platform_browser/messagechannel/sender.test.ts

@@ -89,6 +89,7 @@ describe('platform_browser/messagechannel/sender', () => {
 
     it('should timeout if it doesnt see an ACK', async () => {
       serviceWorker.addEventListener('message', (_event: Event) => {
+        // eslint-disable-next-line @typescript-eslint/no-floating-promises
         Promise.resolve().then(() => {
           pendingTimeouts[_TimeoutDuration.ACK]();
         });
@@ -110,6 +111,7 @@ describe('platform_browser/messagechannel/sender', () => {
         }
       ];
       serviceWorker.addEventListener('message', (event: Event) => {
+        // eslint-disable-next-line @typescript-eslint/no-floating-promises
         Promise.resolve().then(() => {
           pendingTimeouts[_TimeoutDuration.ACK]();
         });

packages-exp/auth-exp/src/platform_browser/popup_redirect.test.ts

@@ -41,11 +41,8 @@ import {
   PopupRedirectResolver
 } from '../model/popup_redirect';
 import * as authWindow from './auth_window';
-import * as mockFetch from '../../test/helpers/mock_fetch';
 import * as gapiLoader from './iframe/gapi';
 import { browserPopupRedirectResolver } from './popup_redirect';
-import { mockEndpoint } from '../../test/helpers/api/helper';
-import { Endpoint } from '../api';
 
 use(chaiAsPromised);
 use(sinonChai);
@@ -57,7 +54,6 @@ describe('platform_browser/popup_redirect', () => {
   let iframeSendStub: sinon.SinonStub;
 
   beforeEach(async () => {
-    mockFetch.setUp();
     auth = await testAuth();
     resolver = new (browserPopupRedirectResolver as SingletonInstantiator<
       PopupRedirectResolver
@@ -88,7 +84,6 @@ describe('platform_browser/popup_redirect', () => {
 
   afterEach(() => {
     sinon.restore();
-    mockFetch.tearDown();
     delete authWindow._window().gapi;
   });
 
@@ -125,15 +120,11 @@ describe('platform_browser/popup_redirect', () => {
       );
     });
 
-    it('throws an error if authDomain is unspecified', async () => {
-       await resolver._initialize(auth);
-      
-      mockEndpoint(Endpoint.GET_PROJECT_CONFIG, {
-        authorizedDomains: []
-      });
+    it('validates the origin', async () => {
+      await resolver._initialize(auth);
 
-      expect(resolver._openPopup(auth, provider, event)
-      ).to.be.rejectedWith(FirebaseError, 'auth/auth-domain-config-required');
+      await resolver._openPopup(auth, provider, event);
+      expect(validateOrigin._validateOrigin).to.have.been.calledWith(auth);
     });
 
     it('throws an error if apiKey is unspecified', async () => {

packages-exp/auth-exp/src/platform_browser/strategies/phone.test.ts

@@ -421,7 +421,7 @@ describe('platform_browser/strategies/phone', () => {
     let idToken: string;
 
     beforeEach(() => {
-      idToken = makeJWT({exp: '200', iat: '100'})
+      idToken = makeJWT({exp: '200', iat: '100'});
       reloadMock = mockEndpoint(Endpoint.GET_ACCOUNT_INFO, {
         users: [{ uid: 'uid' }]
       });