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

Unified Diff: README.md

Issue 29328998: Issue 132 - Add REAMDE to adblockplustests repository (Closed)
Patch Set: Created Oct. 8, 2015, 4:19 p.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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: README.md
diff --git a/README.md b/README.md
new file mode 100644
index 0000000000000000000000000000000000000000..8501c610915a2d163f11172f114de972ec286705
--- /dev/null
+++ b/README.md
@@ -0,0 +1,64 @@
+# Adblock Plus Tests
+
+## Introduction
+
+The `adblockplustests` repository contains the test suite for the Adblock Plus
+Firefox extension `adblockplus`. This test suite is itself a separate Firefox
Wladimir Palant 2015/10/09 17:35:01 "for the Adblock Plus Firefox extension `adblockpl
kzar 2015/10/12 11:06:07 Done.
+extension which interacts with the Adblock Plus extension in order to test it.
+(It is therefore assumed that the tests should pass when the latest available
+revisions of `adblockplus` and `adblockplustests` are used in combination.)
+
+Some of the tests in this test suite are also re-used by the Chrome/Opera/Safari
+version of Adblock Plus `adblockpluschrome`. They are included in the
+repository as a dependency, and they are run from within the extension using its
+in-built test runner. (The dependency is managed by the `ensure_dependencies.py`
+script and the expected revision of `adblockplustests` is therefore specified
+explicitly.)
Wladimir Palant 2015/10/09 17:35:01 Please don't document technical details, these can
kzar 2015/10/12 11:06:08 Done.
+
+
+## Usage
+
+### Firefox
+
+To test your changes to Adblock Plus for Firefox you will need to build and
+install both your modified version of the extension and the test suite. For
+each project you will need to type the following:
+
+ ./build.py build
+
+This will create a build with a name in the form
+`adblockplus[tests]-1.2.3.nnnn.xpi`, which you can then manually add to Firefox.
+
+Alternatively, to speed up the process, you can install the
+[Extension Auto-Installer](https://addons.mozilla.org/addon/autoinstaller)
+extension. Assuming that Extension Auto-Installer is configured to use port 8888
+(the default value), you can build and install in one step by running:
+
+ ./build.py autoinstall 8888
+
+Once both the Adblock Plus and Adblock Plus Tests extensions have been built and
+installed you can run the test suite by opening the Firefox Add-ons Manager,
+pressing the "Preferences" button for the Adblock Plus Tests extension and then
+pressing the "Run" button.
+
+Things to note:
+
+- The test suite can take some time to run, sometimes several minutes.
+- If you re-install the Adblock Plus Tests extension you will need to re-load
+ the preferences page before pressing "Run" again. Otherwise the tests will not
+ re-start.
+- The final storage tests often get stuck.
+- Some tests will fail intermittently with a `NS_ERROR_FAILURE` message.
+- Several `Content policy: HTML embed (Java)` tests are currently failing.
Wladimir Palant 2015/10/09 17:35:01 I'd not document specific test failures, these can
kzar 2015/10/12 11:06:08 Done.
+
+
+### Chrome
Sebastian Noack 2015/10/09 12:00:09 IMO this section is irrelevant and redundant here.
kzar 2015/10/09 14:39:42 Yea, maybe you're right. I wasn't sure either way
Wladimir Palant 2015/10/09 17:35:01 I agree with Sebastian, this is unnecessary. The n
kzar 2015/10/12 11:06:07 Done.
+
+To test your changes to Adblock Plus for Chrome, go to the extension's options
+page, open the JavaScript Console and type:
+
+ location.href = "qunit/index.html";
+
+The unit tests should then run automatically in the page. For more information
+about building and testing the Chrome extension please refer to [the project's
+README file](https://hg.adblockplus.org/adblockpluschrome/file/tip/README.md).
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld