From bf09089f64cec31680b0bb63b2ebaa81e210c492 Mon Sep 17 00:00:00 2001 From: Ricky Date: Wed, 10 Apr 2024 11:40:22 -0400 Subject: [PATCH] Remove Scheduler.log from ReactSuspenseFuzz-test (#28812) These test don't `assertLog` or `waitFor` so we don't need to `Scheduler.log`. Ideally we would, but since they're fuzzers it's a bit difficult to know what the expected log is from the helper. Since this doesn't regress current test behavior, we can improve them after this to unblock https://github.com/facebook/react/pull/28737 --- .../src/__tests__/ReactSuspenseFuzz-test.internal.js | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/packages/react-reconciler/src/__tests__/ReactSuspenseFuzz-test.internal.js b/packages/react-reconciler/src/__tests__/ReactSuspenseFuzz-test.internal.js index 9e16297c5c..aa206a682d 100644 --- a/packages/react-reconciler/src/__tests__/ReactSuspenseFuzz-test.internal.js +++ b/packages/react-reconciler/src/__tests__/ReactSuspenseFuzz-test.internal.js @@ -115,24 +115,19 @@ describe('ReactSuspenseFuzz', () => { setTimeout(() => { cache.set(fullText, fullText); pendingTasks.delete(task); - Scheduler.log(task.label); resolve(); }, delay); }, }; cache.set(fullText, thenable); - Scheduler.log(`Suspended! [${fullText}]`); throw thenable; } else if (typeof resolvedText.then === 'function') { - const thenable = resolvedText; - Scheduler.log(`Suspended! [${fullText}]`); - throw thenable; + throw resolvedText; } } else { resolvedText = fullText; } - Scheduler.log(resolvedText); return resolvedText; }