Skip to content

test: add advanced databinding harness tests#137

Merged
mfazekas merged 2 commits intomainfrom
test/add-advanced-databinding-harness
Feb 17, 2026
Merged

test: add advanced databinding harness tests#137
mfazekas merged 2 commits intomainfrom
test/add-advanced-databinding-harness

Conversation

@mfazekas
Copy link
Collaborator

@mfazekas mfazekas commented Feb 11, 2026

Add advanced databinding harness tests to main and bump react-native-harness to alpha.25. Guard negative indices in viewModelByIndex and createInstanceByIndex on both iOS and Android. Wrap Android viewModelByIndex/viewModelByName in try-catch since the Rive SDK throws JNI exceptions for invalid lookups. Add ANDROID_AVD env var support to harness config.

mfazekas and others added 2 commits February 11, 2026 12:31
Guard negative indices in viewModelByIndex and createInstanceByIndex to prevent UInt crash. Bump react-native-harness from alpha.20 to alpha.25.

Generated with [Claude Code](https://claude.ai/code)
via [Happy](https://happy.engineering)

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Happy <yesreply@happy.engineering>
Wrap viewModelByIndex/viewModelByName in try-catch on Android since the
Rive SDK throws JNI exceptions for invalid lookups. Add ANDROID_AVD env
var support to harness config.

Generated with [Claude Code](https://claude.ai/code)
via [Happy](https://happy.engineering)

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Happy <yesreply@happy.engineering>
@mfazekas mfazekas marked this pull request as ready for review February 11, 2026 20:31
@mfazekas mfazekas requested a review from HayesGordon February 16, 2026 09:35
Copy link
Contributor

@HayesGordon HayesGordon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you!

@mfazekas mfazekas merged commit 36bdf8d into main Feb 17, 2026
8 checks passed
@mfazekas mfazekas deleted the test/add-advanced-databinding-harness branch February 17, 2026 11:48
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