Merge pull request #78 from microsoft/users/jepoisso/objstoreprovider-versionerror

IndexedDBProvider upgradeCallback upon VersionError

Author: eliranek1

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@microsoft/objectstoreprovider",
-  "version": "0.8.5",
+  "version": "0.8.6",
   "description": "A cross-browser object store library",
   "author": "DataStack Team [email protected]",
   "scripts": {

src/IndexedDbProvider.ts

@@ -559,6 +559,20 @@ export class IndexedDbProvider extends DbProvider {
         });
       },
       (err) => {
+        // Invoke the upgradeCallback with error details
+        if (this._upgradeCallback) {
+          this._upgradeCallback({
+            status: "Error",
+            isCopyRequired: false,
+            upgradeSteps,
+            ...upgradeMetadata,
+            errorName: err?.target?.error?.name || "Unknown",
+            errorMessage: err
+              ? `${err?.message} ${err?.target?.error} ${err?.target?.error?.name}`
+              : "Unknown error occurred during upgrade",
+          });
+        }
+
         if (
           err &&
           err.type === "error" &&
@@ -583,19 +597,6 @@ export class IndexedDbProvider extends DbProvider {
           }
         );
 
-        // Invoke the upgradeCallback with error details
-        if (this._upgradeCallback) {
-          this._upgradeCallback({
-            status: "Error",
-            isCopyRequired: false,
-            upgradeSteps,
-            ...upgradeMetadata,
-            errorMessage: err
-              ? `${err?.message} ${err?.target?.error} ${err?.target?.error?.name}`
-              : "Unknown error occurred during upgrade",
-          });
-        }
-
         return Promise.reject<void>(err);
       }
     );

src/ObjectStoreProvider.ts

@@ -117,6 +117,7 @@ export type UpgradeDetails = UpgradeMetadata & {
   status: UpgradeStatus;
   upgradeSteps: UpgradeStep[];
   isCopyRequired: boolean;
+  errorName?: string;
   errorMessage?: string;
 };