Merge pull request #1497 from Hirobreak/mig-logs

migration logs

Author: javiteri95

electron_app/src/database/DBEmigrations/20200123161254-multipleAccounts.js

@@ -458,7 +458,10 @@ module.exports = {
 
       await transaction.commit();
     } catch (ex) {
-      logger.error(ex);
+      logger.error({
+        message: 'Migration Multiple Accounts',
+        error: ex
+      });
       await transaction.rollback();
       throw new Error(ex.toString());
     }

electron_app/src/database/DBEmigrations/20200422145912-customerType.js

@@ -1,5 +1,6 @@
 'use strict';
 const { Table } = require('../DBEmodel');
+const logger = require('../../logger');
 
 const migrateAccount = async (queryInterface, Sequelize, transaction) => {
   const accountDef = await queryInterface.describeTable(Table.ACCOUNT, {
@@ -24,7 +25,10 @@ module.exports = {
       await migrateAccount(queryInterface, Sequelize, transaction);
       await transaction.commit();
     } catch (ex) {
-      console.error(ex);
+      logger.error({
+        message: 'Migration Customer Type',
+        error: ex
+      });
       await transaction.rollback();
       throw new Error(ex.toString());
     }

electron_app/src/database/DBEmigrations/20200512111423_addBlockAllColumn.js

@@ -1,5 +1,6 @@
 'use strict';
 const { Table, Email } = require('../DBEmodel');
+const logger = require('../../logger');
 
 const migrateBlockRemoteContent = async (
   queryInterface,
@@ -79,7 +80,10 @@ module.exports = {
       await addUniqueIndexes(queryInterface, transaction);
       await transaction.commit();
     } catch (ex) {
-      console.error(ex);
+      logger.error({
+        message: 'Migration Block All Column',
+        error: ex
+      });
       await transaction.rollback();
       throw new Error(ex.toString());
     }

electron_app/src/database/DBEmigrations/20200701180920_addDefaultAddressId.js

@@ -1,5 +1,6 @@
 'use strict';
 const { Table } = require('../DBEmodel');
+const logger = require('../../logger');
 
 const migrateAccount = async (queryInterface, Sequelize, transaction) => {
   const accountDef = await queryInterface.describeTable(Table.ACCOUNT, {
@@ -24,7 +25,10 @@ module.exports = {
       await migrateAccount(queryInterface, Sequelize, transaction);
       await transaction.commit();
     } catch (ex) {
-      console.error(ex);
+      logger.error({
+        message: 'Migration Default Address',
+        error: ex
+      });
       await transaction.rollback();
       throw new Error(ex.toString());
     }

electron_app/src/database/DBEmodel.js

@@ -559,6 +559,10 @@ const initDatabaseEncrypted = async (
     }
   });
 
+  logger.info(
+    `Database Current Version : ${JSON.stringify(localVersion.toJSON())}`
+  );
+
   const emailDef = await Email.describe();
   if (emailDef.accountId && localVersion.value === CURRENT_VERSION) return;
 
@@ -592,6 +596,8 @@ const initDatabaseEncrypted = async (
       break;
   }
 
+  logger.info(`Migrations to perform : ${migrationFiles.toString()}`);
+
   try {
     const migrationPath = path.join(__dirname, '/DBEmigrations');
     const migrator = new umzug({
@@ -619,7 +625,10 @@ const initDatabaseEncrypted = async (
       }
     );
   } catch (ex) {
-    logger.error(ex);
+    logger.error({
+      message: 'Migrating Database',
+      error: ex
+    });
   }
 };