Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(218)

Issue 3410041: Put the DocTestCase at the end of suite because they drop the database (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 6 months ago by ced
Modified:
15 years, 5 months ago
Reviewers:
yangoon1
Visibility:
Public.

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -1 line) Patch
M trytond/tests/test_tryton.py View 2 chunks +10 lines, -1 line 2 comments Download

Messages

Total messages: 2
yangoon1
http://codereview.appspot.com/3410041/diff/1/trytond/tests/test_tryton.py File trytond/tests/test_tryton.py (right): http://codereview.appspot.com/3410041/diff/1/trytond/tests/test_tryton.py#newcode239 trytond/tests/test_tryton.py:239: return unittest.TestSuite(tests) shouldn't this return suite_? in __main__ modules_suite ...
15 years, 6 months ago (2010-12-02 18:14:24 UTC) #1
ced
15 years, 6 months ago (2010-12-02 18:21:02 UTC) #2
http://codereview.appspot.com/3410041/diff/1/trytond/tests/test_tryton.py
File trytond/tests/test_tryton.py (right):

http://codereview.appspot.com/3410041/diff/1/trytond/tests/test_tryton.py#new...
trytond/tests/test_tryton.py:239: return unittest.TestSuite(tests)
On 2010/12/02 18:14:25, yangoon wrote:
> shouldn't this return suite_?
> in __main__ modules_suite is still called the same way

No the goals is to create a new suite with DocTestCase at the end.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b