Steve-Dee-Designs
/
woocommerce
mirror of https://github.com/woocommerce/woocommerce.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

QuickLinks: Prevent jsdom "Error: Not implemented: navigation" test output (https://github.com/woocommerce/woocommerce-admin/pull/4547)

This commit is contained in:
Matt Sherman 2020-06-11 07:58:09 -04:00 committed by GitHub
parent 227a499343
commit bea82c93cd
2 changed files with 29 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
plugins/woocommerce-admin/client/quick-links/test/index.js
View File

@ -56,7 +56,12 @@ describe( 'QuickLinks', () => {
const recordEvent = jest.fn();
render(
<QuickLinks getSetting={ () => {} } recordEvent={ recordEvent } />
<QuickLinks
getSetting={ () => {} }
recordEvent={ recordEvent }
// Prevent jsdom "Error: Not implemented: navigation" in test output
onItemClick={ () => false }
/>
);
userEvent.click(
@ -76,7 +81,12 @@ describe( 'QuickLinks', () => {
const recordEvent = jest.fn();
render(
<QuickLinks getSetting={ () => {} } recordEvent={ recordEvent } />
<QuickLinks
getSetting={ () => {} }
recordEvent={ recordEvent }
// Prevent jsdom "Error: Not implemented: navigation" in test output
onItemClick={ () => false }
/>
);
userEvent.click(
@ -96,7 +106,12 @@ describe( 'QuickLinks', () => {
const recordEvent = jest.fn();
render(
<QuickLinks getSetting={ () => {} } recordEvent={ recordEvent } />
<QuickLinks
getSetting={ () => {} }
recordEvent={ recordEvent }
// Prevent jsdom "Error: Not implemented: navigation" in test output
onItemClick={ () => false }
/>
);
userEvent.click(
@ -118,7 +133,12 @@ describe( 'QuickLinks', () => {
const recordEvent = jest.fn();
render(
<QuickLinks getSetting={ () => {} } recordEvent={ recordEvent } />
<QuickLinks
getSetting={ () => {} }
recordEvent={ recordEvent }
// Prevent jsdom "Error: Not implemented: navigation" in test output
onItemClick={ () => false }
/>
);
userEvent.click(
@ -141,10 +161,6 @@ describe( 'QuickLinks', () => {
<QuickLinks getSetting={ getSetting } recordEvent={ () => {} } />
);
userEvent.click(
screen.getByRole( 'menuitem', { name: 'View my store' } )
);
expect( getSetting ).toHaveBeenCalledWith( 'siteUrl' );
} );
} );

6
plugins/woocommerce-admin/packages/components/src/link/test/index.js
View File

@ -105,7 +105,11 @@ describe( 'Link', () => {
} );
it( 'should support `onClick`', () => {
const clickHandler = jest.fn();
// Prevent jsdom "Error: Not implemented: navigation" in test output
const clickHandler = jest.fn( ( event ) => {
event.preventDefault();
return false;
} );
const { container } = render(
<Link