Remove unnecessary console.warn statements added in #9121

Automattic · Aug 9, 2024 · 4120820 · 4120820
1 parent 28a3248
commit 4120820
Show file tree

Hide file tree

Showing 7 changed files with 7 additions and 36 deletions.
diff --git a/tests/e2e/config/jest.setup.js b/tests/e2e/config/jest.setup.js
@@ -136,11 +136,7 @@ beforeAll( async () => {
 
 	capturePageEventsForTearDown();
 	page.on( 'dialog', async function ( dialog ) {
-		try {
-			await dialog.accept();
-		} catch ( err ) {
-			console.warn( err.message );
-		}
+		await dialog.accept();
 	} );
 	setTestTimeouts();
 	await setupBrowser();

diff --git a/tests/e2e/specs/subscriptions/merchant/merchant-subscriptions-renew-subscription.spec.js b/tests/e2e/specs/subscriptions/merchant/merchant-subscriptions-renew-subscription.spec.js
@@ -62,11 +62,7 @@ describeif( RUN_SUBSCRIPTIONS_TESTS )(
 		afterAll( async () => {
 			page.removeAllListeners( 'dialog' );
 			page.on( 'dialog', async function ( dialog ) {
-				try {
-					await dialog.accept();
-				} catch ( err ) {
-					console.warn( err.message );
-				}
+				await dialog.accept();
 			} );
 			await merchant.logout();
 		} );
@@ -91,11 +87,7 @@ describeif( RUN_SUBSCRIPTIONS_TESTS )(
 				page.removeAllListeners( 'dialog' ),
 				evalAndClick( 'button.save_order' ),
 				page.on( 'dialog', async ( dialog ) => {
-					try {
-						await dialog.accept();
-					} catch ( err ) {
-						console.warn( err.message );
-					}
+					await dialog.accept();
 				} ),
 				uiUnblocked(),
 				page.waitForNavigation( { waitUntil: 'networkidle0' } ),

diff --git a/tests/e2e/specs/wcpay/merchant/merchant-disputes-submit-winning.spec.js b/tests/e2e/specs/wcpay/merchant/merchant-disputes-submit-winning.spec.js
@@ -64,11 +64,7 @@ describe( 'Disputes > Submit winning dispute', () => {
 	afterAll( async () => {
 		page.removeAllListeners( 'dialog' );
 		page.on( 'dialog', async function ( dialog ) {
-			try {
-				await dialog.accept();
-			} catch ( err ) {
-				console.warn( err.message );
-			}
+			await dialog.accept();
 		} );
 		await merchant.logout();
 	} );

diff --git a/tests/e2e/specs/wcpay/merchant/merchant-orders-full-refund.spec.js b/tests/e2e/specs/wcpay/merchant/merchant-orders-full-refund.spec.js
@@ -56,11 +56,7 @@ describe( 'Order > Full refund', () => {
 	afterAll( async () => {
 		page.removeAllListeners( 'dialog' );
 		page.on( 'dialog', async function ( dialog ) {
-			try {
-				await dialog.accept();
-			} catch ( err ) {
-				console.warn( err.message );
-			}
+			await dialog.accept();
 		} );
 		await merchant.logout();
 	} );

diff --git a/tests/e2e/specs/wcpay/merchant/merchant-orders-partial-refund.spec.js b/tests/e2e/specs/wcpay/merchant/merchant-orders-partial-refund.spec.js
@@ -89,11 +89,7 @@ describe.each( dataTable )(
 		afterEach( async () => {
 			page.removeAllListeners( 'dialog' );
 			page.on( 'dialog', async function ( dialog ) {
-				try {
-					await dialog.accept();
-				} catch ( err ) {
-					console.warn( err.message );
-				}
+				await dialog.accept();
 			} );
 		} );
 

diff --git a/tests/e2e/specs/wcpay/merchant/merchant-orders-refund-failures.spec.js b/tests/e2e/specs/wcpay/merchant/merchant-orders-refund-failures.spec.js
@@ -57,11 +57,7 @@ describe( 'Order > Refund Failure', () => {
 	afterAll( async () => {
 		page.removeAllListeners( 'dialog' );
 		page.on( 'dialog', async function ( dialog ) {
-			try {
-				await dialog.accept();
-			} catch ( err ) {
-				console.warn( err.message );
-			}
+			await dialog.accept();
 		} );
 		await merchant.logout();
 	} );

diff --git a/tests/e2e/specs/wcpay/shopper/shopper-klarna-checkout.spec.js b/tests/e2e/specs/wcpay/shopper/shopper-klarna-checkout.spec.js
@@ -132,7 +132,6 @@ describe( 'Klarna checkout', () => {
 			} );
 			readyToBuy = true;
 		} catch ( err ) {
-			console.warn( err );
 			readyToBuy = false;
 		}