From 0e094352853a5ab53c298ef1846f111fe0cfaead Mon Sep 17 00:00:00 2001 From: Lewis Sanchez <87730006+lewis-sanchez@users.noreply.github.com> Date: Thu, 23 Jun 2022 12:19:49 -0700 Subject: [PATCH] Removes vscode download for smoke tests (#19795) * Commenting out for now, but will delete in next commit * Adds SQL carbon edit markers * Uses multiline comments * Formats file * Code review changes * Edit comment moved to same line * Add ts-ignore directive to unused method --- test/smoke/src/main.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/smoke/src/main.ts b/test/smoke/src/main.ts index 54d6bfa1a9..b6917d6579 100644 --- a/test/smoke/src/main.ts +++ b/test/smoke/src/main.ts @@ -238,6 +238,7 @@ async function setupRepository(): Promise { } } +// @ts-ignore ts6133 {{SQL CARBON EDIT}} Not used (see below) async function ensureStableCode(): Promise { if (opts.web || !opts['build']) { return; @@ -290,7 +291,7 @@ async function setup(): Promise { console.log('*** Test data:', testDataPath); console.log('*** Preparing smoketest setup...'); - await ensureStableCode(); + // await ensureStableCode(); {{SQL CARBON EDIT}} Smoketests no longer need to download VS Code since they run against ADS await setupRepository(); console.log('*** Smoketest setup done!\n');