feat: add in-app test runner#106
Merged
Merged
Conversation
Add testing infrastructure following Nitro's pattern with chainable assertions. Initial tests cover viewModel() and replaceViewModel() functionality.
Use iOS 18.6 instead of 18.5, load assets via metro URL instead of require() since Image.resolveAssetSource doesn't work in harness context.
04466e6 to
bf0e3d6
Compare
Remove custom assertion backend and test runner infrastructure in favor of using expect from react-native-harness directly.
Keep only the harness tests in __tests__/rive.harness.ts
HayesGordon
reviewed
Jan 12, 2026
|
|
||
| test-harness-ios: | ||
| if: false # TODO: re-enable once harness timeout issues are resolved | ||
| runs-on: macos-latest |
Contributor
There was a problem hiding this comment.
We should update our runners to use Cirrus:
Suggested change
| runs-on: macos-latest | |
| runs-on: ghcr.io/cirruslabs/macos-runner:sonoma |
HayesGordon
pushed a commit
that referenced
this pull request
Jan 14, 2026
🤖 I have created a release *beep* *boop* --- ## [0.1.5](v0.1.4...v0.1.5) (2026-01-14) ### Features * add in-app test runner ([#106](#106)) ([d7ace51](d7ace51)) * bump native versions for scripting support ([#121](#121)) ([d7119b3](d7119b3)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
mfazekas
added a commit
that referenced
this pull request
Jan 16, 2026
Restructures example apps into organized sections: - **demos/** - User-facing tutorials with marketplace links - **exercisers/** - Developer testing, edge cases, stress tests - **tests/** - Test runner UI (integrated from #106) - **reproducers/** - Bug reproduction (`local/` gitignored for throwaway files) Also wires up the 🔧 wrench menu "Run Tests" to navigate to TestsPage in both example and expo-example. <img width="300" alt="image" src="https://github.com/user-attachments/assets/c9afbfe3-cb68-4957-9256-2bdfb47d22d6" /> <img width="300" alt="image" src="https://github.com/user-attachments/assets/5e33696a-14f1-4d31-8f7e-681ba4a4ec63" />
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
viewModel()andreplaceViewModel()from feat: add viewModel and replaceViewModel for nested ViewModel access #96Test plan