Skip all authorize errors

woocommerce · Aug 13, 2024 · 007190f · 007190f
1 parent 54c1014
commit 007190f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/js/src/data/actions.js b/js/src/data/actions.js
@@ -523,13 +523,8 @@ export function* disconnectAllAccounts() {
 			type: TYPES.DISCONNECT_ACCOUNTS_ALL,
 		};
 	} catch ( error ) {
-		if (
-			error.errors[
-				`${ API_NAMESPACE }/rest-api/authorize`
-			]?.message?.includes(
-				'No token found associated with the client ID and user'
-			)
-		) {
+		// Skip any error related to revoking WPCOM token.
+		if ( error.errors[ `${ API_NAMESPACE }/rest-api/authorize` ] ) {
 			return {
 				type: TYPES.DISCONNECT_ACCOUNTS_ALL,
 			};

diff --git a/js/src/data/test/disconnectAllAccounts.test.js b/js/src/data/test/disconnectAllAccounts.test.js
@@ -35,7 +35,7 @@ describe( 'Disconnect All Accounts', () => {
 		} );
 	} );
 
-	it( 'Ignore the error if its because missing WPCOM token', async () => {
+	it( 'Ignore the error if its thrown from the authorize endpoint', async () => {
 		mockFetch.mockRejectedValue( {
 			errors: {
 				[ `${ API_NAMESPACE }/rest-api/authorize` ]: {
@@ -57,7 +57,7 @@ describe( 'Disconnect All Accounts', () => {
 		expect( response ).toEqual( { type: 'DISCONNECT_ACCOUNTS_ALL' } );
 	} );
 
-	it( 'Throw the error unless its related to a missing WPCOM token', async () => {
+	it( 'Throw the error if it is not related to the authorize endpoint', async () => {
 		mockFetch.mockRejectedValue( {
 			errors: {
 				[ `${ API_NAMESPACE }/ads/connection` ]: {