fix(v2): learn should write a 0 action for renaming, call endMigration for state at the end of up

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

Author: wzrdtales

lib/executors/versioned/v2.js

@@ -55,7 +55,8 @@ const execUnit = {
       await execUnit.down(context, driver, file);
       throw err;
     }
-    return Promise.promisify(context.writeMigrationRecord.bind(context))(file);
+    await Promise.promisify(context.writeMigrationRecord.bind(context))(file);
+    return State.endMigration(context._driver, file, context.internals);
     // end migration, same as start migration
   },
 

lib/learn.js

@@ -157,16 +157,17 @@ class STD {
     this.modS[t] = {};
 
     if (this.schema[t][c].notNull === true) {
-      switch (o.columnStrategies) {
+      switch (o.columnStrategy) {
         case 'delay':
           this.modS[t][c] = this.schema[t][c];
 
           o.passthrough = o.passthrough || {};
           o.passthrough.column =
-            o.passthrough.column || `__dbmrn_${c}_${new Date().toString()}__`;
+            o.passthrough.column ||
+            `__dbmrn_${c}_${new Date().toISOString()}__`;
 
           this.modC.push({
-            t: 1,
+            t: 0,
             a: 'renameColumn',
             c: [t, o.passthrough.column, c]
           });