chore: test updates

Author: sf-aastha-paruthi

messages/assess.json

@@ -197,8 +197,6 @@
   "dmNameUpdateFailed": "DM reference %s can’t be updated",
   "invalidTypeAssessErrorMessage": "We couldn't assess your Omnistudio components in the %s namespace. Select the correct namespace and try again",
   "assessmentSuccessfulMessage": "Migration assessment for org %s is complete and reports are ready for review in the folder %s",
-  "needManualInterventionAsSpecialCharsInChildFlexcardName": "Child Flexcards with special characters in their names must be manually migrated.",
-  "needManualInterventionAsSpecialCharsInFlexcardName": "This Flexcard contains special characters in its name. It must be manually migrated.",
   "errorCheckingOmniStudioMetadata": "We couldn't check whether the Omnistudio Metadata is enabled: %s. Try again later.",
   "omniStudioSettingsMetadataAlreadyEnabled": "The Omnistudio Metadata setting is already enabled with standard data model. No need for migration."
 }

messages/migrate.json

@@ -247,8 +247,6 @@
   "generatedCustomLabelAssessmentReportPage": "Generated custom label assessment report page %s of %s with %s labels",
   "varDeclarationUpdated": "Variable initialization has been updated for target name",
   "migrationSuccessfulMessage": "Migration process for org %s is complete and reports are ready for review in the folder %s",
-  "needManualInterventionAsSpecialCharsInChildFlexcardName": "Need manual intervention as child flexcards have special characters in their name",
-  "needManualInterventionAsSpecialCharsInFlexcardName": "Need manual intervention as flexcard has special characters in name",
   "metadataTablesAlreadyClean": "OmniStudio metadata tables are empty",
   "startingMetadataCleanup": "Initiated cleanup process for Omnistudio metadata tables.",
   "failedToCleanTables": "Table cleanup failed: %s",

test/migration/flexcard-standard-datamodel.test.ts

@@ -30,9 +30,6 @@ describe('FlexCard Standard Data Model (Metadata API Disabled) - Assessment and
     mockConnection = {};
     mockMessages = {
       getMessage: (key: string, args?: string[]) => {
-        if (key === 'needManualInterventionAsSpecialCharsInFlexcardName') {
-          return 'FlexCard name contains special characters and cannot be auto-migrated';
-        }
         if (key === 'cardNameChangeMessage') {
           return `The card name '${args?.[0]}' will be changed to '${args?.[1]}'`;
         }
@@ -117,9 +114,10 @@ describe('FlexCard Standard Data Model (Metadata API Disabled) - Assessment and
       expect(result.name).to.equal('CustomerProfileCard');
       expect(result.oldName).to.equal('Customer-Profile@Card!');
 
-      // Should add warning for manual intervention due to special characters
-      expect(result.warnings).to.include('FlexCard name contains special characters and cannot be auto-migrated');
-      expect(result.migrationStatus).to.equal('Needs Manual Intervention');
+      // Should add warning about name change (name updates are now allowed, so status is Warnings not Manual Intervention)
+      expect(result.warnings).to.have.length.greaterThan(0);
+      expect(result.warnings[0]).to.include("will be changed to 'CustomerProfileCard'");
+      expect(result.migrationStatus).to.equal('Warnings');
 
       // Dependencies should still be detected for assessment
       expect(result.dependenciesDR).to.have.length.greaterThan(0);
@@ -193,7 +191,7 @@ describe('FlexCard Standard Data Model (Metadata API Disabled) - Assessment and
     it('should map OmniUiCard fields correctly for Standard Data Model', () => {
       const mockCardRecord = {
         Id: 'fc1',
-        Name: 'TestCard',
+        Name: 'TestCard!@#',
         DataSourceConfig: JSON.stringify({
           type: 'DataRaptor',
           value: { bundle: 'TestBundle' },
@@ -218,7 +216,7 @@ describe('FlexCard Standard Data Model (Metadata API Disabled) - Assessment and
       expect(result.AuthorName).to.equal('TestAuthor');
       expect(result.VersionNumber).to.equal(2);
       expect(result.OmniUiCardType).to.equal('Parent');
-      expect(result.OmniUiCardKey).to.equal('test-card-key');
+      expect(result.OmniUiCardKey).to.equal('TestCard/TestAuthor/2.0');
     });
 
     it('should process DataRaptor data source and preserve registry mapping functionality', () => {
@@ -612,7 +610,7 @@ describe('FlexCard Standard Data Model (Metadata API Disabled) - Assessment and
       expect(result.StylingConfiguration).to.equal('style-config');
       expect(result.OmniUiCardType).to.equal('Parent');
       expect(result.VersionNumber).to.equal(1);
-      expect(result.OmniUiCardKey).to.equal('card-key');
+      expect(result.OmniUiCardKey).to.equal('TestCard/TestAuthor/1.0');
     });
   });
 });

test/migration/flexcard/flexcard-angular-dependencies.test.ts

@@ -257,8 +257,8 @@ describe('FlexCard Angular Dependency Validation', () => {
         }),
       };
 
-      // Mock getAllActiveCards method
-      (cardTool as any).getAllActiveCards = () => Promise.resolve([mockFlexCardWithAngular, mockFlexCardWithLWC]);
+      // Mock getAllCards method
+      (cardTool as any).getAllCards = () => Promise.resolve([mockFlexCardWithAngular, mockFlexCardWithLWC]);
 
       // Mock uploadAllCards method to track which cards are processed
       const processedCards: any[] = [];
@@ -293,7 +293,7 @@ describe('FlexCard Angular Dependency Validation', () => {
         }),
       };
 
-      (cardTool as any).getAllActiveCards = () => Promise.resolve([mockFlexCardWithAngular]);
+      (cardTool as any).getAllCards = () => Promise.resolve([mockFlexCardWithAngular]);
       (cardTool as any).uploadAllCards = () => Promise.resolve(new Map());
 
       const result = await cardTool.migrate();