Merge lp:~lesniak-szymon-p/openlp/correct_test_readme into lp:openlp
Proposed by
Szymon Leśniak
Status: | Needs review |
---|---|
Proposed branch: | lp:~lesniak-szymon-p/openlp/correct_test_readme |
Merge into: | lp:openlp |
Diff against target: |
24 lines (+4/-4) 1 file modified
tests/README.txt (+4/-4) |
To merge this branch: | bzr merge lp:~lesniak-szymon-p/openlp/correct_test_readme |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Raoul Snyman | Needs Fixing | ||
Review via email: mp+292043@code.launchpad.net |
Description of the change
Documentation stated that you run tests with `nosetests` command. The merge changes it to `nosetests3`.
To post a comment you must log in.
Unmerged revisions
- 2641. By Szymon Leśniak
-
Corrected the mistake
Hi Szymon, thanks for your fix!
Unfortunately, no matter how small the fix, we have a merge policy that you need to add at least 1 unit test before we will merge your fix. You can read up on unit tests on our wiki:
http:// wiki.openlp. org/Development :Unit_Tests
In addition to that, you need to run the code through Jenkins and present the output. If either the functional or the interface tests break, you need to fix them before continuing.
http:// wiki.openlp. org/Development :Running_ Tests
Come chat to me in IRC if you want an easy test to write.