Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup the snapshots in E2E test app #13229

Merged
merged 6 commits into from
May 14, 2024

Conversation

acoates-ms
Copy link
Contributor

@acoates-ms acoates-ms commented May 13, 2024

  • Changed the format of the snapshots for the E2E fabric app to make them more readable.
  • Changed the field name of the children property so that it is always at the bottom of the object. This means that the properties of the parent visual are no longer split between above the children and below them.
  • Use a formatted string for Offset/Size fields so that they do not take up 5 lines in the snaphots.
  • Added a snapshot of the native ComponentView's as an additional set of testing.
Microsoft Reviewers: Open in CodeFlow

@acoates-ms acoates-ms requested a review from a team as a code owner May 13, 2024 19:05
Copy link
Contributor

@chiaramooney chiaramooney left a comment

Choose a reason for hiding this comment

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

Great changes thanks!

@acoates-ms acoates-ms merged commit 00ad953 into microsoft:main May 14, 2024
53 checks passed
@acoates-ms acoates-ms deleted the snapshotcleanup branch May 14, 2024 04:37
acoates-ms added a commit to acoates-ms/react-native-windows that referenced this pull request May 14, 2024
* Cleanup the snapshots

* format

* Process snapshots so that ImageSource Uri's are consistent

* Change files

* fix

* fix
acoates-ms added a commit to acoates-ms/react-native-windows that referenced this pull request May 14, 2024
* Cleanup the snapshots

* format

* Process snapshots so that ImageSource Uri's are consistent

* Change files

* fix

* fix
acoates-ms added a commit that referenced this pull request May 14, 2024
* Cleanup the snapshots in E2E test app (#13229)

* Cleanup the snapshots

* format

* Process snapshots so that ImageSource Uri's are consistent

* Change files

* fix

* fix

* fix(TextInput/isFocused): correctly handle null focused input (#13219)

* Change files

* fix(TextInput/isFocused): correctly handle null focused input

Signed-off-by: Nathanael Bracy <[email protected]>

* fix

---------

Signed-off-by: Nathanael Bracy <[email protected]>
Co-authored-by: Andrew Coates <[email protected]>

* Fixing crash when passing null children to view with no-hide-descendents (#13224)

* Preventing crash when passing null children to view with no-hide-descendents

* Change files

---------

Co-authored-by: Sam Walker <[email protected]>
Co-authored-by: Andrew <[email protected]>

* C++20: use view() instead of str() to avoid a copy, use starts_with (#13218)

* Change files

* C++20: use view() instead of str() to avoid a copy, use starts_with

Signed-off-by: Nathanael Bracy <[email protected]>

* fix

* Apply changefile modifications from code review

Co-authored-by: Jon Thysell <[email protected]>

* Revert package.json

Signed-off-by: Nathanael Bracy <[email protected]>

---------

Signed-off-by: Nathanael Bracy <[email protected]>
Co-authored-by: Andrew Coates <[email protected]>
Co-authored-by: Jon Thysell <[email protected]>

* ViewComponentView should be activatable from rn-win32.dll (#13225)

* ViewComponentView should be activatable from rn-win32.dll

* Change files

* Cleanup the snapshots in E2E test app (#13229)

* Cleanup the snapshots

* format

* Process snapshots so that ImageSource Uri's are consistent

* Change files

* fix

* fix

* lint

* format

* Revert "C++20: use view() instead of str() to avoid a copy, use starts_with (#13218)"

This reverts commit 71335b5.

---------

Signed-off-by: Nathanael Bracy <[email protected]>
Co-authored-by: Nate <[email protected]>
Co-authored-by: Sam Walker <[email protected]>
Co-authored-by: Sam Walker <[email protected]>
Co-authored-by: Jon Thysell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

3 participants