get tests running

Author: jhuleatt

package-lock.json

@@ -1,5 +1,5 @@
 {
-  "version": "4.2.3",
+  "version": "4.3.3",
   "license": "MIT",
   "type": "module",
   "main": "dist/index.umd.cjs",
@@ -15,7 +15,7 @@
     "src"
   ],
   "engines": {
-    "node": ">=14"
+    "node": ">=18"
   },
   "scripts": {
     "start": "firebase emulators:exec --project=rxfire-525a3 \"vitest --ui\"",
@@ -36,8 +36,8 @@
     "docs:fork": "typedoc --options typedoc.json --gitRemote upstream && markdown-toc -i docs/use.md"
   },
   "peerDependencies": {
-    "firebase": "^9.0.0 || next",
-    "react": ">=16 || experimental"
+    "firebase": "^11.0.0",
+    "react": "^18 || ^19"
   },
   "husky": {
     "hooks": {
@@ -71,44 +71,44 @@
     }
   ],
   "devDependencies": {
-    "@rollup/plugin-typescript": "^11.1.1",
-    "@size-limit/preset-small-lib": "^8.2.6",
-    "@testing-library/jest-dom": "^5.16.5",
-    "@testing-library/react": "^14.0.0",
-    "@types/react": "^18.2.0",
-    "@types/react-dom": "^18.2.0",
-    "@types/use-sync-external-store": "^0.0.3",
-    "@typescript-eslint/eslint-plugin": "^5.60.1",
-    "@typescript-eslint/parser": "^5.60.1",
-    "@vitejs/plugin-react": "^4.0.1",
+    "@rollup/plugin-typescript": "^12.1.2",
+    "@size-limit/preset-small-lib": "^11.2.0",
+    "@testing-library/jest-dom": "^6.6.3",
+    "@testing-library/react": "^16.3.0",
+    "@types/react": "^19.1.3",
+    "@types/react-dom": "^19.1.3",
+    "@types/use-sync-external-store": "^1.5.0",
+    "@typescript-eslint/eslint-plugin": "^8.32.0",
+    "@typescript-eslint/parser": "^8.32.0",
+    "@vitejs/plugin-react": "^4.4.1",
     "@vitest/ui": "^3.1.3",
-    "cross-fetch": "^3.1.6",
-    "eslint": "^8.43.0",
-    "eslint-plugin-no-only-tests": "^3.1.0",
-    "eslint-plugin-react": "^7.32.2",
-    "eslint-plugin-react-hooks": "^4.6.0",
-    "firebase": "^9.23.0",
-    "firebase-tools": "^12.4.0",
-    "globalthis": "^1.0.3",
-    "husky": "^8.0.3",
-    "jest-environment-jsdom": "^29.5.0",
+    "cross-fetch": "^4.1.0",
+    "eslint": "^9.26.0",
+    "eslint-plugin-no-only-tests": "^3.3.0",
+    "eslint-plugin-react": "^7.37.5",
+    "eslint-plugin-react-hooks": "^5.2.0",
+    "firebase": "^11.7.1",
+    "firebase-tools": "^14.3.0",
+    "globalthis": "^1.0.4",
+    "husky": "^9.1.7",
+    "jest-environment-jsdom": "^29.7.0",
     "jsdom": "^26.1.0",
     "markdown-toc": "^1.2.0",
-    "prettier": "^2.8.8",
-    "react": "^18.2.0",
-    "react-dom": "^18.2.0",
-    "react-test-renderer": "^18.2.0",
-    "rollup-plugin-visualizer": "^5.9.2",
-    "size-limit": "^8.2.6",
-    "typedoc": "^0.24.8",
-    "typedoc-plugin-markdown": "^3.15.3",
-    "typescript": "^5.1.3",
-    "vite": "^4.3.9",
+    "prettier": "^3.5.3",
+    "react": "^19.1.0",
+    "react-dom": "^19.1.0",
+    "react-test-renderer": "^19.1.0",
+    "rollup-plugin-visualizer": "^5.14.0",
+    "size-limit": "^11.2.0",
+    "typedoc": "^0.28.4",
+    "typedoc-plugin-markdown": "^4.6.3",
+    "typescript": "^5.8.3",
+    "vite": "^6.3.5",
     "vitest": "^3.1.3"
   },
   "dependencies": {
-    "rxfire": "^6.0.3",
-    "rxjs": "^6.6.3 || ^7.0.1",
-    "use-sync-external-store": "^1.2.0"
+    "rxfire": "^6.1.0",
+    "rxjs": "^7.8.2",
+    "use-sync-external-store": "^1.5.0"
   }
 }

package.json

@@ -244,7 +244,7 @@ export function ClaimsCheck({ user, fallback, children, requiredClaims }: Claims
  *
  * Meant for Concurrent mode only (`<FirebaseAppProvider suspense=true />`). [More detail](https://github.com/FirebaseExtended/reactfire/issues/325#issuecomment-827654376).
  */
-export function AuthCheck({ fallback, children, requiredClaims }: AuthCheckProps): JSX.Element {
+export function AuthCheck({ fallback, children, requiredClaims }: AuthCheckProps): React.JSX.Element {
   const { data: user } = useUser<User>();
 
   const suspenseMode = useSuspenseEnabledFromConfigAndContext();

src/auth.tsx

@@ -3,6 +3,8 @@ import { getApps, initializeApp, registerVersion } from 'firebase/app';
 
 import type { FirebaseApp, FirebaseOptions } from 'firebase/app';
 
+import {version as pkgVersion} from '../package.json';
+
 // INVESTIGATE I don't like magic strings, can we have export this in js-sdk?
 const DEFAULT_APP_NAME = '[DEFAULT]';
 
@@ -16,8 +18,7 @@ export interface FirebaseAppProviderProps {
   suspense?: boolean;
 }
 
-// REACTFIRE_VERSION is automatically pulled in from `package.json` by Vite
-export const version = process.env.REACTFIRE_VERSION as string;
+export const version = pkgVersion;
 
 const shallowEq = (a: { [key: string]: any }, b: { [key: string]: any }) => a === b || [...Object.keys(a), ...Object.keys(b)].every((key) => a[key] === b[key]);
 

src/firebaseApp.tsx

@@ -6,7 +6,7 @@ export interface SuspensePerfProps {
   fallback: React.ReactNode;
 }
 
-export function SuspenseWithPerf({ children, traceId, fallback }: SuspensePerfProps): JSX.Element {
+export function SuspenseWithPerf({ children, traceId, fallback }: SuspensePerfProps): React.JSX.Element {
   // TODO: Should this import firebase/performance?
 
   const entries = performance?.getEntriesByName?.(traceId, 'measure') || [];

src/performance.tsx

@@ -37,7 +37,7 @@ export type StorageImageProps = {
   storagePath: string;
   storage?: FirebaseStorage;
   suspense?: boolean;
-  placeHolder?: JSX.Element;
+  placeHolder?: React.JSX.Element;
 };
 
 function StorageFromContext(props: StorageImageProps & React.DetailedHTMLProps<React.ImgHTMLAttributes<HTMLImageElement>, HTMLImageElement>) {
@@ -48,7 +48,7 @@ function StorageFromContext(props: StorageImageProps & React.DetailedHTMLProps<R
   return <INTERNALStorageImage {...props} />;
 }
 
-function INTERNALStorageImage(props: StorageImageProps & React.DetailedHTMLProps<React.ImgHTMLAttributes<HTMLImageElement>, HTMLImageElement>): JSX.Element {
+function INTERNALStorageImage(props: StorageImageProps & React.DetailedHTMLProps<React.ImgHTMLAttributes<HTMLImageElement>, HTMLImageElement>): React.JSX.Element {
   const { storage, storagePath, suspense, placeHolder, ...imgProps } = props;
 
   const reactfireOptions: ReactFireOptions<string> = {

src/storage.tsx

@@ -1,5 +1,5 @@
 import { cleanup, render, waitFor, renderHook, act } from '@testing-library/react';
-import '@testing-library/jest-dom/extend-expect';
+import '@testing-library/jest-dom';
 import * as React from 'react';
 import {
   FirebaseAppProvider,

test/auth.test.tsx

@@ -1,6 +1,6 @@
 import { cleanup, render, waitFor, renderHook } from '@testing-library/react';
 import { initializeApp, deleteApp, getApps } from 'firebase/app';
-import '@testing-library/jest-dom/extend-expect';
+import '@testing-library/jest-dom';
 import * as React from 'react';
 import { useFirebaseApp, FirebaseAppProvider, version } from '../src/index';
 import pkg from '../package.json';

test/firebaseApp.test.tsx

@@ -1,7 +1,7 @@
 import { renderHook, act, cleanup, waitFor } from '@testing-library/react';
 
 import * as React from 'react';
-import '@testing-library/jest-dom/extend-expect';
+import '@testing-library/jest-dom';
 import {
   useFirestoreDoc,
   useFirestoreCollection,

test/firestore.test.tsx

@@ -1,4 +1,4 @@
-import '@testing-library/jest-dom/extend-expect';
+import '@testing-library/jest-dom';
 import { act, cleanup, render, renderHook, waitFor } from '@testing-library/react';
 import * as React from 'react';
 import { of, Subject, BehaviorSubject, throwError } from 'rxjs';
@@ -317,7 +317,7 @@ describe('useObservable', () => {
       expect(refreshedComp).toBeInTheDocument();
 
       // if useObservable doesn't re-emit, the value here will still be "Jeff"
-      expect(refreshedComp).toHaveTextContent('James');
+      await waitFor(() => expect(refreshedComp).toHaveTextContent('James'));
     });
   });
 });