Migrarion to @types/vscode and vscode-test (#1126)

Fix #1058.
This fix includes:
1. Update fot vscode engine version.
2. Migration from vscode modulet to @types/vscode and vscode-test
3. unit-tests.ts script to run unit-tests
4. Unit and integration tests separation
5. Move coverage to independent module
6. Fix unit-tests for vscode 1.38.0
7. Fix git command to check changed files

Author: dgolovin

.vscode/launch.json

@@ -26,10 +26,10 @@
             "runtimeExecutable": "${execPath}",
             "args": [
                 "--extensionDevelopmentPath=${workspaceFolder}",
-                "--extensionTestsPath=${workspaceFolder}/out/test"
+                "--extensionTestsPath=${workspaceFolder}/out/test/unit"
             ],
             "outFiles": [
-                "${workspaceFolder}/out/test/**/*.js"
+                "${workspaceFolder}/out/test/unit/**/*.js"
             ],
             "preLaunchTask": "npm: watch",
             "env": {
@@ -43,15 +43,32 @@
             "runtimeExecutable": "${execPath}",
             "args": [
                 "--extensionDevelopmentPath=${workspaceFolder}",
-                "--extensionTestsPath=${workspaceFolder}/out/test"
+                "--extensionTestsPath=${workspaceFolder}/out/test/unit"
             ],
             "outFiles": [
-                "${workspaceFolder}/out/test/**/*.js"
+                "${workspaceFolder}/out/test/unit/**/*.js"
             ],
             "preLaunchTask": "npm: watch",
             "env": {
                 "VSCOST_TEST_MODE": "coverage"
             }
+        },
+        {
+            "name": "Extension integration Tests Debug",
+            "type": "extensionHost",
+            "request": "launch",
+            "runtimeExecutable": "${execPath}",
+            "args": [
+                "--extensionDevelopmentPath=${workspaceFolder}",
+                "--extensionTestsPath=${workspaceFolder}/out/test/int"
+            ],
+            "outFiles": [
+                "${workspaceFolder}/out/test/int/**/*.js"
+            ],
+            "preLaunchTask": "npm: watch",
+            "env": {
+                "VSCOST_TEST_MODE": "debug"
+            }
         }
     ]
 }

build/unit-tests.ts

@@ -0,0 +1,24 @@
+import * as path from 'path';
+
+import { runTests } from 'vscode-test';
+
+async function main() {
+  try {
+    // The folder containing the Extension Manifest package.json
+    // Passed to `--extensionDevelopmentPath`
+    const extensionDevelopmentPath = path.resolve(__dirname, '../../');
+
+    // The path to the extension test runner script
+    // Passed to --extensionTestsPath
+    const extensionTestsPath = path.resolve(__dirname, '../../out/test/unit/');
+
+    // Download VS Code, unzip it and run the integration test
+    console.log(extensionDevelopmentPath, extensionTestsPath);
+    await runTests({ extensionDevelopmentPath, extensionTestsPath });
+  } catch (err) {
+    console.error(err);
+    process.exit(1);
+  }
+}
+
+main();

build/verify-tools.ts

@@ -39,7 +39,7 @@ async function downloadFileAndCreateSha256(targetFolder: string, fileName: strin
 }
 
 const fileCheckRegex = /\w*tools.json/;
-cp.exec('git diff --name-only master..HEAD', async (error, stdout, stderr) => {
+cp.exec('git diff --name-only origin/master -- .', async (error, stdout, stderr) => {
   if (error) {
     throw error;
   }

coverconfig.json

@@ -1,6 +1,6 @@
 {
     "enabled": true,
-    "relativeSourcePath": "../src",
+    "relativeSourcePath": "../../src",
     "relativeCoverageDir": "../../coverage",
     "ignorePatterns": [
         "**/node_modules/**"

package-lock.json

@@ -893,9 +893,8 @@
     "vscode:prepublish": "npm run build",
     "compile": "tsc -p ./",
     "watch": "tsc -watch -p ./",
-    "postinstall": "node ./node_modules/vscode/bin/install",
     "clean": "rm -rf out || rmdir out /s /q",
-    "test": "npm run clean && npm run compile && npm run verify && node ./node_modules/vscode/bin/test",
+    "test": "npm run clean && npm run compile && npm run verify && node ./out/build/unit-tests.js",
     "update-deps": "node_modules/.bin/ncu --upgrade --loglevel verbose --packageFile package.json && npm update",
     "coverage:upload": "codecov -f coverage/coverage-final.json",
     "build": "npm run clean && tslint -p tslint.json && npm run compile"
@@ -916,6 +915,7 @@
     "@types/string-format": "^2.0.0",
     "@types/tmp": "0.1.0",
     "@types/validator": "^10.11.2",
+    "@types/vscode": "^1.30.0",
     "chai": "^4.2.0",
     "codecov": "^3.5.0",
     "decache": "^4.5.1",
@@ -930,7 +930,7 @@
     "tmp": "0.1.0",
     "tslint": "^5.19.0",
     "typescript": "^3.6.2",
-    "vscode": "^1.1.36",
+    "vscode-test": "^1.2.0",
     "walker": "^1.0.7"
   },
   "dependencies": {

package.json

@@ -489,7 +489,7 @@ class OdoModel {
         return Array.from(this.contextToSettings.values());
     }
 
-    public addContexts(folders: WorkspaceFolder[]) {
+    public addContexts(folders: ReadonlyArray<WorkspaceFolder>) {
         for (const folder of folders) {
             try {
                 const compData = yaml.safeLoad(fs.readFileSync(path.join(folder.uri.fsPath, '.odo', 'config.yaml'), 'utf8')) as odo.Config;