Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move e2e tests to stable #5082

Merged
merged 1 commit into from
Dec 7, 2023
Merged

Move e2e tests to stable #5082

merged 1 commit into from
Dec 7, 2023

Conversation

julieg18
Copy link
Contributor

@julieg18 julieg18 commented Dec 7, 2023

Fixes the failing e2e tests:

  Stopping runner...
      at file:///home/runner/work/vscode-dvc/vscode-dvc/node_modules/@wdio/cli/build/utils.js:59:35
      at processTicksAndRejections (node:internal/process/task_queues:96:5)
      at async Launcher.run (file:///home/runner/work/vscode-dvc/vscode-dvc/node_modules/@wdio/cli/build/launcher.js:94:13)
  HookError [SevereServiceError]: 
  A service failed in the 'onPrepare' hook
  SevereServiceError: Couldn't set up Chromedriver Response code 404 (Not Found)
      at VSCodeServiceLauncher._setupChromedriver (file:///home/runner/work/vscode-dvc/vscode-dvc/node_modules/wdio-vscode-service/src/launcher.ts:235:19)
      at processTicksAndRejections (node:internal/process/task_queues:96:5)
      at async VSCodeServiceLauncher._setupVSCodeDesktop (file:///home/runner/work/vscode-dvc/vscode-dvc/node_modules/wdio-vscode-service/src/launcher.ts:191:72)
      at async VSCodeServiceLauncher.onPrepare (file:///home/runner/work/vscode-dvc/vscode-dvc/node_modules/wdio-vscode-service/src/launcher.ts:107:17)
      at async file:///home/runner/work/vscode-dvc/vscode-dvc/node_modules/@wdio/cli/build/utils.js:43:17
      at async Promise.all (index 0)
      at async Launcher.run (file:///home/runner/work/vscode-dvc/vscode-dvc/node_modules/@wdio/cli/build/launcher.js:94:13)

@julieg18 julieg18 self-assigned this Dec 7, 2023
Copy link

codeclimate bot commented Dec 7, 2023

Code Climate has analyzed commit d342a78 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (85% is the threshold).

This pull request will bring the total coverage in the repository to 95.2% (0.0% change).

View more on Code Climate.

@julieg18 julieg18 marked this pull request as ready for review December 7, 2023 21:03
@mattseddon mattseddon merged commit 5ebfdd5 into main Dec 7, 2023
8 checks passed
@mattseddon mattseddon deleted the move-to-stable-e2e branch December 7, 2023 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants