Merge lp://qastaging/~coreygoldberg/selenium-simple-test/test_html_runner into lp://qastaging/selenium-simple-test

Proposed by Corey Goldberg
Status: Merged
Approved by: Corey Goldberg
Approved revision: 398
Merged at revision: 388
Proposed branch: lp://qastaging/~coreygoldberg/selenium-simple-test/test_html_runner
Merge into: lp://qastaging/selenium-simple-test
Prerequisite: lp://qastaging/~coreygoldberg/selenium-simple-test/update-unit-findcases
Diff against target: 330 lines (+135/-73)
4 files modified
src/sst/htmlrunner.py (+42/-71)
src/sst/runtests.py (+2/-2)
src/sst/tests/test_django_devserver.py (+9/-0)
src/sst/tests/test_html_runner.py (+82/-0)
To merge this branch: bzr merge lp://qastaging/~coreygoldberg/selenium-simple-test/test_html_runner
Reviewer Review Type Date Requested Status
Leo Arias (community) code review Approve
Review via email: mp+160130@code.qastaging.launchpad.net

Commit message

junithtmlrunner module rename and unit test

Description of the change

- renamed HTMLTestRunner module to htmlrunner
- added unit test for htmlrunner (html report module)
- change output in htmlrunner to use stdout instead of stderr for status
- fixed test_django_devserver unittest to patch stdout during test

To post a comment you must log in.
396. By Corey Goldberg

renamed junithtmlrunner module to htmlrunner

397. By Corey Goldberg

changed test files to classes in htmlrunner tests

398. By Corey Goldberg

removed uneccesarry line

Revision history for this message
Leo Arias (elopio) wrote :

+1
/me likes moar tests.

review: Approve (code review)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches