sentry-react-native icon indicating copy to clipboard operation
sentry-react-native copied to clipboard
verified publisher icon getsentry

Run E2E test using a matrix of RN versions

Open marandaneto opened this issue 4 years ago • 3 comments

Helps to catch issues like https://github.com/getsentry/sentry-react-native/pull/1923

marandaneto avatar Dec 03 '21 08:12 marandaneto

we can consider a sample that compiles with different RN versions, it's cheaper and faster than E2E test, it'd already help us out preventing the mentioned issue above.

marandaneto avatar Dec 06 '21 16:12 marandaneto

@marandaneto Yeah, compiling the sample with different versions and just check if it crashes at runtime or not would catch the above issues.

jennmueng avatar Dec 06 '21 16:12 jennmueng

I don't know id that should be new issue, or if it make sense here, but

as the new 5.0.0 and new RN Architectures are coming closer, this issue will be more and more relevant.

Not even we should at least build with RN version matrix but also with the same RN version and new Arch enabled/disabled. What do you think?

krystofwoldrich avatar Sep 16 '22 14:09 krystofwoldrich

The perf tests relate to this as they are a part of e2e tests.

https://github.com/getsentry/sentry-react-native/issues/2545

~Is this still a p1? As we are planning to cut off a lot of older versions in v5.~

Based on the newest info I've researched. We would be able to support from 0.64.0 to the latest (0.70).

The React Native Test app will enable as to have one app for testing multiple RN versions.

krystofwoldrich avatar Nov 02 '22 16:11 krystofwoldrich