Merge lp:~vauxoo/openerp-venezuela-localization/7.0-YAML-rev2-dev-hbto into lp:openerp-venezuela-localization

[hbto [Vauxoo] http://www.vauxoo.com]

[lp:~vauxoo/openerp-venezuela-localization/7.0-YAML-rev2-dev-hbto]

914. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-17

[FIX] Fixing get_journal method on invoice

915. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-17

[MERGE] From lp:~vauxoo/openerp-venezuela-localization/7.0-fixing-demo

916. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-17

[FIX] Cleansing

917. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-17

[FIX] Checking copy method does not pass unneeded args to the new record

918. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-17

[IMP] First test of Customer & Supplier invoice

919. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-17

[FIX] Putting on hold the following modules, by now they are not installable:
l10n_ve_fiscal_book
l10n_ve_generic
l10n_ve_imex
l10n_ve_sale_purchase
l10n_ve_split_invoice
l10n_ve_vat_write_off
l10n_ve_withholding
l10n_ve_withholding_islr
l10n_ve_withholding_iva
l10n_ve_withholding_muni
l10n_ve_withholding_src
ovl

920. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[FIX][l10n_ve_withholding] Cleansing

921. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[FIX][l10n_ve_withholding] Enabling Module & Disabling Test for Module

922. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[IMP][l10n_ve_withholding] Enabled & Added, checking invoice and taxes are OK

923. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[MERGE][l10n_ve_withholding_muni] Cleansing & Enabling Module

924. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[MERGE][l10n_ve_withholding_islr] Cleansing & Enabling Module

925. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[MERGE][l10n_ve_withholding_src] Cleansing & Enabling Module

926. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[MERGE][l10n_ve_withholding_iva] Cleansing & Enabling Module, and Restricting Test

927. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[MERGE][l10n_ve_withholding_iva] From lp:~vauxoo/openerp-venezuela-localization/7.0-fixing-demo

928. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[FIX] Fixing demo data

929. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[FIX][l10n_ve_imex] Enabling Module

930. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[FIX][l10n_ve_split_invoice] Enabling & Scrambling split_invoice

931. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[FIX][l10n_ve_generic] Enabling Module

932. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[FIX][l10n_ve_fiscal_book] Enabling Module

933. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[FIX][l10n_ve_vat_write_off] Enabling Module

934. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[FIX][ovl] Enabling Module

935. By hbto [Vauxoo] http://www.vauxoo.com on 2013-11-18

[MERGE][l10n_ve_sale_purchase] Enabling & Restructuring Module

Unmerged revisions

=== modified file 'l10n_ve_fiscal_book/__openerp__.py'
--- l10n_ve_fiscal_book/__openerp__.py	2013-05-27 16:36:56 +0000
+++ l10n_ve_fiscal_book/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -65,5 +65,5 @@
         "security/ir.model.access.csv"
     ],
     "active": False,
-    "installable": True,
+    "installable": False,
 }

=== modified file 'l10n_ve_fiscal_requirements/__openerp__.py'
--- l10n_ve_fiscal_requirements/__openerp__.py	2013-11-16 17:50:57 +0000
+++ l10n_ve_fiscal_requirements/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -91,20 +91,21 @@
         'demo/demo_partners.xml',
         'demo/journal_demo.xml',
         'demo/demo_invoice.xml',
-        'demo/demo_taxes.yml',
+#        'demo/demo_taxes.yml',
     ],
     'test': [
-        'test/control_number.yml',
-        'test/fr_sale_test.yml',
-        'test/fr_purchase_test.yml',
-        'test/fr_address.yml',
-        'test/fr_vat_test.yml',
-        'test/fr_damaged.yml',
-        'test/fr_debit_note.yml',
-        'test/fr_refund_note.yml',
+        'test/account_customer_invoice.yml',
+        'test/account_supplier_invoice.yml',
+#        'test/control_number.yml',
+#        'test/fr_sale_test.yml',
+#        'test/fr_purchase_test.yml',
+#        'test/fr_address.yml',
+#        'test/fr_vat_test.yml',
+#        'test/fr_damaged.yml',
+#        'test/fr_debit_note.yml',
+#        'test/fr_refund_note.yml',
       ],
     'installable': True,
     'active': False,
 }
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== modified file 'l10n_ve_fiscal_requirements/model/account.py'
--- l10n_ve_fiscal_requirements/model/account.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_fiscal_requirements/model/account.py	2013-11-17 23:21:43 +0000
@@ -24,9 +24,6 @@
 ################################################################################
 
 from openerp.osv import fields, osv
-__TYPES__ = [('sale_debit', 'Sale Debit'),('purchase_debit', 'Purchase Debit')]
-
-__HELP__= " Select 'Sale Debit' for customer debit note journals. Select 'Purchase Debit' for supplier debit note journals."
 
 class account_journal(osv.osv):
     _inherit = 'account.journal'
@@ -45,5 +42,3 @@
                     " Select 'Sale Debit' for customer debit note journals."\
                     " Select 'Purchase Debit' for supplier debit note journals.")
    }
-
-account_journal()

=== modified file 'l10n_ve_fiscal_requirements/model/invoice.py'
--- l10n_ve_fiscal_requirements/model/invoice.py	2013-06-01 18:58:59 +0000
+++ l10n_ve_fiscal_requirements/model/invoice.py	2013-11-17 23:21:43 +0000
@@ -33,21 +33,18 @@
         it does not exist, return false
         """
         
-        if context is None:
-            context = {}
-        journal_type_inv = context.get('journal_type', 'sale')
-        
-        if journal_type_inv in ('sale_debit', 'purchase_debit'):
+        context = context or {}
+        res = super(account_invoice, self)._get_journal(cr, uid, context=context)
+        if res: return res
+        type_inv = context.get('type', 'sale')
+        if type_inv in ('sale_debit', 'purchase_debit'):
             user = self.pool.get('res.users').browse(cr, uid, uid, context=context)
             company_id = context.get('company_id', user.company_id.id)
             journal_obj = self.pool.get('account.journal')
-            res = journal_obj.search(cr, uid, [('type', '=',journal_type_inv),
-                                               ('company_id', '=', company_id)],
-                                                    limit=1)
-            return res and res[0] or False
-        else:
-            return super(account_invoice, self)._get_journal(cr, uid, context=context)
-    
+            domain = [('company_id', '=', company_id),('type', '=',type_inv)]
+            res = journal_obj.search(cr, uid, domain, limit=1)
+        return res and res[0] or False
+
     def _unique_invoice_per_partner(self, cr, uid, ids, context=None):
         """ Return false when it is found 
         that the bill is not out_invoice or out_refund,
@@ -118,6 +115,17 @@
             'child_ids':[],
             'nro_ctrl':None,
             'supplier_invoice_number':None, 
+            'sin_cred': False,
+            # No cleaned in this copy because it is related to the previous
+            # document, if previous document says so this too
+            # 'parent_id':False,
+            'date_document': False,
+            'invoice_printer' : '',
+            'fiscal_printer' : '',
+            # No cleaned in this copy because it is related to the previous
+            # document, if previous document says so this too
+            #'loc_req':False, 
+            'z_report': '',
         })
         return super(account_invoice, self).copy(cr, uid, id, default, context)
 
@@ -129,9 +137,6 @@
         return super(account_invoice, self).write(cr, uid, ids, vals,
                                                     context=context)
 
-account_invoice()
-
-
 class account_invoice_tax(osv.osv):
     _inherit = 'account.invoice.tax'
     _columns = {
@@ -139,6 +144,4 @@
         help="Tax relation to original tax, to be able to take off all data from invoices."),
     }
 
-account_invoice_tax()
-
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== modified file 'l10n_ve_fiscal_requirements/model/partner.py'
--- l10n_ve_fiscal_requirements/model/partner.py	2013-11-16 04:03:30 +0000
+++ l10n_ve_fiscal_requirements/model/partner.py	2013-11-17 23:21:43 +0000
@@ -308,5 +308,4 @@
             # force full VIES online check
             self.update_rif(cr, uid, ids, context=context)
         return True
-res_partner()
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added file 'l10n_ve_fiscal_requirements/test/account_customer_invoice.yml'
--- l10n_ve_fiscal_requirements/test/account_customer_invoice.yml	1970-01-01 00:00:00 +0000
+++ l10n_ve_fiscal_requirements/test/account_customer_invoice.yml	2013-11-17 23:21:43 +0000
@@ -0,0 +1,29 @@
+-
+  In order to test account invoice I create a new customer invoice
+-
+  I set the context that will be used for the encoding of all the invoices of this file
+-
+  !context
+    'type': 'out_invoice'
+-
+  I create a customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer0, view: account.invoice_form}:
+    payment_term: account.account_payment_term_advance
+    journal_id: account.sales_journal
+    partner_id: base.res_partner_3
+    reference_type: none
+    name: 'Test Customer Invoice - l10n_ve_fiscal_requirements'
+    invoice_line:
+      - product_id: product.product_product_5
+        quantity: 10.0
+-
+  I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer0}:
+    - state == 'draft'
+-
+  I check that the customer invoice is a "Customer Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_customer0, string: This is not a customer invoice}:
+    - type == 'out_invoice'

=== added file 'l10n_ve_fiscal_requirements/test/account_supplier_invoice.yml'
--- l10n_ve_fiscal_requirements/test/account_supplier_invoice.yml	1970-01-01 00:00:00 +0000
+++ l10n_ve_fiscal_requirements/test/account_supplier_invoice.yml	2013-11-17 23:21:43 +0000
@@ -0,0 +1,35 @@
+-
+  In order to test account invoice I create a new supplier invoice
+-
+  I set the context that will be used for the encoding of all the invoices of this file
+-
+  !context
+    'type': 'in_invoice'
+-
+  I create a supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier0, view: account.invoice_supplier_form }:
+    account_id: account.a_pay
+    company_id: base.main_company
+    currency_id: base.EUR
+    invoice_line:
+      - account_id: account.a_expense
+        name: '[PCSC234] PC Assemble SC234'
+        price_unit: 300.0
+        product_id: product.product_product_3
+        quantity: 10.0
+        uos_id: product.product_uom_unit
+    journal_id: account.expenses_journal
+    partner_id: base.res_partner_12
+    reference_type: none
+    type: 'in_invoice'
+-
+  I check that Initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier0}:
+    - state == 'draft'
+-
+  I check that the supplier invoice is a "Supplier Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier0, string: This is not a supplier invoice}:
+    - type == 'in_invoice'

=== modified file 'l10n_ve_fiscal_requirements/test/fr_damaged.yml'
--- l10n_ve_fiscal_requirements/test/fr_damaged.yml	2013-11-14 20:14:21 +0000
+++ l10n_ve_fiscal_requirements/test/fr_damaged.yml	2013-11-17 23:21:43 +0000
@@ -109,7 +109,6 @@
   !python {model: account.journal}: |
     aj_ids = self.search(cr, uid, [('type', '=','sale')])
     assert aj_ids, 'There are no Journals'
-
     self.write(cr, uid, aj_ids, {'update_posted': True})
 -
   6.13 Verify that the journal was properly updated
@@ -171,4 +170,3 @@
       pass
     finally:
       cr.sql_log = l
-  

=== modified file 'l10n_ve_generic/__openerp__.py'
--- l10n_ve_generic/__openerp__.py	2012-09-24 12:47:27 +0000
+++ l10n_ve_generic/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -39,7 +39,7 @@
     "update_xml" : ['account_tax_code.xml',"account_chart.xml",
                     'account_tax.xml','l10n_chart_ve_wizard.xml'],
     "auto_install": False,
-    "installable": True,
+    "installable": False,
     'images': ['images/config_chart_l10n_ve.jpeg','images/l10n_ve_chart.jpeg'],
 }
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== modified file 'l10n_ve_imex/__openerp__.py'
--- l10n_ve_imex/__openerp__.py	2013-04-29 16:31:11 +0000
+++ l10n_ve_imex/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -55,5 +55,5 @@
         'workflow/customs_form.xml',
     ],
     "active": False,
-    "installable": True,
+    "installable": False,
 }

=== modified file 'l10n_ve_sale_purchase/__openerp__.py'
--- l10n_ve_sale_purchase/__openerp__.py	2013-09-18 23:55:08 +0000
+++ l10n_ve_sale_purchase/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -41,7 +41,7 @@
     ],
     'demo' : [],
     'active' : False,
-    'installable': True,
+    'installable': False,
     'test': [
 #        'test/purchase_order.yml',
 #        'test/purchase_picking.yml',

=== modified file 'l10n_ve_split_invoice/__openerp__.py'
--- l10n_ve_split_invoice/__openerp__.py	2013-11-14 20:44:23 +0000
+++ l10n_ve_split_invoice/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -58,5 +58,5 @@
         'test/spl_test.yml',
             ],
     "active": False,
-    "installable": True,
+    "installable": False,
 }

=== modified file 'l10n_ve_vat_write_off/__openerp__.py'
--- l10n_ve_vat_write_off/__openerp__.py	2013-05-02 17:01:41 +0000
+++ l10n_ve_vat_write_off/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -13,6 +13,6 @@
         "view/l10n_ve_vat_write_off.xml",
     ],
     "active": False,
-    "installable": True,
+    "installable": False,
 }
 

=== modified file 'l10n_ve_withholding/__openerp__.py'
--- l10n_ve_withholding/__openerp__.py	2013-09-18 23:55:08 +0000
+++ l10n_ve_withholding/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -49,9 +49,9 @@
     ],
     'demo_xml': [],
     'test': [
-#        'test/wh_pay_invoice.yml'
+        'test/wh_pay_invoice.yml'
         ],
-    'installable': True,
+    'installable': False,
     'active': False,
 }
 

=== modified file 'l10n_ve_withholding_islr/__openerp__.py'
--- l10n_ve_withholding_islr/__openerp__.py	2013-11-14 20:14:21 +0000
+++ l10n_ve_withholding_islr/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -74,5 +74,5 @@
 #            'test/test_duplicate.yml',
         ],
     "active": False,
-    "installable": True
+    "installable": False,
 }

=== modified file 'l10n_ve_withholding_iva/__openerp__.py'
--- l10n_ve_withholding_iva/__openerp__.py	2013-11-17 02:03:44 +0000
+++ l10n_ve_withholding_iva/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -72,20 +72,20 @@
             ],
 
     'test': [
-#            'test/iva_purchase.yml',
-#            'test/iva_purchase_wh.yml',
-#            'test/iva_purchase_nret1.yml',
-#            'test/iva_purchase_nret2.yml',
-#            'test/iva_purchase_nret3.yml',
-#            'test/iva_sale.yml',
-#            'test/iva_sale_wh.yml',
-#            'test/iva_test_invoice.yml',
-#            'test/iva_test_wh.yml',
-#            'test/iva_test_wh_2.yml',
-#            'test/iva_test_wh_3.yml',
-#            'test/iva_test_wh_4.yml',
+            'test/iva_purchase.yml',
+            'test/iva_purchase_wh.yml',
+            'test/iva_purchase_nret1.yml',
+            'test/iva_purchase_nret2.yml',
+            'test/iva_purchase_nret3.yml',
+            'test/iva_sale.yml',
+            'test/iva_sale_wh.yml',
+            'test/iva_test_invoice.yml',
+            'test/iva_test_wh.yml',
+            'test/iva_test_wh_2.yml',
+            'test/iva_test_wh_3.yml',
+            'test/iva_test_wh_4.yml',
         ],
-    'installable': True,
+    'installable': False,
     'active': False,
 }
 

=== modified file 'l10n_ve_withholding_iva/model/wh_iva.py'
--- l10n_ve_withholding_iva/model/wh_iva.py	2013-07-23 14:23:34 +0000
+++ l10n_ve_withholding_iva/model/wh_iva.py	2013-11-17 23:21:43 +0000
@@ -725,6 +725,7 @@
         fortnight_str = {'True': ' - Second Fortnight)',
                         'False': ' - First Fortnight)'}
         for awi_brw in self.browse(cr, uid, ids, context=context):
+            if not awi_brw.wh_lines: return False
             if awi_brw.type in ['out_invoice']: return True
             for awil_brw in awi_brw.wh_lines:
                 awil_period_id, awil_fortnight = per_obj.find_fortnight(

=== modified file 'l10n_ve_withholding_iva/test/iva_purchase_nret1.yml'
--- l10n_ve_withholding_iva/test/iva_purchase_nret1.yml	2013-01-15 15:16:32 +0000
+++ l10n_ve_withholding_iva/test/iva_purchase_nret1.yml	2013-11-17 23:21:43 +0000
@@ -32,7 +32,7 @@
     date_invoice: !eval time.strftime('%Y-%m-%d')
     partner_id: base.res_partner_23
     reference_type: none
-    journal_id: account.sales_journal
+    journal_id: account.expenses_journal
     wh_iva: True
     invoice_line:
       - product_id: product.product_product_3

=== modified file 'l10n_ve_withholding_iva/test/iva_purchase_nret2.yml'
--- l10n_ve_withholding_iva/test/iva_purchase_nret2.yml	2013-01-15 15:16:32 +0000
+++ l10n_ve_withholding_iva/test/iva_purchase_nret2.yml	2013-11-17 23:21:43 +0000
@@ -42,7 +42,7 @@
     date_invoice: !eval time.strftime('%Y-%m-%d')
     partner_id: base.res_partner_23
     reference_type: none
-    journal_id: account.sales_journal
+    journal_id: account.expenses_journal
     wh_iva: True
     invoice_line:
       - product_id: product.product_product_3

=== modified file 'l10n_ve_withholding_iva/test/iva_purchase_nret3.yml'
--- l10n_ve_withholding_iva/test/iva_purchase_nret3.yml	2013-01-15 15:16:32 +0000
+++ l10n_ve_withholding_iva/test/iva_purchase_nret3.yml	2013-11-17 23:21:43 +0000
@@ -52,7 +52,7 @@
     date_invoice: !eval time.strftime('%Y-%m-%d')
     partner_id: base.res_partner_23
     reference_type: none
-    journal_id: account.sales_journal
+    journal_id: account.expenses_journal
     wh_iva: True
     invoice_line:
       - product_id: product.product_product_3

=== modified file 'l10n_ve_withholding_iva/test/iva_purchase_wh.yml'
--- l10n_ve_withholding_iva/test/iva_purchase_wh.yml	2013-01-15 15:16:32 +0000
+++ l10n_ve_withholding_iva/test/iva_purchase_wh.yml	2013-11-17 23:21:43 +0000
@@ -26,13 +26,14 @@
 -
   2.4 I create a purchase invoice
 -
-  !record {model: account.invoice, id: test_purchase_iva02}:
-    type: out_invoice
+  !record {model: account.invoice, id: test_purchase_iva02, view: account.invoice_supplier_form, context:{'default_type': 'in_invoice', 'type': 'in_invoice', 'journal_type': 'purchase'}}:
+    type: in_invoice
+    state: draft
     company_id: base.main_company
     date_invoice: !eval time.strftime('%Y-%m-%d')
     partner_id: base.res_partner_23
     reference_type: none
-    journal_id: account.sales_journal
+    journal_id: 
     wh_iva: True
     invoice_line:
       - product_id: product.product_product_3
@@ -50,10 +51,19 @@
         invoice_line_tax_id:
         - l10n_ve_fiscal_requirements.iva_purchase1
 -
-  2.5 I check that Initially customer invoice is in the "Draft" state
+  2.5 I check that Initially supplier invoice is in the "Draft" state
 -
-  !assert {model: account.invoice, id: test_purchase_iva02}:
-    - state == 'draft'
+  !python {model: account.invoice}: |
+    ap_obj=self.pool.get('account.period')
+    import time
+    period = ap_obj.find(cr, uid, time.strftime('%Y/%m/%d'), context={'company_id':ref("base.main_company")})
+    assert period, 'There aren\'t periods for this date'
+    ap_obj.write(cr, uid, period, {'special':False})
+    self.write(cr,uid,ref('test_purchase_iva02'),{'journal_id':ref('account.expenses_journal'),'type':'in_invoice'})
+    ai_brw = self.browse(cr,uid,ref('test_purchase_iva02'))
+    assert ai_brw.type == 'in_invoice', "Not the intended type. It should be 'in_invoice' not '%s'"%ai_brw.type
+    assert ai_brw.state == 'draft', "Not the intended state. It should be 'draft' not '%s'"%ai_brw.state
+    assert ai_brw.journal_id.id == ref('account.expenses_journal'), "Not the intended Journal. It should be 'Expenses Journal - (test)' not '%s'"%ai_brw.journal_id.name
 -
   2.6 I change the state of invoice to "Proforma2" by clicking PRO-FORMA button
 -
@@ -76,6 +86,7 @@
   2.10 Set control number for the invoice
 -
   !python {model: account.invoice}: |
-    self.write(cr, uid, [ref("test_purchase_iva02")], {'nro_ctrl':'iva002', 'journal_id':ref("account.expenses_journal")})
+    self.write(cr, uid, [ref("test_purchase_iva02")], {'nro_ctrl':'iva002'})
     ai_brw = self.browse(cr, uid, ref("test_purchase_iva02"))
-    assert ai_brw.nro_ctrl=='iva002' and ai_brw.journal_id.id == ref("account.expenses_journal"), 'Invoice wasn\'t updated properly'
+    assert ai_brw.nro_ctrl=='iva002', "Not the intended Control Number. It should be 'iva002' not '%s'"%ai_brw.nro_ctrl
+    assert ai_brw.journal_id.id == ref('account.expenses_journal'), "Not the intended Journal. It should be 'Expenses Journal - (test)' not '%s'"%ai_brw.journal_id.name

=== modified file 'l10n_ve_withholding_iva/test/iva_sale_wh.yml'
--- l10n_ve_withholding_iva/test/iva_sale_wh.yml	2013-01-17 19:19:09 +0000
+++ l10n_ve_withholding_iva/test/iva_sale_wh.yml	2013-11-17 23:21:43 +0000
@@ -109,11 +109,13 @@
     ap_obj=self.pool.get('account.period')
     period = ap_obj.find(cr, uid, time.strftime('%Y/%m/%d'), context={'company_id':ref("base.main_company")})
     assert period, 'There aren\'t periods for this date'
+    ap_obj.write(cr, uid, period, {'special':False})
     ap_brw = ap_obj.browse(cr, uid, period[0])
     if ap_brw.state == 'done':
         ap.action_draft(cr, uid, period)
         ap_brw = ap.browse(cr, uid, period[0])
     assert ap_brw.state == 'draft', 'Period is not open as it should be'
+    assert ap_brw.special == False, 'Period is special. It should be'
 
     self.write(cr, uid, ref("wh_iva_sale01"), {'period_id':period[0]})
 -

=== modified file 'l10n_ve_withholding_iva/test/iva_test_wh_2.yml'
--- l10n_ve_withholding_iva/test/iva_test_wh_2.yml	2013-01-18 19:55:38 +0000
+++ l10n_ve_withholding_iva/test/iva_test_wh_2.yml	2013-11-17 23:21:43 +0000
@@ -10,28 +10,55 @@
 -
   10.2 I generate a withholding vat customer
 -
-  !record {model: account.wh.iva, id: wh_iva_test02}:
+  !record {model: account.wh.iva, id: wh_iva_test02, view: view_wh_iva_form_customer, context: {'type':'out_invoice'}}:
     journal_id: wh_sales_journal
     partner_id: base.res_partner_23
     number: 222444333000
     account_id: account.a_recv
     name: 'IVA wh vat customer Vauxoo test 2'
     type: out_invoice
+    state: draft
 -
   10.3 I check withholding vat customer
 -
-  !assert {model: account.wh.iva, id: wh_iva_test02, string: Wh was not saved}:
-    - state == 'draft'
+  !python {model: account.wh.iva}: |
+    values = {}
+    rp_obj = self.pool.get('res.partner')
+    rp_brw = rp_obj.browse(cr,uid,ref("base.res_partner_23"))
+    res = self.onchange_partner_id(cr, uid, [ref('wh_iva_test02')], 'out_invoice', rp_brw.id)
+    values['wh_lines'] = [(0,0,{'invoice_id':ref('test_iva01'), 'name':'N/A', 'wh_iva_rate':rp_brw.wh_iva_rate})]
+    values['account_id'] = res['value']['account_id']
+    values['type'] = 'out_invoice'
+    self.write(cr,uid,ref('wh_iva_test02'),values)
+    awi_brw = self.browse(cr,uid,ref('wh_iva_test02'))
+    assert awi_brw.type == 'out_invoice', "Not the intended type. It should be 'out_invoice' not '%s'"%awi_brw.type
+    assert awi_brw.state == 'draft', "Not the intended state. It should be 'draft' not '%s'"%awi_brw.state
+    assert awi_brw.wh_lines, "There are no lines to withhold"
+    self.compute_amount_wh(cr,uid,[ref('wh_iva_test02')])
 -
   10.4 I confirm the withholding vat customer
 -
-  !workflow {model: account.wh.iva, action: wh_iva_confirmed, ref: wh_iva_test02}
+  !python {model: account.wh.iva}: |
+    import netsvc
+    from openerp.osv import osv
+    wf_service = netsvc.LocalService("workflow")
+    try:
+      self.check_wh_lines_fortnights(cr, uid, [ref('wh_iva_test02')])
+      self.check_invoice_nro_ctrl(cr, uid, [ref('wh_iva_test02')])
+      self.check_vat_wh(cr, uid, [ref('wh_iva_test02')])
+      self.check_wh_taxes(cr, uid, [ref('wh_iva_test02')])
+      self.check_wh_taxes(cr, uid, [ref('wh_iva_test02')])
+      assert False, "An exception should have been raised, unwithholdable taxes  are not allowed to reach here!"
+    except osv.except_osv:
+      # exception was raised as expected
+      pass
 -
   10.5 I check withholding vat is not confirmed
   # TODO: Check this test case, it fails. It means that something in the wh_iva_confirmed action is wrong.
 -
-  !assert {model: account.wh.iva, id: wh_iva_test02, string: Could not confirm the wh}:
-    - state != 'confirmed'
+  !python {model: account.wh.iva}: |
+    awi_brw = self.browse(cr,uid,ref('wh_iva_test02'))
+    assert awi_brw.state == 'draft', "Not the intended state. It should be 'draft' not '%s'"%awi_brw.state
 -
   10.6 I try to mark it as done
 -

=== modified file 'l10n_ve_withholding_iva/test/iva_test_wh_3.yml'
--- l10n_ve_withholding_iva/test/iva_test_wh_3.yml	2013-01-18 19:55:38 +0000
+++ l10n_ve_withholding_iva/test/iva_test_wh_3.yml	2013-11-17 23:21:43 +0000
@@ -16,11 +16,15 @@
     account_id: account.a_recv
     name: 'IVA wh vat customer Vauxoo test 3'
     type: out_invoice
+    state: draft
 -
   11.3 I check withholding vat customer
 -
-  !assert {model: account.wh.iva, id: wh_iva_test03, string: Wh was not saved}:
-    - state == 'draft'
+  !python {model: account.wh.iva}: |
+    self.write(cr,uid,ref('wh_iva_test03'),{'type':'out_invoice'})
+    awi_brw = self.browse(cr,uid,ref('wh_iva_test03'))
+    assert awi_brw.type == 'out_invoice', "Not the intended type. It should be 'out_invoice' not '%s'"%awi_brw.type
+    assert awi_brw.state == 'draft', "Not the intended state. It should be 'draft' not '%s'"%awi_brw.state
 -
   11.4 Before validating let's add the period
 -
@@ -43,8 +47,9 @@
   11.7 I check withholding vat is not confirmed
   # TODO: Check this test case, it fails. It means that something in the wh_iva_confirmed action is wrong.
 -
-  !assert {model: account.wh.iva, id: wh_iva_test03, string: Could not confirm the wh}:
-    - state != 'confirmed'
+  !python {model: account.wh.iva}: |
+    awi_brw = self.browse(cr,uid,ref('wh_iva_test03'))
+    assert awi_brw.state == 'draft', "Not the intended state. It should be 'draft' not '%s'"%awi_brw.state
 -
   11.8 I try to mark it as done
 -
@@ -52,5 +57,6 @@
 -
   11.9 I check withholding vat is not done
 -
-  !assert {model: account.wh.iva, id: wh_iva_test01, string: Wh cannot be in done state}:
-    - state != 'done'
+  !python {model: account.wh.iva}: |
+    awi_brw = self.browse(cr,uid,ref('wh_iva_test03'))
+    assert awi_brw.state == 'draft', "Not the intended state. It should be 'draft' not '%s'"%awi_brw.state

=== modified file 'l10n_ve_withholding_iva/test/iva_test_wh_4.yml'
--- l10n_ve_withholding_iva/test/iva_test_wh_4.yml	2013-01-18 19:55:38 +0000
+++ l10n_ve_withholding_iva/test/iva_test_wh_4.yml	2013-11-17 23:21:43 +0000
@@ -16,11 +16,15 @@
     account_id: account.a_recv
     name: 'IVA wh vat customer Vauxoo test 3'
     type: out_invoice
+    state: draft
 -
   12.3 I check withholding vat customer
 -
-  !assert {model: account.wh.iva, id: wh_iva_test04, string: Wh was not saved}:
-    - state == 'draft'
+  !python {model: account.wh.iva}: |
+    self.write(cr,uid,ref('wh_iva_test04'),{'type':'out_invoice'})
+    awi_brw = self.browse(cr,uid,ref('wh_iva_test04'))
+    assert awi_brw.type == 'out_invoice', "Not the intended type. It should be 'out_invoice' not '%s'"%awi_brw.type
+    assert awi_brw.state == 'draft', "Not the intended state. It should be 'draft' not '%s'"%awi_brw.state
 -
   12.4 Before validating let's add the period
 -

=== modified file 'l10n_ve_withholding_muni/__openerp__.py'
--- l10n_ve_withholding_muni/__openerp__.py	2013-01-30 20:27:24 +0000
+++ l10n_ve_withholding_muni/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -47,7 +47,7 @@
     ],
     'demo_xml': [],
     'test': [],
-    'installable': True,
+    'installable': False,
     'active': False,
 }
 

=== modified file 'l10n_ve_withholding_src/__openerp__.py'
--- l10n_ve_withholding_src/__openerp__.py	2013-04-18 21:03:12 +0000
+++ l10n_ve_withholding_src/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -67,5 +67,5 @@
         
     ],
     "active": False,
-    "installable": True
+    "installable": False,
 }

=== modified file 'ovl/__openerp__.py'
--- ovl/__openerp__.py	2013-11-14 06:07:44 +0000
+++ ovl/__openerp__.py	2013-11-17 23:21:43 +0000
@@ -87,5 +87,5 @@
     "images" : [],
     "auto_install": False,
     "application": True,
-    "installable": True,
+    "installable": False,
 }