Merge d6dae69 into 20f5746

packages/compare/src/compare.ts

@@ -137,6 +137,7 @@ function compareResults(current: MeasureResults, baseline: MeasureResults | null
   const compared: CompareEntry[] = [];
   const added: AddedEntry[] = [];
   const removed: RemovedEntry[] = [];
+  const currentWarnings: string[] = [];
 
   names.forEach((name) => {
     const currentEntry = current.entries[name];
@@ -149,6 +150,10 @@ function compareResults(current: MeasureResults, baseline: MeasureResults | null
     } else if (baselineEntry) {
       removed.push({ name, type: baselineEntry.type, baseline: baselineEntry });
     }
+
+    if (currentEntry?.warnings) {
+      currentWarnings.push(...currentEntry.warnings.map((warning) => `${currentEntry.name}: ${warning}`));
+    }
   });
 
   const withCurrent = [...compared, ...added];
@@ -171,7 +176,7 @@ function compareResults(current: MeasureResults, baseline: MeasureResults | null
   return {
     metadata: { current: current.metadata, baseline: baseline?.metadata },
     errors,
-    warnings,
+    warnings: [...warnings, ...currentWarnings],
     significant,
     meaningless,
     countChanged,

packages/compare/src/type-schemas.ts

@@ -50,4 +50,6 @@ export const MeasureEntryScheme = z.object({
   counts: z.array(z.number()),
 
   issues: z.optional(RenderIssuesScheme),
+
+  warnings: z.optional(z.array(z.string())),
 });

packages/measure/src/measure-renders.tsx

@@ -63,6 +63,7 @@ async function measureRendersInternal(
   applyRenderPolyfills();
 
   const runResults: RunResult[] = [];
+  const warnings: string[] = [];
   let hasTooLateRender = false;
 
   const renderJsonTrees: ElementJsonTree[] = [];
@@ -121,8 +122,11 @@ async function measureRendersInternal(
 
   if (hasTooLateRender) {
     const testName = expect.getState().currentTestName;
+    warnings.push(
+      'Test still re-renders after test scenario finished. Update your code to wait for all renders to finish.'
+    );
     logger.warn(
-      `test "${testName}" still re-renders after test scenario finished.\n\nPlease update your code to wait for all renders to finish.`
+      `test "${testName}" still re-renders after test scenario finished.\n\nUpdate your code to wait for all renders to finish.`
     );
   }
 
@@ -134,6 +138,7 @@ async function measureRendersInternal(
       initialUpdateCount: initialRenderCount - 1,
       redundantUpdates: detectRedundantUpdates(renderJsonTrees, initialRenderCount),
     },
+    warnings: warnings.length > 0 ? warnings : undefined,
   };
 }
 

packages/measure/src/types.ts

@@ -28,6 +28,12 @@ export interface MeasureResults {
 
   /** Array of measured render/execution count for each run */
   counts: number[];
+
+  /** Errors that occurred during the test */
+  errors?: string[];
+
+  /** Warnings that occurred during the test */
+  warnings?: string[];
 }
 
 export interface MeasureRendersResults extends MeasureResults {

test-apps/native/src/OtherTest.perf.tsx

@@ -26,7 +26,7 @@ const AsyncComponent = () => {
 };
 
 jest.setTimeout(600_000);
-test('Other Component', async () => {
+test('AsyncComponent', async () => {
   const scenario = async () => {
     const button = screen.getByText('Action');
 
@@ -38,7 +38,7 @@ test('Other Component', async () => {
   await measureRenders(<AsyncComponent />, { scenario, runs: 10 });
 });
 
-test('Other Component 20 run', async () => {
+test('AsyncComponent 20 runs', async () => {
   const scenario = async () => {
     const button = screen.getByText('Action');
 
@@ -49,3 +49,14 @@ test('Other Component 20 run', async () => {
 
   await measureRenders(<AsyncComponent />, { scenario, runs: 20 });
 });
+
+test('AsyncComponent - no wait', async () => {
+  const scenario = async () => {
+    const button = screen.getByText('Action');
+
+    fireEvent.press(button);
+    fireEvent.press(button);
+  };
+
+  await measureRenders(<AsyncComponent />, { scenario });
+});