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

Tweak some logic in our cart Integration test. (https://github.com/woocommerce/woocommerce-blocks/pull/2778) 

* Fix Cart test async logic

* tweak test

* refactor test to not call response twice

* add comment for why we're calling fetch twice

* fix duplicate calls

* reset empty test to normal value

* switch to mockResponse

* convert comment to todo block

Co-authored-by: Darren Ethier <darren@roughsmootheng.in>
This commit is contained in:
Seghir Nadir 2020-07-02 20:10:19 +01:00 committed by GitHub
parent 76ab006219
commit 81570d733c
1 changed files with 21 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
plugins/woocommerce-blocks/assets/js/blocks/cart-checkout/cart/test/block.js
View File

@ -13,7 +13,7 @@ import { defaultCartState } from '../../../../data/default-states';
describe( 'Testing cart', () => {
beforeEach( async () => {
fetchMock.mockResponseOnce( ( req ) => {
fetchMock.mockResponse( ( req ) => {
if ( req.url.match( /wc\/store\/cart/ ) ) {
return Promise.resolve( JSON.stringify( previewCart ) );
}
@ -22,6 +22,9 @@ describe( 'Testing cart', () => {
await dispatch( storeKey ).invalidateResolutionForStore();
await dispatch( storeKey ).receiveCart( defaultCartState );
} );
afterEach( () => {
fetchMock.resetMocks();
} );
it( 'renders cart if there are items in the cart', async () => {
render(
<CartBlock
@ -32,14 +35,21 @@ describe( 'Testing cart', () => {
} }
/>
);
await waitFor( () => {
expect(
screen.getByText( /Proceed to Checkout/ )
).toBeInTheDocument();
} );
await waitFor( () => expect( fetchMock ).toHaveBeenCalled() );
expect(
await screen.getByText( /Proceed to Checkout/i )
).toBeInTheDocument();
/**
* @todo Investigate extra POST request on initial cart render.
*
* We have an unfixed bug in our test in which the full cart triggers a POST
* request to `wc/store/cart/update-item` causing the fetch to be called twice.
*/
expect( fetchMock ).toHaveBeenCalledTimes( 2 );
} );
it( 'renders empty cart if there are no items in the cart', async () => {
fetchMock.mockResponseOnce( ( req ) => {
fetchMock.mockResponse( ( req ) => {
if ( req.url.match( /wc\/store\/cart/ ) ) {
return Promise.resolve( JSON.stringify( defaultCartState ) );
}
@ -53,6 +63,9 @@ describe( 'Testing cart', () => {
} }
/>
);
expect( await screen.findByText( /Empty Cart/ ) ).toBeInTheDocument();
await waitFor( () => expect( fetchMock ).toHaveBeenCalled() );
expect( await screen.getByText( /Empty Cart/i ) ).toBeInTheDocument();
expect( fetchMock ).toHaveBeenCalledTimes( 1 );
} );
} );