Merge lp:~donkirkby/openobject-server/test-count into lp:openobject-server/5.0

Proposed by Don Kirkby
Status: Merged
Merge reported by: Christophe Simonis (OpenERP)
Merged at revision: not available
Proposed branch: lp:~donkirkby/openobject-server/test-count
Merge into: lp:openobject-server/5.0
Prerequisite: lp:~donkirkby/openobject-server/test-context
Diff against target: 19 lines (+11/-0)
1 file modified
bin/addons/base/base_test.xml (+11/-0)
To merge this branch: bzr merge lp:~donkirkby/openobject-server/test-count
Reviewer Review Type Date Requested Status
Christophe Simonis (OpenERP) Pending
OpenERP Core Team Pending
Review via email: mp+21688@code.launchpad.net

Description of the change

Add a failing XML test for bug lp:541545 and then fix it by adding the count attribute to the XML test schema.

To post a comment you must log in.
1996. By Don Kirkby

merge 5.0.9 from test-context branch and remove my fix because Christophe Simonis duplicated it. Leave the test case in place.

Revision history for this message
Don Kirkby (donkirkby) wrote :

Asked Christophe to review this merge proposal, because he committed the same fix. Maybe the test case should still be merged.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bin/addons/base/base_test.xml'
2--- bin/addons/base/base_test.xml 2010-04-30 18:07:27 +0000
3+++ bin/addons/base/base_test.xml 2010-04-30 18:07:27 +0000
4@@ -17,4 +17,15 @@
5 <test expr="str(rate)">0.632</test>
6 </assert>
7 </data>
8+
9+ <!-- Test count attribute for assertions -->
10+ <data>
11+ <assert
12+ model="res.currency.rate"
13+ search="[('currency_id', '=', ref('INR'))]"
14+ count="3"
15+ string="Rate entries for Indian rupee">
16+ <test expr="True"/>
17+ </assert>
18+ </data>
19 </openerp>