fix: do not clear storage when interaction required

OKTA-391537
<<<Jenkins Check-In of Tested SHA: 5bb7ae7 for [email protected]>>>
Artifact: okta-auth-js
Files changed count: 6
PR Link: "#742"

Author: aarongranick-okta

.gitignore

@@ -9,3 +9,4 @@ testenv
 junit.xml
 yarn-error.*
 coverage
+testenv.yml

lib/oidc/parseFromUrl.ts

@@ -12,7 +12,7 @@
  *
  */
 import { AuthSdkError } from '../errors';
-import { urlParamsToObject } from './util';
+import { isInteractionRequiredError, urlParamsToObject } from './util';
 import { ParseFromUrlOptions, TokenResponse, CustomUrls, TransactionMeta } from '../types';
 import { isString } from '../util';
 import { handleOAuthResponse } from './handleOAuthResponse';
@@ -79,8 +79,15 @@ export function parseFromUrl(sdk, options: string | ParseFromUrlOptions): Promis
         responseMode === 'query' ? removeSearch(sdk) : removeHash(sdk);
       }
       return handleOAuthResponse(sdk, oauthParams, res, urls)
-        .finally(() => {
+        .catch(err => {
+          if (!isInteractionRequiredError(err)) {
+            sdk.transactionManager.clear();
+          }
+          throw err;
+        })
+        .then(res => {
           sdk.transactionManager.clear();
+          return res;
         });
     });
 }