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

fix(jest): fix mock systemtime setup #531

Merged
merged 1 commit into from
Oct 3, 2022

Conversation

tthvo
Copy link
Member

@tthvo tthvo commented Oct 3, 2022

Fixes #530

Properly set up Jest mock system timer and added callback to reset to mock date to real after DatetimePicker tests.

The issue is that DateTimePicker should be imported after systemtime mock to receive the mocks.

@tthvo tthvo added the fix label Oct 3, 2022
@tthvo tthvo requested review from andrewazores and maxcao13 October 3, 2022 05:52
@tthvo tthvo added the test label Oct 3, 2022
@tthvo
Copy link
Member Author

tthvo commented Oct 3, 2022

Sorry I didn't notice this requirements while testing in September :((

@tthvo tthvo changed the title fix(tests): fix mock systemtime setup fix(jest): fix mock systemtime setup Oct 3, 2022
@andrewazores andrewazores merged commit 62c3fe4 into cryostatio:main Oct 3, 2022
@tthvo tthvo deleted the fix-mock-systemtime branch October 6, 2022 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Task] Jest mock system timer is not properly set up
2 participants