Skip to content

Commit ac55840

Browse files
committed
Skip Binary Module tests for release pipeline
1 parent c09dd94 commit ac55840

File tree

1 file changed

+13
-7
lines changed

1 file changed

+13
-7
lines changed

test/features/DebugSession.test.ts

+13-7
Original file line numberDiff line numberDiff line change
@@ -430,12 +430,9 @@ describe("DebugSessionFeature", () => {
430430
});
431431

432432
describe("DebugSessionFeature E2E", function slowTests() {
433-
this.slow(20000); // Will warn if test takes longer than 10s and show red if longer than 20s
434-
this.timeout(30000);
433+
this.slow(20 * 1000); // Will warn if test takes longer than 10s and show red if longer than 20s
434+
this.timeout(5 * 60 * 1000); // Give it five minutes, some CI is slow!
435435

436-
if (process.platform == "darwin") {
437-
this.timeout(60000); // The MacOS test runner is sloooow in Azure Devops
438-
}
439436
before(async () => {
440437
// Registers and warms up the debug adapter and the PowerShell Extension Terminal
441438
await ensureEditorServicesIsConnected();
@@ -446,7 +443,9 @@ describe("DebugSessionFeature E2E", function slowTests() {
446443
let startDebugSession: DebugSession;
447444
let stopDebugSession: DebugSession;
448445
const interactiveSessionConfig = defaultDebugConfigurations[DebugConfig.InteractiveSession];
449-
// Asserts dont seem to fire in this event or the event doesnt resolve in the test code flow, so we need to "extract" the values for later use by the asserts
446+
// Asserts don't seem to fire in this event or the event doesn't resolve
447+
// in the test code flow, so we need to "extract" the values for later
448+
// use by the asserts
450449

451450
const startDebugEvent = debug.onDidStartDebugSession((newDebugSession) => {
452451
startDebugEvent.dispose();
@@ -470,11 +469,18 @@ describe("DebugSessionFeature E2E", function slowTests() {
470469

471470
describe("Binary Modules", () => {
472471
let binaryModulePath: Uri;
473-
before(async () => {
472+
473+
before(async function binarySetup() {
474+
if (process.env.BUILD_SOURCEBRANCHNAME === "release") {
475+
// The binary modules tests won't work in the release pipeline
476+
// due to dependency requirements.
477+
this.skip();
478+
}
474479
binaryModulePath = Uri.joinPath(workspace.workspaceFolders![0].uri, "BinaryModule");
475480
BuildBinaryModuleMock();
476481
await ensureEditorServicesIsConnected();
477482
});
483+
478484
afterEach(async () => {
479485
// Cleanup E2E testing state
480486
await debug.stopDebugging(undefined);

0 commit comments

Comments
 (0)