Skip to content

Conversation

@jwalk088
Copy link
Contributor

@jwalk088 jwalk088 commented Aug 3, 2021

WHY are these changes introduced?

This updates the Skeleton and Spinner tests to use the new react-testing framework

  • SkeletonBodyTest
  • SkeletonDisplayTest
  • SkeletonPage
  • SkeletonThumbnail
  • Spinner

WHAT is this pull request doing?

Updates tests using {mountWithAppProvider} from test-utilities/legacy to use {mountWithApp} from test-utilities

@jwalk088 jwalk088 requested review from a team and alex-page August 3, 2021 17:47
@ghost
Copy link

ghost commented Aug 3, 2021

👋 Thanks for opening your first pull request. A contributor should give feedback soon. If you haven’t already, please check out the contributing guidelines.

@jwalk088 jwalk088 force-pushed the polaris-skeleton-test-mod branch from b7efe0b to d1abc8c Compare August 3, 2021 17:50
@github-actions
Copy link
Contributor

github-actions bot commented Aug 3, 2021

size-limit report

Path Size
cjs 142.53 KB (0%)
esm 96.3 KB (0%)
esnext 139.5 KB (0%)
css 33.75 KB (0%)

@jwalk088 jwalk088 force-pushed the polaris-skeleton-test-mod branch from d1abc8c to 886284e Compare August 3, 2021 17:54
@jwalk088 jwalk088 force-pushed the polaris-skeleton-test-mod branch from 886284e to 2168d49 Compare August 10, 2021 14:12
@jwalk088 jwalk088 merged commit aba13f7 into main Aug 10, 2021
@jwalk088 jwalk088 deleted the polaris-skeleton-test-mod branch August 10, 2021 14:44
@ghost
Copy link

ghost commented Aug 10, 2021

🎉 Thanks for your contribution to Polaris React!

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.

3 participants