@react-native-firebase/app 21.7.4 → 21.8.0

package/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [21.8.0](https://github.com/invertase/react-native-firebase/compare/v21.7.4...v21.8.0) (2025-02-10)
+
+### Bug Fixes
+
+- do not ship unit tests in released packages ([e71dadf](https://github.com/invertase/react-native-firebase/commit/e71dadfc1c0cad2e89c94100913af31ddf7d9c91))
+
 ## [21.7.4](https://github.com/invertase/react-native-firebase/compare/v21.7.3...v21.7.4) (2025-02-08)
 
 **Note:** Version bump only for package @react-native-firebase/app

package/android/src/reactnative/java/io/invertase/firebase/app/ReactNativeFirebaseVersion.java

@@ -18,5 +18,5 @@ package io.invertase.firebase.app;
  */
 // generated file - do not modify or commit
 public class ReactNativeFirebaseVersion {
-  public static String VERSION = "21.7.4";
+  public static String VERSION = "21.8.0";
 }

package/ios/RNFBApp/RNFBVersion.m

@@ -18,4 +18,4 @@
 #import <React/RCTVersion.h>
 
 // generated file - do not modify or commit
-NSString* const RNFBVersionString = @"21.7.4";
+NSString* const RNFBVersionString = @"21.8.0";

package/lib/internal/RNFBNativeEventEmitter.js

@@ -37,24 +37,24 @@ class RNFBNativeEventEmitter extends NativeEventEmitter {
       this.ready = true;
     }
     RNFBAppModule.eventsAddListener(eventType);
-    if (global.RNFBDebug) {
+    if (globalThis.RNFBDebug) {
       // eslint-disable-next-line no-console
       console.debug(`[RNFB-->Event][👂] ${eventType} -> listening`);
     }
     const listenerDebugger = (...args) => {
-      if (global.RNFBDebug) {
+      if (globalThis.RNFBDebug) {
         // eslint-disable-next-line no-console
         console.debug(`[RNFB<--Event][📣] ${eventType} <-`, JSON.stringify(args[0]));
         // Possible leaking test if events are still being received after the test.
         // This is not super accurate but it's better than nothing, e.g. if doing setup/teardown
         // logic outside of a test this may cause false positives.
-        if (global.RNFBTest && !global.RNFBDebugInTestLeakDetection) {
+        if (globalThis.RNFBTest && !globalThis.RNFBDebugInTestLeakDetection) {
           // eslint-disable-next-line no-console
           console.debug(
             `[TEST--->Leak][💡] Possible leaking test detected! An event (☝️) ` +
               `was received outside of any running tests which may indicates that some ` +
               `listeners/event subscriptions that have not been unsubscribed from in your ` +
-              `test code. The last test that ran was: "${global.RNFBDebugLastTest}".`,
+              `test code. The last test that ran was: "${globalThis.RNFBDebugLastTest}".`,
           );
         }
       }

package/lib/internal/nativeModuleAndroidIos.js

@@ -9,7 +9,7 @@ import { NativeModules } from 'react-native';
  */
 export function getReactNativeModule(moduleName) {
   const nativeModule = NativeModules[moduleName];
-  if (!global.RNFBDebug) {
+  if (!globalThis.RNFBDebug) {
     return nativeModule;
   }
   return new Proxy(nativeModule, {

package/lib/internal/nativeModuleWeb.js

@@ -10,11 +10,12 @@ export function getReactNativeModule(moduleName) {
   if (!nativeModule) {
     throw new Error(`Native module ${moduleName} is not registered.`);
   }
-  if (!global.RNFBDebug) {
+  if (!globalThis.RNFBDebug) {
     return nativeModule;
   }
   return new Proxy(nativeModule, {
     ownKeys(target) {
+      // FIXME - test in new arch context - I don't think Object.keys works
       return Object.keys(target);
     },
     get: (_, name) => {

package/lib/version.js

@@ -1,2 +1,2 @@
 // Generated by genversion.
-module.exports = '21.7.4';
+module.exports = '21.8.0';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-firebase/app",
-  "version": "21.7.4",
+  "version": "21.8.0",
   "author": "Invertase <oss@invertase.io> (http://invertase.io)",
   "description": "A well tested, feature rich Firebase implementation for React Native, supporting iOS & Android. Individual module support for Admob, Analytics, Auth, Crash Reporting, Cloud Firestore, Database, Dynamic Links, Functions, Messaging (FCM), Remote Config, Storage and more.",
   "main": "lib/index.js",
@@ -90,5 +90,5 @@
       "firebaseAppDistributionGradle": "5.1.0"
     }
   },
-  "gitHead": "82c4c7391fee6597726e5363d78400f11a644e47"
+  "gitHead": "14ebd5c2d2e0f284397c331f909e533c47f2ec62"
 }

package/__tests__/app.test.ts

@@ -1,96 +0,0 @@
-import { describe, expect, it, jest } from '@jest/globals';
-import { checkV9Deprecation } from '../lib/common/unitTestUtils';
-import firebase, {
-  deleteApp,
-  registerVersion,
-  onLog,
-  getApps,
-  initializeApp,
-  getApp,
-  setLogLevel,
-} from '../lib';
-
-describe('App', function () {
-  describe('modular', function () {
-    it('`deleteApp` function is properly exposed to end user', function () {
-      expect(deleteApp).toBeDefined();
-    });
-
-    it('`registerVersion` function is properly exposed to end user', function () {
-      expect(registerVersion).toBeDefined();
-    });
-
-    it('`onLog` function is properly exposed to end user', function () {
-      expect(onLog).toBeDefined();
-    });
-
-    it('`getApps` function is properly exposed to end user', function () {
-      expect(getApps).toBeDefined();
-    });
-
-    it('`initializeApp` function is properly exposed to end user', function () {
-      expect(initializeApp).toBeDefined();
-    });
-
-    it('`getApp` function is properly exposed to end user', function () {
-      expect(getApp).toBeDefined();
-    });
-
-    it('`setLogLevel` function is properly exposed to end user', function () {
-      expect(setLogLevel).toBeDefined();
-    });
-  });
-
-  describe('`console.warn` only called for non-modular API', function () {
-    it('deleteApp', function () {
-      // this test has a slightly special setup
-      // @ts-ignore test
-      jest.spyOn(getApp(), '_deleteApp').mockImplementation(() => Promise.resolve(null));
-      checkV9Deprecation(
-        () => {}, // no modular replacement
-        () => getApp().delete(), // modular getApp(), then non-modular to check
-      );
-    });
-
-    it('getApps', function () {
-      checkV9Deprecation(
-        () => getApps(),
-        () => firebase.apps,
-      );
-    });
-
-    it('getApp', function () {
-      checkV9Deprecation(
-        () => getApp(),
-        () => firebase.app(),
-      );
-    });
-
-    it('setLogLevel', function () {
-      checkV9Deprecation(
-        () => setLogLevel('debug'),
-        () => firebase.setLogLevel('debug'),
-      );
-    });
-
-    it('FirebaseApp.toString()', function () {
-      checkV9Deprecation(
-        () => {}, // no modular replacement
-        () => getApp().toString(), // modular getApp(), then non-modular to check
-      );
-    });
-
-    it('FirebaseApp.extendApp()', function () {
-      checkV9Deprecation(
-        // no modular replacement for this one so no modular func to send in
-        () => {},
-        // modular getApp(), then non-modular to check
-        () => {
-          const app = getApp();
-          (app as any).extendApp({ some: 'property' });
-          return;
-        },
-      );
-    });
-  });
-});