fix: do not reload state on rollback

Signed-off-by: Tobias Gurtzick <[email protected]>

Author: wzrdtales

lib/executors/versioned/v2.js

@@ -58,7 +58,7 @@ const execUnit = {
         'An error occured. No alternative failure strategy defined. Rolling back!',
         err
       );
-      await execUnit.down(context, driver, file);
+      await execUnit.down(context, driver, file, { abort: true });
       throw err;
     }
     await Promise.promisify(context.writeMigrationRecord.bind(context))(file);
@@ -104,9 +104,13 @@ const execUnit = {
     // end migration, same as start migration
   },
 
-  down: async function (context, driver, file) {
+  down: async function (context, driver, file, { abort } = {}) {
+    // if we get the abort signal this means we are in a rollback routine
+    // which means in turn we want to avoid reloading the state
+    if (abort === true) {
+      await State.startMigration(context._pdriver, file, context.internals);
+    }
     // start migration, see up comments
-    await State.startMigration(context._pdriver, file, context.internals);
     await TranslateState(context._driver, file, driver, context.internals, context._pdriver);
     await State.endMigration(context._pdriver, file, context.internals);
     return Promise.promisify(context.deleteMigrationRecord.bind(context))(file);