Bump 'max' placeholder to latest VS Code 1.83.0

Signed-off-by: Dominik Jelinek <[email protected]>
redhat-developer · Oct 5, 2023 · 2bd561b · 2bd561b
1 parent 8f1a515
commit 2bd561b
Show file tree

Hide file tree

Showing 5 changed files with 27 additions and 33 deletions.
diff --git a/.github/workflows/insiders.yml b/.github/workflows/insiders.yml
@@ -29,7 +29,7 @@ jobs:
       - name: Setup Node
         uses: actions/setup-node@v3
         with:
-          node-version: 18.15.0
+          node-version: 18.15.x
           cache: npm
 
       - name: Run Tests (macOS, windows)

diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -17,7 +17,7 @@ jobs:
     strategy:
       matrix:
         os: [ ubuntu-latest, macos-latest, windows-latest ]
-        version:  [ min, 1.81.1, max ]
+        version:  [ min, 1.82.3, max ]
       fail-fast: false
 
     env:
@@ -30,7 +30,7 @@ jobs:
       - name: Setup Node
         uses: actions/setup-node@v3
         with:
-          node-version: 18.15.0
+          node-version: 18.15.x
           cache: npm
 
       - name: Run Tests (macOS, windows)

diff --git a/page-objects/src/components/editor/SettingsEditor.ts b/page-objects/src/components/editor/SettingsEditor.ts
@@ -26,21 +26,12 @@ export class SettingsEditor extends Editor {
      * @returns Promise resolving to a Setting object if found, undefined otherwise
      */
     async findSetting(title: string, ...categories: string[]): Promise<Setting> {
-        const category = categories.join(' › ');
+        const category = categories.join('>');
         const searchBox = await this.findElement(SettingsEditor.locators.Editor.inputArea);
         await searchBox.sendKeys(Key.chord(SettingsEditor.ctlKey, 'a'));
-        await searchBox.sendKeys(`${category}: ${title}`);
+        await searchBox.sendKeys(`${category}>${title}`);
 
-        let setting!: Setting;
-        const items = await this._getSettingItems();
-
-        for (const item of items) {
-            try {
-                return await (await this.createSetting(item, title, category)).wait();
-            } catch (err) {
-            }
-        }
-        return setting;
+        return await this._getSettingItem(title);
     }
 
     /**
@@ -56,24 +47,12 @@ export class SettingsEditor extends Editor {
         await searchBox.sendKeys(Key.chord(SettingsEditor.ctlKey, 'a'));
         await searchBox.sendKeys(id);
 
-        let setting!: Setting;
-        const items = await this._getSettingItems();
-
-        for (const item of items) {
-            try {
-                const category = (await (await item.findElement(SettingsEditor.locators.SettingsEditor.settingCategory)).getText()).replace(':', '');
-                const title = await (await item.findElement(SettingsEditor.locators.SettingsEditor.settingLabel)).getText();
-                return await (await this.createSetting(item, title, category)).wait();
-            } catch (err) {
-            }
-        }
-        return setting;
+        return await this._getSettingItem();
     }
 
-    private async _getSettingItems(): Promise<WebElement[]> {
+    private async _getSettingItem(title: string = ''): Promise<Setting> {
         const count = await this.findElement(SettingsEditor.locators.SettingsEditor.itemCount);
         let textCount = await count.getText();
-
         await this.getDriver().wait(async() => {
             await new Promise(res => setTimeout(res, 1500));
             const text = await count.getText();
@@ -84,7 +63,22 @@ export class SettingsEditor extends Editor {
             return true;
         });
 
-        return await this.findElements(SettingsEditor.locators.SettingsEditor.itemRow);
+        let setting!: Setting;
+        const items = await this.findElements(SettingsEditor.locators.SettingsEditor.itemRow);
+        for (const item of items) {
+            try {
+                const _category = (await (await item.findElement(SettingsEditor.locators.SettingsEditor.settingCategory)).getText()).replace(':', '');
+                const _title = await (await item.findElement(SettingsEditor.locators.SettingsEditor.settingLabel)).getText();
+                if(title.length > 0) {
+                    if(title !== _title) {
+                        continue;
+                    }
+                }
+                return await (await this.createSetting(item, _title, _category)).wait();
+            } catch (err) {
+            }
+        }
+        return setting;
     }
 
     /**

diff --git a/src/extester.ts b/src/extester.ts
@@ -26,8 +26,8 @@ export const DEFAULT_SETUP_OPTIONS = {
     installDependencies: false
 }
 
-export const VSCODE_VERSION_MIN = '1.80.2';
-export const VSCODE_VERSION_MAX = '1.82.3';
+export const VSCODE_VERSION_MIN = '1.81.1';
+export const VSCODE_VERSION_MAX = '1.83.0';
 
 /**
  * VSCode Extension Tester

diff --git a/test/test-project/src/test/editor/settingsEditor-test.ts b/test/test-project/src/test/editor/settingsEditor-test.ts
@@ -21,7 +21,7 @@ describe('Settings Editor', function () {
 
     it('findSetting by ID works', async function () {
         this.timeout(15000);
-        const setting = await editor.findSettingByID('testProject.general.helloWorld');
+        const setting = await editor.findSettingByID('workbench.editor.enablePreview');
         expect(setting).not.undefined;
     });