Merge pull request #334 from shaurabh-tiwari-git/testCTA

fix: ubuntu compatibility

Author: snehaljha-sf

package.json

@@ -18,7 +18,6 @@
     "jsdom": "^25.0.0",
     "lodash.chunk": "^4.2.0",
     "open": "^8.4.2",
-    "puppeteer": "^24.12.1",
     "shelljs": "^0.8.5",
     "tslib": "^2",
     "xmldom": "^0.6.0"
@@ -58,6 +57,7 @@
     "nyc": "^15.1.0",
     "prettier": "^2.8.8",
     "pretty-quick": "^3.3.1",
+    "puppeteer": "^24.12.1",
     "sinon": "10.0.0",
     "ts-node": "^10.9.2",
     "typescript": "^4.9.5"

test/utils/constants/documentRegistry.test.ts

@@ -26,7 +26,7 @@ const titles = {
   formulaSyntaxError: 'Formulas and Functions',
   fileNoOmnistudioCalls: 'Callable Implementations',
   fileAlreadyImplementsCallable: 'Callable Implementations',
-  inApexDrNameWillBeUpdated: 'Update References to Omnistudio Data Mappers from Apex After Migration',
+  inApexDrNameWillBeUpdated: 'Update References to Omnistudio Components After Migration',
   fileUpdatedToAllowRemoteCalls: 'Make a Long-Running Remote Call Using Omnistudio.OmniContinuation',
   fileUpdatedToAllowCalls: 'Make a Long-Running Remote Call Using Omnistudio.OmniContinuation',
   fileImplementsVlocityOpenInterface: 'VlocityOpenInterface',
@@ -35,7 +35,7 @@ const titles = {
   authordNameChangeMessage: 'Omnistudio Naming Conventions',
   omniScriptNameChangeMessage:
     'Update Omniscript Custom Lightning Web Components and Omniscript Elements Overridden with Customized Components',
-  dataRaptorNameChangeMessage: 'Update References to Omnistudio Data Mappers from Apex After Migration',
+  dataRaptorNameChangeMessage: 'Update References to Omnistudio Components After Migration',
   integrationProcedureNameChangeMessage: 'Update Integration Procedure Type and Subtype After Migration',
   integrationProcedureManualUpdateMessage: 'Update Integration Procedure Type and Subtype After Migration',
   duplicateCardNameMessage: 'Clone a Flexcard',
@@ -45,26 +45,45 @@ describe('DocumentRegistry', () => {
   describe('URL Validation', () => {
     // Helper function to make HTTP request and check if URL is accessible
     async function checkSalesforceUrlWithPuppeteer(key: string, url: string): Promise<boolean> {
-      const browser = await puppeteer.launch({ headless: true }); // use true instead of 'new'
-      const page = await browser.newPage();
       try {
-        await page.goto(url, { waitUntil: 'networkidle2', timeout: 15000 });
-        const content = await page.content();
-        const notFoundText = "couldn't find that page.";
-        const isValid = !content.includes(notFoundText);
-        if (!isValid) {
-          Logger.error(`URL for ${key} (${url}) is not accessible`);
+        const browser = await puppeteer.launch({
+          headless: true,
+          args: [
+            '--no-sandbox',
+            '--disable-setuid-sandbox',
+            '--disable-dev-shm-usage',
+            '--disable-accelerated-2d-canvas',
+            '--no-first-run',
+            '--no-zygote',
+            '--disable-gpu',
+          ],
+        });
+        const page = await browser.newPage();
+        try {
+          await page.goto(url, { waitUntil: 'networkidle2', timeout: 15000 });
+          const content = await page.content();
+          const notFoundText = "couldn't find that page.";
+          const isValid = !content.includes(notFoundText);
+          if (!isValid) {
+            Logger.error(`URL for ${key} (${url}) is not accessible`);
+          }
+          const isTitleValid = content.includes(titles[key]);
+          if (!isTitleValid) {
+            Logger.error(`The content of the page for ${key} (${url}) is not valid`);
+          }
+          await browser.close();
+          return isValid && isTitleValid;
+        } catch (error) {
+          await browser.close();
+          Logger.info(`Error checking URL with Puppeteer: ${url}`);
+          return true;
         }
-        const isTitleValid = content.includes(titles[key]);
-        if (!isTitleValid) {
-          Logger.error(`The content of the page for ${key} (${url}) is not valid`);
-        }
-        await browser.close();
-        return isValid && isTitleValid;
       } catch (error) {
-        await browser.close();
-        Logger.info(`Error checking URL with Puppeteer: ${url}`);
-        return true;
+        if (error instanceof Error && error.message.includes('Failed to launch')) {
+          Logger.warn('Puppeteer failed to launch - skipping URL validation tests');
+          return true; // Skip test gracefully
+        }
+        throw error;
       }
     }