-
Notifications
You must be signed in to change notification settings - Fork 105
feat: Update browser-tools orb so that Chrome for testing and Edge browsers are installed in cypress/install #530
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
Conversation
circleci-orb
|
||||||||||||||||||||||||||||||||||||||||||||||
| Project |
circleci-orb
|
| Branch Review |
bump-browser-tools
|
| Run status |
|
| Run duration | 00m 11s |
| Commit |
|
| Committer | Jennifer Shehane |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
|
|
1
|
|
|
0
|
|
|
0
|
|
|
0
|
|
|
0
|
| View all changes introduced in this branch ↗︎ | |
UI Coverage
100%
|
|
|---|---|
|
|
0
|
|
|
3
|
Accessibility
90.82%
|
|
|---|---|
|
|
0 critical
0 serious
3 moderate
0 minor
|
|
|
5
|
Tests for review
cypress/e2e/login.cy.ts • 1 failed test
| Test | Artifacts | |
|---|---|---|
| Authentication > Logs in |
Test Replay
Screenshots
|
|
|
Chrome for Testing isn't installing due to this issue: CircleCI-Public/browser-tools-orb#137 I could theoretically bump our version and get built in Edge support, but I think I'll wait a bit to see if this PR merges. We also want the version to be optional which is covered here: CircleCI-Public/browser-tools-orb#138 |
|
Fixed and made some requests to browser-tools orb which they addressed in 2.1.0: https://github.com/CircleCI-Public/browser-tools-orb/releases/tag/v2.1.0 |
|
This PR is trying to run This won't work, and is the reason for CI failures, because |
|
@MikeMcC399 Yah I'm a bit confused by these tests - I thought it was meant to test itself - like the install browsers part, but it's just using our browsers image... |
|
This PR is hitting multiple issues including the fact that Cypress doesn't find the Chrome for Testing installation from CircleCI. This may need fixes in I will dig in deeper and report back what I can find out and if I have any suggestions. |
| Cypress runs by default in the Electron browser. Use this flag to install additional browsers to run your tests in. | ||
| This is only needed if you are passing the `--browser` flag in your `cypress-command`. | ||
| This parameter leverages the `circleci/browser-tools` orb and includes Chrome and FireFox. | ||
| This parameter leverages the `circleci/browser-tools` orb and includes Chrome, Chrome for testing, FireFox, and Edge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| This parameter leverages the `circleci/browser-tools` orb and includes Chrome, Chrome for testing, FireFox, and Edge. | |
| This parameter leverages the `circleci/browser-tools` orb and includes Chrome, Chrome for Testing, FireFox, and Edge. |
https://developer.chrome.com/blog/chrome-for-testing/ brands it as "Chrome for Testing"
| Cypress runs by default in the Electron browser. Use this flag to install additional browsers to run your tests in. | ||
| This is only needed if you are passing the `--browser` flag in your `cypress-command`. | ||
| This parameter leverages the `circleci/browser-tools` orb and includes Chrome and FireFox. | ||
| This parameter leverages the `circleci/browser-tools` orb and includes Chrome, Chrome for testing, FireFox and Edge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| This parameter leverages the `circleci/browser-tools` orb and includes Chrome, Chrome for testing, FireFox and Edge. | |
| This parameter leverages the `circleci/browser-tools` orb and includes Chrome, Chrome for Testing, FireFox and Edge. |
| docker: | ||
| - image: cypress/browsers:22.16.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| docker: | |
| - image: cypress/browsers:22.16.0 | |
| executor: | |
| name: cypress/default |
| run-ct-tests-in-chrome: | ||
| executor: | ||
| name: cypress/default | ||
| node-version: "22.14.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe delete this line and take the default node-version ?
| orbs: | ||
| node: circleci/node@7 | ||
| browser-tools: circleci/browser-tools@1.5.3 | ||
| browser-tools: circleci/browser-tools@2.1.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| browser-tools: circleci/[email protected].0 | |
| browser-tools: circleci/[email protected].1 |
This is the minimum version required that allows Cypress to detect Chrome for Testing.
See https://github.com/CircleCI-Public/browser-tools-orb/releases/tag/v2.1.1
| - browser-tools/install-browser-tools | ||
| - browser-tools/install_browser_tools: | ||
| install_chrome_for_testing: true | ||
| install_chrome_for_testing_driver: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| install_chrome_for_testing_driver: true | |
| install_edge: true |
The driver isn't needed, however Edge is missing
|
I suggest to close this PR as there are / were several issues with it and it would be tricky to fix everything here in a transparent way that lends itself to reviewing. Instead, I'm two thirds into replacing it with:
and I would follow up with a final PR to add Chrome for Testing. The remaining issue is that in https://github.com/cypress-io/circleci-orb/blob/master/.circleci/test-deploy.yml the The CircleCI issue preventing Chrome for Testing from being used by Cypress was resolved in CircleCI-Public/browser-tools-orb#141 In parallel I'm also submitting some PRs to the https://github.com/CircleCI-Public/browser-tools-orb to fix their documentation. |
combined, replace this PR |
Close #528