tests - disable terminal integration tests for web

This commit is contained in:
Benjamin Pasero
2020-03-19 09:37:11 +01:00
parent 9e89ada95f
commit c0eea2a712
16 changed files with 181 additions and 187 deletions

View File

@@ -6,7 +6,8 @@
import { window, Pseudoterminal, EventEmitter, TerminalDimensions, workspace, ConfigurationTarget, Disposable, UIKind, env } from 'vscode';
import { doesNotThrow, equal, ok, deepEqual, throws } from 'assert';
suite('window namespace tests', () => {
// TODO@Daniel flaky tests (https://github.com/microsoft/vscode/issues/92826)
((env.uiKind === UIKind.Web) ? suite.skip : suite)('vscode API - terminal', () => {
suiteSetup(async () => {
// Disable conpty in integration tests because of https://github.com/microsoft/vscode/issues/76548
await workspace.getConfiguration('terminal.integrated').update('windowsEnableConpty', false, ConfigurationTarget.Global);
@@ -19,8 +20,8 @@ suite('window namespace tests', () => {
disposables.length = 0;
});
// TODO@Daniel flaky test (https://github.com/microsoft/vscode/issues/92826)
((env.uiKind === UIKind.Web) ? test.skip : test)('sendText immediately after createTerminal should not throw', (done) => {
test('sendText immediately after createTerminal should not throw', (done) => {
disposables.push(window.onDidOpenTerminal(term => {
try {
equal(terminal, term);
@@ -34,8 +35,7 @@ suite('window namespace tests', () => {
doesNotThrow(terminal.sendText.bind(terminal, 'echo "foo"'));
});
// TODO@Daniel done called multiple times (https://github.com/microsoft/vscode/issues/92826)
((env.uiKind === UIKind.Web) ? test.skip : test)('onDidCloseTerminal event fires when terminal is disposed', (done) => {
test('onDidCloseTerminal event fires when terminal is disposed', (done) => {
disposables.push(window.onDidOpenTerminal(term => {
try {
equal(terminal, term);
@@ -235,8 +235,7 @@ suite('window namespace tests', () => {
});
suite('window.onDidWriteTerminalData', () => {
// TODO@Daniel done called multiple times (https://github.com/microsoft/vscode/issues/92826)
((env.uiKind === UIKind.Web) ? test.skip : test)('should listen to all future terminal data events', (done) => {
test('should listen to all future terminal data events', (done) => {
const openEvents: string[] = [];
const dataEvents: { name: string, data: string }[] = [];
const closeEvents: string[] = [];