Adds missing entries in the unittest ioc container for extract var tests (microsoft#11493)

karthiknadig · web-flow · commit 9348a370a73a · 2020-04-29T12:43:12.000-07:00
* Adds missing entries in the unittest ioc container for extract var tests

* Use registerInterpreterStorageTypes
diff --git a/src/test/ciConstants.ts b/src/test/ciConstants.ts
@@ -10,7 +10,8 @@ export const PYTHON_VIRTUAL_ENVS_LOCATION = process.env.PYTHON_VIRTUAL_ENVS_LOCA
 export const IS_APPVEYOR = process.env.APPVEYOR === 'true';
 export const IS_TRAVIS = process.env.TRAVIS === 'true';
 export const IS_VSTS = process.env.TF_BUILD !== undefined;
-export const IS_CI_SERVER = IS_TRAVIS || IS_APPVEYOR || IS_VSTS;
+export const IS_GITHUB_ACTIONS = process.env.GITHUB_ACTIONS === 'true';
+export const IS_CI_SERVER = IS_TRAVIS || IS_APPVEYOR || IS_VSTS || IS_GITHUB_ACTIONS;
 
 // Control JUnit-style output logging for reporting purposes.
 let reportJunit: boolean = false;
diff --git a/src/test/refactor/extension.refactor.extract.var.test.ts b/src/test/refactor/extension.refactor.extract.var.test.ts
@@ -89,6 +89,7 @@ suite('Variable Extraction', () => {
         ioc.registerCommonTypes();
         ioc.registerProcessTypes();
         ioc.registerVariableTypes();
+        ioc.registerInterpreterStorageTypes();
         ioc.registerMockInterpreterTypes();
 
         ioc.serviceManager.addSingleton<ICondaService>(ICondaService, CondaService);
@@ -192,13 +193,15 @@ suite('Variable Extraction', () => {
     }
 
     // This test fails on linux (text document not getting updated in time)
-    if (!IS_CI_SERVER) {
-        test('Extract Variable (end to end)', async () => {
-            const startPos = new Position(234, 29);
-            const endPos = new Position(234, 38);
-            await testingVariableExtractionEndToEnd(false, startPos, endPos);
-        });
-    }
+    test('Extract Variable (end to end)', async function () {
+        if (!IS_CI_SERVER) {
+            // tslint:disable-next-line:no-invalid-this
+            return this.skip();
+        }
+        const startPos = new Position(234, 29);
+        const endPos = new Position(234, 38);
+        await testingVariableExtractionEndToEnd(false, startPos, endPos);
+    });
 
     test('Extract Variable fails if whole string not selected (end to end)', async () => {
         const startPos = new Position(234, 20);
