Skip to content

test: add advanced databinding harness tests#137

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

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

Conversation

@mfazekas

@mfazekas mfazekas commented Feb 11, 2026

Copy link
Copy Markdown
Collaborator

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

@HayesGordon HayesGordon left a comment

Copy link
Copy Markdown
Contributor

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