Skip to content

Commit

Permalink
Disable flaky test (6766) (#6826)
Browse files Browse the repository at this point in the history
* Disable flaky test (6766)

* lint

Co-authored-by: Stephen Weatherford <Stephen.Weatherford.com>
  • Loading branch information
StephenWeatherford authored May 12, 2022
1 parent 76684d4 commit 2d37ef2
Showing 1 changed file with 14 additions and 14 deletions.
28 changes: 14 additions & 14 deletions src/vscode-bicep/src/test/e2e/emptyConfigFileSnippets.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,8 @@ import vscode, {
import path from "path";
import fse from "fs-extra";
import {
executeAcceptSelectedSuggestion,
executeCloseAllEditors,
executeCompletionItemProvider,
executeSelectNextSuggestion,
} from "./commands";
import {} from "fs";
import { createUniqueTempFolder } from "../utils/createUniqueTempFolder";
Expand Down Expand Up @@ -74,18 +72,20 @@ describe("empty config file snippets", (): void => {
normalizeLineEndings(expectedAfterInsertion)
);

// Verify that the snippet placed VS Code into an "insertion" state with the dropdown for the first rule open to show
// the available diagnostic levels (the current one should be "warning").
// Verify this by moving down to the next suggestion ("off") and selecting it
const expectedAfterSelectingOffInsteadOfWarning =
expectedAfterInsertion.replace(/warning/, "off");
await executeSelectNextSuggestion();
await executeAcceptSelectedSuggestion();
const textAfterSelectingOffInsteadOfWarningtext =
editor.document.getText();
expect(
normalizeLineEndings(textAfterSelectingOffInsteadOfWarningtext)
).toBe(normalizeLineEndings(expectedAfterSelectingOffInsteadOfWarning));
// TODO: DISABLED (FLAKY) - see https://github.com/Azure/bicep/issues/6766
//
// // Verify that the snippet placed VS Code into an "insertion" state with the dropdown for the first rule open to show
// // the available diagnostic levels (the current one should be "warning").
// // Verify this by moving down to the next suggestion ("off") and selecting it
// const expectedAfterSelectingOffInsteadOfWarning =
// expectedAfterInsertion.replace(/warning/, "off");
// await executeSelectNextSuggestion();
// await executeAcceptSelectedSuggestion();
// const textAfterSelectingOffInsteadOfWarningtext =
// editor.document.getText();
// expect(
// normalizeLineEndings(textAfterSelectingOffInsteadOfWarningtext)
// ).toBe(normalizeLineEndings(expectedAfterSelectingOffInsteadOfWarning));
} finally {
fse.rmdirSync(tempFolder, {
recursive: true,
Expand Down

0 comments on commit 2d37ef2

Please sign in to comment.