fix default var and actually write the state

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

Author: wzrdtales

lib/executors/versioned/v2.js

@@ -73,7 +73,7 @@ const execUnit = {
       chain.addChain(AddConventions);
     }
     chain.addChain(Learn);
-    // chain.addChain(StateTravel);
+    chain.addChain(StateTravel);
 
     await State.startMigration(context._driver, file, context.internals);
     // startMigration - needs secondary instance since we can not afford to
@@ -99,7 +99,6 @@ const execUnit = {
       await execUnit.down(context, driver, file);
       throw err;
     }
-    await Promise.promisify(context.writeMigrationRecord.bind(context))(file);
     await State.endMigration(context._driver, file, context.internals);
     log.verbose(`[fix] current schema`, util.inspect(context.internals.schema, false, null, true));
     // end migration, same as start migration

lib/state.js

@@ -47,6 +47,7 @@ module.exports = {
 
         log.info(`[state] Created a backup of ${internals.migrationState} by renaming table to ${newName}`);
         await driver._createKV(internals.migrationState);
+        await driver._insertKV(internals.migrationState, SSTATE, '{}');
       }
     }
 

lib/walker.js

@@ -64,7 +64,7 @@ Walker.prototype = {
     return this._driver._createList(this.internals.migrationTable);
   },
 
-  createMigrationsTable: function (options) {
+  createMigrationsTable: function (options = {}) {
     if (
       typeof this._driver._createList !== 'function' ||
       typeof this._driver._getList !== 'function' ||