More CI fixes (#158)

- Bring back retries for E2E tests
- Bump versions

Author: fjakobs

.github/workflows/tests.yml

@@ -116,12 +116,15 @@ jobs:
               working-directory: packages/databricks-vscode
 
             - name: Integration Tests
-              run: |
-                  yarn workspace databricks-vscode run test:integ:clean
-                  yarn install --immutable
-                  yarn build
-                  cd packages/databricks-vscode
-                  yarn run test:integ
+              uses: nick-fields/retry@v2
+              with:
+                  max_attempts: 3
+                  retry_wait_seconds: 2
+                  timeout_minutes: 10
+                  retry_on: any
+                  command: |
+                      cd packages/databricks-vscode
+                      yarn run test:integ
 
             - name: Upload test logs
               if: always()

package.json

@@ -30,11 +30,11 @@
     "homepage": "https://github.com/databricks/databricks-vscode#readme",
     "packageManager": "yarn@3.2.1",
     "devDependencies": {
-        "@typescript-eslint/eslint-plugin": "^5.37.0",
-        "@typescript-eslint/parser": "^5.37.0",
-        "eslint": "^8.23.1",
+        "@typescript-eslint/eslint-plugin": "^5.42.0",
+        "@typescript-eslint/parser": "^5.42.0",
+        "eslint": "^8.26.0",
         "eslint-config-prettier": "^8.5.0",
         "ts-mockito": "^2.6.1",
-        "typescript": "^4.8.3"
+        "typescript": "^4.8.4"
     }
 }

packages/databricks-sdk-js/package.json

@@ -40,23 +40,23 @@
         "@types/chai-as-promised": "^7.1.5",
         "@types/chai-spies": "^1.0.3",
         "@types/ini": "^1.3.31",
-        "@types/mocha": "^9.1.1",
-        "@types/node": "^18.8.2",
+        "@types/mocha": "^10.0.0",
+        "@types/node": "^18.11.9",
         "@types/tmp": "^0.2.3",
         "@types/uuid": "^8.3.4",
         "chai": "^4.3.6",
         "chai-as-promised": "^7.1.1",
         "chai-spies": "^1.0.0",
-        "eslint": "^8.23.1",
-        "mocha": "^10.0.0",
+        "eslint": "^8.26.0",
+        "mocha": "^10.1.0",
         "nyc": "^15.1.0",
         "prettier": "^2.7.1",
         "tmp-promise": "^3.0.3",
-        "ts-loader": "^9.3.1",
+        "ts-loader": "^9.4.1",
         "ts-mocha": "^10.0.0",
         "ts-mockito": "^2.6.1",
         "ts-node": "^10.9.1",
-        "typescript": "^4.8.3",
+        "typescript": "^4.8.4",
         "uuid": "^9.0.0"
     },
     "nyc": {
@@ -78,4 +78,4 @@
         ],
         "report-dir": "coverage"
     }
-}
+}

packages/databricks-vscode-types/package.json

@@ -18,8 +18,8 @@
     "watch": "tsc -w"
   },
   "devDependencies": {
-    "@types/vscode": "^1.69.1",
-    "typescript": "^4.8.3"
+    "@types/vscode": "^1.73.0",
+    "typescript": "^4.8.4"
   },
   "dependencies": {
     "@databricks/databricks-sdk": "workspace:^"

packages/databricks-vscode/package.json

@@ -502,7 +502,6 @@
         "fix": "prettier . --write",
         "test:lint": "eslint src --ext ts && prettier . -c",
         "test:unit": "yarn run build && node ./out/test/runTest.js",
-        "test:integ:clean": "yarn run clean && rm -rf /tmp/vscode-test-databricks /tmp/databricks-vscode-test-extensions",
         "test:integ:prepare": "CODE_VERSION=1.69.1 && yarn run package && ts-node src/test/vscode-extension-tester-patch/cli.ts get-vscode --code_version $CODE_VERSION --storage /tmp/vscode-test-databricks && ts-node src/test/vscode-extension-tester-patch/cli.ts get-chromedriver --code_version $CODE_VERSION --storage /tmp/vscode-test-databricks",
         "test:integ:install-vsix": "ts-node src/test/vscode-extension-tester-patch/cli.ts install-vsix --storage /tmp/vscode-test-databricks -f databricks-vscode*.vsix -e /tmp/databricks-vscode-test-extensions",
         "test:integ:run": "yarn run build && node out/test/e2e/scripts/e2e.js -e /tmp/databricks-vscode-test-extensions --storage /tmp/vscode-test-databricks --code_settings src/test/e2e/settings.json 'out/**/*.e2e.js'",
@@ -514,34 +513,34 @@
     "dependencies": {
         "@databricks/databricks-sdk": "*",
         "@databricks/databricks-vscode-types": "workspace:^",
-        "@vscode/debugadapter": "^1.57.0",
-        "@vscode/webview-ui-toolkit": "^1.0.1"
+        "@vscode/debugadapter": "^1.58.0",
+        "@vscode/webview-ui-toolkit": "^1.1.0"
     },
     "devDependencies": {
         "@istanbuljs/nyc-config-typescript": "^1.0.2",
         "@types/fs-extra": "^9.0.13",
         "@types/glob": "^8.0.0",
-        "@types/mocha": "^9.1.1",
-        "@types/node": "^18.7.17",
+        "@types/mocha": "^10.0.0",
+        "@types/node": "^18.11.9",
         "@types/tmp": "^0.2.3",
-        "@types/vscode": "^1.69.0",
-        "@typescript-eslint/eslint-plugin": "^5.37.0",
-        "@typescript-eslint/parser": "^5.37.0",
-        "@vscode/test-electron": "^2.1.5",
-        "esbuild": "^0.15.7",
-        "eslint": "^8.23.1",
+        "@types/vscode": "^1.69.1",
+        "@typescript-eslint/eslint-plugin": "^5.42.0",
+        "@typescript-eslint/parser": "^5.42.0",
+        "@vscode/test-electron": "^2.2.0",
+        "esbuild": "^0.15.13",
+        "eslint": "^8.26.0",
         "fs-extra": "^10.1.0",
         "glob": "^8.0.3",
-        "mocha": "^10.0.0",
+        "mocha": "^10.1.0",
         "mock-require": "^3.0.3",
         "nyc": "^15.1.0",
         "prettier": "^2.7.1",
         "tmp-promise": "^3.0.3",
         "ts-mockito": "^2.6.1",
         "ts-node": "^10.9.1",
-        "typescript": "^4.8.3",
-        "vsce": "^2.11.0",
-        "vscode-extension-tester": "^4.4.1",
+        "typescript": "^4.8.4",
+        "vsce": "^2.13.0",
+        "vscode-extension-tester": "5.1.x",
         "winston": "^3.8.2"
     },
     "nyc": {

packages/databricks-vscode/src/test/e2e/configure.e2e.ts

@@ -27,7 +27,7 @@ describe("Configure Databricks Extension", async function () {
     let clusterId: string;
     let periodicRunners = new Map<string, PeriodicRunner>();
 
-    this.timeout(10 * 60 * 1000);
+    this.timeout(3 * 60 * 1000);
 
     before(async function () {
         browser = VSBrowser.instance;

packages/databricks-vscode/src/test/vscode-extension-tester-patch/download.ts

@@ -30,6 +30,7 @@ function isGotResponseOk(response: Response) {
 
 const options: OptionsOfTextResponseBody & {isStream?: undefined} = {
     headers: {
+        // eslint-disable-next-line @typescript-eslint/naming-convention
         "user-agent": "nodejs",
     },
     agent: {