Skip to content

Commit 81c4e04

Browse files
committed
Conflicts resolved
1 parent c8e9063 commit 81c4e04

File tree

7 files changed

+23
-152
lines changed

7 files changed

+23
-152
lines changed

src/commands/omnistudio/migration/assess.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,8 @@ import { DebugTimer } from '../../../utils';
1111
import { Logger } from '../../../utils/logger';
1212
import OmnistudioRelatedObjectMigrationFacade from '../../../migration/related/OmnistudioRelatedObjectMigrationFacade';
1313
import { OmnistudioOrgDetails, OrgUtils } from '../../../utils/orgUtils';
14-
import { OrgPreferences } from '../../../utils/orgPreferences';
14+
import { OrgPreferences } from '../../../utils/orgpreferences';
15+
import { Constants } from '../../../utils/constants/stringContants';
1516

1617
Messages.importMessagesDirectory(__dirname);
1718
const messages = Messages.loadMessages('@salesforce/plugin-omnistudio-migration-tool', 'assess');
@@ -113,7 +114,7 @@ export default class Assess extends OmniStudioBaseCommand {
113114
} catch (error) {
114115
this.ux.log(error.message);
115116
}
116-
await AssessmentReporter.generate(assesmentInfo, conn.instanceUrl, orgs);
117+
await AssessmentReporter.generate(assesmentInfo, conn.instanceUrl, orgs, assessOnly, objectsToProcess);
117118
return assesmentInfo;
118119
}
119120

src/utils/reportGenerator/reportGenerator.ts

Lines changed: 3 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -22,23 +22,8 @@ export function generateHtmlTable<T>(reportFrameworkParameters: ReportFrameworkP
2222
const indexedKey: string = reportFrameworkParameters.indexedKey;
2323
const showMigrationBanner: boolean = reportFrameworkParameters.showMigrationBanner;
2424

25-
<<<<<<< HEAD
2625
const transformedHeader: TableHeaderCell[][] = transform(headerColumns);
2726
const tableId = `report-table-${reportTableInstance++}`;
28-
=======
29-
export function generateHtmlTable<T>(
30-
headerRows: HeaderColumn[],
31-
columns: Array<TableColumn<T>>,
32-
rows: T[],
33-
reportHeader: ReportHeader[],
34-
filters: Filter[] = [],
35-
tableClass = 'slds-table slds-table_cell-buffer slds-table_bordered slds-table_striped slds-table_col-bordered',
36-
ariaLabel = '',
37-
rollbackFlags?: string[],
38-
rollbackFlagName?: string
39-
): string {
40-
const transformedHeader: TableHeaderCell[][] = transform(headerRows);
41-
>>>>>>> c749de5 (generateRollback flags logic moved to reportGenerator.ts itself)
4227

4328
const thead = `
4429
<thead>
@@ -199,7 +184,6 @@ export function generateHtmlTable<T>(
199184
</div>
200185
`;
201186

202-
<<<<<<< HEAD
203187
const ctaSumm = `
204188
${ctaSummary
205189
.map(
@@ -217,12 +201,11 @@ export function generateHtmlTable<T>(
217201
<div class="slds-text-heading_large"> ${reportHeaderLabel} Report </div>
218202
${ctaSummary && ctaSummary.length > 0 ? ctaButton : ''}
219203
</div>`;
220-
=======
204+
221205
let rollbackFlagsHtml = '';
222-
if (rollbackFlags && rollbackFlagName) {
223-
rollbackFlagsHtml = generateRollbackFlagsHtml(rollbackFlags, rollbackFlagName);
206+
if (reportFrameworkParameters.rollbackFlags && reportFrameworkParameters.rollbackFlagName) {
207+
rollbackFlagsHtml = generateRollbackFlagsHtml(reportFrameworkParameters.rollbackFlags, reportFrameworkParameters.rollbackFlagName);
224208
}
225-
>>>>>>> c749de5 (generateRollback flags logic moved to reportGenerator.ts itself)
226209

227210
return `
228211
<div class="report-wrapper">
@@ -250,13 +233,10 @@ export function generateHtmlTable<T>(
250233
${ctaSumm}
251234
</div>
252235
</div>
253-
<<<<<<< HEAD
254236
<script src="./reportGeneratorUtility.js" defer></script>
255237
<link rel="stylesheet" href="./reportGenerator.css">
256238
</div>
257-
=======
258239
${rollbackFlagsHtml}
259-
>>>>>>> c749de5 (generateRollback flags logic moved to reportGenerator.ts itself)
260240
`;
261241
}
262242

src/utils/resultsbuilder/DRAssessmentReporter.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,11 +128,13 @@ export class DRAssessmentReporter {
128128
headerColumns: headerColumn,
129129
columns,
130130
rows: dataRaptorAssessmentInfos,
131-
orgDetails: org,
131+
orgDetails: orgDetails,
132132
filters,
133133
ctaSummary: [],
134134
reportHeaderLabel: 'Data Mapper Assessment',
135135
showMigrationBanner: true,
136+
rollbackFlags,
137+
rollbackFlagName: 'RollbackDRChanges',
136138
};
137139
// Render table
138140
const tableHtml = generateHtmlTable(reportFrameworkParameters);

src/utils/resultsbuilder/IPAssessmentReporter.ts

Lines changed: 3 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,11 @@
11
import { IPAssessmentInfo } from '../interfaces';
22
import { generateHtmlTable } from '../reportGenerator/reportGenerator';
3-
<<<<<<< HEAD
43
import {
54
HeaderColumn,
65
ReportFrameworkParameters,
76
ReportHeaderFormat,
87
TableColumn,
98
} from '../reportGenerator/reportInterfaces';
10-
=======
11-
import { HeaderColumn, ReportHeaderFormat, TableColumn } from '../reportGenerator/reportInterfaces';
12-
>>>>>>> c749de5 (generateRollback flags logic moved to reportGenerator.ts itself)
139
import { reportingHelper } from './reportingHelper';
1410

1511
export class IPAssessmentReporter {
@@ -111,27 +107,17 @@ export class IPAssessmentReporter {
111107
const reportFrameworkParameters: ReportFrameworkParameters<IPAssessmentInfo> = {
112108
headerColumns: headerColumn,
113109
columns,
114-
<<<<<<< HEAD
115110
rows: ipAssessmentInfos,
116-
orgDetails: org,
111+
orgDetails: orgDetails,
117112
filters: [],
118113
ctaSummary: [],
119114
reportHeaderLabel: 'Integration Procedure Assessment',
120115
showMigrationBanner: true,
116+
rollbackFlags,
117+
rollbackFlagName: 'RollbackIPChanges',
121118
};
122119
// Render table
123120
const tableHtml = generateHtmlTable(reportFrameworkParameters);
124121
return `${tableHtml}`;
125-
=======
126-
ipAssessmentInfos,
127-
orgDetails,
128-
undefined,
129-
'Integration Procedure Assessment',
130-
undefined,
131-
rollbackFlags,
132-
'RollbackIPChanges'
133-
);
134-
return `<div class="slds-text-heading_large">Integration Procedure Assessment Report</div>${tableHtml}`;
135-
>>>>>>> c749de5 (generateRollback flags logic moved to reportGenerator.ts itself)
136122
}
137123
}

src/utils/resultsbuilder/OSAssessmentReporter.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -185,11 +185,13 @@ export class OSAssessmentReporter {
185185
headerColumns: headerColumn,
186186
columns,
187187
rows: osAssessmentInfos,
188-
orgDetails: org,
188+
orgDetails: orgDetails,
189189
filters,
190190
ctaSummary,
191191
reportHeaderLabel: 'Omniscript Assessment',
192192
showMigrationBanner: true,
193+
rollbackFlags,
194+
rollbackFlagName: 'RollbackOSChanges',
193195
};
194196

195197
// Render table

src/utils/resultsbuilder/assessmentReporter.ts

Lines changed: 8 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ import { ReportHeaderFormat } from '../reportGenerator/reportInterfaces';
77
import { OmnistudioOrgDetails } from '../orgUtils';
88
import { Constants } from '../constants/stringContants';
99
import { pushAssestUtilites } from '../file/fileUtil';
10-
import { OrgPreferences } from '../orgpreferences';
1110
import { OSAssessmentReporter } from './OSAssessmentReporter';
1211
import { ApexAssessmentReporter } from './ApexAssessmentReporter';
1312
import { IPAssessmentReporter } from './IPAssessmentReporter';
@@ -33,35 +32,11 @@ export class AssessmentReporter {
3332
const lwcAssessmentFilePath = basePath + '/lwc_assessment.html';
3433
const orgDetails: ReportHeaderFormat[] = this.formattedOrgDetails(omnistudioOrgDetails);
3534

36-
<<<<<<< HEAD
3735
if (!assessOnly) {
3836
this.createDocument(
3937
omniscriptAssessmentFilePath,
40-
OSAssessmentReporter.generateOSAssesment(result.omniAssessmentInfo.osAssessmentInfos, instanceUrl, orgDetails)
38+
OSAssessmentReporter.generateOSAssesment(result.omniAssessmentInfo.osAssessmentInfos, instanceUrl, orgDetails, omnistudioOrgDetails.rollbackFlags)
4139
);
42-
=======
43-
this.createDocument(
44-
omniscriptAssessmentFilePath,
45-
OSAssessmentReporter.generateOSAssesment(
46-
result.omniAssessmentInfo.osAssessmentInfos,
47-
instanceUrl,
48-
orgDetails,
49-
omnistudioOrgDetails.rollbackFlags
50-
)
51-
);
52-
this.createDocument(
53-
flexcardAssessmentFilePath,
54-
this.generateCardAssesment(result.flexCardAssessmentInfos, instanceUrl)
55-
);
56-
this.createDocument(
57-
integrationProcedureAssessmentFilePath,
58-
IPAssessmentReporter.generateIPAssesment(result.omniAssessmentInfo.ipAssessmentInfos, instanceUrl, orgDetails, omnistudioOrgDetails.rollbackFlags)
59-
);
60-
this.createDocument(
61-
dataMapperAssessmentFilePath,
62-
DRAssessmentReporter.generateDRAssesment(result.dataRaptorAssessmentInfos, instanceUrl, orgDetails, omnistudioOrgDetails.rollbackFlags)
63-
);
64-
>>>>>>> 451bd60 (Comments addressed)
6540

6641
this.createDocument(
6742
flexcardAssessmentFilePath,
@@ -79,12 +54,12 @@ export class AssessmentReporter {
7954

8055
this.createDocument(
8156
integrationProcedureAssessmentFilePath,
82-
IPAssessmentReporter.generateIPAssesment(result.omniAssessmentInfo.ipAssessmentInfos, instanceUrl, orgDetails)
57+
IPAssessmentReporter.generateIPAssesment(result.omniAssessmentInfo.ipAssessmentInfos, instanceUrl, orgDetails, omnistudioOrgDetails.rollbackFlags)
8358
);
8459

8560
this.createDocument(
8661
dataMapperAssessmentFilePath,
87-
DRAssessmentReporter.generateDRAssesment(result.dataRaptorAssessmentInfos, instanceUrl, orgDetails)
62+
DRAssessmentReporter.generateDRAssesment(result.dataRaptorAssessmentInfos, instanceUrl, orgDetails, omnistudioOrgDetails.rollbackFlags)
8863
);
8964
} else {
9065
switch (assessOnly) {
@@ -94,7 +69,8 @@ export class AssessmentReporter {
9469
OSAssessmentReporter.generateOSAssesment(
9570
result.omniAssessmentInfo.osAssessmentInfos,
9671
instanceUrl,
97-
orgDetails
72+
orgDetails,
73+
omnistudioOrgDetails.rollbackFlags
9874
)
9975
);
10076
break;
@@ -112,15 +88,16 @@ export class AssessmentReporter {
11288
IPAssessmentReporter.generateIPAssesment(
11389
result.omniAssessmentInfo.ipAssessmentInfos,
11490
instanceUrl,
115-
orgDetails
91+
orgDetails,
92+
omnistudioOrgDetails.rollbackFlags
11693
)
11794
);
11895
break;
11996

12097
case Constants.DataMapper:
12198
this.createDocument(
12299
dataMapperAssessmentFilePath,
123-
DRAssessmentReporter.generateDRAssesment(result.dataRaptorAssessmentInfos, instanceUrl, orgDetails)
100+
DRAssessmentReporter.generateDRAssesment(result.dataRaptorAssessmentInfos, instanceUrl, orgDetails, omnistudioOrgDetails.rollbackFlags)
124101
);
125102
break;
126103

@@ -169,19 +146,6 @@ export class AssessmentReporter {
169146
// location: 'lwc_assessment.html',
170147
// },
171148
];
172-
<<<<<<< HEAD
173-
// Check rollback flags
174-
const enabledFlags = omnistudioOrgDetails.rollbackFlags || [];
175-
if (enabledFlags.length > 0) {
176-
const rollbackFlagsReportPath = basePath + '/rollback_flags_report.html';
177-
this.createDocument(rollbackFlagsReportPath, this.generateRollbackFlagsReport(enabledFlags));
178-
nameUrls.push({
179-
name: 'Rollback Flags Report',
180-
location: 'rollback_flags_report.html',
181-
});
182-
}
183-
=======
184-
>>>>>>> 451bd60 (Comments addressed)
185149

186150
await this.createMasterDocument(nameUrls, basePath);
187151
pushAssestUtilites('javascripts', basePath);

src/utils/resultsbuilder/index.ts

Lines changed: 0 additions & 64 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import fs from 'fs';
22
import open from 'open';
3-
<<<<<<< HEAD
43
import { Logger } from '@salesforce/core';
54
import { pushAssestUtilites } from '../file/fileUtil';
65
import { ApexAssessmentInfo, MigratedObject, MigratedRecordInfo, RelatedObjectAssesmentInfo } from '../interfaces';
@@ -26,10 +25,6 @@ const apexConstants = { componentName: 'apex', title: 'Apex Classes Migration Re
2625
// const lwcConstants = { componentName: 'lwc', title: 'LWC Components Migration Result' };
2726
const migrationResultCSSfileName = 'reportGenerator.css';
2827
const migrationReportHTMLfileName = 'migration_report.html';
29-
=======
30-
import { ApexAssessmentInfo, LWCAssessmentInfo, MigratedObject, RelatedObjectAssesmentInfo } from '../interfaces';
31-
import { OmnistudioOrgDetails } from '../orgUtils';
32-
>>>>>>> 451bd60 (Comments addressed)
3328

3429
export class ResultsBuilder {
3530
private static logger: Logger = new Logger('ResultsBuilder');
@@ -38,11 +33,7 @@ export class ResultsBuilder {
3833
results: MigratedObject[],
3934
relatedObjectMigrationResult: RelatedObjectAssesmentInfo,
4035
instanceUrl: string,
41-
<<<<<<< HEAD
4236
orgDetails: OmnistudioOrgDetails
43-
=======
44-
omnistudioOrgDetails: OmnistudioOrgDetails
45-
>>>>>>> 451bd60 (Comments addressed)
4637
): Promise<void> {
4738
this.logger.info('Generating directories');
4839
fs.mkdirSync(resultsDir, { recursive: true });
@@ -75,7 +66,6 @@ export class ResultsBuilder {
7566
const details: ComponentDetail[] = [];
7667

7768
for (const result of results) {
78-
<<<<<<< HEAD
7969
const resultConstants = this.getResultConstant(result.name);
8070
details.push({
8171
name: resultConstants.componentName,
@@ -85,57 +75,6 @@ export class ResultsBuilder {
8575
error: result.data?.filter((record) => record.status === 'Error').length || 0,
8676
skip: result.data?.filter((record) => record.status === 'Skipped').length || 0,
8777
});
88-
=======
89-
htmlBody += '<br />' + this.generateResult(result, instanceUrl);
90-
}
91-
htmlBody += '<br />' + this.generateApexAssesment(relatedObjectMigrationResult.apexAssessmentInfos);
92-
// TODO: Uncomment code once MVP for migration is completed
93-
// htmlBody += '<br />' + this.generateLwcAssesment(relatedObjectMigrationResult.lwcAssessmentInfos);
94-
95-
// Add rollback flags report if there are any enabled flags
96-
const enabledFlags = omnistudioOrgDetails.rollbackFlags || [];
97-
if (enabledFlags.length > 0) {
98-
htmlBody += '<br />' + this.generateRollbackFlagsReport(enabledFlags);
99-
}
100-
101-
const doc = this.generateDocument(htmlBody);
102-
const fileUrl = process.cwd() + '/migrationresults.html';
103-
fs.writeFileSync(fileUrl, doc);
104-
await open('file://' + fileUrl);
105-
}
106-
107-
private static generateDocument(resultsAsHtml: string): string {
108-
const document = `
109-
<html>
110-
<head>
111-
<title>OmniStudio Migration Results</title>
112-
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/design-system/2.17.5/styles/salesforce-lightning-design-system.min.css" />
113-
</head>
114-
<body>
115-
<div style="margin: 20px;">
116-
<div class="slds-text-heading_large">OmniStudio Migration Results</div>
117-
${resultsAsHtml}
118-
</div>
119-
</div>
120-
</body>
121-
</html>
122-
`;
123-
return document;
124-
}
125-
126-
private static generateResult(migrationResult: MigratedObject, instanceUrl: string): string {
127-
let errorsBody = '';
128-
let tableBody = '';
129-
130-
if (migrationResult.errors) {
131-
errorsBody = migrationResult.errors
132-
.map((e) => `<li class="slds-item slds-text-color_destructive">${e}</li>`)
133-
.join('');
134-
errorsBody = `
135-
<div style="margin-block: 15px">
136-
<ul class="slds-list_dotted">${errorsBody}</ul>
137-
</div>`;
138-
>>>>>>> 451bd60 (Comments addressed)
13978
}
14079

14180
details.push({
@@ -409,7 +348,6 @@ export class ResultsBuilder {
409348
// this.generateReportForLwcResult(result.lwcAssessmentInfos, instanceUrl, orgDetails);
410349
}
411350

412-
<<<<<<< HEAD
413351
private static generateReportForApexResult(
414352
result: ApexAssessmentInfo[],
415353
instanceUrl: string,
@@ -588,7 +526,6 @@ export class ResultsBuilder {
588526
// </html>`;
589527
// fs.writeFileSync(resultsDir + '/' + lwcConstants.componentName + '.html', html);
590528
// }
591-
=======
592529
private static generateRollbackFlagsReport(enabledFlags: string[]): string {
593530
return `
594531
<div class="slds-box" style="background-color: white">
@@ -604,5 +541,4 @@ export class ResultsBuilder {
604541
</div>
605542
</div>`;
606543
}
607-
>>>>>>> 451bd60 (Comments addressed)
608544
}

0 commit comments

Comments
 (0)