npm - @testsmith/testblocks - Versions diffs - 0.9.0 → 0.9.2 - Mend

@testsmith/testblocks 0.9.0 → 0.9.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (22) hide show

package/dist/cli/executor.js +29 -18
package/dist/cli/index.js +9 -2
package/dist/cli/reporters/HTMLReporter.js +21 -8
package/dist/cli/reporters/JUnitReporter.js +16 -8
package/dist/cli/reporters/types.d.ts +1 -0
package/dist/client/assets/{index-Ivy7T1Qk.js → index-CVn_B7zc.js} +71 -71
package/dist/client/assets/{index-Ivy7T1Qk.js.map → index-CVn_B7zc.js.map} +1 -1
package/dist/client/index.html +1 -1
package/dist/core/blocks/playwright/assertions.js +10 -20
package/dist/core/blocks/playwright/interactions.js +13 -15
package/dist/core/blocks/playwright/navigation.js +2 -4
package/dist/core/blocks/playwright/retrieval.js +3 -6
package/dist/core/blocks/playwright/types.d.ts +4 -0
package/dist/core/blocks/playwright/utils.d.ts +4 -0
package/dist/core/blocks/playwright/utils.js +8 -0
package/dist/core/types.d.ts +1 -0
package/dist/server/executor.js +30 -17
package/dist/server/globals.d.ts +4 -0
package/dist/server/globals.js +7 -0
package/dist/server/index.js +4 -2
package/dist/server/startServer.js +6 -4
package/package.json +1 -1

package/dist/client/index.html CHANGED Viewed

@@ -16,7 +16,7 @@
         overflow: hidden;
       }
     </style>
-    <script type="module" crossorigin src="/assets/index-Ivy7T1Qk.js"></script>
+    <script type="module" crossorigin src="/assets/index-CVn_B7zc.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/index-C5yUtTzz.css">
   </head>
   <body>

package/dist/core/blocks/playwright/assertions.js CHANGED Viewed

@@ -14,7 +14,6 @@ exports.assertionBlocks = [
         tooltip: 'Assert that an element is visible (auto-waits)',
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -22,7 +21,7 @@ exports.assertionBlocks = [
             const page = context.page;
             const selector = (0, utils_1.resolveSelector)(params, context);
             const displaySelector = (0, utils_1.getDisplaySelector)(params, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const expect = await (0, utils_1.getExpect)();
             const locator = page.locator(selector);
             context.logger.info(`Asserting ${displaySelector} is visible`);
@@ -43,7 +42,6 @@ exports.assertionBlocks = [
         tooltip: 'Assert that an element is not visible (auto-waits)',
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -51,7 +49,7 @@ exports.assertionBlocks = [
             const page = context.page;
             const selector = (0, utils_1.resolveSelector)(params, context);
             const displaySelector = (0, utils_1.getDisplaySelector)(params, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const expect = await (0, utils_1.getExpect)();
             const locator = page.locator(selector);
             context.logger.info(`Asserting ${displaySelector} is not visible`);
@@ -73,7 +71,6 @@ exports.assertionBlocks = [
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
             { name: 'TEXT', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -82,7 +79,7 @@ exports.assertionBlocks = [
             const selector = (0, utils_1.resolveSelector)(params, context);
             const displaySelector = (0, utils_1.getDisplaySelector)(params, context);
             const expectedText = (0, utils_1.resolveVariables)(params.TEXT, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const expect = await (0, utils_1.getExpect)();
             const locator = page.locator(selector);
             context.logger.info(`Asserting ${displaySelector} contains "${expectedText}"`);
@@ -104,7 +101,6 @@ exports.assertionBlocks = [
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
             { name: 'TEXT', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -113,7 +109,7 @@ exports.assertionBlocks = [
             const selector = (0, utils_1.resolveSelector)(params, context);
             const displaySelector = (0, utils_1.getDisplaySelector)(params, context);
             const expectedText = (0, utils_1.resolveVariables)(params.TEXT, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const expect = await (0, utils_1.getExpect)();
             const locator = page.locator(selector);
             context.logger.info(`Asserting ${displaySelector} text equals "${expectedText}"`);
@@ -134,14 +130,13 @@ exports.assertionBlocks = [
         tooltip: 'Assert that current URL contains expected value (auto-waits)',
         inputs: [
             { name: 'TEXT', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
         execute: async (params, context) => {
             const page = context.page;
             const expectedText = (0, utils_1.resolveVariables)(params.TEXT, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const expect = await (0, utils_1.getExpect)();
             // Escape special regex characters and create a regex pattern
             const escapedText = expectedText.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
@@ -164,14 +159,13 @@ exports.assertionBlocks = [
         tooltip: 'Assert that page title contains expected value (auto-waits)',
         inputs: [
             { name: 'TEXT', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
         execute: async (params, context) => {
             const page = context.page;
             const expectedText = (0, utils_1.resolveVariables)(params.TEXT, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const expect = await (0, utils_1.getExpect)();
             // Escape special regex characters and create a regex pattern
             const escapedText = expectedText.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
@@ -193,7 +187,6 @@ exports.assertionBlocks = [
         tooltip: 'Assert that an element is enabled (auto-waits)',
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -201,7 +194,7 @@ exports.assertionBlocks = [
             const page = context.page;
             const selector = (0, utils_1.resolveSelector)(params, context);
             const displaySelector = (0, utils_1.getDisplaySelector)(params, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const expect = await (0, utils_1.getExpect)();
             const locator = page.locator(selector);
             context.logger.info(`Asserting ${displaySelector} is enabled`);
@@ -223,7 +216,6 @@ exports.assertionBlocks = [
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
             { name: 'EXPECTED', type: 'field', fieldType: 'checkbox', default: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -232,7 +224,7 @@ exports.assertionBlocks = [
             const selector = (0, utils_1.resolveSelector)(params, context);
             const displaySelector = (0, utils_1.getDisplaySelector)(params, context);
             const expected = params.EXPECTED;
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const expect = await (0, utils_1.getExpect)();
             const locator = page.locator(selector);
             context.logger.info(`Asserting ${displaySelector} is ${expected ? 'checked' : 'unchecked'}`);
@@ -261,7 +253,6 @@ exports.assertionBlocks = [
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
             { name: 'VALUE', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -270,7 +261,7 @@ exports.assertionBlocks = [
             const selector = (0, utils_1.resolveSelector)(params, context);
             const displaySelector = (0, utils_1.getDisplaySelector)(params, context);
             const expectedValue = (0, utils_1.resolveVariables)(params.VALUE, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const expect = await (0, utils_1.getExpect)();
             const locator = page.locator(selector);
             context.logger.info(`Asserting ${displaySelector} has value "${expectedValue}"`);
@@ -294,7 +285,6 @@ exports.assertionBlocks = [
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
             { name: 'VALUE', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -303,7 +293,7 @@ exports.assertionBlocks = [
             const selector = (0, utils_1.resolveSelector)(params, context);
             const displaySelector = (0, utils_1.getDisplaySelector)(params, context);
             const expectedValue = (0, utils_1.resolveVariables)(params.VALUE, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const expect = await (0, utils_1.getExpect)();
             const locator = page.locator(selector);
             context.logger.info(`Asserting ${displaySelector} value contains "${expectedValue}"`);

package/dist/core/blocks/playwright/interactions.js CHANGED Viewed

@@ -11,19 +11,23 @@ exports.interactionBlocks = [
         type: 'web_click',
         category: 'Web',
         color: '#E91E63',
-        tooltip: 'Click on an element (auto-waits for element)',
+        tooltip: 'Click on an element (auto-waits for element to be visible and stable)',
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
         execute: async (params, context) => {
             const page = context.page;
             const selector = (0, utils_1.resolveSelector)(params, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             context.logger.info(`Clicking: ${selector}`);
             const locator = page.locator(selector);
+            // Wait for element to be visible and stable before clicking
+            await locator.waitFor({ state: 'visible', timeout });
+            // Scroll into view to ensure element is in viewport
+            await locator.scrollIntoViewIfNeeded({ timeout });
+            // Click with retry on intercept (handles overlays/animations)
             await locator.click({ timeout });
             return {
                 _summary: params.SELECTOR,
@@ -40,7 +44,6 @@ exports.interactionBlocks = [
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
             { name: 'VALUE', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -49,7 +52,7 @@ exports.interactionBlocks = [
             const selector = (0, utils_1.resolveSelector)(params, context);
             const rawValue = params.VALUE;
             const value = (0, utils_1.resolveVariables)(rawValue, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             context.logger.info(`Filling ${selector} with "${value}"`);
             const locator = page.locator(selector);
             await locator.fill(value, { timeout });
@@ -71,7 +74,6 @@ exports.interactionBlocks = [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
             { name: 'TEXT', type: 'field', fieldType: 'text', required: true },
             { name: 'DELAY', type: 'field', fieldType: 'number', default: 50 },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -80,7 +82,7 @@ exports.interactionBlocks = [
             const selector = (0, utils_1.resolveSelector)(params, context);
             const text = (0, utils_1.resolveVariables)(params.TEXT, context);
             const delay = params.DELAY;
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             context.logger.info(`Typing "${text}" into ${selector}`);
             const locator = page.locator(selector);
             await locator.pressSequentially(text, { delay, timeout });
@@ -102,7 +104,6 @@ exports.interactionBlocks = [
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
             { name: 'KEY', type: 'field', fieldType: 'dropdown', options: [['Enter', 'Enter'], ['Tab', 'Tab'], ['Escape', 'Escape'], ['Backspace', 'Backspace'], ['ArrowUp', 'ArrowUp'], ['ArrowDown', 'ArrowDown'], ['ArrowLeft', 'ArrowLeft'], ['ArrowRight', 'ArrowRight']] },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -110,7 +111,7 @@ exports.interactionBlocks = [
             const page = context.page;
             const selector = (0, utils_1.resolveSelector)(params, context);
             const key = params.KEY;
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             context.logger.info(`Pressing ${key} on ${selector}`);
             const locator = page.locator(selector);
             await locator.press(key, { timeout });
@@ -130,7 +131,6 @@ exports.interactionBlocks = [
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
             { name: 'VALUE', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -138,7 +138,7 @@ exports.interactionBlocks = [
             const page = context.page;
             const selector = (0, utils_1.resolveSelector)(params, context);
             const value = (0, utils_1.resolveVariables)(params.VALUE, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             context.logger.info(`Selecting "${value}" in ${selector}`);
             const locator = page.locator(selector);
             await locator.selectOption(value, { timeout });
@@ -158,7 +158,6 @@ exports.interactionBlocks = [
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
             { name: 'ACTION', type: 'field', fieldType: 'dropdown', options: [['Check', 'check'], ['Uncheck', 'uncheck']] },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -166,7 +165,7 @@ exports.interactionBlocks = [
             const page = context.page;
             const selector = (0, utils_1.resolveSelector)(params, context);
             const action = params.ACTION;
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             context.logger.info(`${action === 'check' ? 'Checking' : 'Unchecking'} ${selector}`);
             const locator = page.locator(selector);
             if (action === 'check') {
@@ -190,14 +189,13 @@ exports.interactionBlocks = [
         tooltip: 'Hover over an element (auto-waits)',
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
         execute: async (params, context) => {
             const page = context.page;
             const selector = (0, utils_1.resolveSelector)(params, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             context.logger.info(`Hovering over ${selector}`);
             const locator = page.locator(selector);
             await locator.hover({ timeout });

package/dist/core/blocks/playwright/navigation.js CHANGED Viewed

@@ -40,7 +40,6 @@ exports.navigationBlocks = [
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
             { name: 'STATE', type: 'field', fieldType: 'dropdown', options: [['Visible', 'visible'], ['Hidden', 'hidden'], ['Attached', 'attached'], ['Detached', 'detached']] },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
@@ -48,7 +47,7 @@ exports.navigationBlocks = [
             const page = context.page;
             const selector = (0, utils_1.resolveSelector)(params, context);
             const state = params.STATE;
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             context.logger.info(`Waiting for ${selector} to be ${state}`);
             await page.waitForSelector(selector, { state, timeout });
             return {
@@ -66,14 +65,13 @@ exports.navigationBlocks = [
         tooltip: 'Wait for URL to match',
         inputs: [
             { name: 'URL', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         previousStatement: true,
         nextStatement: true,
         execute: async (params, context) => {
             const page = context.page;
             const url = (0, utils_1.resolveVariables)(params.URL, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             context.logger.info(`Waiting for URL to match: ${url}`);
             await page.waitForURL(url, { timeout });
             return {

package/dist/core/blocks/playwright/retrieval.js CHANGED Viewed

@@ -14,13 +14,12 @@ exports.retrievalBlocks = [
         tooltip: 'Get text content of an element (auto-waits)',
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         output: { type: 'String' },
         execute: async (params, context) => {
             const page = context.page;
             const selector = (0, utils_1.resolveSelector)(params, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const locator = page.locator(selector);
             await locator.waitFor({ state: 'visible', timeout });
             const text = await locator.textContent({ timeout });
@@ -43,14 +42,13 @@ exports.retrievalBlocks = [
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
             { name: 'ATTRIBUTE', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         output: { type: 'String' },
         execute: async (params, context) => {
             const page = context.page;
             const selector = (0, utils_1.resolveSelector)(params, context);
             const attribute = params.ATTRIBUTE;
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const locator = page.locator(selector);
             await locator.waitFor({ state: 'attached', timeout });
             const value = await locator.getAttribute(attribute, { timeout });
@@ -72,13 +70,12 @@ exports.retrievalBlocks = [
         tooltip: 'Get current value of an input field (auto-waits)',
         inputs: [
             { name: 'SELECTOR', type: 'field', fieldType: 'text', required: true },
-            { name: 'TIMEOUT', type: 'field', fieldType: 'number', default: 30000 },
         ],
         output: { type: 'String' },
         execute: async (params, context) => {
             const page = context.page;
             const selector = (0, utils_1.resolveSelector)(params, context);
-            const timeout = params.TIMEOUT;
+            const timeout = (0, utils_1.getTimeout)(context);
             const locator = page.locator(selector);
             await locator.waitFor({ state: 'visible', timeout });
             const value = await locator.inputValue({ timeout });

package/dist/core/blocks/playwright/types.d.ts CHANGED Viewed

@@ -4,6 +4,7 @@
 export interface PlaywrightLocator {
     click(options?: {
         timeout?: number;
+        force?: boolean;
     }): Promise<void>;
     fill(value: string, options?: {
         timeout?: number;
@@ -51,6 +52,9 @@ export interface PlaywrightLocator {
         state?: 'attached' | 'detached' | 'visible' | 'hidden';
         timeout?: number;
     }): Promise<void>;
+    scrollIntoViewIfNeeded(options?: {
+        timeout?: number;
+    }): Promise<void>;
 }
 export interface PlaywrightPage {
     goto(url: string, options?: {

package/dist/core/blocks/playwright/utils.d.ts CHANGED Viewed

@@ -1,4 +1,8 @@
 import { ExecutionContext } from '../../types';
+/**
+ * Get the global web timeout from context
+ */
+export declare function getTimeout(context: ExecutionContext): number;
 export declare function getExpect(): Promise<any>;
 /**
  * Execute a web assertion with soft assertion support

package/dist/core/blocks/playwright/utils.js CHANGED Viewed

@@ -33,11 +33,19 @@ var __importStar = (this && this.__importStar) || (function () {
     };
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.getTimeout = getTimeout;
 exports.getExpect = getExpect;
 exports.executeWebAssertion = executeWebAssertion;
 exports.resolveVariables = resolveVariables;
 exports.resolveSelector = resolveSelector;
 exports.getDisplaySelector = getDisplaySelector;
+const DEFAULT_WEB_TIMEOUT = 30000;
+/**
+ * Get the global web timeout from context
+ */
+function getTimeout(context) {
+    return context.webTimeout ?? DEFAULT_WEB_TIMEOUT;
+}
 // Import Playwright's expect for assertions with auto-waiting
 // We use dynamic import with string concatenation to prevent Vite from
 // trying to bundle the playwright package (which is Node.js-only)

package/dist/core/types.d.ts CHANGED Viewed

@@ -106,6 +106,7 @@ export interface ExecutionContext {
     testIdAttribute?: string;
     softAssertions?: boolean;
     softAssertionErrors?: SoftAssertionError[];
+    webTimeout?: number;
 }
 export interface Logger {
     info(message: string, data?: unknown): void;

package/dist/server/executor.js CHANGED Viewed

@@ -196,7 +196,32 @@ class TestExecutor {
             plugins: this.plugins,
             testIdAttribute: this.options.testIdAttribute,
             procedures: mergedProcedures,
+            webTimeout: this.options.timeout,
         };
+        // 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) {
+                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) {
+            sharedContext.logger.info('All tests disabled, skipping hooks');
+            await this.cleanup();
+            return results;
+        }
         let beforeAllFailed = false;
         try {
             // Run beforeAll hooks
@@ -211,22 +236,8 @@ class TestExecutor {
             }
             // Only run tests if beforeAll succeeded
             if (!beforeAllFailed) {
-                // Run each test with beforeEach/afterEach
-                for (const test of testFile.tests) {
-                    // Skip disabled tests
-                    if (test.disabled) {
-                        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 with beforeEach/afterEach
+                for (const test of enabledTests) {
                     // Load data from file if specified
                     let testData = test.data;
                     if (test.dataFile && !testData) {
@@ -416,6 +427,7 @@ class TestExecutor {
             // Enable soft assertions if configured on the test
             softAssertions: test.softAssertions || false,
             softAssertionErrors: [],
+            webTimeout: this.options.timeout,
         };
         // Run beforeTest hooks
         for (const plugin of this.plugins.values()) {
@@ -507,6 +519,7 @@ class TestExecutor {
             // Enable soft assertions if configured on the test
             softAssertions: test.softAssertions || false,
             softAssertionErrors: [],
+            webTimeout: this.options.timeout,
         };
         // Inject data values into variables
         for (const [key, value] of Object.entries(dataSet.values)) {
@@ -720,7 +733,7 @@ class TestExecutor {
         if (status === 'failed' && isWebStep && context.page) {
             try {
                 const page = context.page;
-                const screenshotBuffer = await page.screenshot({ fullPage: false });
+                const screenshotBuffer = await page.screenshot({ fullPage: true });
                 screenshot = `data:image/png;base64,${screenshotBuffer.toString('base64')}`;
             }
             catch (screenshotErr) {

package/dist/server/globals.d.ts CHANGED Viewed

@@ -56,6 +56,10 @@ export declare function getGlobalProcedures(): Record<string, ProcedureDefinitio
  * Get the configured test ID attribute (defaults to 'data-testid')
  */
 export declare function getTestIdAttribute(): string;
+/**
+ * Get the configured global timeout in milliseconds (defaults to 30000)
+ */
+export declare function getGlobalTimeout(): number;
 /**
  * Set the test ID attribute and persist to globals.json
  */

package/dist/server/globals.js CHANGED Viewed

@@ -47,6 +47,7 @@ exports.getGlobals = getGlobals;
 exports.getGlobalVariables = getGlobalVariables;
 exports.getGlobalProcedures = getGlobalProcedures;
 exports.getTestIdAttribute = getTestIdAttribute;
+exports.getGlobalTimeout = getGlobalTimeout;
 exports.setTestIdAttribute = setTestIdAttribute;
 exports.discoverSnippets = discoverSnippets;
 exports.loadSnippet = loadSnippet;
@@ -118,6 +119,12 @@ function getGlobalProcedures() {
 function getTestIdAttribute() {
     return loadedGlobals.testIdAttribute || 'data-testid';
 }
+/**
+ * Get the configured global timeout in milliseconds (defaults to 30000)
+ */
+function getGlobalTimeout() {
+    return loadedGlobals.timeout || 30000;
+}
 /**
  * Set the test ID attribute and persist to globals.json
  */

package/dist/server/index.js CHANGED Viewed

@@ -384,7 +384,8 @@ app.post('/api/reports/html', (req, res) => {
             summary: {
                 totalTests: results.length,
                 passed: results.filter(r => r.status === 'passed').length,
-                failed: results.filter(r => r.status !== 'passed').length,
+                failed: results.filter(r => r.status !== 'passed' && r.status !== 'skipped').length,
+                skipped: results.filter(r => r.status === 'skipped').length,
                 duration: results.reduce((sum, r) => sum + r.duration, 0),
             },
             testFiles: [{
@@ -420,7 +421,8 @@ app.post('/api/reports/junit', (req, res) => {
             summary: {
                 totalTests: results.length,
                 passed: results.filter(r => r.status === 'passed').length,
-                failed: results.filter(r => r.status !== 'passed').length,
+                failed: results.filter(r => r.status !== 'passed' && r.status !== 'skipped').length,
+                skipped: results.filter(r => r.status === 'skipped').length,
                 duration: results.reduce((sum, r) => sum + r.duration, 0),
             },
             testFiles: [{

package/dist/server/startServer.js CHANGED Viewed

@@ -244,7 +244,7 @@ async function startServer(options = {}) {
             const testIdAttr = (0, globals_1.getTestIdAttribute)();
             const executor = new executor_1.TestExecutor({
                 headless: req.query.headless !== 'false',
-                timeout: Number(req.query.timeout) || 30000,
+                timeout: Number(req.query.timeout) || (0, globals_1.getGlobalTimeout)(),
                 variables: globalVars,
                 procedures: globalProcs,
                 testIdAttribute: testIdAttr,
@@ -294,7 +294,7 @@ async function startServer(options = {}) {
             const testIdAttr = (0, globals_1.getTestIdAttribute)();
             const executor = new executor_1.TestExecutor({
                 headless: req.query.headless !== 'false',
-                timeout: Number(req.query.timeout) || 30000,
+                timeout: Number(req.query.timeout) || (0, globals_1.getGlobalTimeout)(),
                 variables: globalVars,
                 procedures: globalProcs,
                 testIdAttribute: testIdAttr,
@@ -429,7 +429,8 @@ async function startServer(options = {}) {
                 summary: {
                     totalTests: results.length,
                     passed: results.filter(r => r.status === 'passed').length,
-                    failed: results.filter(r => r.status !== 'passed').length,
+                    failed: results.filter(r => r.status !== 'passed' && r.status !== 'skipped').length,
+                    skipped: results.filter(r => r.status === 'skipped').length,
                     duration: results.reduce((sum, r) => sum + r.duration, 0),
                 },
                 testFiles: [{
@@ -464,7 +465,8 @@ async function startServer(options = {}) {
                 summary: {
                     totalTests: results.length,
                     passed: results.filter(r => r.status === 'passed').length,
-                    failed: results.filter(r => r.status !== 'passed').length,
+                    failed: results.filter(r => r.status !== 'passed' && r.status !== 'skipped').length,
+                    skipped: results.filter(r => r.status === 'skipped').length,
                     duration: results.reduce((sum, r) => sum + r.duration, 0),
                 },
                 testFiles: [{

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@testsmith/testblocks",
-  "version": "0.9.0",
+  "version": "0.9.2",
   "description": "Visual test automation tool with Blockly - API and Playwright testing",
   "author": "Roy de Kleijn",
   "license": "MIT",