review comments addressed

Author: tkotthakota-adobe

src/tasks/demo-url-processor/handler.js

@@ -35,13 +35,12 @@ function getImsTenantId(imsOrgId, organization, context, log) {
   } catch (error) {
     log.error('Error loading IMS_ORG_TENANT_ID_MAPPINGS mapping:', error.message);
   }
-
   if (!name) {
-    log.error('Organization name is missing, using fallback tenant ID');
-    return 'unknown-tenant';
+    log.error('Organization name is missing, using default tenant ID');
+    return context.env.DEFAULT_TENANT_ID;
+  } else {
+    return name.toLowerCase().replace(/\s+/g, '');
   }
-
-  return name.toLowerCase().replace(/\s+/g, '');
 }
 
 /**

src/tasks/opportunity-status-processor/handler.js

@@ -24,8 +24,10 @@ function getOpportunityTitle(opportunityType) {
   const opportunityTitles = {
     cwv: 'Core Web Vitals',
     'meta-tags': 'SEO Meta Tags',
-    'broken-back-links': 'Broken Back Links',
-    'broken-links': 'Broken Links',
+    'broken-backlinks': 'Broken Backlinks',
+    'broken-internal-links': 'Broken Internal Links',
+    'alt-text': 'Alt Text',
+    sitemap: 'Sitemap',
   };
 
   // Check if the opportunity type exists in our map

test/tasks/demo-url-processor/demo-url-processor.test.js

@@ -160,11 +160,14 @@ describe('Demo URL Processor', () => {
         // name property is missing
       });
 
+      // Set default tenant ID
+      context.env.DEFAULT_TENANT_ID = 'default-tenant';
+
       await runDemoUrlProcessor(message, context);
 
       // Should log error about missing name and use fallback
-      expect(context.log.error.calledWith('Organization name is missing, using fallback tenant ID')).to.be.true;
-      const expectedDemoUrl = 'https://example.com?organizationId=test-org-id#/@unknown-tenant/sites-optimizer/sites/test-site-id/home';
+      expect(context.log.error.calledWith('Organization name is missing, using default tenant ID')).to.be.true;
+      const expectedDemoUrl = 'https://example.com?organizationId=test-org-id#/@default-tenant/sites-optimizer/sites/test-site-id/home';
       expect(context.log.info.calledWith(`Setup complete! Access your demo environment here: ${expectedDemoUrl}`)).to.be.true;
     });
   });