Update migrations to support future migrations when migrating from th…

…is as the initial point
w3c · Aug 1, 2023 · f5478f4 · f5478f4
1 parent 252e896
commit f5478f4
Show file tree

Hide file tree

Showing 4 changed files with 460 additions and 30 deletions.
diff --git a/server/migrations/20230523163856-remove-OtherUnexpectedBehaviorText.js b/server/migrations/20230523163856-remove-OtherUnexpectedBehaviorText.js
@@ -7,6 +7,7 @@ const {
 const conflictsResolver = require('../resolvers/TestPlanReport/conflictsResolver');
 const BrowserLoader = require('../models/loaders/BrowserLoader');
 const AtLoader = require('../models/loaders/AtLoader');
+const { TEST_PLAN_REPORT_ATTRIBUTES } = require('../models/services/helpers');
 
 module.exports = {
     up: queryInterface => {
@@ -138,14 +139,20 @@ module.exports = {
                 }
             }
 
+            // Exclude certain attributes called in testPlanReport query;
+            // needed to support future migrations
+            const testPlanReportAttributes = TEST_PLAN_REPORT_ATTRIBUTES.filter(
+                e => !['approvedAt'].includes(e)
+            );
+
             for (let i = 0; i < testPlanReportsData.length; i++) {
                 const testPlanReportId = testPlanReportsData[i].id;
                 const status = testPlanReportsData[i].status;
                 if (status === 'DRAFT') {
                     let updateParams = {};
                     const testPlanReport = await getTestPlanReportById(
                         testPlanReportId,
-                        null,
+                        testPlanReportAttributes,
                         null,
                         ['id', 'tests']
                     );