fix: mvp fixes

Author: shaurabh-tiwari-git

…/utils/constants/callToActionMessages.ts src/utils/constants/documentRegistry.tssrc/utils/constants/callToActionMessages.ts renamed to src/utils/constants/documentRegistry.ts src/utils/constants/callToActionMessages.ts renamed to src/utils/constants/documentRegistry.ts

@@ -1,4 +1,4 @@
-export const callToActionMessages = {
+export const documentRegistry = {
   errorNoOrgResults:
     'https://help.salesforce.com/s/articleView?id=xcloud.os_migrate_get_moving_run_the_omnistudio_migration_tool.htm&type=5',
   couldNotDeactivateOmniProcesses:

src/utils/resultsbuilder/ApexAssessmentReporter.ts

@@ -74,7 +74,7 @@ export class ApexAssessmentReporter {
           1,
           false,
           undefined,
-          apexAssessmentInfo.infos
+          apexAssessmentInfo.infos ? reportingHelper.decorateErrors(apexAssessmentInfo.infos) : []
         ),
         createRowDataParam(
           'errors',
@@ -84,7 +84,7 @@ export class ApexAssessmentReporter {
           1,
           false,
           undefined,
-          apexAssessmentInfo.warnings
+          apexAssessmentInfo.warnings ? reportingHelper.decorateErrors(apexAssessmentInfo.warnings) : []
         ),
       ],
     }));

src/utils/resultsbuilder/DRAssessmentReporter.ts

@@ -146,7 +146,7 @@ export class DRAssessmentReporter {
           1,
           false,
           undefined,
-          dataRaptorAssessmentInfo.warnings
+          dataRaptorAssessmentInfo.warnings ? reportingHelper.decorateErrors(dataRaptorAssessmentInfo.warnings) : []
         ),
         createRowDataParam(
           'customFunctionDependencies',

src/utils/resultsbuilder/IPAssessmentReporter.ts

@@ -70,7 +70,7 @@ export class IPAssessmentReporter {
           1,
           false,
           undefined,
-          ipAssessmentInfo.warnings
+          ipAssessmentInfo.warnings ? reportingHelper.decorateErrors(ipAssessmentInfo.warnings) : []
         ),
         createRowDataParam(
           'integrationProcedureDependencies',

src/utils/resultsbuilder/OSAssessmentReporter.ts

@@ -80,7 +80,7 @@ export class OSAssessmentReporter {
           1,
           false,
           undefined,
-          info.warnings
+          info.warnings ? reportingHelper.decorateErrors(info.warnings) : []
         ),
         createRowDataParam(
           'omniScriptDependencies',

src/utils/resultsbuilder/index.ts

@@ -10,7 +10,7 @@ import { TemplateParser } from '../templateParser/generate';
 import { createFilterGroupParam, createRowDataParam } from '../reportGenerator/reportUtil';
 import { FileDiffUtil } from '../lwcparser/fileutils/FileDiffUtil';
 import { Logger } from '../logger';
-
+import { reportingHelper } from './reportingHelper';
 const resultsDir = path.join(process.cwd(), 'migration_report');
 // const lwcConstants = { componentName: 'lwc', title: 'LWC Components Migration Result' };
 const migrationReportHTMLfileName = 'dashboard.html';
@@ -139,7 +139,16 @@ export class ResultsBuilder {
             createRowDataParam('name', item.name, true, 1, 1, false),
             createRowDataParam('migratedId', item.migratedId, false, 1, 1, true, `${instanceUrl}/${item.migratedId}`),
             createRowDataParam('migratedName', item.migratedName, false, 1, 1, false),
-            createRowDataParam('status', item.status, false, 1, 1, false),
+            createRowDataParam(
+              'status',
+              item.status,
+              false,
+              1,
+              1,
+              false,
+              undefined,
+              reportingHelper.decorateStatus(item.status)
+            ),
             createRowDataParam(
               'errors',
               item.errors ? 'Has Errors' : 'Has No Errors',
@@ -148,7 +157,7 @@ export class ResultsBuilder {
               1,
               false,
               undefined,
-              item.errors
+              item.errors ? reportingHelper.decorateErrors(item.errors) : []
             ),
             createRowDataParam(
               'warnings',
@@ -158,7 +167,7 @@ export class ResultsBuilder {
               1,
               false,
               undefined,
-              item.warnings || []
+              item.warnings ? reportingHelper.decorateErrors(item.warnings) : []
             ),
           ],
         })),
@@ -263,7 +272,7 @@ export class ResultsBuilder {
             1,
             false,
             undefined,
-            item.warnings
+            item.warnings ? reportingHelper.decorateErrors(item.warnings) : []
           ),
         ],
       })),

src/utils/resultsbuilder/reportingHelper.ts

@@ -2,7 +2,7 @@ import { Messages } from '@salesforce/core';
 import { nameLocation, oldNew } from '../interfaces';
 import { CTASummary } from '../reportGenerator/reportInterfaces';
 import { IPAssessmentInfo, OSAssessmentInfo, DataRaptorAssessmentInfo, ApexAssessmentInfo } from '../interfaces';
-import { callToActionMessages } from '../constants/callToActionMessages';
+import { documentRegistry } from '../constants/documentRegistry';
 import { Logger } from '../logger';
 
 Messages.importMessagesDirectory(__dirname);
@@ -37,14 +37,33 @@ export class reportingHelper {
     }
   }
 
+  public static decorateErrors(errors: string[]): string[] {
+    if (!errors || errors.length === 0) return [];
+    const errorMessages: string[] = [];
+    for (const error of errors) {
+      errorMessages.push('<div class="slds-text-color_error">' + error + '</div>');
+    }
+    return errorMessages;
+  }
+
+  public static decorateStatus(status: string): string {
+    if (status === 'Can be Automated' || status === 'Complete') {
+      return '<div class="slds-text-color_success">' + status + '</div>';
+    }
+    if (status === 'Skipped') {
+      return '<div class="text-warning">' + status + '</div>';
+    }
+    return '<div class="slds-text-color_error">' + status + '</div>';
+  }
+
   public static getCallToAction(
     assessmentInfos: Array<IPAssessmentInfo | OSAssessmentInfo | DataRaptorAssessmentInfo | ApexAssessmentInfo>
   ): CTASummary[] {
     const callToAction = [];
     assessmentInfos.forEach((assessmentInfo) => {
       if (assessmentInfo.warnings && assessmentInfo.warnings.length > 0) {
         for (const info of assessmentInfo.warnings) {
-          for (const key of Object.keys(callToActionMessages)) {
+          for (const key of Object.keys(documentRegistry)) {
             const value = assessMessages.getMessage(key);
             if (
               typeof value === 'string' &&
@@ -99,8 +118,8 @@ export class reportingHelper {
   }
 
   private static getLink(key: string): string {
-    if (callToActionMessages[key]) {
-      return callToActionMessages[key] as string;
+    if (documentRegistry[key]) {
+      return documentRegistry[key] as string;
     }
     Logger.logVerbose(`No link found for ${key}`);
     return undefined;