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
=== modified file 'bin/addons/base/base_test.xml'
--- bin/addons/base/base_test.xml 2010-04-30 18:07:27 +0000
+++ bin/addons/base/base_test.xml 2010-04-30 18:07:27 +0000
@@ -17,4 +17,15 @@
17 <test expr="str(rate)">0.632</test>17 <test expr="str(rate)">0.632</test>
18 </assert>18 </assert>
19 </data>19 </data>
20
21 <!-- Test count attribute for assertions -->
22 <data>
23 <assert
24 model="res.currency.rate"
25 search="[('currency_id', '=', ref('INR'))]"
26 count="3"
27 string="Rate entries for Indian rupee">
28 <test expr="True"/>
29 </assert>
30 </data>
20</openerp>31</openerp>