test(app): fix all app e2e deprecation warnings

Author: mikehardy

packages/app/e2e/app.e2e.js

@@ -17,6 +17,16 @@
 
 describe('modular', function () {
   describe('firebase v8 compat', function () {
+    beforeEach(async function beforeEachTest() {
+      // @ts-ignore
+      globalThis.RNFB_SILENCE_MODULAR_DEPRECATION_WARNINGS = true;
+    });
+
+    afterEach(async function afterEachTest() {
+      // @ts-ignore
+      globalThis.RNFB_SILENCE_MODULAR_DEPRECATION_WARNINGS = false;
+    });
+
     it('it should allow read the default app from native', function () {
       if (Platform.other) return; // Not supported on non-native platforms.
       // app is created in tests app before all hook
@@ -201,21 +211,20 @@ describe('modular', function () {
     });
 
     it('it should initialize dynamic apps', async function () {
-      const { initializeApp, getApps, getApp } = modular;
+      const { initializeApp, getApps, getApp, deleteApp } = modular;
 
-      const appCount = firebase.apps.length;
+      const appCount = getApps().length;
       const name = `testscoreapp${FirebaseHelpers.id}`;
       const platformAppConfig = FirebaseHelpers.app.config();
       const newApp = await initializeApp(platformAppConfig, name);
       newApp.name.should.equal(name);
-      newApp.toString().should.equal(name);
       newApp.options.apiKey.should.equal(platformAppConfig.apiKey);
 
       const apps = getApps();
 
       should.equal(apps.includes(getApp(name)), true);
       should.equal(apps.length, appCount + 1);
-      return newApp.delete();
+      return deleteApp(newApp);
     });
 
     it('should error if dynamic app initialization values are incorrect', async function () {
@@ -257,8 +266,8 @@ describe('modular', function () {
       } catch (e) {
         e.code.should.containEql('app/unknown');
         e.message.should.containEql('Configuration fails');
-        should.equal(firebase.apps.length, appCount);
-        should.equal(firebase.apps.includes('myname'), false);
+        should.equal(getApps().length, appCount);
+        should.equal(getApps().includes('myname'), false);
       }
     });
 
@@ -270,7 +279,6 @@ describe('modular', function () {
       const newApp = await initializeApp(platformAppConfig, name);
 
       newApp.name.should.equal(name);
-      newApp.toString().should.equal(name);
       newApp.options.apiKey.should.equal(platformAppConfig.apiKey);
 
       await deleteApp(newApp);

packages/app/e2e/asyncStorage.e2e.js

@@ -16,8 +16,9 @@ describe('firebase.setReactNativeAsyncStorage()', function () {
   });
 
   it('throws if asyncStorage is not an object', function () {
+    const { setReactNativeAsyncStorage } = modular;
     try {
-      firebase.setReactNativeAsyncStorage(123);
+      setReactNativeAsyncStorage(123);
       return Promise.reject(new Error('Did not throw an Error.'));
     } catch (error) {
       error.message.should.containEql("'asyncStorage' must be an object");
@@ -26,8 +27,9 @@ describe('firebase.setReactNativeAsyncStorage()', function () {
   });
 
   it('throws if asyncStorage.setItem is not a function', function () {
+    const { setReactNativeAsyncStorage } = modular;
     try {
-      firebase.setReactNativeAsyncStorage({ setItem: 123 });
+      setReactNativeAsyncStorage({ setItem: 123 });
       return Promise.reject(new Error('Did not throw an Error.'));
     } catch (error) {
       error.message.should.containEql("'asyncStorage.setItem' must be a function");
@@ -36,8 +38,9 @@ describe('firebase.setReactNativeAsyncStorage()', function () {
   });
 
   it('throws if asyncStorage.getItem is not a function', function () {
+    const { setReactNativeAsyncStorage } = modular;
     try {
-      firebase.setReactNativeAsyncStorage({ setItem: sinon.spy(), getItem: 123 });
+      setReactNativeAsyncStorage({ setItem: sinon.spy(), getItem: 123 });
       return Promise.reject(new Error('Did not throw an Error.'));
     } catch (error) {
       error.message.should.containEql("'asyncStorage.getItem' must be a function");
@@ -46,8 +49,9 @@ describe('firebase.setReactNativeAsyncStorage()', function () {
   });
 
   it('throws if asyncStorage.removeItem is not a function', function () {
+    const { setReactNativeAsyncStorage } = modular;
     try {
-      firebase.setReactNativeAsyncStorage({
+      setReactNativeAsyncStorage({
         setItem: sinon.spy(),
         getItem: sinon.spy(),
         removeItem: 123,
@@ -60,13 +64,14 @@ describe('firebase.setReactNativeAsyncStorage()', function () {
   });
 
   it('sets the async storage instance', async function () {
+    const { setReactNativeAsyncStorage } = modular;
     isMemoryStorage().should.equal(true);
 
     const setItemSpy = sinon.spy();
     const getItemSpy = sinon.spy();
     const removeItemSpy = sinon.spy();
 
-    firebase.setReactNativeAsyncStorage({
+    setReactNativeAsyncStorage({
       setItem: setItemSpy,
       getItem: getItemSpy,
       removeItem: removeItemSpy,
@@ -84,10 +89,11 @@ describe('firebase.setReactNativeAsyncStorage()', function () {
   });
 
   it('works with @react-native-async-storage/async-storage', async function () {
+    const { setReactNativeAsyncStorage } = modular;
     isMemoryStorage().should.equal(true);
     const key = Date.now().toString();
     const value = 'bar';
-    firebase.setReactNativeAsyncStorage(asyncStorage);
+    setReactNativeAsyncStorage(asyncStorage);
     isMemoryStorage().should.equal(false);
 
     // Through our internals,

packages/app/e2e/utils.e2e.js

@@ -19,34 +19,44 @@ describe('utils()', function () {
   if (Platform.other) return; // Not supported on non-native platforms.
 
   describe('namespace', function () {
+    beforeEach(async function beforeEachTest() {
+      // @ts-ignore
+      globalThis.RNFB_SILENCE_MODULAR_DEPRECATION_WARNINGS = true;
+    });
+
+    afterEach(async function afterEachTest() {
+      // @ts-ignore
+      globalThis.RNFB_SILENCE_MODULAR_DEPRECATION_WARNINGS = false;
+    });
+
     it('accessible from firebase.app()', function () {
       const app = firebase.app();
       should.exist(app.utils);
       app.utils().app.should.equal(app);
     });
-  });
 
-  describe('isRunningInTestLab', function () {
-    it('returns true or false', function () {
-      should.equal(firebase.utils().isRunningInTestLab, false);
+    describe('isRunningInTestLab', function () {
+      it('returns true or false', function () {
+        should.equal(firebase.utils().isRunningInTestLab, false);
+      });
     });
-  });
 
-  describe('playServicesAvailability', function () {
-    it('returns isAvailable and Play Service status', async function () {
-      const playService = await firebase.utils().playServicesAvailability;
-      //iOS always returns { isAvailable: true, status: 0}
-      should(playService.isAvailable).equal(true);
-      should(playService.status).equal(0);
+    describe('playServicesAvailability', function () {
+      it('returns isAvailable and Play Service status', async function () {
+        const playService = await firebase.utils().playServicesAvailability;
+        //iOS always returns { isAvailable: true, status: 0}
+        should(playService.isAvailable).equal(true);
+        should(playService.status).equal(0);
+      });
     });
-  });
 
-  describe('getPlayServicesStatus', function () {
-    it('returns isAvailable and Play Service status', async function () {
-      const status = await firebase.utils().getPlayServicesStatus();
-      //iOS always returns { isAvailable: true, status: 0}
-      should(status.isAvailable).equal(true);
-      should(status.status).equal(0);
+    describe('getPlayServicesStatus', function () {
+      it('returns isAvailable and Play Service status', async function () {
+        const status = await firebase.utils().getPlayServicesStatus();
+        //iOS always returns { isAvailable: true, status: 0}
+        should(status.isAvailable).equal(true);
+        should(status.status).equal(0);
+      });
     });
   });
 });