@testsmith/testblocks 0.8.9 → 0.9.1

package/dist/cli/executor.js

@@ -93,29 +93,39 @@ class TestExecutor {
         }
         // Create base context for hooks
         const baseContext = this.createBaseContext(testFile.variables);
+        // Check if there are any enabled tests
+        const enabledTests = testFile.tests.filter(t => !t.disabled);
+        const hasEnabledTests = enabledTests.length > 0;
+        // Add skipped results for disabled tests first
+        for (const test of testFile.tests) {
+            if (test.disabled) {
+                console.log(`  Skipping (disabled): ${test.name}`);
+                results.push({
+                    testId: test.id,
+                    testName: test.name,
+                    status: 'skipped',
+                    duration: 0,
+                    steps: [],
+                    error: { message: 'Test is disabled' },
+                    startedAt: new Date().toISOString(),
+                    finishedAt: new Date().toISOString(),
+                });
+            }
+        }
+        // Only run hooks and tests if there are enabled tests
+        if (!hasEnabledTests) {
+            console.log('  All tests disabled, skipping hooks');
+            await this.cleanup();
+            return results;
+        }
         try {
             // Run beforeAll hooks
             if (testFile.beforeAll) {
                 const steps = this.extractStepsFromBlocklyState(testFile.beforeAll);
                 await this.runSteps(steps, baseContext, 'beforeAll');
             }
-            // Run each test - pass baseContext variables so beforeAll state persists
-            for (const test of testFile.tests) {
-                // Skip disabled tests
-                if (test.disabled) {
-                    console.log(`  Skipping (disabled): ${test.name}`);
-                    results.push({
-                        testId: test.id,
-                        testName: test.name,
-                        status: 'skipped',
-                        duration: 0,
-                        steps: [],
-                        error: { message: 'Test is disabled' },
-                        startedAt: new Date().toISOString(),
-                        finishedAt: new Date().toISOString(),
-                    });
-                    continue;
-                }
+            // Run each enabled test - pass baseContext variables so beforeAll state persists
+            for (const test of enabledTests) {
                 // Check if test has data-driven sets
                 if (test.data && test.data.length > 0) {
                     // Run test for each data set
@@ -563,7 +573,7 @@ class TestExecutor {
         let screenshot;
         if (status === 'failed' && this.page) {
             try {
-                const buffer = await this.page.screenshot({ type: 'png' });
+                const buffer = await this.page.screenshot({ type: 'png', fullPage: true });
                 screenshot = `data:image/png;base64,${buffer.toString('base64')}`;
             }
             catch (screenshotError) {

package/dist/cli/index.js

@@ -447,7 +447,7 @@ program
                 'test:ci': 'testblocks run tests/**/*.testblocks.json -r console,html,junit -o reports',
             },
             devDependencies: {
-                '@testsmith/testblocks': '^0.8.9',
+                '@testsmith/testblocks': '^0.9.1',
             },
         };
         fs.writeFileSync(packagePath, JSON.stringify(packageJson, null, 2));

package/dist/cli/reporters/utils.d.ts

@@ -5,6 +5,10 @@
  * Generate timestamp string for filenames (e.g., 2024-01-15T14-30-45)
  */
 export declare function getTimestamp(): string;
+/**
+ * Strip ANSI escape codes from a string
+ */
+export declare function stripAnsi(str: string): string;
 /**
  * Escape XML special characters
  */

package/dist/cli/reporters/utils.js

@@ -4,6 +4,7 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getTimestamp = getTimestamp;
+exports.stripAnsi = stripAnsi;
 exports.escapeXml = escapeXml;
 exports.escapeHtml = escapeHtml;
 exports.formatStepType = formatStepType;
@@ -16,11 +17,18 @@ exports.formatStepOutput = formatStepOutput;
 function getTimestamp() {
     return new Date().toISOString().replace(/[:.]/g, '-').slice(0, 19);
 }
+/**
+ * Strip ANSI escape codes from a string
+ */
+function stripAnsi(str) {
+    // eslint-disable-next-line no-control-regex
+    return str.replace(/\x1B(?:[@-Z\\-_]|\[[0-?]*[ -/]*[@-~])/g, '');
+}
 /**
  * Escape XML special characters
  */
 function escapeXml(str) {
-    return str
+    return stripAnsi(str)
         .replace(/&/g, '&')
         .replace(/</g, '&lt;')
         .replace(/>/g, '&gt;')
@@ -31,7 +39,7 @@ function escapeXml(str) {
  * Escape HTML special characters
  */
 function escapeHtml(str) {
-    return str
+    return stripAnsi(str)
         .replace(/&/g, '&amp;')
         .replace(/</g, '&lt;')
         .replace(/>/g, '&gt;')