chore(crashlytics): deprecation warning for v8 API ahead of future major release (#8107)

russellwheatley · mikehardy · commit 3279cc196848 · 2025-02-03T07:41:44.000-05:00
diff --git a/packages/app/lib/common/index.js b/packages/app/lib/common/index.js
@@ -103,6 +103,64 @@ export function tryJSONStringify(data) {
   }
 }
 
+// Used to indicate if there is no corresponding modular function
+const NO_REPLACEMENT = true;
+
+const mapOfDeprecationReplacements = {
+  crashlytics: {
+    checkForUnsentReports: 'checkForUnsentReports()',
+    crash: 'crash()',
+    deleteUnsentReports: 'deleteUnsentReports()',
+    didCrashOnPreviousExecution: 'didCrashOnPreviousExecution()',
+    log: 'log()',
+    setAttribute: 'setAttribute()',
+    setAttributes: 'setAttributes()',
+    setUserId: 'setUserId()',
+    recordError: 'recordError()',
+    sendUnsentReports: 'sendUnsentReports()',
+    setCrashlyticsCollectionEnabled: 'setCrashlyticsCollectionEnabled()',
+  },
+};
+
+const v8deprecationMessage =
+  'This v8 method is deprecated and will be removed in the next major release ' +
+  'as part of move to match Firebase Web modular v9 SDK API.';
+
+export function deprecationConsoleWarning(moduleName, methodName, isModularMethod) {
+  if (!isModularMethod) {
+    const moduleMap = mapOfDeprecationReplacements[moduleName];
+    if (moduleMap) {
+      const replacementMethodName = moduleMap[methodName];
+      // only warn if it is mapped and purposefully deprecated
+      if (replacementMethodName) {
+        const message = createMessage(moduleName, methodName);
+
+        // eslint-disable-next-line no-console
+        console.warn(message);
+      }
+    }
+  }
+}
+
+export function createMessage(moduleName, methodName, uniqueMessage = '') {
+  if (uniqueMessage.length > 0) {
+    // Unique deprecation message used for testing
+    return uniqueMessage;
+  }
+
+  const moduleMap = mapOfDeprecationReplacements[moduleName];
+  if (moduleMap) {
+    const replacementMethodName = moduleMap[methodName];
+    if (replacementMethodName) {
+      if (replacementMethodName !== NO_REPLACEMENT) {
+        return v8deprecationMessage + ` Please use \`${replacementMethodName}\` instead.`;
+      } else {
+        return v8deprecationMessage;
+      }
+    }
+  }
+}
+
 export const MODULAR_DEPRECATION_ARG = 'react-native-firebase-modular-method-call';
 
 export function warnIfNotModularCall(args, replacementMethodName, noAlternative) {
diff --git a/packages/app/lib/common/unitTestUtils.ts b/packages/app/lib/common/unitTestUtils.ts
@@ -1,10 +1,46 @@
+// @ts-nocheck
 import { expect, jest } from '@jest/globals';
+import { createMessage } from './index';
 
 export const checkV9Deprecation = (modularFunction: () => void, nonModularFunction: () => void) => {
   const consoleWarnSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
+  consoleWarnSpy.mockRestore();
   modularFunction();
   expect(consoleWarnSpy).not.toHaveBeenCalled();
+  consoleWarnSpy.mockClear();
+  const consoleWarnSpy2 = jest.spyOn(console, 'warn').mockImplementation(() => {});
   nonModularFunction();
-  expect(consoleWarnSpy).toHaveBeenCalledTimes(1);
-  consoleWarnSpy.mockRestore();
+
+  expect(consoleWarnSpy2).toHaveBeenCalledTimes(1);
+  consoleWarnSpy2.mockClear();
+};
+
+export type CheckV9DeprecationFunction = (
+  modularFunction: () => void,
+  nonModularFunction: () => void,
+  methodName: string,
+  uniqueMessage: string = '',
+) => void;
+
+export const createCheckV9Deprecation = (moduleName: string): CheckV9DeprecationFunction => {
+  return (
+    modularFunction: () => void,
+    nonModularFunction: () => void,
+    methodName: string,
+    uniqueMessage = '',
+  ) => {
+    const consoleWarnSpy = jest.spyOn(console, 'warn').mockImplementation(() => {});
+    consoleWarnSpy.mockReset();
+    modularFunction();
+    expect(consoleWarnSpy).not.toHaveBeenCalled();
+    consoleWarnSpy.mockReset();
+    const consoleWarnSpy2 = jest.spyOn(console, 'warn').mockImplementation(warnMessage => {
+      const message = createMessage(moduleName, methodName, uniqueMessage);
+      expect(message).toMatch(warnMessage);
+    });
+    nonModularFunction();
+
+    expect(consoleWarnSpy2).toHaveBeenCalledTimes(1);
+    consoleWarnSpy2.mockReset();
+  };
 };
diff --git a/packages/app/lib/internal/registry/namespace.js b/packages/app/lib/internal/registry/namespace.js
@@ -15,7 +15,7 @@
  *
  */
 
-import { isString } from '../../common';
+import { isString, MODULAR_DEPRECATION_ARG, deprecationConsoleWarning } from '../../common';
 import FirebaseApp from '../../FirebaseApp';
 import SDK_VERSION from '../../version';
 import { DEFAULT_APP_NAME, KNOWN_NAMESPACES } from '../constants';
@@ -170,10 +170,10 @@ function getOrCreateModuleForRoot(moduleNamespace) {
     }
 
     if (!APP_MODULE_INSTANCE[_app.name][moduleNamespace]) {
-      APP_MODULE_INSTANCE[_app.name][moduleNamespace] = new ModuleClass(
-        _app,
-        NAMESPACE_REGISTRY[moduleNamespace],
+      const module = createDeprecationProxy(
+        new ModuleClass(_app, NAMESPACE_REGISTRY[moduleNamespace]),
       );
+      APP_MODULE_INSTANCE[_app.name][moduleNamespace] = module;
     }
 
     return APP_MODULE_INSTANCE[_app.name][moduleNamespace];
@@ -277,6 +277,28 @@ export function getFirebaseRoot() {
   return createFirebaseRoot();
 }
 
+function createDeprecationProxy(instance) {
+  return new Proxy(instance, {
+    get(target, prop, receiver) {
+      const originalMethod = target[prop];
+      if (prop === 'constructor') {
+        return target.constructor;
+      }
+      if (typeof originalMethod === 'function') {
+        return function (...args) {
+          const isModularMethod = args.includes(MODULAR_DEPRECATION_ARG);
+          const moduleName = receiver._config.namespace;
+
+          deprecationConsoleWarning(moduleName, prop, isModularMethod);
+
+          return originalMethod.apply(target, args);
+        };
+      }
+      return Reflect.get(target, prop, receiver);
+    },
+  });
+}
+
 /**
  *
  * @param options
diff --git a/packages/crashlytics/__tests__/crashlytics.test.ts b/packages/crashlytics/__tests__/crashlytics.test.ts
@@ -1,5 +1,10 @@
-import { describe, expect, it } from '@jest/globals';
-
+import { describe, expect, it, jest, beforeEach } from '@jest/globals';
+// @ts-ignore test
+import FirebaseModule from '../../app/lib/internal/FirebaseModule';
+import {
+  createCheckV9Deprecation,
+  CheckV9DeprecationFunction,
+} from '../../app/lib/common/unitTestUtils';
 import {
   firebase,
   getCrashlytics,
@@ -79,4 +84,132 @@ describe('Crashlytics', function () {
       expect(setCrashlyticsCollectionEnabled).toBeDefined();
     });
   });
+
+  describe('test `console.warn` is called for RNFB v8 API & not called for v9 API', function () {
+    let checkV9Deprecation: CheckV9DeprecationFunction;
+
+    beforeEach(function () {
+      checkV9Deprecation = createCheckV9Deprecation('crashlytics');
+
+      // @ts-ignore test
+      jest.spyOn(FirebaseModule.prototype, 'native', 'get').mockImplementation(() => {
+        return new Proxy(
+          {},
+          {
+            get: () => jest.fn(),
+          },
+        );
+      });
+    });
+
+    it('checkForUnsentReports', function () {
+      const crashlytics = getCrashlytics();
+      checkV9Deprecation(
+        () => checkForUnsentReports(crashlytics),
+        () => crashlytics.checkForUnsentReports(),
+        'checkForUnsentReports',
+      );
+    });
+
+    it('crash', function () {
+      const crashlytics = getCrashlytics();
+      checkV9Deprecation(
+        () => crash(crashlytics),
+        () => crashlytics.crash(),
+        'crash',
+      );
+    });
+
+    it('deleteUnsentReports', function () {
+      const crashlytics = getCrashlytics();
+      checkV9Deprecation(
+        () => deleteUnsentReports(crashlytics),
+        () => crashlytics.deleteUnsentReports(),
+        'deleteUnsentReports',
+      );
+    });
+
+    it('didCrashOnPreviousExecution', function () {
+      const crashlytics = getCrashlytics();
+      checkV9Deprecation(
+        () => didCrashOnPreviousExecution(crashlytics),
+        () => crashlytics.didCrashOnPreviousExecution(),
+        'didCrashOnPreviousExecution',
+      );
+    });
+
+    it('log', function () {
+      const crashlytics = getCrashlytics();
+      checkV9Deprecation(
+        () => log(crashlytics, 'message'),
+        () => crashlytics.log('message'),
+        'log',
+      );
+    });
+
+    it('setAttribute', function () {
+      const crashlytics = getCrashlytics();
+      checkV9Deprecation(
+        () => setAttribute(crashlytics, 'name', 'value'),
+        () => crashlytics.setAttribute('name', 'value'),
+        'setAttribute',
+      );
+    });
+
+    it('setAttributes', function () {
+      const crashlytics = getCrashlytics();
+      checkV9Deprecation(
+        () => setAttributes(crashlytics, {}),
+        () => crashlytics.setAttributes({}),
+        'setAttributes',
+      );
+    });
+
+    it('setUserId', function () {
+      const crashlytics = getCrashlytics();
+      checkV9Deprecation(
+        () => setUserId(crashlytics, 'id'),
+        () => crashlytics.setUserId('id'),
+        'setUserId',
+      );
+    });
+
+    it('recordError', function () {
+      const crashlytics = getCrashlytics();
+      checkV9Deprecation(
+        () => recordError(crashlytics, new Error(), 'name'),
+        () => crashlytics.recordError(new Error(), 'name'),
+        'recordError',
+      );
+    });
+
+    it('sendUnsentReports', function () {
+      const crashlytics = getCrashlytics();
+      checkV9Deprecation(
+        () => sendUnsentReports(crashlytics),
+        () => crashlytics.sendUnsentReports(),
+        'sendUnsentReports',
+      );
+    });
+
+    it('setCrashlyticsCollectionEnabled', function () {
+      const crashlytics = getCrashlytics();
+      checkV9Deprecation(
+        () => setCrashlyticsCollectionEnabled(crashlytics, true),
+        () => crashlytics.setCrashlyticsCollectionEnabled(true),
+        'setCrashlyticsCollectionEnabled',
+      );
+    });
+
+    it('isCrashlyticsCollectionEnabled', function () {
+      const crashlytics = getCrashlytics();
+      checkV9Deprecation(
+        // swapped order here because we're deprecating the modular method and keeping the property on Crashlytics instance
+        () => crashlytics.isCrashlyticsCollectionEnabled,
+        () => isCrashlyticsCollectionEnabled(crashlytics),
+        '',
+        '`isCrashlyticsCollectionEnabled()` is deprecated, please use `Crashlytics.isCrashlyticsCollectionEnabled` property instead',
+      );
+    });
+  });
 });
diff --git a/packages/crashlytics/lib/index.js b/packages/crashlytics/lib/index.js
@@ -51,6 +51,7 @@ class FirebaseCrashlyticsModule extends FirebaseModule {
   }
 
   get isCrashlyticsCollectionEnabled() {
+    // Purposefully did not deprecate this as I think it should remain a property rather than a method.
     return this._isCrashlyticsCollectionEnabled;
   }
 
diff --git a/packages/crashlytics/lib/modular/index.js b/packages/crashlytics/lib/modular/index.js

Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,7 @@ class FirebaseCrashlyticsModule extends FirebaseModule {`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`get isCrashlyticsCollectionEnabled() {`
	`54`	`+ // Purposefully did not deprecate this as I think it should remain a property rather than a method.`
`54`	`55`	`return this._isCrashlyticsCollectionEnabled;`
`55`	`56`	`}`
`56`	`57`