-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Bring back test util docs #5891
Conversation
Build successful! 🎉 |
Please see the following section in the testing docs for more information on how to simulate this action in your | ||
test suite. | ||
|
||
// TODO need to also mock the client rect? |
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.
Does this need to be revisited before merging?
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.
Ah, thought I got rid of all of those, lemme check
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.
Double checked, the section in the main testing docs that is linked should be sufficient along with the link to our tests IMO. Can always add more if need be if people still have trouble. Thanks for catching this one!
Build successful! 🎉 |
Build successful! 🎉 |
Build successful! 🎉 |
Build successful! 🎉 |
Build successful! 🎉 |
## API Changes
unknown top level export { type: 'any' } @react-aria/test-utilsDEFAULT_LONG_PRESS_TIME-
+triggerLongPress {
+ element: HTMLElement
+ opts: any
+ returnVal: undefined
+} triggerLongPress-
+installMouseEvent {
+ returnVal: undefined
+} installMouseEvent-
+installPointerEvent {
+ returnVal: undefined
+} @react-spectrum/test-utilsDEFAULT_LONG_PRESS_TIME-
+triggerLongPress {
+ element: HTMLElement
+ opts: any
+ returnVal: undefined
+} triggerLongPress-
+installMouseEvent {
+ returnVal: undefined
+} installMouseEvent-
+installPointerEvent {
+ returnVal: undefined
+} installPointerEvent-
+simulateMobile {
+ width: number
+ returnVal: undefined
+} pointerMap-
+simulateDesktop {
+ width: number
+ returnVal: undefined
+} |
Turns out I just needed to not delete the dev/util file
✅ Pull Request Checklist:
📝 Test Instructions:
🧢 Your Project: