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

Use web implementation for gesture handlers in Jest #2747

Closed

Conversation

piaskowyk
Copy link
Member

@piaskowyk piaskowyk commented Dec 15, 2021

Description

I want to add support for gesture-handler events in Jest. I want to make it user-friendly, so I am adding changes to react-native-gesture-handler. During this, I noticed that reanimated sometimes tried to use native implementation instead of web implementation. So I changed it to prevent mocking Platform.OS.

Related PR: software-mansion/react-native-gesture-handler#1762

@piaskowyk piaskowyk marked this pull request as ready for review December 16, 2021 19:50
@piaskowyk piaskowyk closed this Jan 27, 2022
piaskowyk pushed a commit that referenced this pull request Jan 27, 2022
## Description
Extracted from #2747.
RNGH needs to use useAnimatedGestureHandler in Jest environment so we want to use web implementation there.

## Checklist

- [x] Ensured that CI passes
aeddi pushed a commit to aeddi/react-native-reanimated that referenced this pull request Mar 22, 2022
## Description
Extracted from software-mansion#2747.
RNGH needs to use useAnimatedGestureHandler in Jest environment so we want to use web implementation there.

## Checklist

- [x] Ensured that CI passes
@piaskowyk piaskowyk deleted the @piaskowyk/jest-improvements-for-gesture-handler branch May 17, 2022 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants