Fix analytics and functions Saucelabs tests

Author: hsubox76

packages/analytics/index.test.ts

@@ -88,7 +88,8 @@ describe('FirebaseAnalytics instance tests', () => {
         currency: 'USD'
       });
       // Clear event stack of async FID call.
-      await fidDeferred.promise;
+      // For IE: Need to wrap it in another function as FID fetch is wrapped in initializeGAId.
+      await (() => fidDeferred.promise);
       expect(gtagStub).to.have.been.calledWith('js');
       expect(gtagStub).to.have.been.calledWith(
         GtagCommand.CONFIG,
@@ -158,7 +159,8 @@ describe('FirebaseAnalytics instance tests', () => {
         currency: 'USD'
       });
       // Clear event stack of async FID call.
-      await fidDeferred.promise;
+      // For IE: Need to wrap it in another function as FID fetch is wrapped in initializeGAId.
+      await (() => fidDeferred.promise);
       expect(gtagStub).to.have.been.calledWith('js');
       expect(gtagStub).to.have.been.calledWith(
         GtagCommand.CONFIG,

packages/functions/karma.conf.js

@@ -15,16 +15,14 @@
  * limitations under the License.
  */
 
-const karma = require('karma');
-const path = require('path');
 const karmaBase = require('../../config/karma.base');
 
 const files = [`test/**/*`];
 
 module.exports = function(config) {
   const karmaConfig = Object.assign({}, karmaBase, {
     // files to load into karma
-    files: files,
+    files,
     // frameworks to use
     // available frameworks: https://npmjs.org/browse/keyword/karma-adapter
     frameworks: ['mocha']

packages/functions/test/browser/callable.test.ts

@@ -43,6 +43,10 @@ describe('Firebase Functions > Call', () => {
   // TODO(klimt): Move this to the cross-platform tests and delete this file,
   // once instance id works there.
   it('instance id', async () => {
+    if (!('Notification' in self)) {
+      console.log('No Notification API: skipping instance id test.');
+      return;
+    }
     // mock firebase messaging
     const messagingMock: FirebaseMessaging = ({
       getToken: async () => 'iid'