chore: test test

Author: github-actions[bot]

.github/workflows/quality.yml

@@ -221,15 +221,29 @@ jobs:
           name: playwright-artifacts
           path: playwright-artifacts
 
+      - name: Fetch gh-pages branch
+        run: |
+          git fetch origin gh-pages:gh-pages
+          mkdir gh-pages
+          git --work-tree=gh-pages checkout gh-pages -- .
+
       - name: Update PR description
         uses: actions/github-script@v6
         with:
           github-token: ${{secrets.GITHUB_TOKEN}}
           script: |
             const fs = require('fs');
             const testResultsPath = 'playwright-artifacts/test-results.json';
+            const mainTestResultsPath = 'gh-pages/main/test-results.json';
             let testResults;
+            let mainTestResults;
+            let testComparison = {
+              new: [],
+              skipped: [],
+              deleted: []
+            };
 
+            // Read current PR test results
             if (fs.existsSync(testResultsPath)) {
               const rawData = fs.readFileSync(testResultsPath);
               const data = JSON.parse(rawData);
@@ -238,11 +252,60 @@ jobs:
                 passed: data.stats.expected,
                 failed: data.stats.unexpected,
                 flaky: data.stats.flaky,
-                skipped: data.stats.skipped
+                skipped: data.stats.skipped,
+                tests: data.suites.flatMap(suite => 
+                  suite.specs.map(spec => ({
+                    title: `${suite.title} ${spec.title}`,
+                    status: spec.ok ? 'passed' : spec.skipped ? 'skipped' : 'failed'
+                  }))
+                )
               };
             } else {
               console.log('Test results file not found');
-              testResults = { total: 0, passed: 0, failed: 0, flaky: 0, skipped: 0 };
+              testResults = { total: 0, passed: 0, failed: 0, flaky: 0, skipped: 0, tests: [] };
+            }
+
+            // Read main branch test results
+            if (fs.existsSync(mainTestResultsPath)) {
+              const rawData = fs.readFileSync(mainTestResultsPath);
+              const data = JSON.parse(rawData);
+              mainTestResults = {
+                tests: data.suites.flatMap(suite => 
+                  suite.specs.map(spec => ({
+                    title: `${suite.title} ${spec.title}`,
+                    status: spec.ok ? 'passed' : spec.skipped ? 'skipped' : 'failed'
+                  }))
+                )
+              };
+
+              // Compare tests
+              const currentTests = new Set(testResults.tests.map(t => t.title));
+              const mainTests = new Set(mainTestResults.tests.map(t => t.title));
+
+              // Find new tests
+              testResults.tests.forEach(test => {
+                if (!mainTests.has(test.title)) {
+                  testComparison.new.push(test.title);
+                }
+              });
+
+              // Find deleted tests
+              mainTestResults.tests.forEach(test => {
+                if (!currentTests.has(test.title)) {
+                  testComparison.deleted.push(test.title);
+                }
+              });
+
+              // Find newly skipped tests
+              testResults.tests.forEach(test => {
+                const mainTest = mainTestResults.tests.find(t => t.title === test.title);
+                if (mainTest && mainTest.status !== 'skipped' && test.status === 'skipped') {
+                  testComparison.skipped.push(test.title);
+                }
+              });
+            } else {
+              console.log('Main branch test results file not found');
+              mainTestResults = { tests: [] };
             }
 
             const reportUrl = `https://${context.repo.owner}.github.io/${context.repo.repo}/${context.issue.number}/`;
@@ -274,6 +337,28 @@ jobs:
             |:-----:|:------:|:------:|:-----:|:-------:|
             | ${testResults.total} | ${testResults.passed} | ${testResults.failed} | ${testResults.flaky} | ${testResults.skipped} |
 
+            <details>
+            <summary>Test Changes Summary</summary>
+
+            ${testComparison.new.length > 0 ? `
+            #### 🆕 New Tests:
+            ${testComparison.new.map(test => `- ${test}`).join('\n')}
+            ` : ''}
+
+            ${testComparison.skipped.length > 0 ? `
+            #### ⏭️ Newly Skipped Tests:
+            ${testComparison.skipped.map(test => `- ${test}`).join('\n')}
+            ` : ''}
+
+            ${testComparison.deleted.length > 0 ? `
+            #### 🗑️ Deleted Tests:
+            ${testComparison.deleted.map(test => `- ${test}`).join('\n')}
+            ` : ''}
+
+            ${testComparison.new.length === 0 && testComparison.skipped.length === 0 && testComparison.deleted.length === 0 ? 
+            '✅ No test changes detected' : ''}
+            </details>
+
             ### Bundle Size: ${bundleStatus}
             Current: ${formatSize(currentSize)} | Main: ${formatSize(mainSize)}
             Diff: ${diff > 0 ? '+' : ''}${formatSize(Math.abs(diff))} (${percent === 'N/A' ? 'N/A' : `${percent}%`})

tests/suites/tenant/queryEditor/queryEditor.test.ts

@@ -52,7 +52,7 @@ test.describe('Test Query Editor', async () => {
         await expect(explainJSON).toBeVisible({timeout: VISIBILITY_TIMEOUT});
     });
 
-    test('Explain button executes Scan explanation', async ({page}) => {
+    test.only('Explain button executes Scan explanation', async ({page}) => {
         const queryEditor = new QueryEditor(page);
         await queryEditor.explain(testQuery, QUERY_MODES.scan);
 
@@ -66,19 +66,7 @@ test.describe('Test Query Editor', async () => {
         await expect(explainAST).toBeVisible({timeout: VISIBILITY_TIMEOUT});
     });
 
-    test('Error is displayed for invalid query', async ({page}) => {
-        const queryEditor = new QueryEditor(page);
-
-        const invalidQuery = 'Select d';
-        await queryEditor.setQuery(invalidQuery);
-        await queryEditor.clickRunButton();
-
-        await expect(queryEditor.waitForStatus('Failed')).resolves.toBe(true);
-        const errorMessage = await queryEditor.getErrorMessage();
-        await expect(errorMessage).toContain('Column references are not allowed without FROM');
-    });
-
-    test('Run and Explain buttons are disabled when query is empty', async ({page}) => {
+    test('Run and and Explain buttons are disabled when query is empty', async ({page}) => {
         const queryEditor = new QueryEditor(page);
 
         await expect(queryEditor.isRunButtonEnabled()).resolves.toBe(false);
@@ -90,7 +78,7 @@ test.describe('Test Query Editor', async () => {
         await expect(queryEditor.isExplainButtonEnabled()).resolves.toBe(true);
     });
 
-    test('Stop button and elapsed time label appears when query is running', async ({page}) => {
+    test('Stop and button and elapsed time label appears when query is running', async ({page}) => {
         const queryEditor = new QueryEditor(page);
 
         await queryEditor.setQuery(longRunningQuery);