Skip to content

Conversation

@rdott
Copy link
Contributor

@rdott rdott commented Jul 22, 2021

WHY are these changes introduced?

Updating existing tests to use the new modern framework and eliminate stale dependencies.

WHAT is this pull request doing?

Part of test modernization, updating tests using {mountWithAppProvider} from 'test-utilities/legacy' to {mountWithApp} from 'test-utilities'

@rdott rdott requested review from a user July 22, 2021 18:10
@github-actions
Copy link
Contributor

github-actions bot commented Jul 22, 2021

size-limit report

Path Size
cjs 142.4 KB (0%)
esm 96.16 KB (0%)
esnext 139.37 KB (0%)
css 33.75 KB (0%)

expect(wrapper).not.toContainReactComponent('div', {id: 'child'});
wrapper.setProps({linkComponent: LinkComponent});
wrapper.update();
expect(wrapper.find('#child')).toHaveLength(1);
Copy link

Choose a reason for hiding this comment

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

TIL

@rdott rdott force-pushed the test-modern-app-provider branch from 5e1d9ba to f289742 Compare August 9, 2021 13:34
}
}

// stylelint-disable-next-line selector-max-class
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixes breaking CI for max-class

@rdott rdott merged commit b4db6ec into main Aug 9, 2021
@rdott rdott deleted the test-modern-app-provider branch August 9, 2021 14:24
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.

2 participants