Skip to content

Commit

Permalink
fix: tests not included by default (#6306)
Browse files Browse the repository at this point in the history
<!-- Thanks for submitting a pull request! We appreciate you spending
the time to work on these changes. Please follow the template so that
the reviewers can easily understand what the code changes affect. -->

## Summary

Forgor.

## Test plan

<!-- Provide a minimal but complete code snippet that can be used to
test out this change along with instructions how to run it and a
description of the expected behavior. -->
  • Loading branch information
tjzel authored Jul 22, 2024
1 parent c5a50f5 commit 5cd9cea
Showing 1 changed file with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,13 @@ export default function RuntimeTestsRunner({ tests }: RuntimeTestRunnerProps) {
const [finished, setFinished] = useState<boolean>(false);

const testSelectionCallbacks = useRef<Set<() => void>>(new Set());

useEffect(() => {
tests.forEach(testData => {
!testData.skipByDefault && testSelectionCallbacks.current.add(testData.importTest);
});
}, [tests]);

useEffect(() => {
if (renderLock) {
renderLock.unlock();
Expand Down

0 comments on commit 5cd9cea

Please sign in to comment.