fix(apextests): add error handling to replace

This is probably an incorrect fix, will fix as more details emerge. Just return the original one in case replacement fails
dxatscale · Sep 14, 2023 · a08d135 · a08d135
1 parent 56644a6
commit a08d135
Showing 1 changed file with 13 additions and 2 deletions.
diff --git a/packages/core/src/apextest/TriggerApexTests.ts b/packages/core/src/apextest/TriggerApexTests.ts
@@ -457,20 +457,31 @@ export default class TriggerApexTests {
     private fixBadNamespaceClassFullNames(testResult: any): any {
         let modifiedTestResult = _.cloneDeep(testResult);
 
+        try
+        {
         modifiedTestResult.tests = modifiedTestResult.tests.map((test) => {
             return {
                 ...test,
                 ...{
-                    fullName: test.fullName.replace('__', '.'),
+                    fullName: test.fullName?.replace('__', '.'),
                     apexClass: {
                         ...test.apexClass,
                         ...{
-                            fullName: test.apexClass.fullName.replace('__', '.'),
+                            fullName: test.apexClass?.fullName?.replace('__', '.'),
                         },
                     },
                 },
             };
         });
+        }catch(error)
+        {
+            SFPLogger.log(
+                `Unable to fix bad namespace class full names due to ${error}`,
+                LoggerLevel.DEBUG,
+                this.fileLogger
+            );
+            modifiedTestResult = _.cloneDeep(testResult);
+        }
 
         return modifiedTestResult;
     }