Merge lp:~openerp-venezuela/openerp-venezuela-localization/l10n_ve_fiscal_req-debit_credit-dev_sabrina into lp:openerp-venezuela-localization

[Sabrina Romero - http://www.vauxoo.com]

[FIX] Changes for compatibility with debit_credit_note dependency. Removing redundancy.

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

review: Needs Fixing

[lp:~openerp-venezuela/openerp-venezuela-localization/l10n_ve_fiscal_req-debit_credit-dev_sabrina]

975. By Sabrina Romero - http://www.vauxoo.com on 2014-01-27

[FIX] Methods belonging to OVL were removed of debit_credit_note module.

[Sabrina Romero - http://www.vauxoo.com (sabrina)]

[FIX] Methods belonging to OVL were removed of debit_credit_note module.

review: Needs Resubmitting

[lp:~openerp-venezuela/openerp-venezuela-localization/l10n_ve_fiscal_req-debit_credit-dev_sabrina]

976. By Sabrina Romero - http://www.vauxoo.com on 2014-01-27

[ADD] Refund tests added for customer and supplier.

[Sabrina Romero - http://www.vauxoo.com (sabrina)]

[ADD] Refund tests added for customer and supplier.

review: Needs Resubmitting

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

It LGTM,

I will make a last revision and after testing the whole OVL

with this new proposal and if everything goes fine I will

merge it today.

Best Regards.

review: Approve

[lp:~openerp-venezuela/openerp-venezuela-localization/l10n_ve_fiscal_req-debit_credit-dev_sabrina]

977. By Sabrina Romero - http://www.vauxoo.com on 2014-01-30

[FIX] amount_total correct assignment.

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

LGTM

review: Approve

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

Will be merged when AV70 fixess in Sabrina's branches are merged

[lp:~openerp-venezuela/openerp-venezuela-localization/l10n_ve_fiscal_req-debit_credit-dev_sabrina]

978. By Sabrina Romero - http://www.vauxoo.com on 2014-02-11

[FIX] Using browse method instead read and splitting date removed.

[Sabrina Romero - http://www.vauxoo.com (sabrina)]

[FIX] Using browse method instead read and splitting date removed.
Tested to review 978, working!

review: Needs Resubmitting

Unmerged revisions

978. By Sabrina Romero - http://www.vauxoo.com on 2014-02-11

[FIX] Using browse method instead read and splitting date removed.

977. By Sabrina Romero - http://www.vauxoo.com on 2014-01-30

[FIX] amount_total correct assignment.

976. By Sabrina Romero - http://www.vauxoo.com on 2014-01-27

[ADD] Refund tests added for customer and supplier.

975. By Sabrina Romero - http://www.vauxoo.com on 2014-01-27

[FIX] Methods belonging to OVL were removed of debit_credit_note module.

974. By Sabrina Romero - http://www.vauxoo.com on 2014-01-23

[FIX] Changes for compatibility with debit_credit_note dependency.

=== modified file 'l10n_ve_fiscal_requirements/__openerp__.py'
--- l10n_ve_fiscal_requirements/__openerp__.py	2014-01-20 13:03:50 +0000
+++ l10n_ve_fiscal_requirements/__openerp__.py	2014-02-11 00:22:34 +0000
@@ -96,6 +96,8 @@
     'test': [
         'test/account_customer_invoice.yml',
         'test/account_supplier_invoice.yml',
+        'test/account_customer_refund.yml',
+        'test/account_supplier_refund.yml',
         'test/fr_vat_search_test.yml',
         'test/fr_ut_test.yml',
         'test/fr_vat_test.yml',
 
=== added file 'l10n_ve_fiscal_requirements/test/account_customer_refund.yml'
--- l10n_ve_fiscal_requirements/test/account_customer_refund.yml	1970-01-01 00:00:00 +0000
+++ l10n_ve_fiscal_requirements/test/account_customer_refund.yml	2014-02-11 00:22:34 +0000
@@ -0,0 +1,602 @@
+-
+  I set the context that will be used for the encoding of all the invoices of this file
+-
+  !context
+    'type': 'out_invoice'
+-
+  1.- In order to test account refund with "Create a draft refund" method
+-
+  1.1.- I create a customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer0, view: account.invoice_form}: &customer_invoice
+    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 - debit_credit_note'
+    invoice_line:
+      - product_id: product.product_product_5
+        price_unit: 850.0
+        quantity: 1.0
+-       
+  1.2.- I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer0}:
+    - state == 'draft'
+-
+  1.3.- 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'
+-
+  1.4.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_customer0}
+-
+  1.5.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_customer0}:
+    - state == 'open'
+-
+  1.6.- I create a customer refund using the method "Create a draft refund" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('account.refund_expenses_journal'),
+        'description': 'Desde el test',
+        'filter_refund': 'refund',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_customer0')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])    
+    assert rfnd_customer_inv_brw[0].state == 'draft', 'The customer refund was not properly created, the state must be Draft.'
+-
+  1.7.- I check the correct asigning of parent_id in the customer refund created, for that
+  the customer refund must have the customer invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    customer_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_customer0'))
+    assert (rfnd_customer_inv_brw[0].parent_id.id == customer_inv_brw.id), 'The customer refund dont have the correct parent_id.'
+-
+  2.- In order to test account refund with "Create refund and reconcile" method
+-
+  2.1.- I create a new customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer1, view: account.invoice_form }:
+    <<: *customer_invoice
+-
+  2.2.- I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer1}:
+    - state == 'draft'
+-
+  2.3.- I check that the customer invoice is a "Customer Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_customer1, string: This is not a customer invoice}:
+    - type == 'out_invoice'
+-
+  2.4.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_customer1}
+-
+  2.5.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_customer1}:
+    - state == 'open'
+-
+  2.6.- I create a customer refund using the method "Create refund and reconcile" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('account.refund_expenses_journal'),
+        'description': 'Desde el test',
+        'filter_refund': 'cancel',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_customer1')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])    
+    assert rfnd_customer_inv_brw[0].state == 'paid', 'The customer refund was not properly created, the state must be Paid.'
+-
+  2.7.- I check the correct asigning of parent_id in the customer refund created, for that
+  the customer refund must have the customer invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    customer_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_customer1'))    
+    assert (rfnd_customer_inv_brw[0].parent_id.id == customer_inv_brw.id), 'The customer refund dont have the correct parent_id.'
+-
+  3.- In order to test account refund with "Create refund, reconcile and create a new draft invoice"
+  method 
+-  
+  3.1.- I create a new customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer2, view: account.invoice_form }:
+    <<: *customer_invoice
+-
+  3.2.- I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer2}:
+    - state == 'draft'
+-
+  3.3.- I check that the customer invoice is a "Customer Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_customer2, string: This is not a customer invoice}:
+    - type == 'out_invoice'
+-
+  3.4.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_customer2}
+-
+  3.5.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_customer2}:
+    - state == 'open'
+-
+  3.6.- I create a customer refund using the method "Create refund, reconcile and create a new draft invoice" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('account.refund_expenses_journal'),
+        'description': 'Desde el test',
+        'filter_refund': 'modify',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_customer2')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])    
+    assert rfnd_customer_inv_brw[0].state == 'paid', 'The customer refund was not properly created, the state must be Paid.'
+-
+  3.7.- I check the correct asigning of parent_id in the customer refund created, for that
+  the customer refund must have the customer invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    customer_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_customer2'))    
+    assert (rfnd_customer_inv_brw[0].parent_id.id == customer_inv_brw.id), 'The customer refund dont have the correct parent_id.'
+-
+  3.8.- I check that the new customer invoice was created in Draft state
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    inv_src = obj_inv.search(cr, uid, [], order='id')
+    customer_inv_brw = obj_inv.browse(cr, uid, inv_src[-1])
+    assert customer_inv_brw.state == 'draft', 'The new customer invoice was not properly created, the state must be Draft.'
+-
+  4.- In order to test account refund with "Create a draft refund" method in a "Paid" invoice
+-
+  4.1.- I create a new partner
+-
+  !record {model: 'res.partner', id: res_partner8}:
+    name: Juan Perez
+    use_parent_address: True
+    function: Tests
+    email: [e-mail]
+-
+  4.2.- I create a Sale Journal.
+-
+  !record {model: account.journal, id: sale_journal_debit}:
+    code: sjou_test
+    company_id: base.main_company
+    name: Sale Journal For Debit Credit Note Tests
+    sequence_id: account.sequence_sale_journal
+    type: sale
+-
+  4.3.- I create a new customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer3}:
+    account_id: account.a_recv
+    company_id: base.main_company
+    currency_id: base.EUR
+    invoice_line:
+      - account_id: account.a_sale
+        name: '[PCSC234] PC Assemble SC234'
+        price_unit: 450.0
+        quantity: 1.0
+        product_id: product.product_product_3
+        uos_id: product.product_uom_unit
+    journal_id: sale_journal_debit
+    partner_id: res_partner8
+-
+  4.4.- I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer3}:
+    - state == 'draft'
+-
+  4.5.- I check that the customer invoice is a "Customer Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_customer3, string: This is not a customer invoice}:
+    - type == 'out_invoice'
+-
+  4.6.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_customer3}
+-
+  4.7.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_customer3}:
+    - state == 'open'
+-
+  4.8.- I check that a payment entry gets created in the account.move.line
+-
+  !python {model: account.invoice}: |
+    acc_id=self.browse(cr, uid, ref("account_invoice_customer3"))
+    assert(acc_id.move_id)
+-
+  4.9.- I create a Cash Journal.
+-
+  !record {model: account.journal, id: cash_journal_debit}:
+    code: cjou_test
+    company_id: base.main_company
+    name: Cash Journal For Debit Credit Note Tests
+    sequence_id: account.sequence_cash_journal
+    type: cash
+-
+  4.10.- I will create and post an account voucher with the amount total 
+  of the customer invoice created for the partner Juan Perez.
+-
+  !python {model: account.voucher}: |
+    import netsvc
+    import time
+    vals = {}
+    amount_total = self.pool.get('account.invoice').browse(cr, uid, [ref('account_invoice_customer3')], context=None)[0].amount_total
+    journal_id = self.default_get(cr, uid, ['journal_id']).get('journal_id',None)
+    res = self.onchange_partner_id(cr, uid, [], ref("res_partner8"), journal_id, 0.0, 1, ttype='receipt', date=False)
+    vals = {
+        'account_id': ref('account.cash'),
+        'amount': amount_total,
+        'company_id': ref('base.main_company'),
+        'journal_id': ref('cash_journal_debit'),
+        'partner_id': ref('res_partner8'),
+        'period_id': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'type': 'receipt',
+    }
+    if not res['value']['line_cr_ids']:
+      res['value']['line_cr_ids'] = [{'type': 'cr', 'account_id': ref('account.a_recv'),}]
+    res['value']['line_cr_ids'][0]['amount'] = amount_total
+    vals['line_cr_ids'] = [(0,0,i) for i in res['value']['line_cr_ids']]
+    id = self.create(cr, uid, vals)
+    voucher_id = self.browse(cr, uid, id)
+    assert (voucher_id.state=='draft'), "Voucher is not in draft state"
+    wf_service = netsvc.LocalService("workflow")
+    wf_service.trg_validate(uid, 'account.voucher', voucher_id.id, 'proforma_voucher', cr)
+-
+  4.11.- I check if the state of the invoice is "Paid"
+-
+  !assert {model: account.invoice, id: account_invoice_customer3}:
+    - state == 'paid'
+-
+  4.12.- I modified the sale journal record so it can cancel invoice
+-
+  !record {model: account.journal, id: sale_journal_debit}:
+    update_posted: 1
+-
+  4.13.- I modified the cash journal record so it can cancel invoice
+-
+  !record {model: account.journal, id: cash_journal_debit}:
+    update_posted: 1
+-
+  4.14.- I create a Sale Refund Journal.
+-
+  !record {model: account.journal, id: sale_refund_journal_debit}:
+    code: csrjou_test
+    company_id: base.main_company
+    name: Sale Refund Journal For Debit Credit Note Tests
+    sequence_id: account.sequence_refund_sales_journal
+    type: sale_refund
+-
+  4.15.- I create a customer refund using the method "Create a draft refund" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('sale_refund_journal_debit'),
+        'description': 'Desde el test',
+        'filter_refund': 'refund',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_customer3')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])
+    assert rfnd_customer_inv_brw[0].state == 'draft', 'The customer refund was not properly created, the state must be Draft.'
+-
+  4.16.- I check the correct asigning of parent_id in the customer refund created, for that
+  the customer refund must have the customer invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    customer_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_customer3'))    
+    assert (rfnd_customer_inv_brw[0].parent_id.id == customer_inv_brw.id), 'The customer refund dont have the correct parent_id.'
+-
+  5.- In order to test account refund with "Create refund and reconcile" method in a "Paid" invoice
+-
+  5.1.- I create a new partner
+-
+  !record {model: 'res.partner', id: res_partner7}:
+    name: Pedro Perez
+    use_parent_address: True
+    function: Tests
+    email: [e-mail]
+-
+  5.2.- I create a new customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer4}:
+    account_id: account.a_recv
+    company_id: base.main_company
+    currency_id: base.EUR
+    invoice_line:
+      - account_id: account.a_sale
+        name: '[PCSC234] PC Assemble SC234'
+        price_unit: 450.0
+        quantity: 1.0
+        product_id: product.product_product_3
+        uos_id: product.product_uom_unit
+    journal_id: sale_journal_debit
+    partner_id: res_partner7
+-
+  5.3.- I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer4}:
+    - state == 'draft'
+-
+  5.4.- I check that the customer invoice is a "Customer Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_customer4, string: This is not a customer invoice}:
+    - type == 'out_invoice'
+-
+  5.5.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_customer4}
+-
+  5.6.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_customer4}:
+    - state == 'open'
+-
+  5.7.- I check that a payment entry gets created in the account.move.line
+-
+  !python {model: account.invoice}: |
+    acc_id=self.browse(cr, uid, ref("account_invoice_customer4"))
+    assert(acc_id.move_id)
+-
+  5.8.- I will create and post an account voucher with the amount total 
+  of the customer invoice created for the partner Pedro Perez.
+-
+  !python {model: account.voucher}: |
+    import netsvc
+    import time
+    vals = {}
+    amount_total = self.pool.get('account.invoice').browse(cr, uid, [ref('account_invoice_customer4')], context=None)[0].amount_total
+    journal_id = self.default_get(cr, uid, ['journal_id']).get('journal_id',None)
+    res = self.onchange_partner_id(cr, uid, [], ref("res_partner7"), journal_id, 0.0, 1, ttype='receipt', date=False)
+    vals = {
+        'account_id': ref('account.cash'),
+        'amount': amount_total,
+        'company_id': ref('base.main_company'),
+        'journal_id': ref('cash_journal_debit'),
+        'partner_id': ref('res_partner7'),
+        'period_id': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'type': 'receipt',
+    }
+    if not res['value']['line_cr_ids']:
+      res['value']['line_cr_ids'] = [{'type': 'cr', 'account_id': ref('account.a_recv'),}]
+    res['value']['line_cr_ids'][0]['amount'] = amount_total
+    vals['line_cr_ids'] = [(0,0,i) for i in res['value']['line_cr_ids']]
+    id = self.create(cr, uid, vals)
+    voucher_id = self.browse(cr, uid, id)
+    assert (voucher_id.state=='draft'), "Voucher is not in draft state"
+    wf_service = netsvc.LocalService("workflow")
+    wf_service.trg_validate(uid, 'account.voucher', voucher_id.id, 'proforma_voucher', cr)
+-
+  5.9.- I check if the state of the invoice is "Paid"
+-
+  !assert {model: account.invoice, id: account_invoice_customer4}:
+    - state == 'paid'
+-
+  5.10.- I create a customer refund using the method "Create refund and reconcile" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('sale_refund_journal_debit'),
+        'description': 'Desde el test',
+        'filter_refund': 'cancel',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_customer4')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])
+    assert rfnd_customer_inv_brw[0].state == 'paid', 'The customer refund was not properly created, the state must be Paid.'
+-
+  5.11.- I check the correct asigning of parent_id in the customer refund created, for that
+  the customer refund must have the customer invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    customer_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_customer4'))    
+    assert (rfnd_customer_inv_brw[0].parent_id.id == customer_inv_brw.id), 'The customer refund dont have the correct parent_id.'
+-
+  6.- In order to test account refund with "Create refund, reconcile and create a new draft invoice"
+  method in a "Paid" invoice
+-
+  6.1.- I create a new partner
+-
+  !record {model: 'res.partner', id: res_partner9}:
+    name: Maria Perez
+    use_parent_address: True
+    function: Tests
+    email: [e-mail]
+-
+  6.2.- I create a new customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer5}:
+    account_id: account.a_recv
+    company_id: base.main_company
+    currency_id: base.EUR
+    invoice_line:
+      - account_id: account.a_sale
+        name: '[PCSC234] PC Assemble SC234'
+        price_unit: 450.0
+        quantity: 1.0
+        product_id: product.product_product_3
+        uos_id: product.product_uom_unit
+    journal_id: sale_journal_debit
+    partner_id: res_partner9
+-
+  6.3.- I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer5}:
+    - state == 'draft'
+-
+  6.4.- I check that the customer invoice is a "Customer Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_customer5, string: This is not a customer invoice}:
+    - type == 'out_invoice'
+-
+  6.5.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_customer5}
+-
+  6.6.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_customer5}:
+    - state == 'open'
+-
+  6.7.- I check that a payment entry gets created in the account.move.line
+-
+  !python {model: account.invoice}: |
+    acc_id=self.browse(cr, uid, ref("account_invoice_customer5"))
+    assert(acc_id.move_id)
+-
+  6.8.- I will create and post an account voucher with the amount total 
+  of the customer invoice created for the partner Maria Perez.
+-
+  !python {model: account.voucher}: |
+    import netsvc
+    import time
+    vals = {}
+    amount_total = self.pool.get('account.invoice').browse(cr, uid, [ref('account_invoice_customer5')], context=None)[0].amount_total
+    journal_id = self.default_get(cr, uid, ['journal_id']).get('journal_id',None)
+    res = self.onchange_partner_id(cr, uid, [], ref("res_partner9"), journal_id, 0.0, 1, ttype='receipt', date=False)
+    vals = {
+        'account_id': ref('account.cash'),
+        'amount': amount_total,
+        'company_id': ref('base.main_company'),
+        'journal_id': ref('cash_journal_debit'),
+        'partner_id': ref('res_partner9'),
+        'period_id': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'type': 'receipt',
+    }
+    if not res['value']['line_cr_ids']:
+      res['value']['line_cr_ids'] = [{'type': 'cr', 'account_id': ref('account.a_recv'),}]
+    res['value']['line_cr_ids'][0]['amount'] = amount_total
+    vals['line_cr_ids'] = [(0,0,i) for i in res['value']['line_cr_ids']]
+    id = self.create(cr, uid, vals)
+    voucher_id = self.browse(cr, uid, id)
+    assert (voucher_id.state=='draft'), "Voucher is not in draft state"
+    wf_service = netsvc.LocalService("workflow")
+    wf_service.trg_validate(uid, 'account.voucher', voucher_id.id, 'proforma_voucher', cr)
+-
+  6.9.- I check if the state of the invoice is "Paid"
+-
+  !assert {model: account.invoice, id: account_invoice_customer5}:
+    - state == 'paid'
+-
+  6.10.- I create a customer refund using the method "Create refund, reconcile and create a new draft invoice" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('sale_refund_journal_debit'),
+        'description': 'Desde el test',
+        'filter_refund': 'modify',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_customer5')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])
+    assert rfnd_customer_inv_brw[0].state == 'paid', 'The customer refund was not properly created, the state must be Paid.'
+-
+  6.11.- I check the correct asigning of parent_id in the customer refund created, for that
+  the customer refund must have the customer invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    customer_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_customer5'))    
+    assert (rfnd_customer_inv_brw[0].parent_id.id == customer_inv_brw.id), 'The customer refund dont have the correct parent_id.'
+-
+  6.12.- I check that the new customer invoice was created in Draft state
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    inv_src = obj_inv.search(cr, uid, [], order='id')
+    customer_inv_brw = obj_inv.browse(cr, uid, inv_src[-1])
+    assert customer_inv_brw.state == 'draft', 'The new customer invoice was not properly created, the state must be Draft.'
 
=== added file 'l10n_ve_fiscal_requirements/test/account_supplier_refund.yml'
--- l10n_ve_fiscal_requirements/test/account_supplier_refund.yml	1970-01-01 00:00:00 +0000
+++ l10n_ve_fiscal_requirements/test/account_supplier_refund.yml	2014-02-11 00:22:34 +0000
@@ -0,0 +1,622 @@
+-
+  I set the context that will be used for the encoding of all the invoices of this file
+-
+  !context
+    'type': 'in_invoice'
+-
+  1.- In order to test account refund with "Create a draft refund" method
+-
+  1.1.- I create a supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier0, view: account.invoice_supplier_form }: &supplier_invoice
+    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'
+-
+  1.2.- I check that Initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier0}:
+    - state == 'draft'
+-
+  1.3.- 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'
+-
+  1.4.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_supplier0}
+-
+  1.5.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier0}:
+    - state == 'open'
+-
+  1.6.- I create a supplier refund using the method "Create a draft refund" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('account.refund_expenses_journal'),
+        'description': 'Desde el test',
+        'nro_ctrl': 'Desde el test',
+        'filter_refund': 'refund',
+    }
+    wzr_id = self.create(cr, uid, values)
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_supplier0')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])    
+    assert rfnd_supplier_inv_brw[0].state == 'draft', 'The supplier refund was not properly created, the state must be Draft.'
+-
+  1.7.- I check the correct asigning of parent_id in the supplier refund created, for that
+  the supplier refund must have the supplier invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    supplier_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_supplier0'))    
+    assert (rfnd_supplier_inv_brw[0].parent_id.id == supplier_inv_brw.id), 'The supplier refund dont have the correct parent_id.'
+-
+  2.- In order to test account refund with "Create refund and reconcile" method
+-
+  2.1.- I create a new supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier1, view: account.invoice_supplier_form }:
+    <<: *supplier_invoice
+-
+  2.2.- I check that Initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier1}:
+    - state == 'draft'
+-
+  2.3.- I check that the supplier invoice is a "Supplier Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier1, string: This is not a supplier invoice}:
+    - type == 'in_invoice'
+-
+  2.4.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_supplier1}
+-
+  2.5.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier1}:
+    - state == 'open'
+-
+  2.6.- I create a supplier refund using the method "Create refund and reconcile" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('account.refund_expenses_journal'),
+        'description': 'Desde el test',
+        'nro_ctrl': 'Desde el test',
+        'filter_refund': 'cancel',
+    }
+    wzr_id = self.create(cr, uid, values)
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_supplier1')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])    
+    assert rfnd_supplier_inv_brw[0].state == 'paid', 'The supplier refund was not properly created, the state must be Paid.'
+-
+  2.7.- I check the correct asigning of parent_id in the supplier refund created, for that
+  the supplier refund must have the supplier invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    supplier_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_supplier1'))    
+    assert (rfnd_supplier_inv_brw[0].parent_id.id == supplier_inv_brw.id), 'The supplier refund dont have the correct parent_id.'
+-
+  3.- In order to test account refund with "Create refund, reconcile and create a new draft invoice"
+  method
+-
+  3.1.- I create a new supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier2, view: account.invoice_supplier_form }:
+    <<: *supplier_invoice
+-
+  3.2.- I check that Initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier2}:
+    - state == 'draft'
+-
+  3.3.- I check that the supplier invoice is a "Supplier Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier2, string: This is not a supplier invoice}:
+    - type == 'in_invoice'
+-
+  3.4.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_supplier2}
+-
+  3.5.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier2}:
+    - state == 'open'
+-
+  3.6.- I create a supplier refund using the method "Create refund, reconcile and create a new draft invoice" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('account.refund_expenses_journal'),
+        'description': 'Desde el test',
+        'nro_ctrl': 'Desde el test',
+        'filter_refund': 'modify',
+    }
+    wzr_id = self.create(cr, uid, values)
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_supplier2')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])    
+    assert rfnd_supplier_inv_brw[0].state == 'paid', 'The supplier refund was not properly created, the state must be Paid.'
+-
+  3.7.- I check the correct asigning of parent_id in the supplier refund created, for that
+  the supplier refund must have the supplier invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    supplier_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_supplier2'))    
+    assert (rfnd_supplier_inv_brw[0].parent_id.id == supplier_inv_brw.id), 'The supplier refund dont have the correct parent_id.'
+-
+  3.8.- I check that the new supplier invoice was created in Draft state
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    inv_src = obj_inv.search(cr, uid, [], order='id')
+    supplier_inv_brw = obj_inv.browse(cr, uid, inv_src[-1])
+    assert supplier_inv_brw.state == 'draft', 'The new supplier invoice was not properly created, the state must be Draft.'
+-
+  4.- In order to test account refund with "Create a draft refund" method in a "Paid" invoice
+-
+  4.1.- I create a new partner
+-
+  !record {model: 'res.partner', id: res_partner10}:
+    name: Juan Garcia
+    use_parent_address: True
+    function: Tests
+    email: [e-mail]
+    supplier: True
+-
+  4.2.- I create a Purchase Journal.
+-
+  !record {model: account.journal, id: purchase_journal_debit}:
+    code: pjou_test
+    company_id: base.main_company
+    name: Purchase Journal For Debit Credit Note Tests
+    sequence_id: account.sequence_purchase_journal
+    type: purchase
+-
+  4.3.- I create a new supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier3, 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: purchase_journal_debit
+    partner_id: res_partner10
+    reference_type: none
+    type: 'in_invoice'
+-
+  4.4.- I check that initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier3}:
+    - state == 'draft'
+-
+  4.5.- I check that the supplier invoice is a "Supplier Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier3, string: This is not a supplier invoice}:
+    - type == 'in_invoice'
+-
+  4.6.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_supplier3}
+-
+  4.7.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier3}:
+    - state == 'open'
+-
+  4.8.- I check that a payment entry gets created in the account.move.line
+-
+  !python {model: account.invoice}: |
+    acc_id=self.browse(cr, uid, ref("account_invoice_supplier3"))
+    assert(acc_id.move_id)
+-
+  4.9.- I create a Bank Journal.
+-
+  !record {model: account.journal, id: bank_journal_debit}:
+    name: Bank Journal For Debit Credit Note Tests
+    code: bjou_test
+    type: bank
+    analytic_journal_id: account.sit
+    sequence_id: account.sequence_bank_journal
+    default_debit_account_id: account.cash
+    default_credit_account_id: account.cash
+    company_id: base.main_company
+-
+  4.10.- I will create and post an account voucher of amount 3450.00 for the partner Juan Garcia.
+-
+  !record {model: account.voucher, id: account_voucher0, view: account_voucher.view_vendor_payment_form}:
+    type: payment
+    account_id: account.cash
+    amount: 3450.00
+    company_id: base.main_company
+    journal_id: bank_journal_debit
+    name: Voucher x
+    narration: PC Assemble SC234
+    line_dr_ids:
+      - account_id: account.cash
+        amount: 3450.00
+        name: Voucher x
+    partner_id: res_partner10
+    reference: none
+-
+  4.11.- I check that initially vendor voucher is in the "Draft" state
+-
+  !assert {model: account.voucher, id: account_voucher0}:
+    - state == 'draft'
+-
+  4.12.- I create voucher by clicking on create button
+-
+  !workflow {model: account.voucher, action: proforma_voucher, ref: account_voucher0}
+-
+  4.13.- I check that the voucher state is "Posted"
+-
+  !assert {model: account.voucher, id: account_voucher0}:
+    - state == 'posted'
+-
+  4.14.- I check if the state of the invoice is "Paid"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier3}:
+    - state == 'paid'
+-
+  4.15.- I modified the sale journal record so it can cancel invoice
+-
+  !record {model: account.journal, id: purchase_journal_debit}:
+    update_posted: 1
+-
+  4.16.- I modified the cash journal record so it can cancel invoice
+-
+  !record {model: account.journal, id: bank_journal_debit}:
+    update_posted: 1
+-
+  4.17.- I create a Purchase Refund Journal.
+-
+  !record {model: account.journal, id: purchase_refund_journal_debit}:
+    code: sprjou_test
+    company_id: base.main_company
+    name: Purchase Refund Journal For Debit Credit Note Tests
+    sequence_id: account.sequence_refund_purchase_journal
+    type: purchase_refund
+-
+  4.18.- I create a supplier refund using the method "Create a draft refund" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('purchase_refund_journal_debit'),
+        'description': 'Desde el test',
+        'nro_ctrl': 'Desde el test',
+        'filter_refund': 'refund',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_supplier3')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])
+    assert rfnd_supplier_inv_brw[0].state == 'draft', 'The supplier refund was not properly created, the state must be Draft.'
+-
+  4.19.- I check the correct asigning of parent_id in the supplier refund created, for that
+  the supplier refund must have the supplier invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    supplier_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_supplier3'))    
+    assert (rfnd_supplier_inv_brw[0].parent_id.id == supplier_inv_brw.id), 'The supplier refund dont have the correct parent_id.'
+-
+  5.- In order to test account refund with "Create refund and reconcile" method in a "Paid" invoice
+-
+  5.1.- I create a new partner
+-
+  !record {model: 'res.partner', id: res_partner11}:
+    name: Maria Garcia
+    use_parent_address: True
+    function: Tests
+    email: [e-mail]
+    supplier: True
+-
+  5.2.- I create a new supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier4, 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: purchase_journal_debit
+    partner_id: res_partner11
+    reference_type: none
+    type: 'in_invoice'
+-
+  5.3.- I check that initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier4}:
+    - state == 'draft'
+-
+  5.4.- I check that the supplier invoice is a "Supplier Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier4, string: This is not a supplier invoice}:
+    - type == 'in_invoice'
+-
+  5.5.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_supplier4}
+-
+  5.6.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier4}:
+    - state == 'open'
+-
+  5.7.- I check that a payment entry gets created in the account.move.line
+-
+  !python {model: account.invoice}: |
+    acc_id=self.browse(cr, uid, ref("account_invoice_supplier4"))
+    assert(acc_id.move_id)
+-
+  5.8.- I will create and post an account voucher of amount 3450.00 for the partner Maria Garcia.
+-
+  !record {model: account.voucher, id: account_voucher1, view: account_voucher.view_vendor_payment_form}:
+    type: payment
+    account_id: account.cash
+    amount: 3450.00
+    company_id: base.main_company
+    journal_id: bank_journal_debit
+    name: Voucher x
+    narration: PC Assemble SC234
+    line_dr_ids:
+      - account_id: account.cash
+        amount: 3450.00
+        name: Voucher x
+    partner_id: res_partner11
+    reference: none
+-
+  5.9.- I check that initially vendor voucher is in the "Draft" state
+-
+  !assert {model: account.voucher, id: account_voucher1}:
+    - state == 'draft'
+-
+  5.10.- I create voucher by clicking on create button
+-
+  !workflow {model: account.voucher, action: proforma_voucher, ref: account_voucher1}
+-
+  5.11.- I check that the voucher state is "Posted"
+-
+  !assert {model: account.voucher, id: account_voucher1}:
+    - state == 'posted'
+-
+  5.12.- I check if the state of the invoice is "Paid"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier4}:
+    - state == 'paid'
+-
+  5.13.- I create a supplier refund using the method "Create refund and reconcile" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('purchase_refund_journal_debit'),
+        'description': 'Desde el test',
+        'nro_ctrl': 'Desde el test',
+        'filter_refund': 'cancel',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_supplier4')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])
+    assert rfnd_supplier_inv_brw[0].state == 'paid', 'The supplier refund was not properly created, the state must be Paid.'
+-
+  5.14.- I check the correct asigning of parent_id in the supplier refund created, for that
+  the supplier refund must have the supplier invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    supplier_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_supplier4'))    
+    assert (rfnd_supplier_inv_brw[0].parent_id.id == supplier_inv_brw.id), 'The supplier refund dont have the correct parent_id.'
+-
+  6.- In order to test account refund with "Create refund, reconcile and create a new draft invoice" 
+  method in a "Paid" invoice
+-
+  6.1.- I create a new partner
+-
+  !record {model: 'res.partner', id: res_partner12}:
+    name: Pablo Garcia
+    use_parent_address: True
+    function: Tests
+    email: [e-mail]
+    supplier: True
+-
+  6.2.- I create a new supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier5, 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: purchase_journal_debit
+    partner_id: res_partner12
+    reference_type: none
+    type: 'in_invoice'
+-
+  6.3.- I check that initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier5}:
+    - state == 'draft'
+-
+  6.4.- I check that the supplier invoice is a "Supplier Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier5, string: This is not a supplier invoice}:
+    - type == 'in_invoice'
+-
+  6.5.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_supplier5}
+-
+  5.6.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier5}:
+    - state == 'open'
+-
+  6.7.- I check that a payment entry gets created in the account.move.line
+-
+  !python {model: account.invoice}: |
+    acc_id=self.browse(cr, uid, ref("account_invoice_supplier5"))
+    assert(acc_id.move_id)
+-
+  6.8.- I will create and post an account voucher of amount 3450.00 for the partner Maria Garcia.
+-
+  !record {model: account.voucher, id: account_voucher2, view: account_voucher.view_vendor_payment_form}:
+    type: payment
+    account_id: account.cash
+    amount: 3450.00
+    company_id: base.main_company
+    journal_id: bank_journal_debit
+    name: Voucher x
+    narration: PC Assemble SC234
+    line_dr_ids:
+      - account_id: account.cash
+        amount: 3450.00
+        name: Voucher x
+    partner_id: res_partner12
+    reference: none
+-
+  6.9.- I check that initially vendor voucher is in the "Draft" state
+-
+  !assert {model: account.voucher, id: account_voucher2}:
+    - state == 'draft'
+-
+  6.10.- I create voucher by clicking on create button
+-
+  !workflow {model: account.voucher, action: proforma_voucher, ref: account_voucher2}
+-
+  6.11.- I check that the voucher state is "Posted"
+-
+  !assert {model: account.voucher, id: account_voucher2}:
+    - state == 'posted'
+-
+  6.12.- I check if the state of the invoice is "Paid"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier5}:
+    - state == 'paid'
+-
+  6.13.- I create a supplier refund using the method "Create refund, reconcile and create a new draft invoice" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('purchase_refund_journal_debit'),
+        'description': 'Desde el test',
+        'nro_ctrl': 'Desde el test',
+        'filter_refund': 'modify',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_supplier5')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])
+    assert rfnd_supplier_inv_brw[0].state == 'paid', 'The supplier refund was not properly created, the state must be Paid.'
+-
+  6.14.- I check the correct asigning of parent_id in the supplier refund created, for that
+  the supplier refund must have the supplier invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    supplier_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_supplier5'))    
+    assert (rfnd_supplier_inv_brw[0].parent_id.id == supplier_inv_brw.id), 'The supplier refund dont have the correct parent_id.'
+-
+  6.15.- I check that the new supplier invoice was created in Draft state
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    inv_src = obj_inv.search(cr, uid, [], order='id')
+    supplier_inv_brw = obj_inv.browse(cr, uid, inv_src[-1])
+    assert supplier_inv_brw.state == 'draft', 'The new supplier invoice was not properly created, the state must be Draft.'
 
=== modified file 'l10n_ve_fiscal_requirements/wizard/account_invoice_refund.py'
--- l10n_ve_fiscal_requirements/wizard/account_invoice_refund.py	2014-01-13 19:38:55 +0000
+++ l10n_ve_fiscal_requirements/wizard/account_invoice_refund.py	2014-02-11 00:22:34 +0000
@@ -99,17 +99,7 @@
                 journal_select = journal_obj._name_search(cr, uid, '', [('type', '=', type)], context=context, limit=None, name_get_uid=1)
                 res['fields'][field]['selection'] = journal_select
         return res
-
-    def _get_orig(self, cr, uid, inv, ref, context={}):
-        """ Return  default origin value
-        """
-        nro_ref = ref
-        if inv.type == 'out_invoice':
-            nro_ref = inv.number
-        orig = 'Devolucion FACT:' +(nro_ref or '') + '- DE FECHA:' + (inv.date_invoice or '') + (' TOTAL:' + str(inv.amount_total) or '')
-        return orig
-
-    
+        
     def cn_iva_validate(self, cr, uid,invoice,context=None):
         """ Validates if retentions have been changes to move the state confirmed and done
         """
@@ -159,20 +149,20 @@
         res_users_obj = self.pool.get('res.users')
         if context is None:
             context = {}
-        for form in  self.read(cr, uid, ids, context=context):
+        for form in self.browse(cr, uid, ids, context=context):
             created_inv = []
             date = False
             period = False
             description = False
             nroctrl = False
             company = res_users_obj.browse(cr, uid, uid, context=context).company_id
-            journal_brw = form.get('journal_id', False)
+            journal_brw = form.journal_id or False
             for inv in inv_obj.browse(cr, uid, context.get('active_ids'), context=context):
                 if inv.state in ['draft', 'proforma2', 'cancel']:
                     raise osv.except_osv(_('Error !'), _('Can not %s draft/proforma/cancel invoice.') % (mode))
                 if inv.reconciled and mode in ('cancel', 'modify'):
                     raise osv.except_osv(_('Error !'), _('Can not %s invoice which is already reconciled, invoice should be unreconciled first. You can only Refund this invoice') % (mode))
-                period = form.get('period') and form.get('period')[0] or False
+                period = form.period.id or False
                 if not period:
 	                #Take period from the current date
                     period = self.pool.get('account.period').find(cr, uid, context=context)
@@ -185,11 +175,11 @@
                 if not journal_brw:
                     journal_id = inv.journal_id.id
                 else:
-                    journal_id=journal_brw[0]
+                    journal_id=journal_brw.id
 
-                if form['date']:
-                    date = form['date']
-                    if not form['period']:
+                if form.date:
+                    date = form.date
+                    if not form.period:
                             cr.execute("select name from ir_model_fields \
                                             where model = 'account.period' \
                                             and name = 'company_id'")
@@ -209,14 +199,14 @@
                     #Take current date
                     #date = inv.date_invoice
                     date = time.strftime('%Y-%m-%d')
-                if form['description']:
-                    description = form['description']
+                if form.description:
+                    description = form.description
                 else:
                     description = inv.name
                 
                 if inv.type in ('in_invoice','in_refund'):
-                    if form['nro_ctrl']:
-                        nroctrl = form['nro_ctrl']
+                    if form.nro_ctrl:
+                        nroctrl = form.nro_ctrl
                     else:
                         raise osv.except_osv(_('Control Number !'), \
                                             _('Missing Control Number on Invoice Refund!'))
@@ -321,19 +311,7 @@
                 orig = self._get_orig(cr, uid, inv, inv.supplier_invoice_number, context)
                 inv_obj.write(cr,uid,created_inv[0],{'origin':inv.origin,'name':wzd_brw.description},context=context)
             return result
-
-    def validate_total_payment_inv(self, cr, uid, ids, context=None):
-        """ Method that validate if invoice is totally paid.
-        @param ids: list of invoices.
-        return: True: if invoice is paid.
-                False: if invoice is not paid.
-        """
-        res = False
-        inv_obj = self.pool.get('account.invoice')
-        for inv in inv_obj.browse(cr, uid, ids, context=context):
-            res = inv.reconciled
-        return res
-
+            
     def validate_wh(self, cr, uid, ids, context=None):
         """ Method that validate if invoice has non-yet processed withholds.
 
@@ -345,36 +323,6 @@
         """
         return True
 
-    def unreconcile_paid_invoices(self, cr, uid, invoiceids, context=None):
-        """ Method that unreconcile the payments of invoice.
-        @param invoiceids: list of invoices.
-        return: True: unreconcile successfully.
-                False: unreconcile unsuccessfully.
-        """
-        inv_obj = self.pool.get('account.invoice')
-        moveline_obj = self.pool.get('account.move.line')
-        voucher_pool = self.pool.get('account.voucher')
-        res = True
-        rec = []
-        mid = []
-        if self.validate_total_payment_inv(cr, uid, invoiceids, context=context):
-            for inv in inv_obj.browse(cr, uid, invoiceids, context=context):
-                movelineids = inv_obj.move_line_id_payment_get(cr, uid,[inv.id])
-                for moveline in moveline_obj.browse(cr, uid, movelineids,context=context):
-                    if moveline.reconcile_id:
-                        rec += [moveline.reconcile_id.id]
-                    if moveline.reconcile_partial_id:
-                        rec += [moveline.reconcile_partial_id.id]
-                movelines = moveline_obj.search(cr, uid, [('|'),('reconcile_id','in',rec),('reconcile_partial_id','in',rec)],context=context)
-                for mids in moveline_obj.browse(cr, uid, movelines, context=context):
-                    mid +=[mids.move_id.id]
-                voucherids = voucher_pool.search(cr, uid,[('move_id','in',mid)])
-            if voucherids:
-                voucher_pool.cancel_voucher(cr, uid, voucherids, context=context)
-            else:
-                res = False
-        return res
-
     def invoice_refund(self, cr, uid, ids, context=None):
         """ Create a invoice refund
         """
@@ -383,9 +331,9 @@
         inv_obj = self.pool.get('account.invoice')
         period_obj = self.pool.get('account.period')
         wzr_brw = self.browse(cr,uid,ids,context=context)[0]
-        date = wzr_brw.date and wzr_brw.date.split('-') 
-        period = wzr_brw and wzr_brw.period and wzr_brw.period.id 
-        period_ids = date and len(date) == 3 and  period_obj.search(cr,uid,[('code','=','%s/%s'%(date[1],date[0]))],context=context)
+        date = wzr_brw.date
+        period = wzr_brw and wzr_brw.period and wzr_brw.period.id
+        period_ids = date and period_obj.search(cr,uid,[('date_start', '<=', date),('date_stop', '>=', date),('special', '=', False)],context=context)
         if period not in period_ids:
             raise osv.except_osv(_('Error !'), \
                                      _('The date should be chosen to belong to the period'))