Skip to content

Commit e941ccf

Browse files
authored
feat: Add support for VS Code 1.89.x (#1293)
Signed-off-by: Dominik Jelinek <[email protected]>
1 parent 28702d7 commit e941ccf

File tree

6 files changed

+13
-5
lines changed

6 files changed

+13
-5
lines changed

.github/workflows/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ jobs:
1717
strategy:
1818
fail-fast: false
1919
matrix:
20-
version: [ min, 1.87.2, max ]
20+
version: [ min, 1.88.1, max ]
2121
uses: ./.github/workflows/template-main.yaml
2222
with:
2323
version: ${{ matrix.version }}

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
<a href="https://www.npmjs.com/package/@redhat-developer/page-objects"><img src="https://img.shields.io/npm/v/@redhat-developer/page-objects?color=orange&label=page%20objects&style=for-the-badge&logo=npm" alt="ExTester Page Objects"/></a>
99
<a href="https://github.com/redhat-developer/vscode-extension-tester/blob/main/LICENSE"><img src="https://img.shields.io/badge/license-Apache%202-yellow.svg?style=for-the-badge&logo=appveyor" alt="License"/></a>
1010
</br>
11-
<a href="https://code.visualstudio.com"><img src="https://img.shields.io/badge/VS_Code-1.88.x-blue?style=for-the-badge&logo=visualstudiocode" alt="Maximal supported version of VS Code"></a>
11+
<a href="https://code.visualstudio.com"><img src="https://img.shields.io/badge/VS_Code-1.89.x-blue?style=for-the-badge&logo=visualstudiocode" alt="Maximal supported version of VS Code"></a>
1212
<a href="https://github.com/redhat-developer/vscode-extension-tester/actions/workflows/main.yml"><img src="https://img.shields.io/github/actions/workflow/status/redhat-developer/vscode-extension-tester/main.yml?label=Main%20CI&style=for-the-badge&logo=githubactions" alt="Main CI"></a>
1313
<a href="https://github.com/redhat-developer/vscode-extension-tester/actions/workflows/insiders.yml"><img src="https://img.shields.io/github/actions/workflow/status/redhat-developer/vscode-extension-tester/insiders.yml?branch=main&label=Insider%20CI&style=for-the-badge&logo=githubactions" alt="Insider CI"></a>
1414
</p><br/>
@@ -39,7 +39,7 @@
3939

4040
|NodeJS|Visual Studio Code|Operating System|
4141
|--|--|--|
42-
|<table style="text-align:center;"> <tr><th>18.x.x</th><th>LTS</th><th>Latest</th></tr><tr><td>✅</td><td>❓</td><td>❓</td></tr><tr><td colspan="3">❓ Best-effort</td></tr> </table>| <table style="text-align:center;"> <tr><th>min</th><th>-</th><th>max</th></tr><tr><td>1.86.x</td><td>1.87.x</td><td>1.88.x</td></tr> </table>| <table style="text-align:center;"> <tr><th>Linux</th><th>Windows</th><th>macOS</th></tr><tr><td>✅</td><td>✅</td><td>⚠️</td></tr><tr><td colspan="3">⚠️ [Known Issues](KNOWN_ISSUES.md#macos-known-limitations-of-native-objects)</td></tr> </table>|
42+
|<table style="text-align:center;"> <tr><th>18.x.x</th><th>LTS</th><th>Latest</th></tr><tr><td>✅</td><td>❓</td><td>❓</td></tr><tr><td colspan="3">❓ Best-effort</td></tr> </table>| <table style="text-align:center;"> <tr><th>min</th><th>-</th><th>max</th></tr><tr><td>1.87.x</td><td>1.88.x</td><td>1.89.x</td></tr> </table>| <table style="text-align:center;"> <tr><th>Linux</th><th>Windows</th><th>macOS</th></tr><tr><td>✅</td><td>✅</td><td>⚠️</td></tr><tr><td colspan="3">⚠️ [Known Issues](KNOWN_ISSUES.md#macos-known-limitations-of-native-objects)</td></tr> </table>|
4343

4444
#### NodeJS Support Policy
4545

packages/extester/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,8 @@
5252
"ui-test"
5353
],
5454
"supportedVersions": {
55-
"vscode-min": "1.86.2",
56-
"vscode-max": "1.88.1",
55+
"vscode-min": "1.87.2",
56+
"vscode-max": "1.89.0",
5757
"nodejs": "18"
5858
},
5959
"dependencies": {

packages/page-objects/src/components/sidebar/tree/debug/VariableSectionItem.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,12 @@ export class VariableSectionItem extends CustomTreeItem {
4141
/**
4242
* Get variable value tooltip.
4343
* @returns a promise resolving to variable value tooltip string
44+
* @deprecated For VS Code 1.89+ this method won't be working any more
4445
*/
4546
async getVariableValueTooltip(): Promise<string> {
47+
if(VariableSectionItem.versionInfo.version >= '1.89.0') {
48+
throw Error(`DEPRECATED METHOD! For VS Code 1.89+ this method won't be working any more.`);
49+
}
4650
const value = await this.findElement(VariableSectionItem.locators.VariableSectionItem.value.constructor);
4751
return VariableSectionItem.locators.VariableSectionItem.value.tooltip(value);
4852
}

tests/test-project/src/test/debug/debug.test.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -204,6 +204,9 @@ describe('Debugging', function () {
204204
});
205205

206206
it('VariableSectionItem.getVariableValueTooltip', async function () {
207+
if(VSBrowser.instance.version >= '1.89.0') {
208+
this.skip();
209+
}
207210
const item = await getNumVariable(view, this.timeout() - 2000);
208211
expect(await item?.getVariableValueTooltip()).equals('5');
209212
});

tests/test-project/src/test/editor/settingsEditor.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,7 @@ describe('Settings Editor', function () {
7373
});
7474

7575
it('getDescription works', async function () {
76+
setting = await editor.findSetting('Title Bar Style', 'Window') as ComboSetting;
7677
const desc = await setting.getDescription();
7778
expect(desc).not.empty;
7879
});

0 commit comments

Comments
 (0)