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

Delta Between Two Patch Sets: README.md

Issue 29367181: Issue 4726 - Add tests for the element hiding emulation content script (Closed) Base URL: https://bitbucket.org/fhd/adblockpluscore
Left Patch Set: Rebased on master, added tests for braces in regexp property selectors Created Dec. 13, 2016, 5:26 p.m.
Right Patch Set: Wrap and indent QUnit.test invocations consistently Created Jan. 19, 2017, 9:55 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « no previous file | chrome/content/elemHideEmulation.js » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 Adblock Plus core 1 Adblock Plus core
2 ================= 2 =================
3 3
4 This repository contains the generic Adblock Plus code that's shared between 4 This repository contains the generic Adblock Plus code that's shared between
5 platforms. This repository is not designed to be used directly, but instead to 5 platforms. This repository is not designed to be used directly, but instead to
6 serve as a dependency for `adblockplus`, `adblockpluschrome` and 6 serve as a dependency for `adblockplus`, `adblockpluschrome` and
7 `libadblockplus`. 7 `libadblockplus`.
8 8
9 Running the unit tests 9 Running the unit tests
10 ---------------------- 10 ----------------------
11 11
12 ### Requirements 12 ### Requirements
kzar 2017/01/09 08:28:42 If switching to this syntax for the headers mind u
Felix Dahlke 2017/01/10 09:16:47 Markdown only has the underlined syntax for level
kzar 2017/01/10 11:05:33 Well you're right the need for the different level
13 13
14 In order to run the unit test suite you need 14 In order to run the unit test suite you need
15 [Node.js 6 or higher](https://nodejs.org/). Once Node.js is installed please run 15 [Node.js 6 or higher](https://nodejs.org/). Once Node.js is installed please run
16 `npm install` in the repository directory in order to install the required 16 `npm install` in the repository directory in order to install the required
17 dependencies. 17 dependencies.
18 18
19 ### Running all tests 19 ### Running all tests
20 20
21 `npm test` will run all tests in the `test` directory of the repository. 21 `npm test` will run all tests in the `test` directory of the repository.
22 22
23 ### Node.js tests 23 ### Running specific tests
kzar 2017/01/09 08:28:42 I'd prefer to call these unit tests rather than No
Felix Dahlke 2017/01/10 09:16:47 I'll rewrite this section when removing the `test-
24 24
25 `npm run test-node` will only run the tests that work in a Node.js environment. 25 You can specify specific test files or directories on the command line, e.g.:
26 `npm test test/synchronizer.js test/browser/elemHideEmulation.html`.
26 27
27 You can specify specific test files on the command line, e.g. 28 ### Running the browser tests in a real browser
28 `npm run test-node test/synchronizer.js`.
29 29
30 ### Browser tests 30 The tests under `test/browser` require a browser environment. `npm test` will
31 31 run these in a headless browser, but you can also run them in a real one by
32 `npm run test-browser` will only run the tests that require a browser 32 opening
33 environment. 33 [test/browser/elemHideEmulation.html](test/browser/elemHideEmulation.html).
34
35 You can run these tests in a real browser by opening `test/browser/index.html`.
LEFTRIGHT

Powered by Google App Engine
This is Rietveld