chore: minor fixes in expsites

Author: sf-aastha-paruthi

messages/assess.json

@@ -183,5 +183,6 @@
   "missingInfo": "Info is missing",
   "errorCheckingGlobalAutoNumber": "We couldn’t check whether the Global Auto Number setting is enabled: %s. Try again later.",
   "errorMigrationMessage": "Error migrating object: %s",
-  "nameMappingUndefined": "Name Mapping is undefined"
+  "nameMappingUndefined": "Name Mapping is undefined",
+  "experienceSiteException": "Exception occurred while processing experience sites"
 }

messages/migrate.json

@@ -195,8 +195,8 @@
   "flexcardStorageProcessingStarted": "Started preparing storage for flexcards",
   "errorWhileProcessingFlexcardStorage": "Error occurred while processing key for flexcard storage",
   "missingInfo": "Info is missing",
-  "incompleteMigrationDetected": "We couldn’t complete the migration process",
   "errorCheckingGlobalAutoNumber": "We couldn’t check whether the Global Auto Number setting is enabled: %s. Try again later.",
   "errorMigrationMessage": "Error migrating object: %s",
-  "nameMappingUndefined": "Name Mapping is undefined"
+  "nameMappingUndefined": "Name Mapping is undefined",
+  "experienceSiteException": "Exception occurred while processing experience sites"
 }

src/commands/omnistudio/migration/assess.ts

@@ -98,7 +98,7 @@ export default class Assess extends OmniStudioBaseCommand {
 
     const namespace = orgs.packageDetails.namespace;
     let projectPath = '';
-    const preMigrate: PreMigrate = new PreMigrate(this.org, namespace, conn, this.logger, messages, this.ux);
+    const preMigrate: PreMigrate = new PreMigrate(namespace, conn, this.logger, messages, this.ux);
     if (relatedObjects) {
       objectsToProcess = relatedObjects.split(',').map((obj) => obj.trim());
       projectPath = await ProjectPathUtil.getProjectPath(messages, true);

src/commands/omnistudio/migration/migrate.ts

@@ -132,7 +132,7 @@ export default class Migrate extends OmniStudioBaseCommand {
     let projectPath: string;
     let objectsToProcess: string[] = [];
     let targetApexNamespace: string;
-    const preMigrate: PreMigrate = new PreMigrate(this.org, namespace, conn, this.logger, messages, this.ux);
+    const preMigrate: PreMigrate = new PreMigrate(namespace, conn, this.logger, messages, this.ux);
     const isExperienceBundleMetadataAPIProgramaticallyEnabled: { value: boolean } = { value: false };
     if (relatedObjects) {
       // To-Do: Add LWC to valid options when GA is released
@@ -452,11 +452,14 @@ export default class Migrate extends OmniStudioBaseCommand {
         let errors: any[] = obj.errors || [];
         errors = errors.concat(recordResults.errors || []);
 
+        obj.status = 'Complete';
+        /*
         obj.status = recordResults?.skipped
           ? messages.getMessage('labelStatusSkipped')
           : !recordResults || recordResults.hasErrors
-          ? messages.getMessage('labelStatusFailed')
-          : messages.getMessage('labelStatusComplete');
+            ? messages.getMessage('labelStatusFailed')
+            : messages.getMessage('labelStatusComplete');
+        */
         obj.errors = errors;
         obj.migratedId = recordResults.id;
         obj.warnings = recordResults.warnings;

src/migration/flexcard.ts

@@ -636,7 +636,7 @@ export class CardMigrationTool extends BaseMigrationTool implements MigrationToo
           flexCardAssessmentInfo.nameMapping === undefined ||
           flexCardAssessmentInfo.nameMapping === null
         ) {
-          Logger.logVerbose(this.messages.getMessage('missingInfo'));
+          Logger.error(this.messages.getMessage('missingInfo'));
           return;
         }
 
@@ -689,13 +689,13 @@ export class CardMigrationTool extends BaseMigrationTool implements MigrationToo
 
         if (newrecord === undefined) {
           value.error = ['Migration Failed'];
-        }
-
-        if (newrecord.hasErrors) {
-          value.error = newrecord.errors;
-          value.migrationSuccess = false;
         } else {
-          value.migrationSuccess = true;
+          if (newrecord.hasErrors) {
+            value.error = newrecord.errors;
+            value.migrationSuccess = false;
+          } else {
+            value.migrationSuccess = true;
+          }
         }
 
         let finalKey = `${oldrecord['Name']}`;

src/migration/omniscript.ts

@@ -537,6 +537,7 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
 
   private updateStorageForOmniscriptAssessment(osAssessmentInfo: OSAssessmentInfo[]): void {
     if (osAssessmentInfo === undefined || osAssessmentInfo === null) {
+      Logger.error(this.messages.getMessage('missingInfo'));
       return;
     }
 
@@ -579,7 +580,7 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
           storage.osStorage.set(finalKey, value);
         }
       } catch (error) {
-        Logger.logVerbose(error);
+        Logger.error(error);
       }
     }
 
@@ -905,11 +906,16 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
             isDuplicate: false,
           };
 
-          if (newrecord.hasErrors) {
-            value.error = newrecord.errors;
+          // New record can be undefined
+          if (newrecord === undefined) {
             value.migrationSuccess = false;
           } else {
-            value.migrationSuccess = true;
+            if (newrecord.hasErrors) {
+              value.error = newrecord.errors;
+              value.migrationSuccess = false;
+            } else {
+              value.migrationSuccess = true;
+            }
           }
 
           let finalKey = `${oldrecord[this.namespacePrefix + 'Type__c']}${

src/migration/premigrate.ts

@@ -1,6 +1,6 @@
 /* eslint-disable */
 
-import { Connection, Messages, Org } from '@salesforce/core';
+import { Connection, Messages } from '@salesforce/core';
 import { UX } from '@salesforce/command';
 import { Logger } from '../utils/logger';
 import { Constants } from '../utils/constants/stringContants';
@@ -9,21 +9,9 @@ import { BaseMigrationTool } from './base';
 import { askStringWithTimeout } from '../utils/promptUtil';
 
 export class PreMigrate extends BaseMigrationTool {
-  private readonly org: Org;
-
   // Source Custom Object Names
-  constructor(org: Org, namespace: string, connection: Connection, logger: Logger, messages: Messages, ux: UX) {
+  constructor(namespace: string, connection: Connection, logger: Logger, messages: Messages, ux: UX) {
     super(namespace, connection, logger, messages, ux);
-    this.org = org;
-  }
-
-  // Just to disable org is unused error coming
-  public printOrgDetails(): void {
-    try {
-      Logger.log(JSON.stringify(this.org));
-    } catch (e) {
-      Logger.log(e);
-    }
   }
 
   public async handleExperienceSitePrerequisites(

src/migration/related/ExperienceSiteMigration.ts

@@ -97,6 +97,7 @@ export class ExperienceSiteMigration extends BaseRelatedObjectMigration {
             Logger.logVerbose(this.messages.getMessage('fileNotHavingWrapper'));
           }
         } catch (err) {
+          this.populateExceptionInfo(file, experienceSitesAssessmentInfo);
           Logger.error(this.messages.getMessage('errorProcessingExperienceSite', [file.name]));
           Logger.error(JSON.stringify(err));
         }
@@ -168,6 +169,25 @@ export class ExperienceSiteMigration extends BaseRelatedObjectMigration {
     return experienceSiteAssessmentInfo;
   }
 
+  private populateExceptionInfo(file: File, experienceSiteAssessmentInfos: ExperienceSiteAssessmentInfo[]): void {
+    try {
+      const experienceSiteAssessmentInfo: ExperienceSiteAssessmentInfo = {
+        name: file.name,
+        warnings: [],
+        errors: ['Unknown error occurred'],
+        infos: [],
+        path: file.location,
+        diff: JSON.stringify([]),
+        hasOmnistudioContent: false,
+        status: 'Errors',
+      };
+
+      experienceSiteAssessmentInfos.push(experienceSiteAssessmentInfo);
+    } catch {
+      Logger.error(this.messages.getMessage('experienceSiteException'));
+    }
+  }
+
   private processRegion(
     region: ExpSiteRegion,
     experienceSiteAssessmentInfo: ExperienceSiteAssessmentInfo,
@@ -290,7 +310,6 @@ export class ExperienceSiteMigration extends BaseRelatedObjectMigration {
     Logger.logVerbose(this.messages.getMessage('processingOmniscriptComponent', [JSON.stringify(component)]));
     // Use storage to find the updated properties
     const targetDataFromStorage: OmniScriptStorage = storage.osStorage.get(targetName.toLowerCase());
-    StorageUtil.printAssessmentStorage();
     Logger.logVerbose(this.messages.getMessage('targetData', [JSON.stringify(targetDataFromStorage)]));
 
     if (this.shouldAddWarning(targetDataFromStorage)) {
@@ -308,7 +327,7 @@ export class ExperienceSiteMigration extends BaseRelatedObjectMigration {
       keysToDelete.forEach((key) => delete currentAttribute[key]);
 
       currentAttribute['direction'] = 'ltr';
-      currentAttribute['display'] = 'Display button to open Omniscript';
+      currentAttribute['display'] = 'Display OmniScript on page';
       currentAttribute['inlineVariant'] = 'brand';
       currentAttribute['language'] =
         targetDataFromStorage.language === undefined ? 'English' : targetDataFromStorage.language;

src/utils/resultsbuilder/index.ts

@@ -251,7 +251,7 @@ export class ResultsBuilder {
       },
       assessmentDate: new Date().toString(),
       total: result.length,
-      filterGroups: [createFilterGroupParam('Filter by Status', 'status', ['Complete', 'Error'])],
+      filterGroups: [createFilterGroupParam('Filter by Status', 'status', ['Can be Automated', 'Error'])],
       headerGroups: [
         {
           header: [
@@ -288,16 +288,7 @@ export class ResultsBuilder {
         data: [
           createRowDataParam('name', item.name, true, 1, 1, false),
           createRowDataParam('path', item.path, false, 1, 1, false),
-          createRowDataParam(
-            'status',
-            item.warnings ? 'Error' : 'Complete', // This is the value which the status key takes
-            false,
-            1,
-            1,
-            false,
-            undefined,
-            item.warnings ? 'Error' : 'Complete' // This is what gets displayed
-          ),
+          createRowDataParam('status', item.status, false, 1, 1, false, undefined, item.status),
           createRowDataParam(
             'diff',
             item.name + 'diff',
@@ -306,7 +297,7 @@ export class ResultsBuilder {
             1,
             false,
             undefined,
-            FileDiffUtil.getDiffHTML(item.diff, item.name)
+            FileDiffUtil.getDiffHTML(item.diff, this.rowId.toString())
           ),
           createRowDataParam(
             'errors',

test/migration/related/ExperienceSiteMigration.test.ts

@@ -203,7 +203,11 @@ describe('ExperienceSiteMigration', () => {
       const result = experienceSiteMigration.processExperienceSites('/test/project', Migrate);
 
       // Assert
-      expect(result).to.be.an('array').that.is.empty;
+      expect(result).to.be.an('array').that.has.length(1);
+      expect(result[0].name).to.equal('error.json');
+      expect(result[0].status).to.equal('Errors');
+      expect(result[0].errors).to.include('Unknown error occurred');
+      expect(result[0].hasOmnistudioContent).to.be.false;
       expect((Logger.error as sinon.SinonStub).called).to.be.true;
     });
   });
@@ -269,7 +273,7 @@ describe('ExperienceSiteMigration', () => {
       expect(component.componentAttributes.subType).to.equal('TestSubtype');
       expect(component.componentAttributes.language).to.equal('English');
       expect(component.componentAttributes.direction).to.equal('ltr');
-      expect(component.componentAttributes.display).to.equal('Display button to open Omniscript');
+      expect(component.componentAttributes.display).to.equal('Display OmniScript on page');
       expect(component.componentAttributes.theme).to.equal('lightning'); // Preserved from original layout
     });