selenium-simple-test

Merge lp://qastaging/~coreygoldberg/selenium-simple-test/pep8-in-unittest into lp://qastaging/selenium-simple-test

  1. pep8-in-unittest
  2. Merge into trunk
Proposed by Corey Goldberg
Status: Merged
Approved by: Corey Goldberg
Approved revision: 400
Merged at revision: 394
Proposed branch: lp://qastaging/~coreygoldberg/selenium-simple-test/pep8-in-unittest
Merge into: lp://qastaging/selenium-simple-test
Diff against target: 90 lines (+45/-5)
4 files modified
ci.sh (+1/-1)
src/sst/tests/test_code_format.py (+39/-0)
tox-acceptance.ini (+1/-1)
tox.ini (+4/-3)
To merge this branch: bzr merge lp://qastaging/~coreygoldberg/selenium-simple-test/pep8-in-unittest
Reviewer Review Type Date Requested Status
Leo Arias (community) code review Approve
Review via email: mp+160766@code.qastaging.launchpad.net

Commit message

added pep8 checks in unit testcase.

also:
edited tox ini's and ci.sh, so they don't suppress nose output. this will make it more apparent if we pollute stdout.

Description of the change

added pep8 checks in unit testcase.

To post a comment you must log in.
lp://qastaging/~coreygoldberg/selenium-simple-test/pep8-in-unittest updated
395. By Corey Goldberg

fixed imports

396. By Corey Goldberg

removed extra print

397. By Corey Goldberg

removed unneeded return

398. By Corey Goldberg

refactor pep8 testcase to use packages, turn on nose output

399. By Corey Goldberg

dont capture output in nose from ci.sh

400. By Corey Goldberg

dont capture output in nose from tox

Revision history for this message
Leo Arias (elopio) :
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