Skip to content

More consistent snapshots on windows#28482

Closed
acoates-ms wants to merge 3 commits into
facebook:masterfrom
acoates-ms:fixTestWindows
Closed

More consistent snapshots on windows#28482
acoates-ms wants to merge 3 commits into
facebook:masterfrom
acoates-ms:fixTestWindows

Conversation

@acoates-ms
Copy link
Copy Markdown
Contributor

Summary

Get jest tests to be runnable on windows, and match current snapshots

Changelog

[Internal] [Fixed] - More consistent snapshots on windows

Test Plan

run yarn test on a windows machine, and hit the test_windows circleci tests

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Microsoft Partner: Microsoft Partner labels Apr 1, 2020
@acoates-ms acoates-ms requested a review from cpojer April 1, 2020 18:51
@analysis-bot
Copy link
Copy Markdown

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 6,605,150 -538
android hermes armeabi-v7a 6,281,209 -1,408
android hermes x86 6,945,174 -1,087
android hermes x86_64 6,875,047 -1,502
android jsc arm64-v8a 8,913,487 -541
android jsc armeabi-v7a 8,552,896 -1,397
android jsc x86 8,737,543 -1,090
android jsc x86_64 9,313,639 -1,518

Base commit: 9b94a54

Copy link
Copy Markdown
Contributor

@cpojer cpojer left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@cpojer has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@analysis-bot
Copy link
Copy Markdown

analysis-bot commented Apr 1, 2020

Platform Engine Arch Size (bytes) Diff
ios - universal 10,831,296 9,875,296

Base commit: 9b94a54

@react-native-bot
Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @acoates-ms in 5b26709.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Apr 1, 2020
@acoates-ms acoates-ms deleted the fixTestWindows branch June 4, 2020 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. p: Microsoft Partner: Microsoft Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants