-
Notifications
You must be signed in to change notification settings - Fork 0
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
test admin dashboard #188
test admin dashboard #188
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes - they look good. I just noticed that there are no tests for the checkbox functionality, though. Could you please add some tests to confirm that the api client update function is called (with the expected date) when checking an unchecked box and that unchecking a box displays the confirmation and makes the appropriate update if the confirmation is OK'd?
also, avoid doing the zero-delay setTimeout thing it's not worth the complexity just to wait for the event to cancel
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than the one little issue with the comment that says the opposite of what it should, I think this is good to go.
mriT1Cell.querySelector("input").dispatchEvent(new MouseEvent("click", {bubbles: true})); | ||
// wait for the async click event handler to resolve | ||
await new Promise(process.nextTick); | ||
// Twilight Sparkle should now have a timestamp for MRI T1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think "now" is probably supposed to be "not" here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch! Just fixed it.
after the MRI T1 checkbox gets unchecked
No description provided.