diff options
| author | Jez Ng | 2012-01-10 01:05:38 +0800 |
|---|---|---|
| committer | Jez Ng | 2012-01-10 01:32:12 +0800 |
| commit | 03728546c28664deaf89b9d2dc8fe5fb630bcd9a (patch) | |
| tree | f8bae4bc65e774217830e54bf7254af3b2912fea | |
| parent | a73f92604fca5380609f271685266cadf376ccbc (diff) | |
| download | vimium-03728546c28664deaf89b9d2dc8fe5fb630bcd9a.tar.bz2 | |
Fix tests and make them easier to run.
Add instructions to the README about running tests.
| -rw-r--r-- | README.markdown | 9 | ||||
| -rw-r--r-- | background_page.html | 7 | ||||
| -rw-r--r-- | test_harnesses/automated.html | 2 |
3 files changed, 17 insertions, 1 deletions
diff --git a/README.markdown b/README.markdown index 1a9294f5..54b584c1 100644 --- a/README.markdown +++ b/README.markdown @@ -124,6 +124,15 @@ discussion on the issues tracker or on the [mailing list][list_url]. If it mirro another browser or in Vim itself, let us know! Once you've picked something to work on, add a comment to the respective issue so others don't duplicate your effort. +Please ensure that existing tests pass. The tests are defined in `test_harnesses/automated.html`. To run the +tests: +1. Go to the extensions page +2. Click on the arrow beside Vimium to expand the menu, and then inspect `background_page.html` in the console +3. Type in `runTests()`. + +We encourage you to write tests for any new features or bugfixes. Contributing more tests for existing +features is great too! + When you're done, send us a pull request on Github. Feel free to include a change to the CREDITS file with your patch. diff --git a/background_page.html b/background_page.html index 7d8f749d..4421ae2f 100644 --- a/background_page.html +++ b/background_page.html @@ -765,6 +765,13 @@ }); } init(); + + /** + * Convenience function for development use. + */ + function runTests() { + open(chrome.extension.getURL('test_harnesses/automated.html')); + } </script> </head> </html> diff --git a/test_harnesses/automated.html b/test_harnesses/automated.html index 9f1b8007..e7d84413 100644 --- a/test_harnesses/automated.html +++ b/test_harnesses/automated.html @@ -117,7 +117,7 @@ }), should("label the hints correctly", function() { - var hintStrings = ["ss", "sa", "sd"]; + var hintStrings = ["ss", "as", "ds"]; for (var i = 0; i < 3; i++) assert.equal(hintStrings[i], linkHints.hintMarkers[i].getAttribute("hintString")); }), |
