Skip to content

fix: e2e tests #5592

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
May 9, 2025
Merged

fix: e2e tests #5592

merged 4 commits into from
May 9, 2025

Conversation

Patrick-Erichsen
Copy link
Collaborator

@Patrick-Erichsen Patrick-Erichsen commented May 9, 2025 •
edited by mrge-io bot
Loading

Summary by mrge

Improved the e2e test workflow to better handle VSCode and chromedriver downloads, permissions, and environment checks.

  • Bug Fixes
    • Added checks and logs to verify downloaded binaries and their versions.
    • Improved permission handling and added symlink creation for chromedriver.
    • Enhanced error messages and directory checks for easier debugging.

@Patrick-Erichsen Patrick-Erichsen requested a review from a team as a code owner May 9, 2025 16:28
@Patrick-Erichsen Patrick-Erichsen requested review from tomasz-stefaniak and removed request for a team May 9, 2025 16:28
@netlify Netlify
Copy link

netlify bot commented May 9, 2025 •
edited
Loading

✅ Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 006e1e4
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/681e500616e59e00087c82a6

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label May 9, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label May 9, 2025
@tomasz-stefaniak tomasz-stefaniak merged commit 01d88a9 into main May 9, 2025
34 checks passed
@tomasz-stefaniak tomasz-stefaniak deleted the pe/e2e-tests-fix branch May 9, 2025 19:50
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PRs May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomasz-stefaniak tomasz-stefaniak tomasz-stefaniak approved these changes

Assignees
No one assigned
Labels
lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
Issues and PRs
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Patrick-Erichsen @tomasz-stefaniak