Merge pull request #857 from vkaracic/fix-remove-user-token-migration

Run remove-user-token migration in a transaction

Author: underscope

server/shared/database/migrations/20190717151916-remove-user-token.js

@@ -6,17 +6,21 @@ const TABLE = 'user';
 
 module.exports = {
   up: async qi => {
-    await qi.removeColumn(TABLE, 'token');
-    await qi.changeColumn(TABLE, 'password', {
-      type: Sequelize.STRING,
-      allowNull: false
+    return qi.sequelize.transaction(async transaction => {
+      await qi.removeColumn(TABLE, 'token', { transaction });
+      await qi.changeColumn(TABLE, 'password', {
+        type: Sequelize.STRING,
+        allowNull: false
+      }, { transaction });
     });
   },
   down: async qi => {
-    await qi.addColumn(TABLE, 'token', { type: Sequelize.STRING, unique: true });
-    await qi.changeColumn(TABLE, 'password', {
-      type: Sequelize.STRING,
-      allowNull: true
+    return qi.sequelize.transaction(async transaction => {
+      await qi.addColumn(TABLE, 'token', { type: Sequelize.STRING, unique: true }, { transaction });
+      await qi.changeColumn(TABLE, 'password', {
+        type: Sequelize.STRING,
+        allowNull: true
+      }, { transaction });
     });
   }
 };