Add forma de pago tests

carlosthe19916 · Dec 4, 2023 · 6a38871 · 6a38871
1 parent f6f2c09
commit 6a38871
Show file tree

Hide file tree

Showing 21 changed files with 224 additions and 42 deletions.
diff --git a/src/enricher/enrich.rs b/src/enricher/enrich.rs
@@ -9,7 +9,7 @@ use crate::enricher::rules::phase1fill::igvtasa::IGVTasaEnrichRule;
 use crate::enricher::rules::phase1fill::invoice::anticipos::InvoiceAnticiposEnrichRule;
 use crate::enricher::rules::phase1fill::invoice::descuentos::InvoiceDescuentosEnrichRule;
 use crate::enricher::rules::phase1fill::invoice::formadepago::{
-    InvoiceFormaDePagoEnrichRule, InvoiceFormaDePagoTotalRule,
+    InvoiceFormaDePagoEnrichRule, InvoiceFormaDePagoTipoRule, InvoiceFormaDePagoTotalRule,
 };
 use crate::enricher::rules::phase1fill::invoice::leyenda::{
     InvoiceLeyendaDetraccionEnrichRule, InvoiceLeyendaDireccionEntregaEnrichRule,
@@ -153,7 +153,7 @@ where
         let mut changed = true;
 
         while changed {
-            let results = vec![
+            let results = [
                 FechaEmisionEnrichRule::fill(self, defaults),
                 FirmanteEnrichRule::fill(self),
                 ICBTasaEnrichRule::fill(self, defaults),
@@ -175,23 +175,23 @@ where
         + InvoiceDescuentosEnrichRule
         + InvoiceFormaDePagoEnrichRule
         + InvoiceFormaDePagoTotalRule
+        + InvoiceFormaDePagoTipoRule
         + InvoiceLeyendaDetraccionEnrichRule
         + InvoiceLeyendaDireccionEntregaEnrichRule
         + InvoiceLeyendaPercepcionEnrichRule
-        + InvoiceFormaDePagoEnrichRule
-        + InvoiceFormaDePagoTotalRule
         + InvoiceTipoComprobanteEnrichRule
         + InvoiceTipoOperacionEnrichRule,
 {
     fn enrich_invoice(&mut self, _: &Defaults) {
         let mut changed = true;
 
         while changed {
-            let results = vec![
+            let results = [
                 InvoiceAnticiposEnrichRule::fill(self),
                 InvoiceDescuentosEnrichRule::fill(self),
                 InvoiceFormaDePagoEnrichRule::fill(self),
                 InvoiceFormaDePagoTotalRule::fill(self),
+                InvoiceFormaDePagoTipoRule::fill(self),
                 InvoiceLeyendaDetraccionEnrichRule::fill(self),
                 InvoiceLeyendaDireccionEntregaEnrichRule::fill(self),
                 InvoiceLeyendaPercepcionEnrichRule::fill(self),
@@ -212,7 +212,7 @@ where
         let mut changed = true;
 
         while changed {
-            let results = vec![
+            let results = [
                 NoteTipoComprobanteAfectadoEnrichRule::fill(self),
                 CreditNoteTipoEnrichRule::fill(self),
             ];
@@ -230,7 +230,7 @@ where
         let mut changed = true;
 
         while changed {
-            let results = vec![
+            let results = [
                 NoteTipoComprobanteAfectadoEnrichRule::fill(self),
                 DebitNoteTipoEnrichRule::fill(self),
             ];
@@ -268,7 +268,7 @@ where
         let mut changed = true;
 
         while changed {
-            let results = vec![DetallesProcessRule::process(self)];
+            let results = [DetallesProcessRule::process(self)];
 
             changed = results.contains(&true);
         }
@@ -307,7 +307,7 @@ where
         let mut changed = true;
 
         while changed {
-            let results = vec![
+            let results = [
                 InvoiceTotalImpuestosSummaryRule::summary(self),
                 InvoiceTotalImporteSummaryRule::summary(self),
                 LeyendaIVAPSummaryRule::summary(self),

diff --git a/src/enricher/rules/phase1fill/detalle/detalles.rs b/src/enricher/rules/phase1fill/detalle/detalles.rs
@@ -36,7 +36,7 @@ where
         self.get_detalles()
             .iter_mut()
             .map(|detalle| {
-                let results = vec![
+                let results = [
                     DetalleICBTasaEnrichRule::fill(detalle, defaults),
                     DetalleIGVTasaEnrichRule::fill(detalle, defaults),
                     DetalleIGVTipoEnrichRule::fill(detalle, defaults),

diff --git a/src/enricher/rules/phase1fill/detalle/igvtasa.rs b/src/enricher/rules/phase1fill/detalle/igvtasa.rs
@@ -1,6 +1,6 @@
 use rust_decimal::Decimal;
 
-use crate::catalogs::{Catalog7, catalog7_value_of_code, Catalog7Group};
+use crate::catalogs::{catalog7_value_of_code, Catalog7, Catalog7Group};
 use crate::enricher::rules::phase1fill::detalle::detalles::DetalleDefaults;
 use crate::models::traits::detalle::igvtasa::{DetalleIGVTasaGetter, DetalleIGVTasaSetter};
 use crate::models::traits::detalle::igvtipo::DetalleIGVTipoGetter;

diff --git a/src/enricher/rules/phase1fill/detalle/precioreferenciatipo.rs b/src/enricher/rules/phase1fill/detalle/precioreferenciatipo.rs
@@ -1,4 +1,4 @@
-use crate::catalogs::{Catalog, Catalog16, catalog7_value_of_code};
+use crate::catalogs::{catalog7_value_of_code, Catalog, Catalog16};
 use crate::enricher::rules::phase1fill::detalle::detalles::DetalleDefaults;
 use crate::models::traits::detalle::igvtipo::DetalleIGVTipoGetter;
 use crate::models::traits::detalle::precioreferenciatipo::{

diff --git a/src/enricher/rules/phase1fill/invoice/anticipos.rs b/src/enricher/rules/phase1fill/invoice/anticipos.rs
@@ -1,11 +1,11 @@
 use log::warn;
 use regex::Regex;
 
-use crate::{BOLETA_SERIE_REGEX, FACTURA_SERIE_REGEX};
 use crate::catalogs::{Catalog, Catalog12, Catalog53};
 use crate::models::traits::invoice::anticipos::{
     AnticipoGetter, AnticipoSetter, InvoiceAnticiposGetter,
 };
+use crate::{BOLETA_SERIE_REGEX, FACTURA_SERIE_REGEX};
 
 pub trait InvoiceAnticiposEnrichRule {
     fn fill(&mut self) -> bool;
@@ -19,7 +19,7 @@ where
         self.get_anticipos()
             .iter_mut()
             .map(|anticipo| {
-                let results = vec![
+                let results = [
                     AnticipoTipoRule::fill(anticipo),
                     AnticipoComprobanteTipoRule::fill(anticipo),
                 ];

diff --git a/src/enricher/rules/phase1fill/invoice/descuentos.rs b/src/enricher/rules/phase1fill/invoice/descuentos.rs
@@ -17,7 +17,7 @@ where
         self.get_descuentos()
             .iter_mut()
             .map(|descuento| {
-                let results = vec![
+                let results = [
                     DescuentoFactorRule::fill(descuento),
                     DescuentoMontoBaseRule::fill(descuento),
                     DescuentoTipoRule::fill(descuento),

diff --git a/src/enricher/rules/phase1fill/invoice/formadepago.rs b/src/enricher/rules/phase1fill/invoice/formadepago.rs
@@ -13,16 +13,20 @@ pub trait InvoiceFormaDePagoTotalRule {
     fn fill(&mut self) -> bool;
 }
 
+pub trait InvoiceFormaDePagoTipoRule {
+    fn fill(&mut self) -> bool;
+}
+
 impl<T> InvoiceFormaDePagoEnrichRule for T
 where
     T: InvoiceFormaDePagoGetter + InvoiceFormaDePagoSetter,
 {
     fn fill(&mut self) -> bool {
         match &self.get_formadepago() {
-            Some(..) => false,
+            Some(_) => false,
             None => {
                 self.set_formadepago(FormaDePago {
-                    tipo: TipoFormaDePago::Contado,
+                    tipo: Some(TipoFormaDePago::Contado),
                     cuotas: vec![],
                     total: None,
                 });
@@ -32,6 +36,31 @@ where
     }
 }
 
+impl<T> InvoiceFormaDePagoTipoRule for T
+where
+    T: InvoiceFormaDePagoGetter + InvoiceFormaDePagoSetter,
+{
+    fn fill(&mut self) -> bool {
+        if let Some(forma_de_pago) = self.get_formadepago() {
+            if forma_de_pago.tipo.is_none() {
+                let tipo = if forma_de_pago.cuotas.is_empty() {
+                    TipoFormaDePago::Contado
+                } else {
+                    TipoFormaDePago::Credito
+                };
+                self.set_formadepago(FormaDePago {
+                    tipo: Some(tipo),
+                    ..forma_de_pago.clone()
+                });
+
+                return true;
+            };
+        };
+
+        false
+    }
+}
+
 impl<T> InvoiceFormaDePagoTotalRule for T
 where
     T: InvoiceFormaDePagoGetter + InvoiceFormaDePagoSetter,

diff --git a/src/enricher/rules/phase1fill/invoice/tipocomprobante.rs b/src/enricher/rules/phase1fill/invoice/tipocomprobante.rs
@@ -1,11 +1,11 @@
 use regex::Regex;
 
-use crate::{BOLETA_SERIE_REGEX, FACTURA_SERIE_REGEX};
 use crate::catalogs::{Catalog, Catalog1};
 use crate::models::traits::invoice::tipocomprobante::{
     InvoiceTipoComprobanteGetter, InvoiceTipoComprobanteSetter,
 };
 use crate::models::traits::serienumero::SerieNumeroGetter;
+use crate::{BOLETA_SERIE_REGEX, FACTURA_SERIE_REGEX};
 
 pub trait InvoiceTipoComprobanteEnrichRule {
     fn fill(&mut self) -> bool;

diff --git a/src/enricher/rules/phase1fill/note/tipocomprobanteafectado.rs b/src/enricher/rules/phase1fill/note/tipocomprobanteafectado.rs
@@ -1,12 +1,12 @@
 use log::warn;
 use regex::Regex;
 
-use crate::{BOLETA_SERIE_REGEX, FACTURA_SERIE_REGEX};
 use crate::catalogs::{Catalog, Catalog1};
 use crate::models::traits::note::tipocomprobanteafectado::{
     NoteTipoComprobanteAfectadoGetter, NoteTipoComprobanteAfectadoSetter,
 };
 use crate::models::traits::serienumero::SerieNumeroGetter;
+use crate::{BOLETA_SERIE_REGEX, FACTURA_SERIE_REGEX};
 
 pub trait NoteTipoComprobanteAfectadoEnrichRule {
     fn fill(&mut self) -> bool;

diff --git a/src/enricher/rules/phase2process/detalle/detalles.rs b/src/enricher/rules/phase2process/detalle/detalles.rs
@@ -22,15 +22,15 @@ where
         self.get_detalles()
             .iter_mut()
             .map(|detalle| {
-                let results = vec![
+                let results = [
                     DetallePrecioProcessRule::process(detalle),
                     DetallePrecioConImpuestosProcessRule::process(detalle),
                     DetallePrecioReferenciaProcessRule::process(detalle),
                 ];
                 if results.contains(&true) {
                     true
                 } else {
-                    let results = vec![
+                    let results = [
                         DetalleICBProcessRule::process(detalle),
                         DetalleICBAplicaProcessRule::process(detalle),
                         DetalleIGVProcessRule::process(detalle),

diff --git a/src/enricher/rules/phase3summary/invoice/detraccion.rs b/src/enricher/rules/phase3summary/invoice/detraccion.rs
@@ -16,7 +16,7 @@ where
     fn summary(&mut self) -> bool {
         match (self.get_totalimporte().clone(), self.get_detraccion()) {
             (Some(total_importe), Some(detraccion)) => {
-                let results = vec![DetraccionMontoRule::summary(detraccion, &total_importe)];
+                let results = [DetraccionMontoRule::summary(detraccion, &total_importe)];
                 results.contains(&true)
             }
             _ => false,

diff --git a/src/enricher/rules/phase3summary/invoice/percepcion.rs b/src/enricher/rules/phase3summary/invoice/percepcion.rs
@@ -19,7 +19,7 @@ where
 {
     fn summary(&mut self) -> bool {
         match (self.get_totalimporte().clone(), self.get_percepcion()) {
-            (Some(total_importe), Some(percepcion)) => vec![
+            (Some(total_importe), Some(percepcion)) => [
                 PerceptionPorcentajeBaseRule::summary(percepcion),
                 PerceptionMontoBaseRule::summary(percepcion, &total_importe),
                 PerceptionMontoRule::summary(percepcion),

diff --git a/src/enricher/rules/phase3summary/invoice/totalimporte.rs b/src/enricher/rules/phase3summary/invoice/totalimporte.rs
@@ -3,7 +3,7 @@ use std::collections::HashMap;
 use rust_decimal::Decimal;
 use rust_decimal_macros::dec;
 
-use crate::catalogs::{Catalog5, Catalog53, catalog53_value_of_code, catalog7_value_of_code};
+use crate::catalogs::{catalog53_value_of_code, catalog7_value_of_code, Catalog5, Catalog53};
 use crate::models::general::TotalImporte;
 use crate::models::traits::detalle::DetallesGetter;
 use crate::models::traits::igv::IGVTasaGetter;

diff --git a/src/enricher/rules/phase3summary/invoice/totalimpuestos.rs b/src/enricher/rules/phase3summary/invoice/totalimpuestos.rs
@@ -3,7 +3,7 @@ use std::collections::HashMap;
 use rust_decimal::Decimal;
 use rust_decimal_macros::dec;
 
-use crate::catalogs::{Catalog5, Catalog53, catalog53_value_of_code, catalog7_value_of_code};
+use crate::catalogs::{catalog53_value_of_code, catalog7_value_of_code, Catalog5, Catalog53};
 use crate::models::general::{Detalle, TotalImpuestos};
 use crate::models::traits::detalle::DetallesGetter;
 use crate::models::traits::invoice::anticipos::InvoiceAnticiposGetter;
@@ -34,7 +34,7 @@ where
                 let gratuito = cal_impuesto_by_tipo(self.get_detalles(), Catalog5::Gratuito);
 
                 // ICB
-                let icb_importe = vec![
+                let icb_importe = [
                     ivap.importe_icb,
                     exportacion.importe_icb,
                     gravado.importe_icb,
@@ -46,7 +46,7 @@ where
                 .fold(dec!(0), |a, b| a + b);
 
                 // ISC
-                let isc_importe = vec![
+                let isc_importe = [
                     ivap.importe_isc,
                     exportacion.importe_isc,
                     gravado.importe_isc,

diff --git a/src/models/general.rs b/src/models/general.rs
@@ -24,7 +24,7 @@ pub enum TipoFormaDePago {
 
 #[derive(Clone, Debug, Serialize)]
 pub struct FormaDePago {
-    pub tipo: TipoFormaDePago,
+    pub tipo: Option<TipoFormaDePago>,
     pub cuotas: Vec<CuotaDePago>,
     pub total: Option<Decimal>,
 }

diff --git a/src/prelude.rs b/src/prelude.rs
@@ -1,8 +1,7 @@
 pub use crate::catalogs::*;
 pub use crate::enricher::enrich::{Defaults, EnrichTrait};
-pub use crate::models::*;
 pub use crate::models::common::*;
 pub use crate::models::general::*;
 pub use crate::models::invoice::*;
+pub use crate::models::*;
 pub use crate::renderer::render_invoice;
-
diff --git a/src/renderer/mod.rs b/src/renderer/mod.rs
@@ -3,11 +3,11 @@ use std::str::FromStr;
 
 use rust_decimal::Decimal;
 use rust_decimal_macros::dec;
-use tera::{Context, Error, from_value, Function, Tera, to_value, Value};
 use tera::helpers::tests::{number_args_allowed, value_defined};
+use tera::{from_value, to_value, Context, Error, Function, Tera, Value};
 
 use crate::models::invoice::Invoice;
-use crate::prelude::{Catalog, catalog7_value_of_code};
+use crate::prelude::{catalog7_value_of_code, Catalog};
 
 fn catalog7_taxcategory() -> impl Function {
     Box::new(
@@ -69,6 +69,16 @@ pub fn currency(value: &Value, _: &HashMap<String, Value>) -> Result<Value, Erro
     }
 }
 
+pub fn format03d(value: &Value, _: &HashMap<String, Value>) -> Result<Value, Error> {
+    match value.as_u64() {
+        Some(number) => {
+            let result = format!("{:03}", number);
+            Ok(to_value(result).unwrap())
+        }
+        None => Err("format03d could not find a string".into()),
+    }
+}
+
 pub fn gt0(value: Option<&Value>, params: &[Value]) -> tera::Result<bool> {
     number_args_allowed("gt0", 0, params.len())?;
     value_defined("gt0", value)?;
@@ -86,6 +96,19 @@ pub fn gt0(value: Option<&Value>, params: &[Value]) -> tera::Result<bool> {
     }
 }
 
+pub fn credito(value: Option<&Value>, params: &[Value]) -> tera::Result<bool> {
+    number_args_allowed("credito", 0, params.len())?;
+    value_defined("credito", value)?;
+
+    match value.and_then(|v| v.as_str()) {
+        Some("Credito") => Ok(true),
+        Some("Contado") => Ok(false),
+        _ => Err(Error::msg(
+            "Tester `gt0` was called on a variable that isn't a number",
+        )),
+    }
+}
+
 lazy_static::lazy_static! {
     pub static ref TEMPLATES: Tera = {
         let mut tera = match Tera::new("src/templates/**/*.xml") {
@@ -99,8 +122,10 @@ lazy_static::lazy_static! {
         tera.register_filter("multiply100", multiply100);
         tera.register_filter("round_decimal", round_decimal);
         tera.register_filter("currency", currency);
+        tera.register_filter("format03d", format03d);
 
         tera.register_tester("gt0", gt0);
+        tera.register_tester("credito", credito);
 
         // tera.autoescape_on(vec![".xml"]);
         tera

diff --git a/src/templates/renderer/invoice.xml b/src/templates/renderer/invoice.xml
@@ -1,4 +1,5 @@
-{%- import "ubl/standard/include/document-line.xml" as macro -%}
+{%- import "ubl/standard/include/document-line.xml" as macros_document_line -%}
+{%- import "ubl/standard/include/address.xml" as macros_address -%}
 <?xml version="1.0" encoding="ISO-8859-1"?>
 <Invoice xmlns="urn:oasis:names:specification:ubl:schema:xsd:Invoice-2"
     {%- include "ubl/standard/include/namespaces.xml" -%}
@@ -40,11 +41,10 @@
     {%- include "ubl/common/signature.xml" %}
     {% include "ubl/standard/include/supplier.xml" %}
     {% include "ubl/standard/include/customer.xml" -%}
-    {%- if direccionEntrega %}
+    {%- if direccion_entrega %}
     <cac:Delivery>
         <cac:DeliveryLocation>
-            <cac:Address>
-                {% include "ubl/standard/include/address.xml" %}
+            <cac:Address>{{ macros_address::address(direccion=direccion_entrega) }}
             </cac:Address>
         </cac:DeliveryLocation>
     </cac:Delivery>
@@ -117,7 +117,7 @@
     {%- for it in detalles %}
     <cac:InvoiceLine>
         <cbc:ID>{{loop.index}}</cbc:ID>
-        <cbc:InvoicedQuantity unitCode="{{it.unidad_medida}}" unitCodeListAgencyName="United Nations Economic Commission for Europe" unitCodeListID="UN/ECE rec 20">{{it.cantidad}}</cbc:InvoicedQuantity>{{ macro::line(item=it) }}
+        <cbc:InvoicedQuantity unitCode="{{it.unidad_medida}}" unitCodeListAgencyName="United Nations Economic Commission for Europe" unitCodeListID="UN/ECE rec 20">{{it.cantidad}}</cbc:InvoicedQuantity>{{ macros_document_line::line(item=it) }}
     </cac:InvoiceLine>
     {%- endfor %}
 </Invoice>