@salesforcedevs/sfdocs-e2e 1.0.7-liquid-alpha.2 → 1.0.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforcedevs/sfdocs-e2e",
-  "version": "1.0.7-liquid-alpha.2",
+  "version": "1.0.7",
   "description": "E2E test suite for SFDocs static portal pages (SSG)",
   "license": "ISC",
   "bin": {

package/src/shared/specs/liquid.test.ts

@@ -7,13 +7,26 @@ describe('Liquid Template Rendering', () => {
     before(async () => {
         try {
             page = new GuidePage('/docs/liquid-project/guide/liquid-quickstart.html');
-            await browser.url(page.path);
+            page.open();
             await page.waitPageLoad();
 
             const content = await $('>>>doc-content-layout');
             const text = await content.getText();
-
-            liquidEnabled = text.includes('Hello from data.yaml');
+            const html = await content.getHTML();
+
+            const hasLiquidOutput = text.includes('Hello from data.yaml');
+            const hasRawSyntax =
+                html.includes('{{ data.greeting }}') ||
+                html.includes('{{ topic }}') ||
+                (html.includes('{{') && html.includes('{%'));
+
+            if (hasRawSyntax) {
+                liquidEnabled = false;
+            } else if (hasLiquidOutput) {
+                liquidEnabled = true;
+            } else {
+                liquidEnabled = false;
+            }
         } catch (e) {
             liquidEnabled = false;
         }
@@ -22,7 +35,7 @@ describe('Liquid Template Rendering', () => {
     beforeEach(async () => {
         if (liquidEnabled) {
             page = new GuidePage('/docs/liquid-project/guide/liquid-quickstart.html');
-            await browser.url(page.path);
+            page.open();
             await page.waitPageLoad();
         }
     });
@@ -37,6 +50,10 @@ describe('Liquid Template Rendering', () => {
 
         expect(text.includes('GitHub for writers')).toBeTruthy();
         expect(text.includes('Hello from data.yaml')).toBeTruthy();
+
+        const html = await content.getHTML();
+        expect(html.includes('{{ topic }}')).toBeFalsy();
+        expect(html.includes('{{ data.greeting }}')).toBeFalsy();
     });
 
     it('Should render included templates from shared directory', async function () {
@@ -59,14 +76,19 @@ describe('Liquid Template Rendering', () => {
         }
 
         const content = await $('>>>doc-content-layout');
+        const html = await content.getHTML();
         const text = await content.getText();
 
-        expect(text.includes('AccountId')).toBeTruthy();
-        expect(text.includes('Type:')).toBeTruthy();
-        expect(text.includes('Properties:')).toBeTruthy();
+        expect(text.includes('create()')).toBeTruthy();
+        expect(text.includes('query()')).toBeTruthy();
+        expect(text.includes('update()')).toBeTruthy();
 
         const listItems = await content.$$('li');
         expect(listItems.length).toBeGreaterThan(5);
+
+        expect(html.includes('{% for')).toBeFalsy();
+        expect(html.includes('{%- for')).toBeFalsy();
+        expect(html.includes('{% endfor')).toBeFalsy();
     });
 
     it('Should not process Liquid syntax inside code blocks', async function () {
@@ -75,14 +97,15 @@ describe('Liquid Template Rendering', () => {
         }
 
         const content = await $('>>>doc-content-layout');
-        const codeBlock = await content.$('dx-code-block');
+        const html = await content.getHTML();
+        const codeBlock = await content.$('code');
 
         expect(await codeBlock.isExisting()).toBeTruthy();
 
-        const codeBlockAttr = await codeBlock.getAttribute('code-block');
-        expect(codeBlockAttr.includes('{{ data.greeting }}')).toBeTruthy();
-        expect(codeBlockAttr.includes('{{ topic }}')).toBeTruthy();
-        expect(codeBlockAttr.includes('{% for item in items %}')).toBeTruthy();
+        const codeHtml = await codeBlock.getHTML();
+        expect(codeHtml.includes('{{ data.greeting }}')).toBeTruthy();
+        expect(codeHtml.includes('{{ topic }}')).toBeTruthy();
+        expect(codeHtml.includes('{% for item in items %}')).toBeTruthy();
 
         const text = await content.getText();
         expect(text.includes('Visit')).toBeTruthy();
@@ -108,66 +131,7 @@ describe('Liquid Template Rendering', () => {
         expect(html.includes('{{ id2 }}')).toBeFalsy();
     });
 
-    it('Should render englishonly block content in production', async function () {
-        if (!liquidEnabled) {
-            this.skip();
-        }
-
-        const content = await $('>>>doc-content-layout');
-        const text = await content.getText();
-        const html = await content.getHTML();
-
-        expect(text.includes('This feature is currently only available in English')).toBeTruthy();
-        expect(html.includes('{% englishonly')).toBeFalsy();
-        expect(html.includes('{% endenglishonly')).toBeFalsy();
-    });
-
-    it('Should render liquid.data string format — single YAML file on root context', async function () {
-        if (!liquidEnabled) {
-            this.skip();
-        }
-
-        const content = await $('>>>doc-content-layout');
-        const text = await content.getText();
-
-        expect(text.includes('Hello from data.yaml')).toBeTruthy();
-    });
-
-    it('Should render liquid.data object format — multiple YAML files each under a named key', async function () {
-        if (!liquidEnabled) {
-            this.skip();
-        }
-
-        const multiDataPage = new GuidePage('/docs/liquid-project/guide/multi-data.html');
-        await browser.url(multiDataPage.path);
-        await multiDataPage.waitPageLoad();
-
-        const content = await $('>>>doc-content-layout');
-        const text = await content.getText();
-        const html = await content.getHTML();
-
-        expect(text.includes('Sample data file to check the multi-include')).toBeTruthy();
-        expect(html.includes('{{ two.summary }}')).toBeFalsy();
-        expect(html.includes('{{ data.one.summary }}')).toBeFalsy();
-    });
-
-    it('Should render the englishonly topic page', async function () {
-        if (!liquidEnabled) {
-            this.skip();
-        }
-
-        const englishOnlyPage = new GuidePage('/docs/liquid-project/guide/english-only-topic.html');
-        await browser.url(englishOnlyPage.path);
-        await englishOnlyPage.waitPageLoad();
-
-        const content = await $('>>>doc-content-layout');
-        const text = await content.getText();
-
-        expect(text.includes('English-Only Topic')).toBeTruthy();
-        expect(text.includes('filters: [englishonly]')).toBeTruthy();
-    });
-
-    it('Should not render raw Liquid syntax when Liquid processing is disabled', async function () {
+    it('Should not process Liquid when frontmatter has liquid.enabled as false', async function () {
         if (!liquidEnabled) {
             this.skip();
         }
@@ -175,7 +139,7 @@ describe('Liquid Template Rendering', () => {
         const disabledPage = new GuidePage(
             '/docs/liquid-project/guide/liquid-learning-resources.html',
         );
-        await browser.url(disabledPage.path);
+        disabledPage.open();
         await disabledPage.waitPageLoad();
 
         const content = await $('>>>doc-content-layout');

package/static-dsc-portal-wdio.conf.js

@@ -18,7 +18,7 @@ exports.config = {
 
     // Exclude tests that require dynamic server features or pages not in static build
     exclude: [
-        // './src/shared/specs/liquid.test.ts',
+        './src/shared/specs/liquid.test.ts',
         './src/portal/specs/docs.api.test.ts',
         './src/portal/specs/docs.test.ts',
         './src/shared/specs/banner.test.ts',

package/static-portal-wdio.conf.js

@@ -18,8 +18,7 @@ exports.config = {
 
     // Exclude tests that require dynamic server features or pages not in static build
     exclude: [
-        // Advanced liquid tests require previewFuture, variables — not in static build
-        './src/shared/specs/liquid-advanced.test.ts',
+        './src/shared/specs/liquid.test.ts',
         // /docs/apis landing page doesn't exist in static builds
         './src/portal/specs/docs.api.test.ts',
         // /docs landing page doesn't exist in static builds

package/src/shared/specs/liquid-advanced.test.ts

@@ -1,71 +0,0 @@
-import GuidePage from '../pageobjects/guide.page';
-
-let page: GuidePage;
-let liquidEnabled = false;
-
-describe('Liquid Template Rendering — Advanced', () => {
-    before(async () => {
-        try {
-            page = new GuidePage('/docs/liquid-project/guide/liquid-quickstart.html');
-            await browser.url(page.path);
-            await page.waitPageLoad();
-
-            const content = await $('>>>doc-content-layout');
-            const text = await content.getText();
-
-            liquidEnabled = text.includes('Hello from data.yaml');
-        } catch (e) {
-            liquidEnabled = false;
-        }
-    });
-
-    beforeEach(async () => {
-        if (liquidEnabled) {
-            page = new GuidePage('/docs/liquid-project/guide/liquid-quickstart.html');
-            await browser.url(page.path);
-            await page.waitPageLoad();
-        }
-    });
-
-    it('Should render repository-level variables from content-config.yml', async function () {
-        if (!liquidEnabled) {
-            this.skip();
-        }
-
-        const content = await $('>>>doc-content-layout');
-        const text = await content.getText();
-        const html = await content.getHTML();
-
-        expect(text.includes('Salesforce Platform')).toBeTruthy();
-        expect(text.includes('62.0')).toBeTruthy();
-        expect(text.includes('sf')).toBeTruthy();
-        expect(html.includes('{{ variables.platform')).toBeFalsy();
-    });
-
-    it('Should render future block content when previewFuture is enabled', async function () {
-        if (!liquidEnabled) {
-            this.skip();
-        }
-
-        const content = await $('>>>doc-content-layout');
-        const text = await content.getText();
-
-        expect(text.includes('Coming Soon')).toBeTruthy();
-    });
-
-    it('Should render the future topic page when previewFuture is enabled', async function () {
-        if (!liquidEnabled) {
-            this.skip();
-        }
-
-        const futurePage = new GuidePage('/docs/liquid-project/guide/future-topic.html');
-        await browser.url(futurePage.path);
-        await futurePage.waitPageLoad();
-
-        const content = await $('>>>doc-content-layout');
-        const text = await content.getText();
-
-        expect(text.includes('Future Feature')).toBeTruthy();
-        expect(text.includes('previewFuture')).toBeTruthy();
-    });
-});