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

Handle Chromium to run Karma #32178

Merged
merged 3 commits into from Nov 17, 2020
Merged

Handle Chromium to run Karma #32178

merged 3 commits into from Nov 17, 2020

Conversation

ffoodd
Copy link
Member

@ffoodd ffoodd commented Nov 17, 2020

I do not have Chrome nor stable Firefox and am always using this locally to run tests.

Could also add Firefox Nightly or ESR but I guess Chromium is a first, better choice.

@ffoodd ffoodd requested a review from Johann-S November 17, 2020 15:13
@ffoodd ffoodd requested a review from a team as a code owner November 17, 2020 15:13
@ffoodd ffoodd requested review from XhmikosR and removed request for Johann-S November 17, 2020 15:13
@XhmikosR
Copy link
Member

XhmikosR commented Nov 17, 2020

@ffoodd don't we need a new karma launcher package for Chromium?

EDIT: to answer my question, this might be already supported by the https://github.com/karma-runner/karma-chrome-launcher package we are using.

@ffoodd
Copy link
Member Author

ffoodd commented Nov 17, 2020

Confirmed: https://github.com/karma-runner/karma-chrome-launcher#readme

Launcher for Google Chrome, Google Chrome Canary and Google Chromium.

The same for Firefox BTW, a single launcher works for ESR, Stable, DevEd and Nightly.

@XhmikosR XhmikosR added this to Inbox in v5.0.0-beta1 via automation Nov 17, 2020
@XhmikosR XhmikosR added this to Inbox in v4.6.0 via automation Nov 17, 2020
@XhmikosR
Copy link
Member

Marked it for backport too since it helps you and doesn't hurt having it :)

I'll try to install Chromium on my VM and try this later to confirm it works.

@XhmikosR XhmikosR added the v4 label Nov 17, 2020
@XhmikosR
Copy link
Member

OK, so I tested this on Windows and Chromium isn't detected. But on Linux Mint it works, so I guess it's fine :)

v5.0.0-beta1 automation moved this from Inbox to Approved Nov 17, 2020
@XhmikosR XhmikosR merged commit 1955ffc into main Nov 17, 2020
v5.0.0-beta1 automation moved this from Approved to Done Nov 17, 2020
@XhmikosR XhmikosR deleted the main-fod-chromium-for-karma branch November 17, 2020 19:42
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@XhmikosR XhmikosR moved this from Inbox to Cherry-picked/Manually backported in v4.6.0 Nov 17, 2020
XhmikosR added a commit that referenced this pull request Nov 17, 2020
Handle Chromium to run Karma
XhmikosR added a commit that referenced this pull request Nov 18, 2020
Handle Chromium to run Karma
XhmikosR added a commit that referenced this pull request Nov 18, 2020
Handle Chromium to run Karma
XhmikosR added a commit that referenced this pull request Nov 19, 2020
Handle Chromium to run Karma
XhmikosR added a commit that referenced this pull request Nov 19, 2020
Handle Chromium to run Karma
XhmikosR added a commit that referenced this pull request Nov 19, 2020
Handle Chromium to run Karma
XhmikosR added a commit that referenced this pull request Nov 20, 2020
Handle Chromium to run Karma
@XhmikosR XhmikosR added the js label Nov 21, 2020
XhmikosR added a commit that referenced this pull request Nov 23, 2020
Handle Chromium to run Karma
@XhmikosR XhmikosR moved this from Cherry-picked/Manually backported to Shipped in v4.6.0 Nov 24, 2020
XhmikosR added a commit that referenced this pull request Nov 24, 2020
Handle Chromium to run Karma
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v4.6.0
Shipped
v5.0.0-beta1
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants