Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reactUtils tests: Start testing useHasStayedTrueForMs. #4997

Merged
merged 1 commit into from
Sep 10, 2021

Commits on Sep 10, 2021

  1. reactUtils tests: Start testing useHasStayedTrueForMs.

    This is our first use of `react-test-renderer`. It piggy-backs on
    our incorporation of Jest's "modern" fake-timer implementation in
    PRs zulip#4754 and zulip#4931. That was handy!
    
    I haven't yet found any test cases that fail with our
    implementation. (And I'd been hoping to, to debug an unexpected
    error!)
    
    But I did try pasting in an earlier iteration of the hook's
    implementation, from zulip#4940, that Greg had found bugs in by reading
    the code. Many of these tests failed on that buggy implementation,
    which is a good sign.
    
    Might as well keep these new tests, then, if they're not an
    unreasonable maintenance burden.
    chrisbobbe committed Sep 10, 2021
    Configuration menu
    Copy the full SHA
    ec99f95 View commit details
    Browse the repository at this point in the history