tweaks.

Author: mdjastrzebski

README.md

@@ -152,7 +152,7 @@ For most use cases, the migration is straightforward:
 import TestRenderer from "react-test-renderer";
 const tree = TestRenderer.create(<MyComponent />);
 
-// After (test-renderer)
+// After (Test Renderer)
 import { createRoot } from "test-renderer";
 const root = createRoot();
 await act(async () => {
@@ -163,12 +163,10 @@ const tree = root.container;
 
 ## Performance Metrics
 
-The library includes optional performance instrumentation using the standard Performance API. All marks and measures are prefixed with `test-renderer/` for easy filtering.
+The library includes optional performance instrumentation using the [Performance API](https://developer.mozilla.org/en-US/docs/Web/API/Performance). All marks and measures are prefixed with `test-renderer/` for easy filtering.
 
 ```tsx
-import { setPerformanceMetricsEnabled } from "test-renderer";
-
-setPerformanceMetricsEnabled(true);
+globalThis.TEST_RENDERER_ENABLE_PROFILING = true;
 
 // Run your tests, then query metrics:
 const marks = performance

src/__tests__/performance.test.tsx

@@ -1,18 +1,17 @@
 import { afterEach, beforeEach, describe, expect, test } from "@jest/globals";
 import * as React from "react";
 
-import { createRoot, setPerformanceMetricsEnabled } from "..";
+import { createRoot } from "..";
 import { act, formatPerfEntries, renderWithAct, unmountWithAct } from "../test-utils/render";
 
 beforeEach(() => {
-  global.IS_REACT_ACT_ENVIRONMENT = true;
-  performance.clearMarks();
-  performance.clearMeasures();
-  setPerformanceMetricsEnabled(false);
+  globalThis.IS_REACT_ACT_ENVIRONMENT = true;
 });
 
 afterEach(() => {
-  setPerformanceMetricsEnabled(false);
+  globalThis.TEST_RENDERER_ENABLE_PROFILING = false;
+  performance.clearMarks();
+  performance.clearMeasures();
 });
 
 function getPerfMarks(): PerformanceMark[] {
@@ -31,7 +30,8 @@ describe("performance metrics", () => {
   });
 
   test("logs marks and measures for render", async () => {
-    setPerformanceMetricsEnabled(true);
+    globalThis.TEST_RENDERER_ENABLE_PROFILING = true;
+
     const root = createRoot();
     await renderWithAct(root, <div>Hello!</div>);
 
@@ -80,7 +80,7 @@ describe("performance metrics", () => {
     const root = createRoot();
     await renderWithAct(root, <TestComponent />);
 
-    setPerformanceMetricsEnabled(true);
+    globalThis.TEST_RENDERER_ENABLE_PROFILING = true;
 
     await act(() => {
       const child = root.container.children[0];
@@ -103,45 +103,15 @@ describe("performance metrics", () => {
     const root = createRoot();
     await renderWithAct(root, <div />);
 
-    setPerformanceMetricsEnabled(true);
-    performance.clearMarks();
-    performance.clearMeasures();
+    globalThis.TEST_RENDERER_ENABLE_PROFILING = true;
+
     await unmountWithAct(root);
 
     const marks = getPerfMarks();
     expect(marks.some((m) => m.name === "test-renderer/unmount:start")).toBe(true);
     expect(marks.some((m) => m.name === "test-renderer/unmount:end")).toBe(true);
     expect(marks.some((m) => m.name === "test-renderer/react/commit:start")).toBe(true);
     expect(marks.some((m) => m.name === "test-renderer/react/commit:end")).toBe(true);
-    expect(formatPerfEntries(marks)).toMatchInlineSnapshot(`
-      "0.00ms: ACT:start
-      0.01ms: ACT:sync start
-      0.06ms: test-renderer/unmount:start
-      0.08ms: test-renderer/reconciler/resolveUpdatePriority (priority=32)
-      0.11ms: test-renderer/reconciler/scheduleMicrotask (id=12)
-      0.12ms: test-renderer/unmount:end
-      0.15ms: test-renderer/reconciler/scheduled microtask:start (id=12)
-      0.16ms: test-renderer/reconciler/scheduled microtask:end (id=12)
-      0.17ms: ACT:sync end
-      0.19ms: test-renderer/reconciler/getRootHostContext
-      0.24ms: test-renderer/reconciler/setCurrentUpdatePriority (priority=2)
-      0.29ms: test-renderer/reconciler/setCurrentUpdatePriority (priority=2)
-      0.30ms: test-renderer/reconciler/prepareForCommit
-      0.30ms: test-renderer/react/commit:start
-      0.51ms: test-renderer/reconciler/removeChildFromContainer (childType="div")
-      0.54ms: test-renderer/react/commit:end
-      0.56ms: test-renderer/reconciler/resetAfterCommit
-      0.56ms: test-renderer/reconciler/setCurrentUpdatePriority (priority=2)
-      0.57ms: test-renderer/reconciler/scheduleMicrotask (id=13)
-      0.58ms: test-renderer/reconciler/setCurrentUpdatePriority (priority=0)
-      0.60ms: test-renderer/reconciler/setCurrentUpdatePriority (priority=32)
-      0.76ms: test-renderer/reconciler/detachDeletedInstance
-      0.88ms: test-renderer/reconciler/setCurrentUpdatePriority (priority=0)
-      0.91ms: test-renderer/reconciler/scheduled microtask:start (id=13)
-      0.91ms: test-renderer/reconciler/scheduled microtask:end (id=13)
-      0.95ms: ACT:async end
-      "
-    `);
 
     const measures = getPerfMeasures();
     expect(measures.some((m) => m.name === "test-renderer/unmount")).toBe(true);

src/index.ts

@@ -1,5 +1,4 @@
 export { createRoot } from "./renderer";
-export { setPerformanceMetricsEnabled } from "./performance";
 
 export type { Root, RootOptions } from "./renderer";
 export type { HostElement, HostElementProps, HostNode } from "./host-element";

src/performance.ts

@@ -2,12 +2,6 @@ declare global {
   var TEST_RENDERER_ENABLE_PROFILING: boolean | undefined;
 }
 
-globalThis.TEST_RENDERER_ENABLE_PROFILING ??= false;
-
-export function setPerformanceMetricsEnabled(enabled: boolean): void {
-  globalThis.TEST_RENDERER_ENABLE_PROFILING = enabled;
-}
-
 export function mark(name: string, details?: Record<string, unknown>): void {
   if (!globalThis.TEST_RENDERER_ENABLE_PROFILING) {
     return;

src/test-utils/render.ts

@@ -2,18 +2,19 @@ import type { ReactElement } from "react";
 import { act as reactAct } from "react";
 
 import type { HostElement } from "../host-element";
+import { mark } from "../performance";
 import type { Root } from "../renderer";
 
 /** @internal */
 export async function act<T>(callback: () => T | Promise<T>): Promise<T> {
-  performance.mark("ACT:start");
+  mark("act:start");
   const result = await reactAct(async () => {
-    performance.mark("ACT:sync start");
+    mark("act callback:start");
     const result = await callback();
-    performance.mark("ACT:sync end");
+    mark("act callback:end");
     return result;
   });
-  performance.mark("ACT:async end");
+  mark("act:end");
   return result;
 }