Skip to content

Commit

Permalink
Remove Scheduler.log from ReactSuspenseFuzz-test (facebook#28812)
Browse files Browse the repository at this point in the history
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 facebook#28737
  • Loading branch information
rickhanlonii authored and AndyPengc12 committed Apr 15, 2024
1 parent b9bf12b commit ba19ae6
Showing 1 changed file with 1 addition and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}

Expand Down

0 comments on commit ba19ae6

Please sign in to comment.