@redhat-developer/page-objects 1.17.0 → 1.18.1

package/out/components/sidebar/extensions/ExtensionsViewSection.js

@@ -65,7 +65,8 @@ class ExtensionsViewSection extends ViewSection_1.ViewSection {
                 throw err;
             }
         }
-        await this.getDriver().wait(selenium_webdriver_1.until.elementIsNotVisible(progress));
+        // Wait for search to complete with a timeout to prevent hanging
+        await this.getDriver().wait(selenium_webdriver_1.until.elementIsNotVisible(progress), 30000);
         const parent = this.enclosingItem;
         const sectionTitle = this.getSectionForCategory(title);
         const section = (await parent.getSection(sectionTitle));
@@ -75,8 +76,16 @@ class ExtensionsViewSection extends ViewSection_1.ViewSection {
         }
         const extensions = await section.getVisibleItems();
         for (const extension of extensions) {
-            if ((await extension.getTitle()) === title) {
-                return extension;
+            try {
+                if ((await extension.getTitle()) === title) {
+                    return extension;
+                }
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
+                }
+                throw e;
             }
         }
         return undefined;
@@ -92,8 +101,14 @@ class ExtensionsViewSection extends ViewSection_1.ViewSection {
         try {
             await textField.findElement(ExtensionsViewSection.locators.ExtensionsViewSection.textField);
             await searchField.sendKeys(selenium_webdriver_1.Key.chord(ExtensionsViewItem_1.ExtensionsViewItem.ctlKey, 'a'), selenium_webdriver_1.Key.BACK_SPACE);
-            await this.getDriver().wait(selenium_webdriver_1.until.elementIsVisible(progress));
-            await this.getDriver().wait(selenium_webdriver_1.until.elementIsNotVisible(progress));
+            // Add timeouts to prevent indefinite waits
+            try {
+                await this.getDriver().wait(selenium_webdriver_1.until.elementIsVisible(progress), 2000);
+            }
+            catch {
+                // Progress indicator may not appear for quick operations
+            }
+            await this.getDriver().wait(selenium_webdriver_1.until.elementIsNotVisible(progress), 10000);
         }
         catch (err) {
             // do nothing, the text field is empty

package/out/components/sidebar/scm/NewScmView.js

@@ -70,8 +70,16 @@ class SingleScmProvider extends ScmView_1.ScmProvider {
             const titlePart = view.getTitlePart();
             const elements = await titlePart.findElements(ScmView_1.ScmView.locators.ScmView.action);
             for (const element of elements) {
-                const title = await element.getAttribute(ScmView_1.ScmView.locators.ScmView.actionLabel);
-                buttons.push(await new ViewTitlePart_1.TitleActionButton(ScmView_1.ScmView.locators.ScmView.actionConstructor(title), titlePart).wait());
+                try {
+                    const title = await element.getAttribute(ScmView_1.ScmView.locators.ScmView.actionLabel);
+                    buttons.push(await new ViewTitlePart_1.TitleActionButton(ScmView_1.ScmView.locators.ScmView.actionConstructor(title), titlePart).wait());
+                }
+                catch (e) {
+                    if (e.name === 'StaleElementReferenceError') {
+                        continue;
+                    }
+                    throw e;
+                }
             }
             names = await Promise.all(buttons.map(async (button) => button.getTitle()));
         }
@@ -108,9 +116,17 @@ class SingleScmProvider extends ScmView_1.ScmProvider {
             const depth = +(await header.getAttribute('aria-level')) + 1;
             const items = await this.findElements(NewScmView.locators.ScmView.itemLevel(depth));
             for (const item of items) {
-                const index = +(await item.getAttribute('data-index'));
-                if (index > startIndex && index <= startIndex + count) {
-                    elements.push(item);
+                try {
+                    const index = +(await item.getAttribute('data-index'));
+                    if (index > startIndex && index <= startIndex + count) {
+                        elements.push(item);
+                    }
+                }
+                catch (e) {
+                    if (e.name === 'StaleElementReferenceError') {
+                        continue;
+                    }
+                    throw e;
                 }
             }
         }
@@ -151,8 +167,16 @@ class MultiScmProvider extends ScmView_1.ScmProvider {
             const headers = await this.enclosingItem.findElements(locator);
             let header;
             for (const item of headers) {
-                if (+(await item.getAttribute('data-index')) > index) {
-                    header = item;
+                try {
+                    if (+(await item.getAttribute('data-index')) > index) {
+                        header = item;
+                    }
+                }
+                catch (e) {
+                    if (e.name === 'StaleElementReferenceError') {
+                        continue;
+                    }
+                    throw e;
                 }
             }
             if (!header) {
@@ -162,9 +186,17 @@ class MultiScmProvider extends ScmView_1.ScmProvider {
             const depth = +(await header.getAttribute('aria-level')) + 1;
             const items = await this.enclosingItem.findElements(NewScmView.locators.ScmView.itemLevel(depth));
             for (const item of items) {
-                const index = +(await item.getAttribute('data-index'));
-                if (index > startIndex && index <= startIndex + count) {
-                    elements.push(item);
+                try {
+                    const index = +(await item.getAttribute('data-index'));
+                    if (index > startIndex && index <= startIndex + count) {
+                        elements.push(item);
+                    }
+                }
+                catch (e) {
+                    if (e.name === 'StaleElementReferenceError') {
+                        continue;
+                    }
+                    throw e;
                 }
             }
         }
@@ -175,9 +207,17 @@ class MultiScmProvider extends ScmView_1.ScmProvider {
         const rows = await this.enclosingItem.findElements(locator);
         const index = +(await this.getAttribute('data-index'));
         for (const row of rows) {
-            if (+(await row.getAttribute('data-index')) > index) {
-                const count = await rows[0].findElement(ScmView_1.ScmChange.locators.ScmView.changeCount);
-                return +(await count.getText());
+            try {
+                if (+(await row.getAttribute('data-index')) > index) {
+                    const count = await rows[0].findElement(ScmView_1.ScmChange.locators.ScmView.changeCount);
+                    return +(await count.getText());
+                }
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
+                }
+                throw e;
             }
         }
         return 0;

package/out/components/sidebar/scm/ScmView.js

@@ -145,10 +145,18 @@ class ScmProvider extends AbstractElement_1.AbstractElement {
             let i = -1;
             elements = await this.findElements(ScmProvider.locators.ScmView.changeItem);
             for (const [index, item] of elements.entries()) {
-                const name = await item.findElement(ScmProvider.locators.ScmView.changeName);
-                if ((await name.getText()) === label) {
-                    i = index + 1;
-                    break;
+                try {
+                    const name = await item.findElement(ScmProvider.locators.ScmView.changeName);
+                    if ((await name.getText()) === label) {
+                        i = index + 1;
+                        break;
+                    }
+                }
+                catch (e) {
+                    if (e.name === 'StaleElementReferenceError') {
+                        continue;
+                    }
+                    throw e;
                 }
             }
             if (i < 0) {

package/out/components/sidebar/tree/TreeSection.js

@@ -57,9 +57,17 @@ class TreeSection extends ViewSection_1.ViewSection {
             if (i + 1 < path.length) {
                 currentItem = undefined;
                 for (const item of items) {
-                    if ((await item.getLabel()) === path[i + 1]) {
-                        currentItem = item;
-                        break;
+                    try {
+                        if ((await item.getLabel()) === path[i + 1]) {
+                            currentItem = item;
+                            break;
+                        }
+                    }
+                    catch (e) {
+                        if (e.name === 'StaleElementReferenceError') {
+                            continue;
+                        }
+                        throw e;
                     }
                 }
             }

package/out/components/sidebar/tree/custom/CustomTreeSection.js

@@ -42,8 +42,16 @@ class GenericCustomTreeSection extends TreeSection_1.TreeSection {
         const container = await this.getContainer();
         const elements = await container.findElements(CustomTreeSection.locators.CustomTreeSection.itemRow);
         for (const element of elements) {
-            if (await element.isDisplayed()) {
-                items.push(new this.itemConstructor(element, this));
+            try {
+                if (await element.isDisplayed()) {
+                    items.push(new this.itemConstructor(element, this));
+                }
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
+                }
+                throw e;
             }
         }
         return items;
@@ -52,11 +60,19 @@ class GenericCustomTreeSection extends TreeSection_1.TreeSection {
         const predicate = typeof labelOrPredicate === 'string' ? async (el) => (await el.getLabel()) === labelOrPredicate : labelOrPredicate;
         const elements = await this.getVisibleItems();
         for (const element of elements) {
-            if (await predicate(element)) {
-                const level = +(await element.getAttribute(CustomTreeSection.locators.ViewSection.level));
-                if (maxLevel < 1 || level <= maxLevel) {
-                    return element;
+            try {
+                if (await predicate(element)) {
+                    const level = +(await element.getAttribute(CustomTreeSection.locators.ViewSection.level));
+                    if (maxLevel < 1 || level <= maxLevel) {
+                        return element;
+                    }
+                }
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
                 }
+                throw e;
             }
         }
         return undefined;

package/out/components/sidebar/tree/debug/WatchSection.js

@@ -30,7 +30,11 @@ class WatchSection extends CustomTreeSection_1.GenericCustomTreeSection {
      */
     async addItem(name) {
         await (await this.getAction(WatchSection.locators.WatchSection.addExpression))?.click();
-        await new Promise((res) => setTimeout(res, 1000));
+        // Wait for input field to appear
+        await this.getWaitHelper().forCondition(async () => {
+            const inputs = await this.findElements(WatchSection.locators.WatchSection.input);
+            return inputs.length > 0;
+        }, { timeout: 3000, pollInterval: 100 });
         const textInput = await this.findElement(WatchSection.locators.WatchSection.input);
         await textInput.clear();
         await textInput.sendKeys(name + selenium_webdriver_1.Key.ENTER);

package/out/components/sidebar/tree/default/DefaultTreeSection.js

@@ -28,7 +28,15 @@ class DefaultTreeSection extends TreeSection_1.TreeSection {
         const items = [];
         const elements = await this.findElements(DefaultTreeSection.locators.DefaultTreeSection.itemRow);
         for (const element of elements) {
-            items.push(await new DefaultTreeItem_1.DefaultTreeItem(element, this).wait());
+            try {
+                items.push(await new DefaultTreeItem_1.DefaultTreeItem(element, this).wait());
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
+                }
+                throw e;
+            }
         }
         return items;
     }

package/out/components/workbench/Notification.js

@@ -97,8 +97,16 @@ class Notification extends ElementWithContextMenu_1.ElementWithContextMenu {
         const buttons = [];
         const elements = await this.findElement(Notification.locators.Notification.actions).findElements(Notification.locators.Notification.action);
         for (const button of elements) {
-            const title = await Notification.locators.Notification.actionLabel.value(button);
-            buttons.push(await new NotificationButton(Notification.locators.Notification.buttonConstructor(title), this).wait());
+            try {
+                const title = await Notification.locators.Notification.actionLabel.value(button);
+                buttons.push(await new NotificationButton(Notification.locators.Notification.buttonConstructor(title), this).wait());
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
+                }
+                throw e;
+            }
         }
         return buttons;
     }

package/out/components/workbench/NotificationsCenter.js

@@ -61,9 +61,17 @@ class NotificationsCenter extends AbstractElement_1.AbstractElement {
         const notifications = [];
         const elements = await this.findElements(NotificationsCenter.locators.NotificationsCenter.row);
         for (const element of elements) {
-            const not = new Notification_1.CenterNotification(element);
-            if (type === Notification_1.NotificationType.Any || (await not.getType()) === type) {
-                notifications.push(await not.wait());
+            try {
+                const not = new Notification_1.CenterNotification(element);
+                if (type === Notification_1.NotificationType.Any || (await not.getType()) === type) {
+                    notifications.push(await not.wait());
+                }
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
+                }
+                throw e;
             }
         }
         return notifications;

package/out/components/workbench/Workbench.js

@@ -88,7 +88,15 @@ class Workbench extends AbstractElement_1.AbstractElement {
         }
         const elements = await container.findElements(Workbench.locators.Workbench.notificationItem);
         for (const element of elements) {
-            notifications.push(await new Notification_1.StandaloneNotification(element).wait());
+            try {
+                notifications.push(await new Notification_1.StandaloneNotification(element).wait());
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
+                }
+                throw e;
+            }
         }
         return notifications;
     }
@@ -105,11 +113,13 @@ class Workbench extends AbstractElement_1.AbstractElement {
      * @returns promise that resolves to a SettingsEditor instance
      */
     async openSettings() {
-        await this.executeCommand('open user settings');
+        await this.executeCommand('Preferences: Open User Settings');
         await new EditorView_1.EditorView().openEditor('Settings');
         await Workbench.driver.wait(selenium_webdriver_1.until.elementLocated(Workbench.locators.Editor.constructor));
-        await new Promise((res) => setTimeout(res, 500));
-        return new SettingsEditor_1.SettingsEditor();
+        // Wait for settings editor to fully load
+        const editor = new SettingsEditor_1.SettingsEditor();
+        await this.getWaitHelper().forStable(editor, { timeout: 2000 });
+        return editor;
     }
     /**
      * Open the VS Code command line prompt

package/out/components/workbench/input/Input.js

@@ -50,7 +50,12 @@ class Input extends AbstractElement_1.AbstractElement {
         }
         const input = await this.findElement(Input.locators.Input.inputBox).findElement(Input.locators.Input.input);
         await this.clear();
-        await new Promise((res) => setTimeout(res, 200));
+        // Wait for clear operation to complete
+        await this.getWaitHelper()
+            .forCondition(async () => (await this.getText()).length === 0, { timeout: 1000, pollInterval: 50 })
+            .catch(() => {
+            /* Text may not be empty if clear didn't work fully */
+        });
         if ((await this.getText()).length > 0) {
             await input.sendKeys(selenium_webdriver_1.Key.END, selenium_webdriver_1.Key.chord(selenium_webdriver_1.Key.SHIFT, selenium_webdriver_1.Key.HOME));
         }
@@ -152,21 +157,29 @@ class Input extends AbstractElement_1.AbstractElement {
         while (!endReached) {
             const picks = await this.getQuickPicks();
             for (const pick of picks) {
-                const lastRow = await this.findElements(Input.locators.DefaultTreeSection.lastRow);
-                if (lastRow.length > 0) {
-                    endReached = true;
-                }
-                else if ((await pick.getAttribute('aria-posinset')) === (await pick.getAttribute('aria-setsize'))) {
-                    endReached = true;
-                }
-                if (typeof indexOrText === 'string') {
-                    const text = await pick.getLabel();
-                    if (text.indexOf(indexOrText) > -1) {
+                try {
+                    const lastRow = await this.findElements(Input.locators.DefaultTreeSection.lastRow);
+                    if (lastRow.length > 0) {
+                        endReached = true;
+                    }
+                    else if ((await pick.getAttribute('aria-posinset')) === (await pick.getAttribute('aria-setsize'))) {
+                        endReached = true;
+                    }
+                    if (typeof indexOrText === 'string') {
+                        const text = await pick.getLabel();
+                        if (text.indexOf(indexOrText) > -1) {
+                            return pick;
+                        }
+                    }
+                    else if (indexOrText === pick.getIndex()) {
                         return pick;
                     }
                 }
-                else if (indexOrText === pick.getIndex()) {
-                    return pick;
+                catch (e) {
+                    if (e.name === 'StaleElementReferenceError') {
+                        continue;
+                    }
+                    throw e;
                 }
             }
             if (!endReached) {
@@ -281,7 +294,15 @@ class QuickPickItem extends AbstractElement_1.AbstractElement {
         const actions = [];
         const elements = await this.findElements(Input.locators.Input.button);
         for (const element of elements) {
-            actions.push(await new QuickInputAction(element, this).wait());
+            try {
+                actions.push(await new QuickInputAction(element, this).wait());
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
+                }
+                throw e;
+            }
         }
         return actions;
     }
@@ -293,8 +314,16 @@ class QuickPickItem extends AbstractElement_1.AbstractElement {
     async getAction(label) {
         const actions = await this.getActions();
         for (const action of actions) {
-            if ((await action.getLabel()) === label) {
-                return action;
+            try {
+                if ((await action.getLabel()) === label) {
+                    return action;
+                }
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
+                }
+                throw e;
             }
         }
     }

package/out/components/workbench/input/InputBox.js

@@ -51,8 +51,16 @@ class InputBox extends __1.Input {
             .findElement(InputBox.locators.InputBox.rows)
             .findElements(InputBox.locators.InputBox.row);
         for (const element of elements) {
-            if (await element.isDisplayed()) {
-                picks.push(await new __1.QuickPickItem(+(await element.getAttribute('data-index')), this).wait());
+            try {
+                if (await element.isDisplayed()) {
+                    picks.push(await new __1.QuickPickItem(+(await element.getAttribute('data-index')), this).wait());
+                }
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
+                }
+                throw e;
             }
         }
         return picks;
@@ -63,8 +71,16 @@ class InputBox extends __1.Input {
             .findElement(InputBox.locators.InputBox.rows)
             .findElements(InputBox.locators.InputBox.row);
         for (const element of elements) {
-            if (await element.isDisplayed()) {
-                picks.push(await new __1.QuickPickItem(+(await element.getAttribute('data-index')), this, true).wait());
+            try {
+                if (await element.isDisplayed()) {
+                    picks.push(await new __1.QuickPickItem(+(await element.getAttribute('data-index')), this, true).wait());
+                }
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
+                }
+                throw e;
             }
         }
         return picks;

package/out/components/workbench/input/QuickOpenBox.js

@@ -44,9 +44,17 @@ class QuickOpenBox extends __1.Input {
         const tree = await this.getDriver().wait(selenium_webdriver_1.until.elementLocated(QuickOpenBox.locators.QuickOpenBox.quickList), 1000);
         const elements = await tree.findElements(QuickOpenBox.locators.QuickOpenBox.row);
         for (const element of elements) {
-            const index = +(await element.getAttribute('aria-posinset'));
-            if (await element.isDisplayed()) {
-                picks.push(await new __1.QuickPickItem(index, this).wait());
+            try {
+                const index = +(await element.getAttribute('aria-posinset'));
+                if (await element.isDisplayed()) {
+                    picks.push(await new __1.QuickPickItem(index, this).wait());
+                }
+            }
+            catch (e) {
+                if (e.name === 'StaleElementReferenceError') {
+                    continue;
+                }
+                throw e;
             }
         }
         return picks;

package/out/errors/ExTesterError.d.ts

@@ -0,0 +1,112 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License", destination); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+/**
+ * Context information for debugging ExTester errors.
+ */
+export interface ErrorContext {
+    /** Name of the page object component (e.g., "TextEditor", "ActivityBar") */
+    component?: string;
+    /** The action being performed when the error occurred */
+    action?: string;
+    /** VS Code version being tested */
+    vscodeVersion?: string;
+    /** String representation of the locator used */
+    locator?: string;
+    /** Additional details about the element or operation */
+    details?: string;
+    /** The original error that caused this error, if any */
+    cause?: Error;
+}
+/**
+ * Base error class for all ExTester-specific errors.
+ * Provides rich context for debugging test failures.
+ */
+export declare class ExTesterError extends Error {
+    readonly context: ErrorContext;
+    readonly timestamp: Date;
+    constructor(message: string, context?: ErrorContext);
+    /**
+     * Get a detailed string representation of the error for logging.
+     */
+    toDetailedString(): string;
+    toString(): string;
+}
+/**
+ * Error thrown when an element cannot be found in the DOM.
+ */
+export declare class ElementNotFoundError extends ExTesterError {
+    constructor(message: string, context?: ErrorContext);
+}
+/**
+ * Error thrown when an element exists but is not visible.
+ */
+export declare class ElementNotVisibleError extends ExTesterError {
+    constructor(message: string, context?: ErrorContext);
+}
+/**
+ * Error thrown when an element cannot be interacted with (e.g., blocked, disabled).
+ */
+export declare class ElementNotInteractableError extends ExTesterError {
+    constructor(message: string, context?: ErrorContext);
+}
+/**
+ * Error thrown when an element reference has become stale (DOM was modified).
+ */
+export declare class StaleElementError extends ExTesterError {
+    constructor(message: string, context?: ErrorContext);
+}
+/**
+ * Error thrown when element recovery/reinitialization fails.
+ */
+export declare class ElementRecoveryError extends ExTesterError {
+    constructor(message: string, context?: ErrorContext);
+}
+/**
+ * Error thrown when a timeout occurs waiting for a condition.
+ */
+export declare class TimeoutError extends ExTesterError {
+    readonly timeoutMs: number;
+    constructor(message: string, timeoutMs: number, context?: ErrorContext);
+    toDetailedString(): string;
+}
+/**
+ * Error thrown when a locator fails to match any elements.
+ */
+export declare class LocatorError extends ExTesterError {
+    constructor(message: string, context?: ErrorContext);
+}
+/**
+ * Error thrown for platform-specific limitations (e.g., macOS native dialogs).
+ */
+export declare class PlatformNotSupportedError extends ExTesterError {
+    readonly platform: string;
+    constructor(message: string, platform?: string, context?: ErrorContext);
+    toDetailedString(): string;
+}
+/**
+ * Error thrown when an operation is not supported for the current VS Code version.
+ */
+export declare class VersionNotSupportedError extends ExTesterError {
+    readonly requiredVersion: string;
+    readonly currentVersion: string;
+    constructor(message: string, requiredVersion: string, currentVersion: string, context?: ErrorContext);
+    toDetailedString(): string;
+}
+/**
+ * Utility to wrap native Selenium errors with ExTester context.
+ */
+export declare function wrapError(originalError: Error, context: ErrorContext): ExTesterError;