Merge lp:~vauxoo/openerp-venezuela-localization/7.0-fr-rev1-kty into lp:openerp-venezuela-localization

Proposed by Katherine Zaoral (Vauxoo)
Status: Merged
Merged at revision: 987
Proposed branch: lp:~vauxoo/openerp-venezuela-localization/7.0-fr-rev1-kty
Merge into: lp:openerp-venezuela-localization
Diff against target: 79 lines (+31/-20)
3 files modified
l10n_ve_fiscal_requirements/__openerp__.py (+8/-1)
l10n_ve_fiscal_requirements/test/fr_ut_test.yml (+2/-2)
l10n_ve_fiscal_requirements/test/fr_vat_test.yml (+21/-17)
To merge this branch: bzr merge lp:~vauxoo/openerp-venezuela-localization/7.0-fr-rev1-kty
Reviewer Review Type Date Requested Status
Moisés López - http://www.vauxoo.com Approve
Katherine Zaoral (Vauxoo) Approve
Review via email: mp+215465@code.launchpad.net

Description of the change

[MERGE] comment the test yaml for search rif until the seniat page service restore the rif consulting functionality.

To post a comment you must log in.
Revision history for this message
Katherine Zaoral (Vauxoo) (kathy-zaoral) wrote :

LGTM

review: Approve
987. By Katherine Zaoral (Vauxoo)

[FIX] bad test yaml using an old unit tax.

988. By Katherine Zaoral (Vauxoo)

[ADD] another test yaml test case have benn comment becuase the seniat disable
the functionality of consult rif for the moment. when this service is enable
again this test case need to be restore.

Revision history for this message
Katherine Zaoral (Vauxoo) (kathy-zaoral) wrote :

The runbot now is green :).
Redgards.

review: Approve
Revision history for this message
Moisés López - http://www.vauxoo.com (moylop260) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'l10n_ve_fiscal_requirements/__openerp__.py'
2--- l10n_ve_fiscal_requirements/__openerp__.py 2014-04-11 17:11:49 +0000
3+++ l10n_ve_fiscal_requirements/__openerp__.py 2014-04-14 14:00:45 +0000
4@@ -102,9 +102,16 @@
5 'test': [
6 'test/account_customer_invoice.yml',
7 'test/account_supplier_invoice.yml',
8- 'test/fr_vat_search_test.yml',
9+ # Note: this test was comment becuase seniat page it is not allowing
10+ # the rif consulting right now, for that reason this test is damage the
11+ # ovl runbot in a red state. When the seniat service go back live this
12+ # yaml test can be uncomment.
13+ #'test/fr_vat_search_test.yml',
14 'test/fr_ut_test.yml',
15 'test/fr_vat_test.yml',
16+ # fr_vat_test last test case was comment because the seniat
17+ # disabled for the moment the consult rif functionality. this case
18+ # need to be uncomment when the seniat enable the function again.
19 'test/fr_tax_test.yml',
20 'test/fr_address.yml',
21 'test/fr_sale_test.yml',
22
23=== modified file 'l10n_ve_fiscal_requirements/test/fr_ut_test.yml'
24--- l10n_ve_fiscal_requirements/test/fr_ut_test.yml 2013-11-19 05:19:31 +0000
25+++ l10n_ve_fiscal_requirements/test/fr_ut_test.yml 2014-04-14 14:00:45 +0000
26@@ -9,5 +9,5 @@
27 Testing Tax Unit Computing
28 -
29 !python {model: l10n.ut }: |
30- assert self.compute(cr, uid, 107, date='2013-12-31') == 1, 'There is something wrong in computing money into tax units'
31- assert self.compute_ut_to_money(cr, uid, 1, date='2013-12-31') == 107, 'There is something wrong in computing tax units into money'
32+ assert self.compute(cr, uid, 127, date='2014-12-31') == 1, 'There is something wrong in computing money into tax units'
33+ assert self.compute_ut_to_money(cr, uid, 1, date='2014-12-31') == 127, 'There is something wrong in computing tax units into money'
34
35=== modified file 'l10n_ve_fiscal_requirements/test/fr_vat_test.yml'
36--- l10n_ve_fiscal_requirements/test/fr_vat_test.yml 2013-12-14 07:43:26 +0000
37+++ l10n_ve_fiscal_requirements/test/fr_vat_test.yml 2014-04-14 14:00:45 +0000
38@@ -105,20 +105,24 @@
39 !assert {model: res.partner, id: base.res_partner_23}:
40 - name == 'Vauxoo'
41 - seniat_updated == False
42--
43- 5.8 I make sure that main company will consult web to check Vat Validity
44--
45- !record { model: res.company, id: base.main_company }:
46- vat_check_vies: True
47--
48- 5.9 Pressing 'Check Validity' Button in Partner Form Again to Update Name
49--
50- !function {model: res.partner, name: button_check_vat, id: base.res_partner_23 }:
51- - model: res.partner
52- eval: "[obj(ref('base.res_partner_23')).id]"
53--
54- 5.10 I check that Partner Name has changed
55--
56- !assert {model: res.partner, id: base.res_partner_23}:
57- - name == 'VAUXOO, C,A'
58- - seniat_updated == True
59+
60+# note: this test cas have been comment because the seniat rif consult have
61+# been disabled for the current moment and this case was returning an error
62+# when running the test case in the vauxoo runbot.
63+#-
64+# 5.8 I make sure that main company will consult web to check Vat Validity
65+#-
66+# !record { model: res.company, id: base.main_company }:
67+# vat_check_vies: True
68+#-
69+# 5.9 Pressing 'Check Validity' Button in Partner Form Again to Update Name
70+#-
71+# !function {model: res.partner, name: button_check_vat, id: base.res_partner_23 }:
72+# - model: res.partner
73+# eval: "[obj(ref('base.res_partner_23')).id]"
74+#-
75+# 5.10 I check that Partner Name has changed
76+#-
77+# !assert {model: res.partner, id: base.res_partner_23}:
78+# - name == 'VAUXOO, C,A'
79+# - seniat_updated == True