Skip to content

Handle Chromium to run Karma#32178

Merged
XhmikosR merged 3 commits into
mainfrom
main-fod-chromium-for-karma
Nov 17, 2020
Merged

Handle Chromium to run Karma#32178
XhmikosR merged 3 commits into
mainfrom
main-fod-chromium-for-karma

Conversation

@ffoodd
Copy link
Copy Markdown
Contributor

@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
Copy Markdown
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
Copy Markdown
Contributor 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
Copy link
Copy Markdown
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
Copy Markdown
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 :)

@XhmikosR XhmikosR merged commit 1955ffc into main Nov 17, 2020
@XhmikosR XhmikosR deleted the main-fod-chromium-for-karma branch November 17, 2020 19:42
Copy link
Copy Markdown

@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 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 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

None yet

Development

Successfully merging this pull request may close these issues.

2 participants