Skip to content

Fix ReactImagePropertyTest SoLoader failures - #19607

Closed
AndrewJack wants to merge 1 commit into
react:masterfrom
AndrewJack:fix/failing_android_tests
Closed

Fix ReactImagePropertyTest SoLoader failures#19607
AndrewJack wants to merge 1 commit into
react:masterfrom
AndrewJack:fix/failing_android_tests

Conversation

@AndrewJack

@AndrewJackAndrewJack commented Jun 7, 2018

Copy link
Copy Markdown
Contributor

Fixes#18637 & #19309

Test Plan

Check Android ReactImagePropertyTest tests pass.

Related PRs

Release Notes

[ANDROID] [BUGFIX] [Unit Tests] - Fix ReactImagePropertyTest SoLoader failure

@facebook-github-botfacebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 7, 2018
@AndrewJackAndrewJack changed the title Fix SoLoader crash in test modeFix ReactImagePropertyTest SoLoader failuresJun 7, 2018
@AndrewJack

Copy link
Copy Markdown
ContributorAuthor

Looks like this fixes the final test_android failure @hramos

@hramos

Copy link
Copy Markdown
Contributor

You're a hero, @AndrewJack

@facebook-github-botfacebook-github-bot added the Import Started This pull request has been imported. This does not imply the PR has been approved. label Jun 7, 2018

@facebook-github-botfacebook-github-bot 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.

@hramos is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

marafat added a commit to plexinc/fb-react-native that referenced this pull request Jun 8, 2018
* master:
Fix NullPointerException when emiting event using UIManagerModule
Fixed concurrency issue in remote debugger
Fix ReactImagePropertyTest SoLoader failures (react#19607)
Remove unused include. (react#19548)
Update Danger token (react#19606)
Modified deepFreezeAndThrowOnMutationInDev to use Object.prototype.ha… (react#19598)
Change error message on interpolation (react#19571)
Bump Prettier to 1.13.4 on xplat
Fix/security issues (react#19373)
Add open source Flow declaration for Metro module
Use correct library reference for libfishhook.a in RCTWebSocket (react#19579)
Enable proguard for Fabric in release builds
Fix events not working after closing and navigating back to Fabric screen in FB4A
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BugCLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.Import StartedThis pull request has been imported. This does not imply the PR has been approved.Platform: AndroidAndroid applications.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] Failing test_android: java.lang.NullPointerException in 4 Android tests

4 participants

@AndrewJack@hramos@facebook-github-bot@react-native-bot