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

Issue 818046: Testcases account account_invoice (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 4 months ago by udono
Modified:
9 years, 1 month ago
Reviewers:
ced, yangoon1
CC:
bch, timitos
Visibility:
Public.

Description

Removed to http://codereview.appspot.com/2550042 Only tests/test_tryton.py should be reviewed. The other files are unnecessary for later implementation, since the content of tests/test_tryton.py will be moved to the account and account_invoice module. You can download the complete patch and store it into a folder named test_tryton under trytond/modules. Change into trytond/modules/test_tryton/tests and run: $ sudo su tryton -c "python test_tryton.py"

Patch Set 1 #

Total comments: 16

Patch Set 2 : Fix issues and add tests #

Patch Set 3 : new Version: more tests, fix issues #

Unified diffs Side-by-side diffs Delta from patch set Stats (+609 lines, -180 lines) Patch
M tests/test_tryton.py View 1 2 1 chunk +609 lines, -180 lines 0 comments Download

Messages

Total messages: 9
ced
http://codereview.appspot.com/818046/diff/1/12 File tests/test_tryton.py (right): http://codereview.appspot.com/818046/diff/1/12#newcode34 tests/test_tryton.py:34: def test0010createcompany(self): Why not rely on company creation of ...
12 years, 4 months ago (2010-04-09 18:35:25 UTC) #1
udono
http://codereview.appspot.com/818046/diff/1/12 File tests/test_tryton.py (right): http://codereview.appspot.com/818046/diff/1/12#newcode34 tests/test_tryton.py:34: def test0010createcompany(self): On 2010/04/09 18:35:27, ced wrote: > Why ...
12 years, 4 months ago (2010-04-09 19:48:43 UTC) #2
ced
http://codereview.appspot.com/818046/diff/1/12 File tests/test_tryton.py (right): http://codereview.appspot.com/818046/diff/1/12#newcode57 tests/test_tryton.py:57: def test0020createfiscalyear2010(self): On 2010/04/09 19:48:43, udono wrote: > On ...
12 years, 4 months ago (2010-04-09 20:42:55 UTC) #3
udono
http://codereview.appspot.com/818046/diff/1/12 File tests/test_tryton.py (right): http://codereview.appspot.com/818046/diff/1/12#newcode44 tests/test_tryton.py:44: }) On 2010/04/09 19:48:43, udono wrote: > On 2010/04/09 ...
12 years, 4 months ago (2010-04-10 01:28:33 UTC) #4
udono
Should I split the tests and put them in the appropriate test scrips for account ...
12 years, 3 months ago (2010-04-29 13:41:15 UTC) #5
ced
On 2010/04/29 13:41:15, udono wrote: > Should I split the tests and put them in ...
12 years, 3 months ago (2010-04-29 20:53:40 UTC) #6
udono
On 2010/04/29 20:53:40, ced wrote: > On 2010/04/29 13:41:15, udono wrote: > > Should I ...
12 years, 3 months ago (2010-04-30 08:45:33 UTC) #7
ced
On 2010/04/30 08:45:33, udono wrote: > On 2010/04/29 20:53:40, ced wrote: > > On 2010/04/29 ...
12 years, 3 months ago (2010-05-24 16:22:42 UTC) #8
yangoon1
12 years, 2 months ago (2010-05-25 09:32:48 UTC) #9

          
Sign in to reply to this message.

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