-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Create shared mock for scheduler #4313
Conversation
@captbaritone has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@captbaritone has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Ignore mocks dir when testing dependencies Don't use __mocks__ Revert change to testDependencies tfoo
@captbaritone has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@captbaritone has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Co-authored-by: Eoin Hennessy <eoin.hennessy@gmail.com>
@captbaritone has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@captbaritone merged this pull request in cc1daa1. |
The version of scheduler we get internally is always the latest. However, on GitHub we get the version is a transitive dependency from
react-test-renderer
.Some methods in the unstable_mock have been renamed between these two versions. This mock file provides a centralized place to reconcile those differences so that the same tests can work both internally and on GitHub.
I also tried manually upgrading
react-test-renderer
to the@experimental
tag in the hopes of getting GitHub to use the same version we have internally, and then transitively inherit the more recent version ofscheduler
. Unfortunately that created many test failures for reasons I don't currently fully understand.