Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: README.md

Issue 29720661: Issue 6391 - Allow running the browser unit tests with Firefox (Closed) Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Review comments addressed. Created May 18, 2018, 12:02 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « .hgignore ('k') | package.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: README.md
===================================================================
--- a/README.md
+++ b/README.md
@@ -25,16 +25,26 @@
You can specify specific test files or directories on the command line, e.g.:
`npm test test/synchronizer.js test/browser/elemHideEmulation.js`.
### Running the browser tests in a real browser
The tests under `test/browser` require a browser environment. `npm test` will
run these in a headless browser, with each module being loaded in a new frame.
+The default is to run in both Chromium and Firefox. You can select which
+runners to use by setting the BROWSER_TEST_RUNNERS environment. Possible
+runners (separated by a ',') are, the default being "chromium,firefox":
+
+- "chromium_remote": Chromium 60 (using the remote interface)
+- "chromium": Chrome 63 (using WebDriver)
+- "firefox": Firefox 60
+
+You can't set a specific version of the browser at runtime.
+
Linting
-------
You can lint the code using [ESLint](http://eslint.org).
eslint *.js chrome lib test
You will need to set up ESLint and our configuration first, see
« no previous file with comments | « .hgignore ('k') | package.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld