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
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.
Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

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.

review: Needs Fixing

Unmerged revisions

2641. By Szymon Leśniak

Corrected the mistake

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/README.txt'
2--- tests/README.txt 2012-12-05 18:52:31 +0000
3+++ tests/README.txt 2016-04-15 19:34:22 +0000
4@@ -19,16 +19,16 @@
5
6 To run the tests, navigate to the root directory of the OpenLP project, and then run the following command::
7
8- nosetests -v tests
9+ nosetests3 -v tests
10
11 Or, to run only the functional tests, run the following command::
12
13- nosetests -v tests/functional
14+ nosetests3 -v tests/functional
15
16 Or, to run only a particular test suite within a file, run the following command::
17
18- nosetests -v tests/functional/test_applocation.py
19+ nosetests3 -v tests/functional/test_applocation.py
20
21 Finally, to only run a particular test, run the following command::
22
23- nosetests -v tests/functional/test_applocation.py:TestAppLocation.get_frozen_path_test
24+ nosetests3 -v tests/functional/test_applocation.py:TestAppLocation.get_frozen_path_test