Fix: Formateo PEP8 (79 cols) y limpieza flake8 en tests y examples

- Líneas >79 cols reajustadas (asserts, strings CUDE/CUDS, comentarios)
- Elimina imports y variables muertas (F401/F841), E712 (== False -> is False)
- Reordena imports en docs/conf.py

CONTRIBUTING.rst

@@ -69,7 +69,7 @@ Ready to contribute? Here's how to set up `facho` for local development.
     $ python3 -mvenv facho-venv && source facho-venv/bin/activate
     $ cd facho/
     $ pre-commit install
-    $ python setup.py develop
+    $ pip install -e .
 
 4. Create a branch for local development::
 
@@ -81,7 +81,7 @@ Ready to contribute? Here's how to set up `facho` for local development.
    tests, including testing other Python versions with tox::
 
     $ flake8 facho tests
-    $ python setup.py test or py.test
+    $ py.test
     $ tox
 
    To get flake8 and tox, just pip install them into your virtualenv.
@@ -97,10 +97,10 @@ Ready to contribute? Here's how to set up `facho` for local development.
 Using docker
 ------------
 
-1. make -f Makefile.dev build
+1. make -f Makefile.dev dev-setup
 2. make -f Makefile.dev dev-shell
-3. make -f Makefile.dev python3.8 setup.py develop
-4. make -f Makefile.dev python3.8 setup.py test
+3. make -f Makefile.dev test
+4. make -f Makefile.dev tox
 
 Pull Request Guidelines
 -----------------------

Dockerfile

@@ -7,34 +7,45 @@ RUN apt install software-properties-common -y \
     && add-apt-repository ppa:deadsnakes/ppa
 
 RUN apt-get install -y --no-install-recommends \
-  python3.9 python3.9-distutils python3.9-dev \
   python3.10 python3.10-distutils python3.10-dev \
   python3.11 python3.11-distutils python3.11-dev \
   python3.12 python3-setuptools python3.12-dev \
+  python3.13 python3-setuptools python3.13-dev \
   wget \
   ca-certificates
 
 RUN wget https://bootstrap.pypa.io/get-pip.py \
-  && python3.9 get-pip.py pip==23.2.1 --break-system-packages \
   && python3.10 get-pip.py pip==23.2.1 --break-system-packages \
   && python3.11 get-pip.py pip==23.2.1 --break-system-packages \
   && python3.12 get-pip.py pip==23.2.1  --break-system-packages \
+  && python3.13 get-pip.py pip==23.2.1  --break-system-packages \
   && rm get-pip.py
 
 RUN apt-get install -y --no-install-recommends \
+	gcc \
+	build-essential \
+    	pkg-config \
     	libxml2-dev \
     	libxmlsec1-dev \
-        build-essential \
+    	xmlsec1 \
+   	libxml2-dev \
+    	libxslt-dev \
         zip
 
-RUN python3.9 --version
 RUN python3.10 --version
 RUN python3.11 --version
 RUN python3.12 --version
+RUN python3.13 --version
+
 
-RUN pip3.9 install setuptools setuptools-rust
 RUN pip3.10 install setuptools setuptools-rust
 RUN pip3.11 install setuptools setuptools-rust --break-system-packages
 RUN pip3.12 install setuptools setuptools-rust --break-system-packages
+RUN pip3.13 install setuptools setuptools-rust --break-system-packages
 
 RUN pip3 install tox pytest --break-system-packages
+
+WORKDIR /app
+COPY pyproject.toml README.rst ./
+COPY facho ./facho
+RUN pip3.13 install -e . --break-system-packages

Makefile

@@ -80,9 +80,8 @@ release: dist ## package and upload a release
 	twine upload dist/*
 
 dist: clean ## builds source and wheel package
-	python setup.py sdist
-	python setup.py bdist_wheel
+	python -m build
 	ls -l dist
 
 install: clean ## install the package to the active Python's site-packages
-	python setup.py install
+	pip install .

Makefile.dev

@@ -15,7 +15,7 @@ dev-shell:
 	docker run --rm -ti -v "$(PWD):/app" -w /app --name facho-cli facho bash
 
 test:
-	docker run -t -v $(PWD):/app -w /app facho sh -c 'cd /app; python3.12 setup.py test'
+	docker run -t -v $(PWD):/app -w /app facho sh -c 'cd /app; python3.13 -m pytest -vv'
 
 tox:
 	docker run -it -v $(PWD)/:/app -w /app facho tox

docs/conf.py

@@ -18,11 +18,11 @@
 # relative to the documentation root, use os.path.abspath to make it
 # absolute, like shown here.
 #
+import facho
 import os
 import sys
 sys.path.insert(0, os.path.abspath('..'))
 
-import facho
 
 # -- General configuration ---------------------------------------------
 

docs/installation.rst

@@ -44,7 +44,7 @@ Once you have a copy of the source, you can install it with:
 
 .. code-block:: console
 
-    $ python setup.py install
+    $ pip install .
 
 
 .. _Github repo: https://github.com/bit4bit/facho

examples/generate-invoice-from-cli.py

@@ -11,13 +11,18 @@ from facho.fe import form_xml
 from facho.fe import fe
 
 # importar otras necesarias
-from datetime import datetime, date
+from datetime import datetime
 
 # Datos del fomulario del SET de pruebas
-INVOICE_AUTHORIZATION = '181360000001'  # Número suministrado por la Dian en el momento de la creación del SET de Pruebas
-ID_SOFTWARE = '57bcb6d1-c591-5a90-b80a-cb030ec91440' #Id suministrado por la Dian en el momento de la creación del SET de Pruebas  
-PIN = '19642' #Número creado por la empresa para poder crear el SET de pruebas
-CLAVE_TECNICA = 'fc9eac422eba16e21ffd8c5f94b3f30a6e38162d' ##Id suministrado por la Dian en el momento de la creación del SET de Pruebas  
+# Número suministrado por la Dian en el momento de la creación del SET de
+# Pruebas
+INVOICE_AUTHORIZATION = '181360000001'
+# Id suministrado por la Dian en el momento de la creación del SET de Pruebas
+ID_SOFTWARE = '57bcb6d1-c591-5a90-b80a-cb030ec91440'
+# Número creado por la empresa para poder crear el SET de pruebas
+PIN = '19642'
+# Id suministrado por la Dian en el momento de la creación del SET de Pruebas
+CLAVE_TECNICA = 'fc9eac422eba16e21ffd8c5f94b3f30a6e38162d'
 
 
 # callback que retonar las extensiones XML necesarias
@@ -26,15 +31,24 @@ CLAVE_TECNICA = 'fc9eac422eba16e21ffd8c5f94b3f30a6e38162d' ##Id suministrado por
 # muchos de los valores usados son obtenidos
 # del servicio web de la DIAN.
 def extensions(inv):
-    security_code = fe.DianXMLExtensionSoftwareSecurityCode(ID_SOFTWARE, PIN, inv.invoice_ident)
+    security_code = fe.DianXMLExtensionSoftwareSecurityCode(
+        ID_SOFTWARE, PIN, inv.invoice_ident)
     authorization_provider = fe.DianXMLExtensionAuthorizationProvider()
     cufe = fe.DianXMLExtensionCUFE(inv, CLAVE_TECNICA, fe.AMBIENTE_PRUEBAS)
-    software_provider = fe.DianXMLExtensionSoftwareProvider('nit_empresa', 'dígito_verificación', ID_SOFTWARE)
-    inv_authorization = fe.DianXMLExtensionInvoiceAuthorization(INVOICE_AUTHORIZATION,
-                                                                datetime(2019, 1, 19),#Datos toamdos de 
-                                                                datetime(2030, 1, 19),#la configuración 
+    software_provider = fe.DianXMLExtensionSoftwareProvider(
+        'nit_empresa', 'dígito_verificación', ID_SOFTWARE)
+    inv_authorization = fe.DianXMLExtensionInvoiceAuthorization(
+        INVOICE_AUTHORIZATION,
+        # Datos tomados de la configuración
+        datetime(2019, 1, 19),
+        datetime(2030, 1, 19),
         'SETP', 990000000, 995000000)  # del SET de pruebas
-    return [security_code, authorization_provider, cufe, software_provider, inv_authorization]
+    return [
+        security_code,
+        authorization_provider,
+        cufe,
+        software_provider,
+        inv_authorization]
 
 
 def invoice():

examples/generate-nomina-from-cli.py

@@ -1,9 +1,11 @@
 
 from facho import fe
 
+
 def extensions(nomina):
     return []
 
+
 def nomina():
     nomina = fe.nomina.DIANNominaIndividual()
 

facho/cli.py

@@ -1,39 +1,37 @@
 # -*- coding: utf-8 -*-
-import sys
-import base64
 import warnings
 
 import click
 
 import logging.config
 
-logging.config.dictConfig({
-    'version': 1,
-    'formatters': {
-        'verbose': {
-            'format': '%(name)s: %(message)s'
-        }
-    },
-    'handlers': {
-        'console': {
-            'level': 'DEBUG',
-            'class': 'logging.StreamHandler',
-            'formatter': 'verbose',
+logging.config.dictConfig(
+    {
+        "version": 1,
+        "formatters": {"verbose": {"format": "%(name)s: %(message)s"}},
+        "handlers": {
+            "console": {
+                "level": "DEBUG",
+                "class": "logging.StreamHandler",
+                "formatter": "verbose",
             },
         },
-    'loggers': {
-        'zeep.transports': {
-            'level': 'DEBUG',
-            'propagate': True,
-            'handlers': ['console'],
+        "loggers": {
+            "zeep.transports": {
+                "level": "DEBUG",
+                "propagate": True,
+                "handlers": ["console"],
+            },
         },
     }
-})
+)
+
 
 def disable_ssl():
     # MACHETE
     import ssl
-    if getattr(ssl, '_create_unverified_context', None):
+
+    if getattr(ssl, "_create_unverified_context", None):
         ssl._create_default_https_context = ssl._create_unverified_context
         warnings.warn("be sure!! ssl disable")
     else:
@@ -41,99 +39,122 @@ def disable_ssl():
 
 
 @click.command()
-@click.option('--nit', required=True)
-@click.option('--nit-proveedor', required=True)
-@click.option('--id-software', required=True)
-@click.option('--username', required=True)
-@click.option('--password', required=True)
-def consultaResolucionesFacturacion(nit, nit_proveedor, id_software, username, password):
+@click.option("--nit", required=True)
+@click.option("--nit-proveedor", required=True)
+@click.option("--id-software", required=True)
+@click.option("--username", required=True)
+@click.option("--password", required=True)
+def consultaResolucionesFacturacion(
+    nit, nit_proveedor, id_software, username, password
+):
     from facho.fe.client import dian
-    client_dian = dian.DianClient(username,
-                                  password)
-    resp = client_dian.request(dian.ConsultaResolucionesFacturacionPeticion(
+
+    client_dian = dian.DianClient(username, password)
+    resp = client_dian.request(
+        dian.ConsultaResolucionesFacturacionPeticion(
             nit, nit_proveedor, id_software
-    ))
+        )
+    )
     print(str(resp))
 
 
 @click.command()
-@click.option('--private-key', required=True)
-@click.option('--public-key', required=True)
-@click.option('--habilitacion/--produccion', default=False)
-@click.option('--password')
-@click.option('--test-setid', required=True)
-@click.argument('filename', required=True)
-@click.argument('zipfile', type=click.Path(exists=True))
-def soap_send_test_set_async(private_key, public_key, habilitacion, password, test_setid, filename, zipfile):
+@click.option("--private-key", required=True)
+@click.option("--public-key", required=True)
+@click.option("--habilitacion/--produccion", default=False)
+@click.option("--password")
+@click.option("--test-setid", required=True)
+@click.argument("filename", required=True)
+@click.argument("zipfile", type=click.Path(exists=True))
+def soap_send_test_set_async(
+    private_key,
+    public_key,
+    habilitacion,
+    password,
+    test_setid,
+    filename,
+    zipfile,
+):
     from facho.fe.client import dian
 
-    client = dian.DianSignatureClient(private_key, public_key, password=password)
+    client = dian.DianSignatureClient(
+        private_key, public_key, password=password
+    )
     req = dian.SendTestSetAsync
     if habilitacion:
         req = dian.Habilitacion.SendTestSetAsync
-    resp = client.request(req(
+    resp = client.request(
+        req(
             filename,
-        open(zipfile, 'rb').read(),
+            open(zipfile, "rb").read(),
             test_setid,
-    ))
+        )
+    )
     print(resp)
 
+
 @click.command()
-@click.option('--private-key', required=True)
-@click.option('--public-key', required=True)
-@click.option('--habilitacion/--produccion', default=False)
-@click.option('--password')
-@click.argument('filename', required=True)
-@click.argument('zipfile', type=click.Path(exists=True))
-def soap_send_bill_async(private_key, public_key, habilitacion, password, filename, zipfile):
+@click.option("--private-key", required=True)
+@click.option("--public-key", required=True)
+@click.option("--habilitacion/--produccion", default=False)
+@click.option("--password")
+@click.argument("filename", required=True)
+@click.argument("zipfile", type=click.Path(exists=True))
+def soap_send_bill_async(
+    private_key, public_key, habilitacion, password, filename, zipfile
+):
     from facho.fe.client import dian
 
-    client = dian.DianSignatureClient(private_key, public_key, password=password)
+    client = dian.DianSignatureClient(
+        private_key, public_key, password=password
+    )
     req = dian.SendBillAsync
     if habilitacion:
         req = dian.Habilitacion.SendBillAsync
-    resp = client.request(req(
-        filename,
-        open(zipfile, 'rb').read()
-    ))
+    resp = client.request(req(filename, open(zipfile, "rb").read()))
     print(resp)
 
+
 @click.command()
-@click.option('--private-key', required=True)
-@click.option('--public-key', required=True)
-@click.option('--habilitacion/--produccion', default=False)
-@click.option('--password')
-@click.argument('filename', required=True)
-@click.argument('zipfile', type=click.Path(exists=True))
-def soap_send_bill_sync(private_key, public_key, habilitacion, password, filename, zipfile):
+@click.option("--private-key", required=True)
+@click.option("--public-key", required=True)
+@click.option("--habilitacion/--produccion", default=False)
+@click.option("--password")
+@click.argument("filename", required=True)
+@click.argument("zipfile", type=click.Path(exists=True))
+def soap_send_bill_sync(
+    private_key, public_key, habilitacion, password, filename, zipfile
+):
     from facho.fe.client import dian
 
-    client = dian.DianSignatureClient(private_key, public_key, password=password)
+    client = dian.DianSignatureClient(
+        private_key, public_key, password=password
+    )
     req = dian.SendBillSync
     if habilitacion:
         req = dian.Habilitacion.SendBillSync
-    resp = client.request(req(
-        filename,
-        open(zipfile, 'rb').read()
-    ))
+    resp = client.request(req(filename, open(zipfile, "rb").read()))
     print(resp)
 
+
 @click.command()
-@click.option('--private-key', required=True)
-@click.option('--public-key', required=True)
-@click.option('--habilitacion/--produccion', default=False)
-@click.option('--password')
-@click.option('--track-id', required=True)
-def soap_get_status_zip(private_key, public_key, habilitacion, password, track_id):
+@click.option("--private-key", required=True)
+@click.option("--public-key", required=True)
+@click.option("--habilitacion/--produccion", default=False)
+@click.option("--password")
+@click.option("--track-id", required=True)
+def soap_get_status_zip(
+    private_key, public_key, habilitacion, password, track_id
+):
     from facho.fe.client import dian
 
-    client = dian.DianSignatureClient(private_key, public_key, password=password)
+    client = dian.DianSignatureClient(
+        private_key, public_key, password=password
+    )
     req = dian.GetStatusZip
     if habilitacion:
         req = dian.Habilitacion.GetStatusZip
-    resp = client.request(req(
-        trackId = track_id
-    ))
+    resp = client.request(req(trackId=track_id))
 
     print("StatusCode:", resp.StatusCode)
     print("StatusDescription:", resp.StatusDescription)
@@ -143,68 +164,78 @@ def soap_get_status_zip(private_key, public_key, habilitacion, password, track_i
 
 
 @click.command()
-@click.option('--private-key', required=True)
-@click.option('--public-key', required=True)
-@click.option('--habilitacion/--produccion', default=False)
-@click.option('--password')
-@click.option('--track-id', required=True)
-def soap_get_status(private_key, public_key, habilitacion, password, track_id):
+@click.option("--private-key", required=True)
+@click.option("--public-key", required=True)
+@click.option("--habilitacion/--produccion", default=False)
+@click.option("--password")
+@click.option("--track-id", required=True)
+def soap_get_status(
+    private_key, public_key, habilitacion, password, track_id
+):
     from facho.fe.client import dian
 
-    client = dian.DianSignatureClient(private_key, public_key, password=password)
+    client = dian.DianSignatureClient(
+        private_key, public_key, password=password
+    )
     req = dian.GetStatus
     if habilitacion:
         req = dian.Habilitacion.GetStatus
-    resp = client.request(req(
-        trackId = track_id
-    ))
+    resp = client.request(req(trackId=track_id))
     print(resp)
 
+
 @click.command()
-@click.option('--private-key', required=True)
-@click.option('--public-key', required=True)
-@click.option('--habilitacion/--produccion', default=False)
-@click.option('--password')
-@click.option('--nit', required=True)
-@click.option('--nit-proveedor', required=True)
-@click.option('--id-software', required=True)
-def soap_get_numbering_range(private_key,
+@click.option("--private-key", required=True)
+@click.option("--public-key", required=True)
+@click.option("--habilitacion/--produccion", default=False)
+@click.option("--password")
+@click.option("--nit", required=True)
+@click.option("--nit-proveedor", required=True)
+@click.option("--id-software", required=True)
+def soap_get_numbering_range(
+    private_key,
     public_key,
     habilitacion,
     password,
-                             nit, nit_proveedor, id_software):
+    nit,
+    nit_proveedor,
+    id_software,
+):
     from facho.fe.client import dian
 
-    client = dian.DianSignatureClient(private_key, public_key, password=password)
+    client = dian.DianSignatureClient(
+        private_key, public_key, password=password
+    )
     req = dian.GetNumberingRange
     if habilitacion:
         req = dian.Habilitacion.GetNumberingRange
-    resp = client.request(req(
-        nit, nit_proveedor, id_software
-    ))
+    resp = client.request(req(nit, nit_proveedor, id_software))
     print(resp)
 
+
 @click.command()
-@click.argument('invoice_path')
+@click.argument("invoice_path")
 def validate_invoice(invoice_path):
     warnings.warn("!! NO APROBADO FUNCIONAMIENTO")
 
     from facho.fe.data.dian import XSD
-    content = open(invoice_path, 'r').read()
+
+    content = open(invoice_path, "r").read()
     # TODO donde ubicar esta responsabilidad?
     # esto es requerido por el XSD de la DIAN
     content = content.replace(
         'xmlns:fe="http://www.dian.gov.co/contratos/facturaelectronica/v1"',
-        'xmlns:fe="urn:oasis:names:specification:ubl:schema:xsd:Invoice-2"'
+        'xmlns:fe="urn:oasis:names:specification:ubl:schema:xsd:Invoice-2"',
     )
     XSD.validate(content, XSD.UBLInvoice)
 
 
 @click.command()
-@click.argument('nomina_path')
+@click.argument("nomina_path")
 def validate_nominaindividual(nomina_path):
     from facho.fe.data.dian import XSD
-    content = open(nomina_path, 'r').read()
+
+    content = open(nomina_path, "r").read()
     content = content.replace(
         'xmlns="http://www.dian.gov.co/contratos/facturaelectronica/v1"',
         'xmlns="dian:gov:co:facturaelectronica:NominaIndividual"',
@@ -213,35 +244,55 @@ def validate_nominaindividual(nomina_path):
 
 
 @click.command()
-@click.option('--private-key', type=click.Path(exists=True))
-@click.option('--passphrase')
-@click.option('--ssl/--no-ssl', default=False)
-@click.option('--use-cache-policy/--no-use-cache-policy', default=False)
-@click.argument('xmlfile', type=click.Path(exists=True), required=True)
-@click.argument('output', required=True)
-def sign_xml(private_key, passphrase, xmlfile, ssl=True, use_cache_policy=False, output=None):
+@click.option("--private-key", type=click.Path(exists=True))
+@click.option("--passphrase")
+@click.option("--ssl/--no-ssl", default=False)
+@click.option("--use-cache-policy/--no-use-cache-policy", default=False)
+@click.argument("xmlfile", type=click.Path(exists=True), required=True)
+@click.argument("output", required=True)
+def sign_xml(
+    private_key,
+    passphrase,
+    xmlfile,
+    ssl=True,
+    use_cache_policy=False,
+    output=None,
+):
     if not ssl:
         disable_ssl()
 
     from facho import fe
+
     if use_cache_policy:
         warnings.warn("xades using cache policy")
 
-    signer = fe.DianXMLExtensionSigner(private_key, passphrase=passphrase, localpolicy=use_cache_policy)
-    document = open(xmlfile, 'r').read().encode('utf-8')
-    with open(output, 'w') as f:
+    signer = fe.DianXMLExtensionSigner(
+        private_key, passphrase=passphrase, localpolicy=use_cache_policy
+    )
+    document = open(xmlfile, "r").read().encode("utf-8")
+    with open(output, "w") as f:
         f.write(signer.sign_xml_string(document))
 
+
 @click.command()
-@click.option('--private-key', type=click.Path(exists=True))
-@click.option('--generate/--validate', default=False)
-@click.option('--passphrase')
-@click.option('--ssl/--no-ssl', default=False)
-@click.option('--sign/--no-sign', default=False)
-@click.option('--use-cache-policy/--no-use-cache-policy', default=False)
-@click.argument('scriptname', type=click.Path(exists=True), required=True)
-@click.argument('output', required=True)
-def generate_invoice(private_key, passphrase, scriptname, generate=False, ssl=True, sign=False, use_cache_policy=False, output=None):
+@click.option("--private-key", type=click.Path(exists=True))
+@click.option("--generate/--validate", default=False)
+@click.option("--passphrase")
+@click.option("--ssl/--no-ssl", default=False)
+@click.option("--sign/--no-sign", default=False)
+@click.option("--use-cache-policy/--no-use-cache-policy", default=False)
+@click.argument("scriptname", type=click.Path(exists=True), required=True)
+@click.argument("output", required=True)
+def generate_invoice(
+    private_key,
+    passphrase,
+    scriptname,
+    generate=False,
+    ssl=True,
+    sign=False,
+    use_cache_policy=False,
+    output=None,
+):
     """
     imprime xml en pantalla.
     SCRIPTNAME espera
@@ -254,13 +305,15 @@ def generate_invoice(private_key, passphrase, scriptname, generate=False, ssl=Tr
 
     import importlib.util
 
-    spec = importlib.util.spec_from_file_location('invoice', scriptname)
+    spec = importlib.util.spec_from_file_location("invoice", scriptname)
     module = importlib.util.module_from_spec(spec)
     spec.loader.exec_module(module)
 
-    import facho.fe.form as form
-    from facho.fe.form_xml import DIANInvoiceXML, DIANWriteSigned, DIANWrite, DIANSupportDocumentXML
-    from facho import fe
+    from facho.fe.form_xml import (
+        DIANWriteSigned,
+        DIANWrite,
+        DIANSupportDocumentXML,
+    )
 
     try:
         invoice_xml = module.document_xml()
@@ -279,24 +332,39 @@ def generate_invoice(private_key, passphrase, scriptname, generate=False, ssl=Tr
             xml.add_extension(extension)
 
         if sign:
-            DIANWriteSigned(xml, output, private_key, passphrase, use_cache_policy)
+            DIANWriteSigned(
+                xml, output, private_key, passphrase, use_cache_policy
+            )
         else:
             DIANWrite(xml, output)
 
+
 @click.command()
-@click.option('--private-key', type=click.Path(exists=True))
-@click.option('--passphrase')
-@click.option('--ssl/--no-ssl', default=False)
-@click.option('--sign/--no-sign', default=False)
-@click.option('--use-cache-policy/--no-use-cache-policy', default=False)
-@click.argument('scriptname', type=click.Path(exists=True), required=True)
-@click.argument('output', required=True)
-def generate_nomina(private_key, passphrase, scriptname, ssl=True, sign=False, use_cache_policy=False, output=None):
+@click.option("--private-key", type=click.Path(exists=True))
+@click.option("--passphrase")
+@click.option("--ssl/--no-ssl", default=False)
+@click.option("--sign/--no-sign", default=False)
+@click.option("--use-cache-policy/--no-use-cache-policy", default=False)
+@click.argument("scriptname", type=click.Path(exists=True), required=True)
+@click.argument("output", required=True)
+def generate_nomina(
+    private_key,
+    passphrase,
+    scriptname,
+    ssl=True,
+    sign=False,
+    use_cache_policy=False,
+    output=None,
+):
     """
     imprime xml en pantalla.
     SCRIPTNAME espera
-     def nomina() -> fe.nomina.NominaIndividual
-     def extensions(fe.nomina.NominaIndividual): -> List[facho.FachoXMLExtension]
+    def nomina() -> (
+    fe.nomina.NominaIndividual
+    )
+    def extensions(
+    fe.nomina.NominaIndividual
+    ): -> List[facho.FachoXMLExtension]
     """
 
     if not ssl:
@@ -304,7 +372,7 @@ def generate_nomina(private_key, passphrase, scriptname, ssl=True, sign=False, u
 
     import importlib.util
 
-    spec = importlib.util.spec_from_file_location('nomina', scriptname)
+    spec = importlib.util.spec_from_file_location("nomina", scriptname)
     module = importlib.util.module_from_spec(spec)
     spec.loader.exec_module(module)
 
@@ -320,59 +388,83 @@ def generate_nomina(private_key, passphrase, scriptname, ssl=True, sign=False, u
         xml.add_extension(extension)
 
     if sign:
-        DIANWriteSigned(xml, output, private_key, passphrase, use_cache_policy, dian_signer=facho.fe.nomina.DianXMLExtensionSigner)
+        DIANWriteSigned(
+            xml,
+            output,
+            private_key,
+            passphrase,
+            use_cache_policy,
+            dian_signer=facho.fe.nomina.DianXMLExtensionSigner,
+        )
     else:
         DIANWrite(xml, output)
 
+
 @click.command()
-@click.option('--private-key', required=True)
-@click.option('--public-key', required=True)
-@click.option('--habilitacion/--produccion', default=False)
-@click.option('--password')
-@click.argument('filename', required=True)
-@click.argument('zipfile', type=click.Path(exists=True))
-def soap_send_nomina_sync(private_key, public_key, habilitacion, password, filename, zipfile):
+@click.option("--private-key", required=True)
+@click.option("--public-key", required=True)
+@click.option("--habilitacion/--produccion", default=False)
+@click.option("--password")
+@click.argument("filename", required=True)
+@click.argument("zipfile", type=click.Path(exists=True))
+def soap_send_nomina_sync(
+    private_key, public_key, habilitacion, password, filename, zipfile
+):
     from facho.fe.client import dian
 
-    client = dian.DianSignatureClient(private_key, public_key, password=password)
+    client = dian.DianSignatureClient(
+        private_key, public_key, password=password
+    )
     req = dian.SendNominaSync
     if habilitacion:
         req = dian.Habilitacion.SendNominaSync
-    resp = client.request(req(
-        open(zipfile, 'rb').read()
-    ))
+    resp = client.request(req(open(zipfile, "rb").read()))
     print(resp)
 
+
 @click.command()
-@click.option('--private-key', type=click.Path(exists=True))
-@click.option('--passphrase')
-@click.option('--ssl/--no-ssl', default=False)
-@click.option('--use-cache-policy/--no-use-cache-policy', default=False)
-@click.argument('xmlfile', type=click.Path(exists=True), required=True)
-def sign_verify_xml(private_key, passphrase, xmlfile, ssl=True, use_cache_policy=False, output=None):
+@click.option("--private-key", type=click.Path(exists=True))
+@click.option("--passphrase")
+@click.option("--ssl/--no-ssl", default=False)
+@click.option("--use-cache-policy/--no-use-cache-policy", default=False)
+@click.argument("xmlfile", type=click.Path(exists=True), required=True)
+def sign_verify_xml(
+    private_key,
+    passphrase,
+    xmlfile,
+    ssl=True,
+    use_cache_policy=False,
+    output=None,
+):
     if not ssl:
         disable_ssl()
 
     from facho.fe import fe
+
     if use_cache_policy:
         warnings.warn("xades using cache policy")
 
     print("THIS ONLY WORKS FOR DOCUMENTS GENERATE WITH FACHO")
-    signer = fe.DianXMLExtensionSignerVerifier(private_key, passphrase=passphrase, localpolicy=use_cache_policy)
-    document = open(xmlfile, 'r').read().encode('utf-8')
+    signer = fe.DianXMLExtensionSignerVerifier(
+        private_key, passphrase=passphrase, localpolicy=use_cache_policy
+    )
+    document = open(xmlfile, "r").read().encode("utf-8")
 
     if signer.verify_string(document):
         print("+OK")
     else:
         print("-INVALID")
 
+
 @click.command()
-@click.option('--software-id')
-@click.option('--software-pin')
-@click.option('--nit')
-@click.option('--dv')
-@click.option('--output-zippath')
-def generate_nomina_habilitacion(software_id, software_pin, nit, dv, output_zippath):
+@click.option("--software-id")
+@click.option("--software-pin")
+@click.option("--nit")
+@click.option("--dv")
+@click.option("--output-zippath")
+def generate_nomina_habilitacion(
+    software_id, software_pin, nit, dv, output_zippath
+):
     from facho import fe
 
     generador = fe.nomina.habilitacion.Habilitacion(
@@ -380,15 +472,17 @@ def generate_nomina_habilitacion(software_id, software_pin, nit, dv, output_zipp
             software_id=software_id,
             software_pin=software_pin,
             nit=nit,
-            dv=dv
+            dv=dv,
         )
     )
     generador.generar(output_zippath)
 
+
 @click.group()
 def main():
     pass
 
+
 main.add_command(consultaResolucionesFacturacion)
 main.add_command(soap_send_test_set_async)
 main.add_command(soap_send_bill_async)

facho/facho.py

@@ -140,12 +140,12 @@ class LXMLBuilder:
         attrs['pretty_print'] = attrs.pop('pretty_print', False)
         attrs['encoding'] = attrs.pop('encoding', 'UTF-8')
 
-        for el in elem.getiterator():
+        for el in elem.iter():
             keys = filter(lambda key: key.startswith('facho_'), el.keys())
             self.remove_attributes(el, keys, exclude=['facho_optional'])
 
             is_optional = el.get('facho_optional', 'False') == 'True'
-            if is_optional and el.getchildren() == [] and el.keys() == [
+            if is_optional and list(el) == [] and el.keys() == [
                     'facho_optional']:
                 el.getparent().remove(el)
 
@@ -156,6 +156,7 @@ class FachoXML:
     """
     Decora XML con funciones de consulta XPATH de un solo elemento
     """
+
     def __init__(self, root, builder=None, nsmap=None, fragment_prefix='',
                  fragment_root_element=None):
         if builder is None:
@@ -291,7 +292,7 @@ class FachoXML:
         # se fuerza la adicion como un nuevo elemento
         if append:
             last_slibing = None
-            for child in parent.getchildren():
+            for child in list(parent):
                 if child.tag == node_tag:
                     last_slibing = child
 

facho/fe/__init__.py

@@ -14,3 +14,22 @@ from .fe import AMBIENTE_PRUEBAS
 from .fe import AMBIENTE_PRODUCCION
 from . import form_xml
 from . import nomina
+
+__all__ = [
+    'FeXML',
+    'fe_from_string',
+    'NAMESPACES',
+    'DianXMLExtensionSigner',
+    'DianXMLExtensionSoftwareSecurityCode',
+    'DianXMLExtensionCUFE',
+    'DianXMLExtensionCUDE',
+    'DianXMLExtensionCUDS',
+    'DianXMLExtensionInvoiceAuthorization',
+    'DianXMLExtensionSoftwareProvider',
+    'DianXMLExtensionAuthorizationProvider',
+    'DianZIP',
+    'AMBIENTE_PRUEBAS',
+    'AMBIENTE_PRODUCCION',
+    'form_xml',
+    'nomina',
+]

facho/fe/client/dian.py

@@ -1,23 +1,13 @@
-from facho import facho
 
 import zeep
 from zeep.wsse.username import UsernameToken
-from .wsse.signature import Signature, BinarySignature
-from zeep.wsa import WsAddressingPlugin
+from .wsse.signature import BinarySignature
 import xmlsec
-import urllib.request
-from datetime import datetime
-from dataclasses import dataclass, asdict, field
-from typing import List
-import http.client
-import hashlib
-import secrets
-import base64
+from dataclasses import dataclass, asdict
 
 
-__all__ = ['DianClient',
-           'ConsultaResolucionesFacturacionPeticion',
-           'ConsultaResolucionesFacturacionRespuesta']
+__all__ = ['DianClient']
+
 
 class SOAPService:
 
@@ -33,6 +23,7 @@ class SOAPService:
     def todict(self):
         return asdict(self)
 
+
 @dataclass
 class GetNumberingRangeResponse:
 
@@ -47,8 +38,7 @@ class GetNumberingRangeResponse:
         ValidateDateTo: str
         TechnicalKey: str
 
-    NumberRangeResponse: List[NumberRangeResponse]
-
+    NumberRangeResponse: list[NumberRangeResponse]
 
     @classmethod
     def fromdict(cls, data):
@@ -91,7 +81,7 @@ class SendBillAsync(SOAPService):
 @dataclass
 class SendTestSetAsyncResponse:
     ZipKey: str
-    ErrorMessageList: List[str]
+    ErrorMessageList: list[str]
 
     @classmethod
     def fromdict(cls, data):
@@ -100,6 +90,7 @@ class SendTestSetAsyncResponse:
             data['ErrorMessageList'] or []
         )
 
+
 @dataclass
 class SendTestSetAsync(SOAPService):
     fileName: str
@@ -115,6 +106,7 @@ class SendTestSetAsync(SOAPService):
     def build_response(self, as_dict):
         return SendTestSetAsyncResponse.fromdict(as_dict)
 
+
 @dataclass
 class SendBillSync(SOAPService):
     fileName: str
@@ -129,12 +121,13 @@ class SendBillSync(SOAPService):
     def build_response(self, as_dict):
         return as_dict
 
+
 @dataclass
 class GetStatusResponse:
     IsValid: bool
     StatusDescription: str
     StatusCode: int
-    ErrorMessage: List[str]
+    ErrorMessage: list[str]
 
     @classmethod
     def fromdict(cls, data):
@@ -162,6 +155,7 @@ class GetStatus(SOAPService):
     def build_response(self, as_dict):
         return GetStatusResponse.fromdict(as_dict)
 
+
 @dataclass
 class GetStatusZip(SOAPService):
     trackId: bytes
@@ -175,6 +169,7 @@ class GetStatusZip(SOAPService):
     def build_response(self, as_dict):
         return GetStatusResponse.fromdict(as_dict[0])
 
+
 @dataclass
 class SendNominaSync(SOAPService):
     contentFile: bytes
@@ -220,6 +215,7 @@ class Habilitacion:
         def get_wsdl(self):
             return Habilitacion.WSDL
 
+
 class DianGateway:
 
     def _open(self, service):
@@ -250,7 +246,11 @@ class DianClient(DianGateway):
         self._password = password
 
     def _open(self, service):
-        return zeep.Client(service.get_wsdl(), wsse=UsernameToken(self._username, self._password))
+        return zeep.Client(
+            service.get_wsdl(),
+            wsse=UsernameToken(
+                self._username,
+                self._password))
 
 
 class DianSignatureClient(DianGateway):
@@ -262,13 +262,10 @@ class DianSignatureClient(DianGateway):
 
     def _open(self, service):
         # RESOLUCCION 0004: pagina 756
-        from zeep.wsse import utils
 
-        client = zeep.Client(service.get_wsdl(), wsse=
-                             BinarySignature(
+        client = zeep.Client(service.get_wsdl(), wsse=BinarySignature(
             self.private_key_path, self.public_key_path, self.password,
             signature_method=xmlsec.Transform.RSA_SHA256,
-                                 digest_method=xmlsec.Transform.SHA256)
-                             ,
+            digest_method=xmlsec.Transform.SHA256),
         )
         return client

facho/fe/client/wsse/signature.py

@@ -9,7 +9,7 @@ module.
 
 """
 import pytz
-from datetime import datetime, timedelta
+from datetime import datetime, timedelta, timezone
 from lxml import etree
 from lxml.etree import QName
 
@@ -70,8 +70,11 @@ class MemorySignature(object):
     def apply(self, envelope, headers):
         key = _make_sign_key(self.key_data, self.cert_data, self.password)
         _sign_envelope_with_key(
-            envelope, key, self.signature_method, self.digest_method, expires_dt=self.expires_dt
-        )
+            envelope,
+            key,
+            self.signature_method,
+            self.digest_method,
+            expires_dt=self.expires_dt)
         return envelope, headers
 
     def verify(self, envelope):
@@ -81,7 +84,7 @@ class MemorySignature(object):
 
 
 class Signature(MemorySignature):
-    """Sign given SOAP envelope with WSSE sig using given key file and cert file."""
+    """Sign given SOAP envelope with WSSE sig using given key and cert file."""
 
     def __init__(
         self,
@@ -101,15 +104,18 @@ class Signature(MemorySignature):
 
 
 class BinarySignature(Signature):
-    """Sign given SOAP envelope with WSSE sig using given key file and cert file.
+    """Sign given SOAP envelope with WSSE sig using given key and cert file.
 
     Place the key information into BinarySecurityElement."""
 
     def apply(self, envelope, headers):
         key = _make_sign_key(self.key_data, self.cert_data, self.password)
         _sign_envelope_with_key_binary(
-            envelope, key, self.signature_method, self.digest_method, expires_dt = self.expires_dt
-        )
+            envelope,
+            key,
+            self.signature_method,
+            self.digest_method,
+            expires_dt=self.expires_dt)
         return envelope, headers
 
 
@@ -219,10 +225,12 @@ def sign_envelope(
     """
     # Load the signing key and certificate.
     key = _make_sign_key(_read_file(keyfile), _read_file(certfile), password)
-    return _sign_envelope_with_key(envelope, key, signature_method, digest_method)
+    return _sign_envelope_with_key(
+        envelope, key, signature_method, digest_method)
+
 
 def get_timestamp(timestamp=None, delta=None):
-    timestamp = timestamp or datetime.utcnow()
+    timestamp = timestamp or datetime.now(timezone.utc)
     if delta:
         timestamp += delta
 
@@ -230,11 +238,15 @@ def get_timestamp(timestamp = None, delta=None):
     timestamp = timestamp.replace(tzinfo=pytz.utc, microsecond=0)
     return timestamp.strftime(format_)
 
+
 def _append_timestamp(security, expires_dt=None):
     if expires_dt is None:
         expires_dt = timedelta(seconds=6000)
 
-    etimestamp = utils.WSU.Timestamp({'{http://docs.oasis-open.org/wss/2004/01/oasis-200401-wss-wssecurity-utility-1.0.xsd}Id': utils.get_unique_id()})
+    etimestamp = utils.WSU.Timestamp({
+        '{http://docs.oasis-open.org/wss/2004/01/'
+        'oasis-200401-wss-wssecurity-utility-1.0.xsd}Id':
+        utils.get_unique_id()})
     etimestamp.append(utils.WSU.Created(get_timestamp()))
     etimestamp.append(utils.WSU.Expires(get_timestamp(delta=expires_dt)))
     security.insert(0, etimestamp)
@@ -243,11 +255,16 @@ def _append_timestamp(security, expires_dt=None):
                                  keep_ns_prefixes=security.nsmap,
                                  top_nsmap=utils.NSMAP)
     else:
-        etree.cleanup_namespaces(header)
+        etree.cleanup_namespaces(security)
 
-def _signature_prepare(envelope, key, signature_method, digest_method, expires_dt=None):
+
+def _signature_prepare(
+        envelope,
+        key,
+        signature_method,
+        digest_method,
+        expires_dt=None):
     """Prepare envelope and sign."""
-    soap_env = detect_soap_env(envelope)
 
     # Create the Signature node.
     signature = xmlsec.template.create(
@@ -278,7 +295,7 @@ def _signature_prepare(envelope, key, signature_method, digest_method, expires_d
     _append_timestamp(security, expires_dt=expires_dt)
 
     timestamp = security.find(QName(ns.WSU, "Timestamp"))
-    if timestamp != None:
+    if timestamp is not None:
         _sign_node(ctx, signature, timestamp, digest_method)
     ctx.sign(signature)
 
@@ -286,18 +303,30 @@ def _signature_prepare(envelope, key, signature_method, digest_method, expires_d
     # KeyInfo. The recipient expects this structure, but we can't rearrange
     # like this until after signing, because otherwise xmlsec won't populate
     # the X509 data (because it doesn't understand WSSE).
-    sec_token_ref = etree.SubElement(key_info, QName(ns.WSSE, "SecurityTokenReference"))
+    sec_token_ref = etree.SubElement(
+        key_info, QName(
+            ns.WSSE, "SecurityTokenReference"))
     return security, sec_token_ref, x509_data
 
 
-def _sign_envelope_with_key(envelope, key, signature_method, digest_method, expires_dt=None):
+def _sign_envelope_with_key(
+        envelope,
+        key,
+        signature_method,
+        digest_method,
+        expires_dt=None):
     _, sec_token_ref, x509_data = _signature_prepare(
         envelope, key, signature_method, digest_method, expires_dt=expires_dt
     )
     sec_token_ref.append(x509_data)
 
 
-def _sign_envelope_with_key_binary(envelope, key, signature_method, digest_method, expires_dt=None):
+def _sign_envelope_with_key_binary(
+        envelope,
+        key,
+        signature_method,
+        digest_method,
+        expires_dt=None):
     security, sec_token_ref, x509_data = _signature_prepare(
         envelope, key, signature_method, digest_method, expires_dt=expires_dt
     )
@@ -353,7 +382,10 @@ def _verify_envelope_with_key(envelope, key):
     ctx = xmlsec.SignatureContext()
 
     # Find each signed element and register its ID with the signing context.
-    refs = signature.xpath("ds:SignedInfo/ds:Reference", namespaces={"ds": ns.DS})
+    refs = signature.xpath(
+        "ds:SignedInfo/ds:Reference",
+        namespaces={
+            "ds": ns.DS})
     for ref in refs:
         # Get the reference URI and cut off the initial '#'
         referenced_id = ref.get("URI")[1:]

facho/fe/data/dian/XSD/__init__.py

@@ -7,10 +7,17 @@ def path_for_xsd(dirname, xsdname):
     data_dir = os.path.dirname(os.path.abspath(__file__))
     return os.path.join(data_dir, dirname, xsdname)
 
-UBLInvoice= xmlschema.XMLSchema(path_for_xsd('maindoc', 'UBL-Invoice-2.1.xsd'))
 
-NominaIndividual = xmlschema.XMLSchema(path_for_xsd('nomina', 'NominaIndividualElectronicaXSDV1.0.6.xsd'))
-NominaIndividualDeAjuste = xmlschema.XMLSchema(path_for_xsd('nomina', 'NominaIndividualDeAjusteElectronicaXSDV1.0.6.xsd'))
+UBLInvoice = xmlschema.XMLSchema(
+    path_for_xsd(
+        'maindoc',
+        'UBL-Invoice-2.1.xsd'))
+
+NominaIndividual = xmlschema.XMLSchema(path_for_xsd(
+    'nomina', 'NominaIndividualElectronicaXSDV1.0.6.xsd'))
+NominaIndividualDeAjuste = xmlschema.XMLSchema(path_for_xsd(
+    'nomina', 'NominaIndividualDeAjusteElectronicaXSDV1.0.6.xsd'))
+
 
 def validate(xml, schema):
     schema.validate(xml)

facho/fe/data/dian/codelist/__init__.py

@@ -35,8 +35,8 @@ class CodeList:
         row = {}
 
         # construir registro...
-        for value in xmlrow.getchildren():
-            row[value.attrib['ColumnRef']] = value.getchildren()[0].text
+        for value in list(xmlrow):
+            row[value.attrib['ColumnRef']] = list(value)[0].text
 
         return row
 
@@ -60,6 +60,7 @@ class CodeList:
 # nombres de variables igual a ./Identification/ShortName
 # TODO: garantizar unica carga en python
 
+
 __all__ = ['TipoOrganizacion',
            'TipoResponsabilidad',
            'TipoAmbiente',
@@ -72,38 +73,88 @@ __all__ = ['TipoOrganizacion',
            'Municipio',
            'Departamento']
 
+
 def path_for_codelist(name):
     return os.path.join(DATA_DIR, name)
 
-TipoOrganizacion = CodeList(path_for_codelist('TipoOrganizacion-2.1.gc'), 'code', 'name')
-TipoResponsabilidad = CodeList(path_for_codelist('TipoResponsabilidad-2.1.gc'), 'code', 'name')\
-    .update(CodeList(path_for_codelist('TipoResponsabilidad-2.1.custom.gc'), 'code', 'name'))
-TipoAmbiente = CodeList(path_for_codelist('TipoAmbiente-2.1.gc'), 'code', 'name')
-TipoDocumento = CodeList(path_for_codelist('TipoDocumento-2.1.gc'), 'code', 'name')
-TipoImpuesto = CodeList(path_for_codelist('TipoImpuesto-2.1.gc'), 'code', 'name')\
-    .update(CodeList(path_for_codelist('TipoImpuesto-2.1.custom.gc'), 'code', 'name'))
-TarifaImpuesto = CodeList(path_for_codelist('TarifaImpuestoINC-2.1.gc'), 'code', 'name')\
-        .update(CodeList(path_for_codelist('TarifaImpuestoIVA-2.1.gc'), 'code', 'name'))\
-        .update(CodeList(path_for_codelist('TarifaImpuestoReteIVA-2.1.gc'), 'code', 'name'))\
-        .update(CodeList(path_for_codelist('TarifaImpuestoReteRenta-2.1.gc'), 'code', 'name'))
-CodigoPrecioReferencia = CodeList(path_for_codelist('CodigoPrecioReferencia-2.1.gc'), 'code', 'name')
+
+TipoOrganizacion = CodeList(path_for_codelist(
+    'TipoOrganizacion-2.1.gc'), 'code', 'name')
+TipoResponsabilidad = CodeList(
+    path_for_codelist('TipoResponsabilidad-2.1.gc'),
+    'code',
+    'name') .update(
+        CodeList(
+            path_for_codelist('TipoResponsabilidad-2.1.custom.gc'),
+            'code',
+            'name'))
+TipoAmbiente = CodeList(path_for_codelist(
+    'TipoAmbiente-2.1.gc'), 'code', 'name')
+TipoDocumento = CodeList(path_for_codelist(
+    'TipoDocumento-2.1.gc'), 'code', 'name')
+TipoImpuesto = CodeList(
+    path_for_codelist('TipoImpuesto-2.1.gc'),
+    'code',
+    'name') .update(
+        CodeList(
+            path_for_codelist('TipoImpuesto-2.1.custom.gc'),
+            'code',
+            'name'))
+TarifaImpuesto = CodeList(
+    path_for_codelist('TarifaImpuestoINC-2.1.gc'),
+    'code',
+    'name') .update(
+        CodeList(
+            path_for_codelist('TarifaImpuestoIVA-2.1.gc'),
+            'code',
+            'name')) .update(
+                CodeList(
+                    path_for_codelist('TarifaImpuestoReteIVA-2.1.gc'),
+                    'code',
+                    'name')) .update(
+                        CodeList(
+                            path_for_codelist(
+                                'TarifaImpuestoReteRenta-2.1.gc'),
+                            'code',
+                            'name'))
+CodigoPrecioReferencia = CodeList(path_for_codelist(
+    'CodigoPrecioReferencia-2.1.gc'), 'code', 'name')
 MediosPago = CodeList(path_for_codelist('MediosPago-2.1.gc'), 'code', 'name')
 FormasPago = CodeList(path_for_codelist('FormasPago-2.1.gc'), 'code', 'name')
-RegimenFiscal = CodeList(path_for_codelist('RegimenFiscal-2.1.custom.gc'), 'code', 'name')
-TipoOperacionNC = CodeList(path_for_codelist('TipoOperacionNC-2.1.gc'), 'code', 'name')
-TipoOperacionNCDS = CodeList(path_for_codelist('TipoOperacionNCDS-2.1.gc'), 'code', 'name')
-TipoOperacionND = CodeList(path_for_codelist('TipoOperacionND-2.1 - copia.gc'), 'code', 'name')
-TipoOperacionF = CodeList(path_for_codelist('TipoOperacionF-2.1.gc'), 'code', 'name')\
-    .update(CodeList(path_for_codelist('TipoOperacionF-2.1.custom.gc'), 'code', 'name'))
+RegimenFiscal = CodeList(path_for_codelist(
+    'RegimenFiscal-2.1.custom.gc'), 'code', 'name')
+TipoOperacionNC = CodeList(path_for_codelist(
+    'TipoOperacionNC-2.1.gc'), 'code', 'name')
+TipoOperacionNCDS = CodeList(path_for_codelist(
+    'TipoOperacionNCDS-2.1.gc'), 'code', 'name')
+TipoOperacionND = CodeList(path_for_codelist(
+    'TipoOperacionND-2.1 - copia.gc'), 'code', 'name')
+TipoOperacionF = CodeList(
+    path_for_codelist('TipoOperacionF-2.1.gc'),
+    'code',
+    'name') .update(
+        CodeList(
+            path_for_codelist('TipoOperacionF-2.1.custom.gc'),
+            'code',
+            'name'))
 Municipio = CodeList(path_for_codelist('Municipio-2.1.gc'), 'code', 'name')
-Departamento = CodeList(path_for_codelist('Departamentos-2.1.gc'), 'code', 'name')
+Departamento = CodeList(path_for_codelist(
+    'Departamentos-2.1.gc'), 'code', 'name')
 Paises = CodeList(path_for_codelist('Paises-2.1.gc'), 'code', 'name')
-TipoIdFiscal = CodeList(path_for_codelist('TipoIdFiscal-2.1.gc'), 'code', 'name')
-CodigoDescuento = CodeList(path_for_codelist('CodigoDescuento-2.1.gc'), 'code', 'name')
-UnidadesMedida = CodeList(path_for_codelist('UnidadesMedida-2.1.gc'), 'code', 'name')
-TipoTrabajador = CodeList(path_for_codelist('TipoTrabajador-2.1.gc'), 'code', 'name')
-SubTipoTrabajador = CodeList(path_for_codelist('SubTipoTrabajador-2.1.gc'), 'code', 'name')
-TipoContrato = CodeList(path_for_codelist('TipoContrato-2.1.gc'), 'code', 'name')
-PeriodoNomina = CodeList(path_for_codelist('PeriodoNomina-2.1.gc'), 'code', 'name')
+TipoIdFiscal = CodeList(path_for_codelist(
+    'TipoIdFiscal-2.1.gc'), 'code', 'name')
+CodigoDescuento = CodeList(path_for_codelist(
+    'CodigoDescuento-2.1.gc'), 'code', 'name')
+UnidadesMedida = CodeList(path_for_codelist(
+    'UnidadesMedida-2.1.gc'), 'code', 'name')
+TipoTrabajador = CodeList(path_for_codelist(
+    'TipoTrabajador-2.1.gc'), 'code', 'name')
+SubTipoTrabajador = CodeList(path_for_codelist(
+    'SubTipoTrabajador-2.1.gc'), 'code', 'name')
+TipoContrato = CodeList(path_for_codelist(
+    'TipoContrato-2.1.gc'), 'code', 'name')
+PeriodoNomina = CodeList(path_for_codelist(
+    'PeriodoNomina-2.1.gc'), 'code', 'name')
 TipoMoneda = CodeList(path_for_codelist('TipoMoneda-2.1.gc'), 'code', 'name')
-IdiomaISO6391 = CodeList(path_for_codelist('Idioma-2.1.gc'), 'iso-639-1', 'name')
+IdiomaISO6391 = CodeList(path_for_codelist(
+    'Idioma-2.1.gc'), 'iso-639-1', 'name')

facho/fe/fe.py

@@ -5,7 +5,6 @@ import uuid
 import xmlsig
 import xades
 from datetime import datetime
-import OpenSSL
 import zipfile
 # import warnings
 import hashlib
@@ -18,36 +17,65 @@ from dateutil import tz
 
 from cryptography.hazmat.primitives.serialization import pkcs12
 
+# Monkey-patch xades/xmlsig para compatibilidad con pyOpenSSL >= 24
+# (PKCS12 fue eliminado de OpenSSL.crypto, pero ambas librerías lo referencian)
+import OpenSSL
+import xades.xades_context
+
+if not hasattr(OpenSSL.crypto, 'PKCS12'):
+    def _patched_load_pkcs12(self, key):
+        if isinstance(key, tuple):
+            self.x509 = key[1]
+            self.public_key = key[1].public_key()
+            self.private_key = key[0]
+        else:
+            raise NotImplementedError("unsupported key type")
+    xades.xades_context.XAdESContext.load_pkcs12 = _patched_load_pkcs12
+
 AMBIENTE_PRUEBAS = codelist.TipoAmbiente.by_name('Pruebas')['code']
 AMBIENTE_PRODUCCION = codelist.TipoAmbiente.by_name('Producción')['code']
 
 
 SCHEME_AGENCY_ATTRS = {
-    'schemeAgencyName': 'CO, DIAN (Dirección de Impuestos y Aduanas Nacionales)',
-    'schemeAgencyID': '195'
-}
+    'schemeAgencyName': 'CO, DIAN (Dirección de Impuestos y Aduanas'
+    ' Nacionales)',
+    'schemeAgencyID': '195'}
 
 
 # RESOLUCION 0001: pagina 516
-POLICY_ID = 'https://facturaelectronica.dian.gov.co/politicadefirma/v2/politicadefirmav2.pdf'
-POLICY_NAME = u'Política de firma para facturas electrónicas de la República de Colombia.'
+POLICY_ID = (
+    'https://facturaelectronica.dian.gov.co/politicadefirma/v2'
+    '/politicadefirmav2.pdf'
+)
+POLICY_NAME = (
+    u'Política de firma para facturas electrónicas de la República de '
+    u'Colombia.'
+)
 
 Bogota = tz.gettz('America/Bogota')
 # NAMESPACES = {
 #     'atd': 'urn:oasis:names:specification:ubl:schema:xsd:AttachedDocument-2',
 #     'nomina': 'dian:gov:co:facturaelectronica:NominaIndividual',
-#     'nominaajuste': 'dian:gov:co:facturaelectronica:NominaIndividualDeAjuste',    
+#     'nominaajuste': 'dian:gov:co:facturaelectronica:'
+#                     'NominaIndividualDeAjuste',
 #     'fe': 'http://www.dian.gov.co/contratos/facturaelectronica/v1',
 #     'xs': 'http://www.w3.org/2001/XMLSchema-instance',
-#     'cac': 'urn:oasis:names:specification:ubl:schema:xsd:CommonAggregateComponents-2',
-#     'cbc': 'urn:oasis:names:specification:ubl:schema:xsd:CommonBasicComponents-2',
-#     'cdt': 'urn:DocumentInformation:names:specification:ubl:colombia:schema:xsd:DocumentInformationAggregateComponents-1',
+#     'cac': 'urn:oasis:names:specification:ubl:schema:xsd:'
+#            'CommonAggregateComponents-2',
+#     'cbc': 'urn:oasis:names:specification:ubl:schema:xsd:'
+#            'CommonBasicComponents-2',
+#     'cdt': 'urn:DocumentInformation:names:specification:ubl:colombia:'
+#            'schema:xsd:DocumentInformationAggregateComponents-1',
 #     'clm54217': 'urn:un:unece:uncefact:codelist:specification:54217:2001',
-#     'clmIANAMIMEMediaType': 'urn:un:unece:uncefact:codelist:specification:IANAMIMEMediaType:2003',
-#     'ext': 'urn:oasis:names:specification:ubl:schema:xsd:CommonExtensionComponents-2',
-#     'qdt': 'urn:oasis:names:specification:ubl:schema:xsd:QualifiedDatatypes-2',
+#     'clmIANAMIMEMediaType': 'urn:un:unece:uncefact:codelist:specification:'
+#                             'IANAMIMEMediaType:2003',
+#     'ext': 'urn:oasis:names:specification:ubl:schema:xsd:'
+#            'CommonExtensionComponents-2',
+#     'qdt': 'urn:oasis:names:specification:ubl:schema:xsd:'
+#            'QualifiedDatatypes-2',
 #     'sts': 'dian:gov:co:facturaelectronica:Structures-2-1',
-#     'udt': 'urn:un:unece:uncefact:data:specification:UnqualifiedDataTypesSchemaModule:2',
+#     'udt': 'urn:un:unece:uncefact:data:specification:'
+#            'UnqualifiedDataTypesSchemaModule:2',
 #     'xsi': 'http://www.w3.org/2001/XMLSchema-instance',
 #     'xades': 'http://uri.etsi.org/01903/v1.3.2#',
 #     'xades141': 'http://uri.etsi.org/01903/v1.4.1#',
@@ -58,12 +86,19 @@ Bogota = tz.gettz('America/Bogota')
 
 NAMESPACES = {
     'fe': 'http://www.dian.gov.co/contratos/facturaelectronica/v1',
-    'cac': 'urn:oasis:names:specification:ubl:schema:xsd:CommonAggregateComponents-2',
-    'cbc': 'urn:oasis:names:specification:ubl:schema:xsd:CommonBasicComponents-2',
-    'ext': 'urn:oasis:names:specification:ubl:schema:xsd:CommonExtensionComponents-2',
+    'cac': (
+        'urn:oasis:names:specification:ubl:schema:xsd'
+        ':CommonAggregateComponents-2'),
+    'cbc': 'urn:oasis:names:specification:ubl:schema:xsd'
+    ':CommonBasicComponents-2',
+    'ext': (
+        'urn:oasis:names:specification:ubl:schema:xsd'
+        ':CommonExtensionComponents-2'),
     'qdt': 'urn:oasis:names:specification:ubl:schema:xsd:QualifiedDatatypes-2',
     'sts': 'dian:gov:co:facturaelectronica:Structures-2-1',
-    'udt': 'urn:un:unece:uncefact:data:specification:UnqualifiedDataTypesSchemaModule:2', 
+    'udt': (
+        'urn:un:unece:uncefact:data:specification'
+        ':UnqualifiedDataTypesSchemaModule:2'),
     'xsi': 'http://www.w3.org/2001/XMLSchema-instance',
     'ds': 'http://www.w3.org/2000/09/xmldsig#',
     'xades': 'http://uri.etsi.org/01903/v1.3.2#',
@@ -105,14 +140,17 @@ class FeXML(FachoXML):
         return super().from_string(document, namespaces=NAMESPACES)
 
     def tostring(self, **kw):
-        # MACHETE(bit4bit) la DIAN espera que la etiqueta raiz no este en un namespace
+        # MACHETE(bit4bit) la DIAN espera que la etiqueta raiz no este en un
+        # namespace
         root_namespace = self.root_namespace()
         root_localname = self.root_localname()
         xmlns_name = {v: k for k, v in NAMESPACES.items()}[root_namespace]
         if root_localname == 'Invoice':
-            urn_oasis = 'urn:oasis:names:specification:ubl:schema:xsd:Invoice-2'
+            urn_oasis = (
+                'urn:oasis:names:specification:ubl:schema:xsd:Invoice-2')
         if root_localname == 'CreditNote':
-            urn_oasis = 'urn:oasis:names:specification:ubl:schema:xsd:CreditNote-2'
+            urn_oasis = (
+                'urn:oasis:names:specification:ubl:schema:xsd:CreditNote-2')
         return super().tostring(**kw)\
             .replace(xmlns_name + ':', '')\
             .replace('xmlns:' + xmlns_name, 'xmlns')\
@@ -135,9 +173,12 @@ class DianXMLExtensionCUDFE(FachoXMLExtension):
 
     def _get_qrcode(self, cufe):
         url_for = {
-            AMBIENTE_PRUEBAS: 'https://catalogo-vpfe-hab.dian.gov.co/document/searchqr?documentkey=',
-            AMBIENTE_PRODUCCION: 'https://catalogo-vpfe.dian.gov.co/document/searchqr?documentkey='
-        }
+            AMBIENTE_PRUEBAS: (
+                'https://catalogo-vpfe-hab.dian.gov.co/document/'
+                'searchqr?documentkey='),
+            AMBIENTE_PRODUCCION: (
+                'https://catalogo-vpfe.dian.gov.co/document/'
+                'searchqr?documentkey=')}
         return url_for[self.tipo_ambiente] + cufe
 
     def build(self, fachoxml):
@@ -148,21 +189,29 @@ class DianXMLExtensionCUDFE(FachoXMLExtension):
 
         if self.schemeName() == "CUDS-SHA384":
             if fachoxml.tag_document() == 'Invoice':
-                fachoxml.set_element('./cbc:ProfileID',
-                        'DIAN 2.1: documento soporte en adquisiciones efectuadas a no obligados a facturar.')
+                fachoxml.set_element(
+                    './cbc:ProfileID',
+                    'DIAN 2.1: documento soporte en adquisiciones '
+                    'efectuadas a no obligados a facturar.')
             else:
-                fachoxml.set_element('./cbc:ProfileID',
-                        'DIAN 2.1: Nota de ajuste al documento soporte en adquisiciones efectuadas a sujetos no obligados a expedir factura o documento equivalente')
+                fachoxml.set_element(
+                    './cbc:ProfileID',
+                    'DIAN 2.1: Nota de ajuste al documento soporte en '
+                    'adquisiciones efectuadas a sujetos no obligados a '
+                    'expedir factura o documento equivalente')
         else:
-            fachoxml.set_element('./cbc:ProfileID', 'DIAN 2.1: Factura Electrónica de Venta')
+            fachoxml.set_element(
+                './cbc:ProfileID',
+                'DIAN 2.1: Factura Electrónica de Venta')
 
-        # #DIAN 1.8.-2021: FAD03
-        # fachoxml.set_element('./cbc:ProfileID', 'DIAN 2.1: Factura Electrónica de Venta')
+        # fachoxml.set_element('./cbc:ProfileID',
+        #                      'DIAN 2.1: Factura Electrónica de Venta')
         fachoxml.set_element(
             './cbc:ProfileExecutionID', self._tipo_ambiente_int())
         # DIAN 1.7.-2020: FAB36
         fachoxml.set_element(
-            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:QRCode',
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:QRCode',
             self._get_qrcode(cufe))
 
     def issue_time(self, datetime_):
@@ -178,7 +227,8 @@ class DianXMLExtensionCUDFE(FachoXMLExtension):
         build_vars['FecFac'] = self.issue_date(invoice.invoice_issue)
         build_vars['HoraFac'] = self.issue_time(invoice.invoice_issue)
         # PAG 601
-        build_vars['ValorBruto'] = invoice.invoice_legal_monetary_total.line_extension_amount
+        build_vars['ValorBruto'] = (
+            invoice.invoice_legal_monetary_total.line_extension_amount)
         build_vars['ValorTotalPagar'
                    ] = invoice.invoice_legal_monetary_total.payable_amount
         ValorImpuestoPara = defaultdict(lambda: form.Amount(0.0))
@@ -190,7 +240,8 @@ class DianXMLExtensionCUDFE(FachoXMLExtension):
                 if subtotal.scheme is not None:
                     # TODO cual es la naturaleza de tax_scheme_ident?
                     codigo_impuesto = subtotal.scheme.code
-                    ValorImpuestoPara.setdefault(codigo_impuesto, form.Amount(0.0))
+                    ValorImpuestoPara.setdefault(
+                        codigo_impuesto, form.Amount(0.0))
                     ValorImpuestoPara[codigo_impuesto] += subtotal.tax_amount
 
         build_vars['ValorImpuestoPara'] = ValorImpuestoPara
@@ -230,21 +281,35 @@ class DianXMLExtensionCUFE(DianXMLExtensionCUDFE):
         CodImpuesto2 = build_vars['CodImpuesto2']
         CodImpuesto3 = build_vars['CodImpuesto3']
         return [
-            '%s' % build_vars['NumFac'],
-            '%s' % build_vars['FecFac'],
-            '%s' % build_vars['HoraFac'],
-            form.Amount(build_vars['ValorBruto']).truncate_as_string(2),
+            '%s' %
+            build_vars['NumFac'],
+            '%s' %
+            build_vars['FecFac'],
+            '%s' %
+            build_vars['HoraFac'],
+            form.Amount(
+                build_vars['ValorBruto']).truncate_as_string(2),
             CodImpuesto1,
-            build_vars['ValorImpuestoPara'].get(CodImpuesto1, form.Amount(0.0)).truncate_as_string(2),
+            build_vars['ValorImpuestoPara'].get(
+                CodImpuesto1,
+                form.Amount(0.0)).truncate_as_string(2),
             CodImpuesto2,
-            build_vars['ValorImpuestoPara'].get(CodImpuesto2, form.Amount(0.0)).truncate_as_string(2),
+            build_vars['ValorImpuestoPara'].get(
+                CodImpuesto2,
+                form.Amount(0.0)).truncate_as_string(2),
             CodImpuesto3,
-            build_vars['ValorImpuestoPara'].get(CodImpuesto3, form.Amount(0.0)).truncate_as_string(2),
+            build_vars['ValorImpuestoPara'].get(
+                    CodImpuesto3,
+                    form.Amount(0.0)).truncate_as_string(2),
             build_vars['ValorTotalPagar'].truncate_as_string(2),
-            '%s' % build_vars['NitOFE'],
-            '%s' % build_vars['NumAdq'],
-            '%s' % build_vars['ClTec'],
-            '%d' % build_vars['TipoAmb'],
+            '%s' %
+            build_vars['NitOFE'],
+            '%s' %
+            build_vars['NumAdq'],
+            '%s' %
+            build_vars['ClTec'],
+            '%d' %
+            build_vars['TipoAmb'],
         ]
 
 
@@ -268,21 +333,39 @@ class DianXMLExtensionCUDE(DianXMLExtensionCUDFE):
         CodImpuesto2 = build_vars['CodImpuesto2']
         CodImpuesto3 = build_vars['CodImpuesto3']
         return [
-            '%s' % build_vars['NumFac'],
-            '%s' % build_vars['FecFac'],
-            '%s' % build_vars['HoraFac'],
-            form.Amount(build_vars['ValorBruto']).truncate_as_string(2),
+            '%s' %
+            build_vars['NumFac'],
+            '%s' %
+            build_vars['FecFac'],
+            '%s' %
+            build_vars['HoraFac'],
+            form.Amount(
+                build_vars['ValorBruto']).truncate_as_string(2),
             CodImpuesto1,
-            form.Amount(build_vars['ValorImpuestoPara'].get(CodImpuesto1, 0.0)).truncate_as_string(2),
+            form.Amount(
+                build_vars['ValorImpuestoPara'].get(
+                    CodImpuesto1,
+                    0.0)).truncate_as_string(2),
             CodImpuesto2,
-            form.Amount(build_vars['ValorImpuestoPara'].get(CodImpuesto2, 0.0)).truncate_as_string(2),
+            form.Amount(
+                    build_vars['ValorImpuestoPara'].get(
+                        CodImpuesto2,
+                        0.0)).truncate_as_string(2),
             CodImpuesto3,
-            form.Amount(build_vars['ValorImpuestoPara'].get(CodImpuesto3, 0.0)).truncate_as_string(2),
-            form.Amount(build_vars['ValorTotalPagar']).truncate_as_string(2),
-            '%s' % build_vars['NitOFE'],
-            '%s' % build_vars['NumAdq'],
-            '%s' % build_vars['Software-PIN'],
-            '%d' % build_vars['TipoAmb'],
+            form.Amount(
+                            build_vars['ValorImpuestoPara'].get(
+                                CodImpuesto3,
+                                0.0)).truncate_as_string(2),
+            form.Amount(
+                build_vars['ValorTotalPagar']).truncate_as_string(2),
+            '%s' %
+            build_vars['NitOFE'],
+            '%s' %
+            build_vars['NumAdq'],
+            '%s' %
+            build_vars['Software-PIN'],
+            '%d' %
+            build_vars['TipoAmb'],
         ]
 
 
@@ -303,20 +386,30 @@ class DianXMLExtensionCUDS(DianXMLExtensionCUDFE):
     def formatVars(self):
         build_vars = self.buildVars()
         CodImpuesto1 = build_vars['CodImpuesto1']
-        CodImpuesto2 = build_vars['CodImpuesto2']
-        CodImpuesto3 = build_vars['CodImpuesto3']
         return [
-            '%s' % build_vars['NumFac'],
-            '%s' % build_vars['FecFac'],
-            '%s' % build_vars['HoraFac'],
-            form.Amount(build_vars['ValorBruto']).truncate_as_string(2),
+            '%s' %
+            build_vars['NumFac'],
+            '%s' %
+            build_vars['FecFac'],
+            '%s' %
+            build_vars['HoraFac'],
+            form.Amount(
+                build_vars['ValorBruto']).truncate_as_string(2),
             CodImpuesto1,
-            form.Amount(build_vars['ValorImpuestoPara'].get(CodImpuesto1, 0.0)).truncate_as_string(2),
-            form.Amount(build_vars['ValorTotalPagar']).truncate_as_string(2),
-            '%s' % build_vars['NitOFE'],
-            '%s' % build_vars['NumAdq'],            
-            '%s' % build_vars['Software-PIN'],
-            '%d' % build_vars['TipoAmb'],
+            form.Amount(
+                build_vars['ValorImpuestoPara'].get(
+                    CodImpuesto1,
+                    0.0)).truncate_as_string(2),
+            form.Amount(
+                    build_vars['ValorTotalPagar']).truncate_as_string(2),
+            '%s' %
+            build_vars['NitOFE'],
+            '%s' %
+            build_vars['NumAdq'],
+            '%s' %
+            build_vars['Software-PIN'],
+            '%d' %
+            build_vars['TipoAmb'],
         ]
 
 
@@ -330,14 +423,19 @@ class DianXMLExtensionSoftwareProvider(FachoXMLExtension):
 
     def build(self, fexml):
         software_provider = fexml.fragment(
-            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:SoftwareProvider')
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:SoftwareProvider')
         provider_id_attrs = SCHEME_AGENCY_ATTRS.copy()
         provider_id_attrs.update({'schemeID': self.dv})
         # DIAN 1.7.-2020: FAB23
         provider_id_attrs.update({'schemeName': '31'})
-        software_provider.set_element('/sts:SoftwareProvider/sts:ProviderID', self.nit,
+        software_provider.set_element(
+            '/sts:SoftwareProvider/sts:ProviderID',
+            self.nit,
             **provider_id_attrs)
-        software_provider.set_element('/sts:SoftwareProvider/sts:SoftwareID', self.id_software,
+        software_provider.set_element(
+            '/sts:SoftwareProvider/sts:SoftwareID',
+            self.id_software,
             **SCHEME_AGENCY_ATTRS)
 
 
@@ -350,7 +448,10 @@ class DianXMLExtensionSoftwareSecurityCode(FachoXMLExtension):
         self.invoice_ident = invoice_ident
 
     def build(self, fexml):
-        dian_path = './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:SoftwareSecurityCode'
+        dian_path = (
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:SoftwareSecurityCode'
+        )
         code = str(self.id_software) + str(self.pin) + str(self.invoice_ident)
         m = hashlib.sha384()
         m.update(code.encode('utf-8'))
@@ -404,8 +505,11 @@ class DianXMLExtensionSigner:
         xml.append(signature)
 
         ref = xmlsig.template.add_reference(
-            signature, xmlsig.constants.TransformSha256, uri="", name="xmldsig-%s-ref0" % (id_uuid)
-        )
+            signature,
+            xmlsig.constants.TransformSha256,
+            uri="",
+            name="xmldsig-%s-ref0" %
+            (id_uuid))
         xmlsig.template.add_transform(ref, xmlsig.constants.TransformEnveloped)
 
         id_keyinfo = "xmldsig-%s-KeyInfo" % (id_uuid)
@@ -424,9 +528,8 @@ class DianXMLExtensionSigner:
 
         id_props = "xmldsig-%s-signedprops" % (id_uuid)
         props_ref = xmlsig.template.add_reference(
-            signature, xmlsig.constants.TransformSha256, uri="#%s" % (id_props),
-            uri_type="http://uri.etsi.org/01903#SignedProperties"
-        )
+            signature, xmlsig.constants.TransformSha256, uri="#%s" %
+            (id_props), uri_type="http://uri.etsi.org/01903#SignedProperties")
         xmlsig.template.add_transform(
             props_ref, xmlsig.constants.TransformInclC14N)
 
@@ -444,9 +547,6 @@ class DianXMLExtensionSigner:
             self._pkcs12_data,
             self._passphrase))
 
-        # ctx.load_pkcs12(OpenSSL.crypto.load_pkcs12(
-        #     self._pkcs12_data,
-        #     self._passphrase))
         if self._localpolicy:
             with mock_xades_policy():
                 ctx.sign(signature)
@@ -470,7 +570,9 @@ class DianXMLExtensionAuthorizationProvider(FachoXMLExtension):
     def build(self, fexml):
         attrs = {'schemeID': '4', 'schemeName': '31'}
         attrs.update(SCHEME_AGENCY_ATTRS)
-        authorization_provider = fexml.fragment('./ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:AuthorizationProvider')
+        authorization_provider = fexml.fragment(
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:AuthorizationProvider')
         authorization_provider.set_element('./sts:AuthorizationProviderID',
                                            '800197268',
                                            **attrs)
@@ -479,12 +581,17 @@ class DianXMLExtensionAuthorizationProvider(FachoXMLExtension):
 class DianXMLExtensionInvoiceSource(FachoXMLExtension):
     # CAB13
     def build(self, fexml):
-        dian_path = '/fe:CreditNote/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:InvoiceSource/cbc:IdentificationCode'
+        dian_path = (
+            '/fe:CreditNote/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionCo'
+            'ntent/sts:DianExtensions/sts:InvoiceSource/cbc:IdentificationCode'
+        )
         fexml.set_element(
             dian_path, 'CO',
             listAgencyID="6",
             listAgencyName="United Nations Economic Commission for Europe",
-            listSchemeURI="urn:oasis:names:specification:ubl:codelist:gc:CountryIdentificationCode-2.1")
+            listSchemeURI=(
+                "urn:oasis:names:specification:ubl:codelist:gc:"
+                "CountryIdentificationCode-2.1"))
 
 
 class DianXMLExtensionInvoiceAuthorization(FachoXMLExtension):
@@ -501,28 +608,38 @@ class DianXMLExtensionInvoiceAuthorization(FachoXMLExtension):
         self.to = to
 
     def build(self, fexml):
-        invoice_control = fexml.fragment('./ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:InvoiceControl')
-        invoice_control.set_element('/sts:InvoiceControl/sts:InvoiceAuthorization', self.authorization)
-        invoice_control.set_element('/sts:InvoiceControl/sts:AuthorizationPeriod/cbc:StartDate',
+        invoice_control = fexml.fragment(
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:InvoiceControl')
+        invoice_control.set_element(
+            '/sts:InvoiceControl/sts:InvoiceAuthorization',
+            self.authorization)
+        invoice_control.set_element(
+            '/sts:InvoiceControl/sts:AuthorizationPeriod/cbc:StartDate',
             self.period_startdate.strftime('%Y-%m-%d'))
-        invoice_control.set_element('/sts:InvoiceControl/sts:AuthorizationPeriod/cbc:EndDate',
+        invoice_control.set_element(
+            '/sts:InvoiceControl/sts:AuthorizationPeriod/cbc:EndDate',
             self.period_enddate.strftime('%Y-%m-%d'))
-        invoice_control.set_element('/sts:InvoiceControl/sts:AuthorizedInvoices/sts:Prefix',
+        invoice_control.set_element(
+            '/sts:InvoiceControl/sts:AuthorizedInvoices/sts:Prefix',
             self.prefix)
-        invoice_control.set_element('/sts:InvoiceControl/sts:AuthorizedInvoices/sts:From',
-                                    self.from_)
-        invoice_control.set_element('/sts:InvoiceControl/sts:AuthorizedInvoices/sts:To',
-                                    self.to)
+        invoice_control.set_element(
+            '/sts:InvoiceControl/sts:AuthorizedInvoices/sts:From', self.from_)
+        invoice_control.set_element(
+            '/sts:InvoiceControl/sts:AuthorizedInvoices/sts:To', self.to)
 
         fexml.set_element(
-            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:InvoiceSource/cbc:IdentificationCode',
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:InvoiceSource/cbc:IdentificationCode',
             'CO',
             # DIAN 1.7.-2020: FAB15
             listAgencyID="6",
             # DIAN 1.7.-2020: FAB16
             listAgencyName="United Nations Economic Commission for Europe",
             # DIAN 1.7.-2020: FAB17
-            listSchemeURI="urn:oasis:names:specification:ubl:codelist:gc:CountryIdentificationCode-2.1")
+            listSchemeURI=(
+                "urn:oasis:names:specification:ubl:codelist:gc:"
+                "CountryIdentificationCode-2.1"))
 
 
 class DianZIP:
@@ -565,7 +682,11 @@ class DianZIP:
 
 class DianXMLExtensionSignerVerifier:
 
-    def __init__(self, pkcs12_path_or_bytes, passphrase=None, localpolicy=True):
+    def __init__(
+            self,
+            pkcs12_path_or_bytes,
+            passphrase=None,
+            localpolicy=True):
         self._pkcs12_path_or_bytes = pkcs12_path_or_bytes
         self._passphrase = None
         self._localpolicy = localpolicy
@@ -590,8 +711,8 @@ class DianXMLExtensionSignerVerifier:
         if isinstance(self._pkcs12_path_or_bytes, str):
             pkcs12_data = open(self._pkcs12_path_or_bytes, 'rb').read()
         ctx = xades.XAdESContext()
-        ctx.load_pkcs12(OpenSSL.crypto.load_pkcs12(pkcs12_data,
-                                                   self._passphrase))
+        ctx.load_pkcs12(pkcs12.load_key_and_certificates(
+            pkcs12_data, self._passphrase))
         try:
             if self._localpolicy:
                 with mock_xades_policy():
@@ -599,5 +720,5 @@ class DianXMLExtensionSignerVerifier:
             else:
                 ctx.verify(signature)
             return True
-        except:
+        except BaseException:
             return False

facho/fe/form/__init__.py

@@ -5,13 +5,14 @@
 # from functools import reduce
 # import copy
 
+from __future__ import annotations
+
 import dataclasses
 from dataclasses import dataclass, field
-from datetime import datetime, date
+from datetime import datetime
 # from collections import defaultdict
 import decimal
 from decimal import Decimal
-import typing
 from ..data.dian import codelist
 
 DECIMAL_PRECISION = 6
@@ -60,7 +61,7 @@ class AmountCollection(Collection):
 
 class Amount:
     def __init__(
-            self, amount: typing.Union[int, float, str, "Amount"],
+            self, amount: int | float | str | "Amount",
             currency: Currency = Currency('COP')):
 
         # DIAN 1.7.-2020: 1.2.3.1
@@ -279,22 +280,11 @@ class Responsability:
                 raise ValueError("code %s not found" % (code))
 
 
-@dataclass
-class TaxScheme:
-    code: str
-    name: str = ''
-
-    def __post_init__(self):
-        if self.code not in codelist.TipoImpuesto:
-            raise ValueError("code not found")
-        self.name = codelist.TipoImpuesto[self.code]['name']
-
-
 @dataclass
 class Party:
     name: str
     ident: str
-    responsability_code: typing.List[Responsability]
+    responsability_code: list[Responsability]
     responsability_regime_code: str
     organization_code: str
     tax_scheme: TaxScheme = field(default_factory=lambda: TaxScheme('01'))
@@ -327,7 +317,7 @@ class TaxScheme:
 @dataclass
 class TaxSubTotal:
     percent: float
-    scheme: typing.Optional[TaxScheme] = None
+    scheme: TaxScheme | None = None
     tax_amount: Amount = field(default_factory=lambda: Amount(0.0))
 
     def calculate(self, invline):
@@ -359,7 +349,7 @@ class TaxTotalOmit(TaxTotal):
 @dataclass
 class WithholdingTaxSubTotal:
     percent: float
-    scheme: typing.Optional[TaxScheme] = None
+    scheme: TaxScheme | None = None
     tax_amount: Amount = field(default_factory=lambda: Amount(0.0))
 
     def calculate(self, invline):
@@ -491,7 +481,7 @@ class AllowanceCharge:
     reason: AllowanceChargeReason = None
 
     # Valor Base para calcular el descuento o el cargo
-    base_amount: typing.Optional[Amount] = field(
+    base_amount: Amount | None = field(
         default_factory=lambda: Amount(0.0))
 
     # Porcentaje: Porcentaje que aplicar.
@@ -536,9 +526,9 @@ class InvoiceLine:
     # ya que al reportar los totales es sobre
     # la factura y el percent es unico por type_code
     # de subtotal
-    tax: typing.Optional[TaxTotal]
-    withholding: typing.Optional[WithholdingTaxTotal]
-    allowance_charge: typing.List[AllowanceCharge] = dataclasses.field(
+    tax: TaxTotal | None
+    withholding: WithholdingTaxTotal | None
+    allowance_charge: list[AllowanceCharge] = dataclasses.field(
         default_factory=list)
 
     def add_allowance_charge(self, charge):

facho/fe/form/query.py

@@ -6,7 +6,10 @@ from .. import form
 from ..fe import fe_from_string
 from datetime import datetime
 
-def billing_reference(xmldocument: str, klass: form.BillingReference) -> form.BillingReference:
+
+def billing_reference(
+        xmldocument: str,
+        klass: form.BillingReference) -> form.BillingReference:
     """
     construye BillingReference desde XMLDOCUMENT
     usando KLASS como clase.

facho/fe/form_xml/__init__.py

@@ -1,7 +1,18 @@
-from .invoice import *
-from .credit_note import *
-from .debit_note import *
-from .utils import *
-from .attached_document import *
-from .support_document import *
-from .support_document_credit_note import *
+from .invoice import DIANInvoiceXML
+from .credit_note import DIANCreditNoteXML
+from .debit_note import DIANDebitNoteXML
+from .utils import DIANWrite, DIANWriteSigned
+from .attached_document import AttachedDocument
+from .support_document import DIANSupportDocumentXML
+from .support_document_credit_note import DIANSupportDocumentCreditNoteXML
+
+__all__ = [
+    'DIANInvoiceXML',
+    'DIANCreditNoteXML',
+    'DIANDebitNoteXML',
+    'DIANWrite',
+    'DIANWriteSigned',
+    'AttachedDocument',
+    'DIANSupportDocumentXML',
+    'DIANSupportDocumentCreditNoteXML',
+]

facho/fe/form_xml/invoice.py

@@ -1,9 +1,18 @@
-from .. import fe
-from ..form import *
 from collections import defaultdict
 
+from .. import fe
+from ..form import (
+    Amount,
+    CreditNoteDocumentReference,
+    DebitNoteDocumentReference,
+    InvoiceDocumentReference,
+    TaxTotalOmit,
+    WithholdingTaxTotalOmit,
+)
+
 __all__ = ['DIANInvoiceXML']
 
+
 class DIANInvoiceXML(fe.FeXML):
     """
     DianInvoiceXML mapea objeto form.Invoice a XML segun
@@ -11,16 +20,31 @@ class DIANInvoiceXML(fe.FeXML):
     """
 
     def __init__(self, invoice, tag_document='Invoice'):
-        super().__init__(tag_document, 'http://www.dian.gov.co/contratos/facturaelectronica/v1')
-        self.placeholder_for('./ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:InvoiceControl')
-        self.placeholder_for('./ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:InvoiceSource')
-        self.placeholder_for('./ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:SoftwareProvider')
-        self.placeholder_for('./ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:SoftwareSecurityCode')
-        self.placeholder_for('./ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:AuthorizationProvider/sts:AuthorizationProviderID')
+        super().__init__(
+            tag_document,
+            'http://www.dian.gov.co/contratos/facturaelectronica/v1')
+        self.placeholder_for(
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:InvoiceControl')
+        self.placeholder_for(
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:InvoiceSource')
+        self.placeholder_for(
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:SoftwareProvider')
+        self.placeholder_for(
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:SoftwareSecurityCode')
+        self.placeholder_for(
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/'
+            'sts:DianExtensions/sts:AuthorizationProvider/'
+            'sts:AuthorizationProviderID')
 
         # ZE02 se requiere existencia para firmar
-        ublextension = self.fragment('./ext:UBLExtensions/ext:UBLExtension', append=True)
-        extcontent = ublextension.find_or_create_element('/ext:UBLExtension/ext:ExtensionContent')
+        ublextension = self.fragment(
+            './ext:UBLExtensions/ext:UBLExtension', append=True)
+        ublextension.find_or_create_element(
+            '/ext:UBLExtension/ext:ExtensionContent')
         self.attach_invoice(invoice)
 
     def set_supplier(fexml, invoice):
@@ -28,72 +52,99 @@ class DIANInvoiceXML(fe.FeXML):
 
         # DIAN 1.7.-2020: CAJ02
         # DIAN 1.7.-2020: FAJ02
-        fexml.set_element('./cac:AccountingSupplierParty/cbc:AdditionalAccountID',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cbc:AdditionalAccountID',
             invoice.invoice_supplier.organization_code)
 
         # DIAN 1.7.-2020: CAJ06
         # DIAN 1.7.-2020: FAJ06
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyName/cbc:Name',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyName/cbc:Name',
             invoice.invoice_supplier.name)
 
         # DIAN 1.7.-2020: CAJ07, CAJ08
         # DIAN 1.7.-2020: FAJ07
-        fexml.placeholder_for('./cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address')
+        fexml.placeholder_for(
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address')
 
         # DIAN 1.7.-2020: FAJ08
         # DIAN 1.7.-2020: CAJ09
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:ID',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:ID',
             invoice.invoice_supplier.address.city.code)
         # DIAN 1.7.-2020: FAJ09
         # DIAN 1.7.-2020: CAJ10
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:CityName',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:CityName',
             invoice.invoice_supplier.address.city.name)
         # DIAN 1.7.-2020: FAJ11
         # DIAN 1.7.-2020: CAJ11
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:CountrySubentity',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:CountrySubentity',
             invoice.invoice_supplier.address.countrysubentity.name)
 
         # DIAN 1.7.-2020: FAJ12
         # DIAN 1.7.-2020: CAJ12
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:CountrySubentityCode',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:CountrySubentityCode',
             invoice.invoice_supplier.address.countrysubentity.code)
         # DIAN 1.7.-2020: FAJ14
         # DIAN 1.7.-2020: CAJ13, CAJ14
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cac:AddressLine/cbc:Line',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cac:AddressLine/cbc:Line',
             invoice.invoice_supplier.address.street)
 
         # DIAN 1.7.-2020: FAJ16
         # DIAN 1.7.-2020: CAJ16, CAJ16
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cac:Country/cbc:IdentificationCode',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cac:Country/cbc:IdentificationCode',
             invoice.invoice_supplier.address.country.code)
 
         # DIAN 1.7.-2020: FAJ17
         # DIAN 1.7.-2020: CAJ17
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cac:Country/cbc:Name',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cac:Country/cbc:Name',
             invoice.invoice_supplier.address.country.name,
             # DIAN 1.7.-2020: FAJ18
             languageID='es')
 
         supplier_company_id_attrs = fe.SCHEME_AGENCY_ATTRS.copy()
-        supplier_company_id_attrs.update({'schemeID': invoice.invoice_supplier.ident.dv,
+        supplier_company_id_attrs.update(
+            {
+                'schemeID': invoice.invoice_supplier.ident.dv,
                 'schemeName': invoice.invoice_supplier.ident.type_fiscal})
 
         # DIAN 1.7.-2020: FAJ19
         # DIAN 1.7.-2020: CAJ19
-        fexml.placeholder_for('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme')
+        fexml.placeholder_for(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme')
 
         # DIAN 1.7.-2020: FAJ20
         # DIAN 1.7.-2020: CAJ20
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cbc:RegistrationName',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cbc'
+            ':RegistrationName',
             invoice.invoice_supplier.legal_name)
 
         # DIAN 1.7.-2020: FAJ21
         # DIAN 1.7.-2020: CAJ21
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cbc:CompanyID',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cbc'
+            ':CompanyID',
             invoice.invoice_supplier.ident,
             # DIAN 1.7.-2020: FAJ22,FAJ23,FAJ24,FAJ25
             **supplier_company_id_attrs)
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cbc:TaxLevelCode',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cbc'
+            ':TaxLevelCode',
             # DIAN 1.7.-2020: FAJ26
             # DIAN 1.7.-2020: CAJ26
             invoice.invoice_supplier.responsability_code,
@@ -102,139 +153,197 @@ class DIANInvoiceXML(fe.FeXML):
             listName=invoice.invoice_supplier.responsability_regime_code)
         # DIAN 1.7.-2020: FAJ28
         # DIAN 1.7.-2020: CAJ28
-        fexml.placeholder_for('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress')
+        fexml.placeholder_for(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress')
 
         # DIAN 1.7.-2020: FAJ29
         # DIAN 1.7.-2020: CAJ29
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cbc:ID',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cbc:ID',
             invoice.invoice_supplier.address.city.code)
 
         # DIAN 1.7.-2020: FAJ30
         # DIAN 1.7.-2020: CAJ30
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cbc:CityName', invoice.invoice_supplier.address.city.name)
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cbc:CityName',
+            invoice.invoice_supplier.address.city.name)
 
         # DIAN 1.7.-2020: FAJ31
         # DIAN 1.7.-2020: CAJ31
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cbc:CountrySubentity',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cbc:CountrySubentity',
             invoice.invoice_supplier.address.countrysubentity.name)
 
         # DIAN 1.7.-2020: FAJ32
         # DIAN 1.7.-2020: CAJ32
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cbc:CountrySubentityCode',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cbc:CountrySubentityCode',
             invoice.invoice_supplier.address.countrysubentity.code)
 
         # DIAN 1.7.-2020: FAJ33,FAJ34
         # DIAN 1.7.-2020: CAJ33,CAJ34
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cac:AddressLine/cbc:Line',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cac:AddressLine/cbc:Line',
             invoice.invoice_supplier.address.street)
 
         # DIAN 1.7.-2020: FAJ35,FAJ36
         # DIAN 1.7.-2020: CAJ35,CAJ36
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cac:Country/cbc:IdentificationCode',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cac:Country/cbc:IdentificationCode',
             invoice.invoice_supplier.address.country.code)
 
         # DIAN 1.7.-2020: FAJ37,FAJ38
         # DIAN 1.7.-2020: CAJ37,CAJ38
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cac:Country/cbc:Name',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cac:Country/cbc:Name',
             invoice.invoice_supplier.address.country.name,
             languageID='es')
 
         # DIAN 1.7.-2020: FAJ39
         # DIAN 1.7.-2020: CAJ39
-        fexml.placeholder_for('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:TaxScheme')
+        fexml.placeholder_for(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':TaxScheme')
 
         # DIAN 1.7.-2020: CAJ40
         # DIAN 1.7.-2020: FAJ40
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:TaxScheme/cbc:ID',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':TaxScheme/cbc:ID',
             invoice.invoice_customer.tax_scheme.code)
 
         # DIAN 1.7.-2020: CAJ41
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:TaxScheme/cbc:Name',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':TaxScheme/cbc:Name',
             invoice.invoice_customer.tax_scheme.name)
 
         # DIAN 1.7.-2020: FAJ42
         # DIAN 1.7.-2020: CAJ42
-        fexml.placeholder_for('./cac:AccountingSupplierParty/cac:Party/cac:PartyLegalEntity')
+        fexml.placeholder_for(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyLegalEntity')
 
         # DIAN 1.7.-2020: FAJ43
         # DIAN 1.7.-2020: CAJ43
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyLegalEntity/cbc:RegistrationName',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyLegalEntity/cbc'
+            ':RegistrationName',
             invoice.invoice_supplier.legal_name)
 
         # DIAN 1.7.-2020: FAJ44,FAJ45,FAJ46,FAJ47,FAJ48
         # DIAN 1.7.-2020: CAJ44,CAJ45,CAJ46,CAJ47,CAJ48
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyLegalEntity/cbc:CompanyID',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyLegalEntity/cbc'
+            ':CompanyID',
             invoice.invoice_supplier.ident,
             **supplier_company_id_attrs)
 
         # DIAN 1.7.-2020: FAJ49
         # DIAN 1.7.-2020: CAJ49
-        fexml.placeholder_for('./cac:AccountingSupplierParty/cac:Party/cac:PartyLegalEntity/cac:CorporateRegistrationScheme')
+        fexml.placeholder_for(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyLegalEntity/cac'
+            ':CorporateRegistrationScheme')
 
         # DIAN 1.7.-2020: FAJ50
         # DIAN 1.7.-2020: CAJ50
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PartyLegalEntity/cac:CorporateRegistrationScheme/cbc:ID',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyLegalEntity/cac'
+            ':CorporateRegistrationScheme/cbc:ID',
             invoice.invoice_ident_prefix)
 
         # DIAN 1.7.-2020: CAJ67
-        fexml.placeholder_for('./cac:AccountingSupplierParty/cac:Party/cac:Contact')
+        fexml.placeholder_for(
+            './cac:AccountingSupplierParty/cac:Party/cac:Contact')
 
         # DIAN 1.7.-2020: FAJ71
         # DIAN 1.7.-2020: CAJ71
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:Contact/cbc:ElectronicMail',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:Contact/cbc'
+            ':ElectronicMail',
             invoice.invoice_supplier.email)
 
-
     def set_customer(fexml, invoice):
         fexml.placeholder_for('./cac:AccountingCustomerParty')
-        fexml.set_element('./cac:AccountingCustomerParty/cbc:AdditionalAccountID',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cbc:AdditionalAccountID',
             invoice.invoice_customer.organization_code)
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyIdentification/cbc:ID',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyIdentification'
+            '/cbc:ID',
             invoice.invoice_customer.ident)
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyName/cbc:Name',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyName/cbc:Name',
             invoice.invoice_customer.name)
 
-        fexml.placeholder_for('./cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation')
+        fexml.placeholder_for(
+            './cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation')
         customer_company_id_attrs = fe.SCHEME_AGENCY_ATTRS.copy()
         # DIAN 1.7.-2020: FAK25
         # DIAN 1.7.-2020: CAK25
-        customer_company_id_attrs.update({'schemeID': invoice.invoice_customer.ident.dv,
+        customer_company_id_attrs.update(
+            {
+                'schemeID': invoice.invoice_customer.ident.dv,
                 'schemeName': invoice.invoice_customer.ident.type_fiscal})
 
         # DIAN 1.7.-2020: FAK07
         # DIAN 1.7.-2020: CAK07
-        fexml.placeholder_for('./cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac:Address')
+        fexml.placeholder_for(
+            './cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address')
 
         # DIAN 1.7.-2020: FAK08
         # DIAN 1.7.-2020: CAK08
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:ID',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:ID',
             invoice.invoice_customer.address.city.code)
         # DIAN 1.7.-2020: FAK09
         # DIAN 1.7.-2020: CAK09
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:CityName', invoice.invoice_customer.address.city.name)
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:CityName',
+            invoice.invoice_customer.address.city.name)
 
         # DIAN 1.7.-2020: FAK11
         # DIAN 1.7.-2020: CAK11
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:CountrySubentity',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:CountrySubentity',
             invoice.invoice_customer.address.countrysubentity.name)
 
         # DIAN 1.7.-2020: FAK12
         # DIAN 1.7.-2020: CAK12
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:CountrySubentityCode',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:CountrySubentityCode',
             invoice.invoice_customer.address.countrysubentity.code)
 
         # DIAN 1.7.-2020: CAK13, CAK14
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac:Address/cac:AddressLine/cbc:Line',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cac:AddressLine/cbc:Line',
             invoice.invoice_customer.address.street)
 
         # DIAN 1.7.-2020: CAK16
         # DIAN 1.7.-2020: FAK16
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac:Address/cac:Country/cbc:IdentificationCode',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cac:Country/cbc:IdentificationCode',
             invoice.invoice_customer.address.country.code)
 
         # DIAN 1.7.-2020: FAK17
         # DIAN 1.7.-2020: CAK17
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac:Address/cac:Country/cbc:Name',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cac:Country/cbc:Name',
             invoice.invoice_customer.address.country.name,
             # DIAN 1.7.-2020: FAK18
             # DIAN 1.7.-2020: CAK18
@@ -242,22 +351,29 @@ class DIANInvoiceXML(fe.FeXML):
 
         # DIAN 1.7.-2020: FAK17,FAK19
         # DIAN 1.7.-2020: CAK19
-        fexml.placeholder_for('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme')
+        fexml.placeholder_for(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme')
 
         # DIAN 1.7.-2020: FAK17,FAK20
         # DIAN 1.7.-2020: CAK20
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cbc:RegistrationName',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cbc'
+            ':RegistrationName',
             invoice.invoice_customer.legal_name)
 
         # DIAN 1.7.-2020: CAK21
         # DIAN 1.7.-2020: FAK21
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cbc:CompanyID',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cbc'
+            ':CompanyID',
             invoice.invoice_customer.ident,
             # DIAN 1.7.-2020: CAK22, CAK23, CAK24, CAK25
             **customer_company_id_attrs)
 
         # DIAN 1.7.-2020: CAK26
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cbc:TaxLevelCode',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cbc'
+            ':TaxLevelCode',
             # DIAN 1.7.-2020: FAK26
             invoice.invoice_customer.responsability_code,
             # DIAN 1.7.-2020: FAK27
@@ -266,98 +382,140 @@ class DIANInvoiceXML(fe.FeXML):
 
         # DIAN 1.7.-2020: FAK28
         # DIAN 1.7.-2020: CAK28
-        fexml.placeholder_for('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress')
+        fexml.placeholder_for(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress')
 
         # DIAN 1.7.-2020: FAK29
         # DIAN 1.7.-2020: CAK29
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cbc:ID',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cbc:ID',
             invoice.invoice_customer.address.city.code)
 
         # DIAN 1.7.-2020: FAK30
         # DIAN 1.7.-2020: CAK30
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cbc:CityName',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cbc:CityName',
             invoice.invoice_customer.address.city.name)
 
         # DIAN 1.7.-2020: FAK31
         # DIAN 1.7.-2020: CAK31
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cbc:CountrySubentity',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cbc:CountrySubentity',
             invoice.invoice_customer.address.countrysubentity.name)
 
         # DIAN 1.7.-2020: FAK32
         # DIAN 1.7.-2020: CAK32
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cbc:CountrySubentityCode',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cbc:CountrySubentityCode',
             invoice.invoice_customer.address.countrysubentity.code)
 
         # DIAN 1.7.-2020: FAK33
         # DIAN 1.7.-2020: CAK33
-        fexml.placeholder_for('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cac:AddressLine')
+        fexml.placeholder_for(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cac:AddressLine')
 
         # DIAN 1.7.-2020: FAK34
         # DIAN 1.7.-2020: CAK34
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cac:AddressLine/cbc:Line',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cac:AddressLine/cbc:Line',
             invoice.invoice_customer.address.street)
 
         # DIAN 1.7.-2020: CAK35
         # DIAN 1.7.-2020: FAK35
-        fexml.placeholder_for('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cac:Country')
+        fexml.placeholder_for(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cac:Country')
 
         # DIAN 1.7.-2020: CAK36
         # DIAN 1.7.-2020: FAK36
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cac:Country/cbc:IdentificationCode',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cac:Country/cbc:IdentificationCode',
             invoice.invoice_customer.address.country.code)
 
         # DIAN 1.7.-2020: CAK37
         # DIAN 1.7.-2020: FAK37
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cac:Country/cbc:Name', invoice.invoice_customer.address.country.name)
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cac:Country/cbc:Name',
+            invoice.invoice_customer.address.country.name)
 
         # DIAN 1.7.-2020: FAK38
         # DIAN 1.7.-2020: CAK38
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:RegistrationAddress/cac:Country/cbc:IdentificationCode',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':RegistrationAddress/cac:Country/cbc:IdentificationCode',
             invoice.invoice_customer.address.country.code,
             languageID='es')
 
         # DIAN 1.7.-2020: CAK39
         # DIAN 1.7.-2020: FAK39
-        fexml.placeholder_for('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:TaxScheme')
+        fexml.placeholder_for(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':TaxScheme')
 
         # DIAN 1.7.-2020: CAK40 Machete Construir Validación
         # DIAN 1.7.-2020: FAK40
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:TaxScheme/cbc:ID',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':TaxScheme/cbc:ID',
             invoice.invoice_customer.tax_scheme.code)
 
         # DIAN 1.7.-2020: FAK41
         # DIAN 1.7.-2020: CAK41 Machete Construir Validación
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:TaxScheme/cbc:Name',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':TaxScheme/cbc:Name',
             invoice.invoice_customer.tax_scheme.name)
         # DIAN 1.7.-2020: FAK42
         # DIAN 1.7.-2020: CAK42
-        fexml.placeholder_for('./cac:AccountingCustomerParty/cac:Party/cac:PartyLegalEntity')
+        fexml.placeholder_for(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyLegalEntity')
 
         # DIAN 1.7.-2020: FAK43
         # DIAN 1.7.-2020: CAK43
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyLegalEntity/cbc:RegistrationName',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyLegalEntity/cbc'
+            ':RegistrationName',
             invoice.invoice_customer.legal_name)
 
         # DIAN 1.7.-2020: CAK44
         # DIAN 1.7.-2020: FAK44,FAK45,FAK46,FAK47,FAK48
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:PartyLegalEntity/cbc:CompanyID',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyLegalEntity/cbc'
+            ':CompanyID',
             invoice.invoice_customer.ident,
             **customer_company_id_attrs)
 
-        fexml.placeholder_for('./cac:AccountingCustomerParty/cac:Party/cac:PartyLegalEntity')
+        fexml.placeholder_for(
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyLegalEntity')
 
         # DIAN 1.7.-2020: FAK55
         # DIAN 1.7.-2020: CAK51, CAK55
-        fexml.set_element('./cac:AccountingCustomerParty/cac:Party/cac:Contact/cbc:ElectronicMail',
+        fexml.set_element(
+            './cac:AccountingCustomerParty/cac:Party/cac:Contact/cbc'
+            ':ElectronicMail',
             invoice.invoice_customer.email)
 
-
     def set_payment_mean(fexml, invoice):
         payment_mean = invoice.invoice_payment_mean
         fexml.set_element('./cac:PaymentMeans/cbc:ID', payment_mean.id)
-        fexml.set_element('./cac:PaymentMeans/cbc:PaymentMeansCode', payment_mean.code)
-        fexml.set_element('./cac:PaymentMeans/cbc:PaymentDueDate', payment_mean.due_at.strftime('%Y-%m-%d'))
-        fexml.set_element('./cac:PaymentMeans/cbc:PaymentID', payment_mean.payment_id)
+        fexml.set_element(
+            './cac:PaymentMeans/cbc:PaymentMeansCode',
+            payment_mean.code)
+        fexml.set_element(
+            './cac:PaymentMeans/cbc:PaymentDueDate',
+            payment_mean.due_at.strftime('%Y-%m-%d'))
+        fexml.set_element(
+            './cac:PaymentMeans/cbc:PaymentID',
+            payment_mean.payment_id)
 
     def set_element_amount_for(fexml, xml, xpath, amount):
         if not isinstance(amount, Amount):
@@ -372,38 +530,48 @@ class DIANInvoiceXML(fe.FeXML):
         fexml.set_element(xpath, amount, currencyID=amount.currency.code)
 
     def set_legal_monetary(fexml, invoice):
-        fexml.set_element_amount('./cac:LegalMonetaryTotal/cbc:LineExtensionAmount',
+        fexml.set_element_amount(
+            './cac:LegalMonetaryTotal/cbc:LineExtensionAmount',
             invoice.invoice_legal_monetary_total.line_extension_amount)
 
-        fexml.set_element_amount('./cac:LegalMonetaryTotal/cbc:TaxExclusiveAmount',
+        fexml.set_element_amount(
+            './cac:LegalMonetaryTotal/cbc:TaxExclusiveAmount',
             invoice.invoice_legal_monetary_total.tax_exclusive_amount)
 
-        fexml.set_element_amount('./cac:LegalMonetaryTotal/cbc:TaxInclusiveAmount',
+        fexml.set_element_amount(
+            './cac:LegalMonetaryTotal/cbc:TaxInclusiveAmount',
             invoice.invoice_legal_monetary_total.tax_inclusive_amount)
 
-        fexml.set_element_amount('./cac:LegalMonetaryTotal/cbc:ChargeTotalAmount',
+        fexml.set_element_amount(
+            './cac:LegalMonetaryTotal/cbc:ChargeTotalAmount',
             invoice.invoice_legal_monetary_total.charge_total_amount)
 
-        fexml.set_element_amount('./cac:LegalMonetaryTotal/cbc:PayableAmount',
+        fexml.set_element_amount(
+            './cac:LegalMonetaryTotal/cbc:PayableAmount',
             invoice.invoice_legal_monetary_total.payable_amount)
 
-
     def _set_invoice_document_reference(fexml, reference):
-        fexml._do_set_billing_reference(reference, 'cac:InvoiceDocumentReference')
+        fexml._do_set_billing_reference(
+            reference, 'cac:InvoiceDocumentReference')
 
     def _set_credit_note_document_reference(fexml, reference):
-        fexml._do_set_billing_reference(reference, 'cac:CreditNoteDocumentReference')
+        fexml._do_set_billing_reference(
+            reference, 'cac:CreditNoteDocumentReference')
 
     def _set_debit_note_document_reference(fexml, reference):
-        fexml._do_set_billing_reference(reference, 'cac:DebitNoteDocumentReference')
+        fexml._do_set_billing_reference(
+            reference, 'cac:DebitNoteDocumentReference')
 
     def _do_set_billing_reference(fexml, reference, tag_document):
         fexml.set_element('./cac:BillingReference/%s/cbc:ID' % (tag_document),
                           reference.ident)
-        fexml.set_element('./cac:BillingReference/cac:InvoiceDocumentReference/cbc:UUID',
+        fexml.set_element(
+            './cac:BillingReference/cac:InvoiceDocumentReference/cbc:UUID',
             reference.uuid,
             schemeName='CUFE-SHA384')
-        fexml.set_element('./cac:BillingReference/cac:InvoiceDocumentReference/cbc:IssueDate',
+        fexml.set_element(
+            './cac:BillingReference/cac:InvoiceDocumentReference/'
+            'cbc:IssueDate',
             reference.date.strftime("%Y-%m-%d"))
 
     def set_billing_reference(fexml, invoice):
@@ -421,7 +589,8 @@ class DIANInvoiceXML(fe.FeXML):
 
     def set_invoice_totals(fexml, invoice):
         tax_amount_for = defaultdict(lambda: defaultdict(lambda: Amount(0.0)))
-        withholding_amount_for = defaultdict(lambda: defaultdict(lambda: Amount(0.0)))
+        withholding_amount_for = defaultdict(
+            lambda: defaultdict(lambda: Amount(0.0)))
         percent_for = defaultdict(lambda: None)
 
         # requeridos para CUFE
@@ -439,9 +608,12 @@ class DIANInvoiceXML(fe.FeXML):
         for invoice_line in invoice.invoice_lines:
             for subtotal in invoice_line.tax.subtotals:
                 if subtotal.scheme is not None:
-                    tax_amount_for[subtotal.scheme.code]['tax_amount'] += subtotal.tax_amount
-                    tax_amount_for[subtotal.scheme.code]['taxable_amount'] += invoice_line.taxable_amount
-                    tax_amount_for[subtotal.scheme.code]['name'] = subtotal.scheme.name
+                    scheme = subtotal.scheme.code
+                    tax_amount_for[scheme][
+                        'tax_amount'] += subtotal.tax_amount
+                    tax_amount_for[scheme][
+                        'taxable_amount'] += invoice_line.taxable_amount
+                    tax_amount_for[scheme]['name'] = subtotal.scheme.name
 
                     # MACHETE ojo InvoiceLine.tax pasar a Invoice
                     percent_for[subtotal.scheme.code] = subtotal.percent
@@ -450,12 +622,16 @@ class DIANInvoiceXML(fe.FeXML):
 
             for subtotal_withholding in invoice_line.withholding.subtotals:
                 if subtotal_withholding.scheme is not None:
-                    withholding_amount_for[subtotal_withholding.scheme.code]['tax_amount'] += subtotal_withholding.tax_amount
-                    withholding_amount_for[subtotal_withholding.scheme.code]['taxable_amount'] += invoice_line.withholding_taxable_amount
+                    scheme = subtotal_withholding.scheme.code
+                    withholding_amount_for[scheme][
+                        'tax_amount'] += subtotal_withholding.tax_amount
+                    withholding_amount_for[scheme][
+                        'taxable_amount'] += (
+                            invoice_line.withholding_taxable_amount)
 
                     # MACHETE ojo InvoiceLine.tax pasar a Invoice
 
-                    percent_for[subtotal_withholding.scheme.code] = subtotal_withholding.percent
+                    percent_for[scheme] = subtotal_withholding.percent
 
                 total_withholding_amount += subtotal_withholding.tax_amount
 
@@ -477,12 +653,14 @@ class DIANInvoiceXML(fe.FeXML):
                                          tax_amount)
 
             # DIAN 1.7.-2020: FAS05
-            fexml.set_element_amount_for(line,
+            fexml.set_element_amount_for(
+                line,
                 '/cac:TaxTotal/cac:TaxSubtotal/cbc:TaxableAmount',
                 amount_of['taxable_amount'])
 
             # DIAN 1.7.-2020: FAU06
-            fexml.set_element_amount_for(line,
+            fexml.set_element_amount_for(
+                line,
                 '/cac:TaxTotal/cac:TaxSubtotal/cbc:TaxAmount',
                 amount_of['tax_amount'])
 
@@ -491,14 +669,19 @@ class DIANInvoiceXML(fe.FeXML):
                 line.set_element('/cac:TaxTotal/cac:TaxSubtotal/cbc:Percent',
                                  percent_for[cod_impuesto])
 
-
             if percent_for[cod_impuesto]:
-                line.set_element('/cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cbc:Percent',
+                line.set_element(
+                    '/cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/'
+                    'cbc:Percent',
                     percent_for[cod_impuesto])
 
-            line.set_element('/cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:ID',
+            line.set_element(
+                '/cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme'
+                '/cbc:ID',
                 cod_impuesto)
-            line.set_element('/cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:Name',
+            line.set_element(
+                '/cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme'
+                '/cbc:Name',
                 amount_of['name'])
 
         for index, item in enumerate(withholding_amount_for.items()):
@@ -506,36 +689,45 @@ class DIANInvoiceXML(fe.FeXML):
             next_append = index > 0
 
             # DIAN 1.7.-2020: FAS01
-            line = fexml.fragment('./cac:WithholdingTaxTotal', append=next_append)
+            line = fexml.fragment(
+                './cac:WithholdingTaxTotal',
+                append=next_append)
             # DIAN 1.7.-2020: FAU06
             tax_amount = amount_of['tax_amount']
-            fexml.set_element_amount_for(line,
-                                         '/cac:WithholdingTaxTotal/cbc:TaxAmount',
-                                         tax_amount)
+            fexml.set_element_amount_for(
+                line, '/cac:WithholdingTaxTotal/cbc:TaxAmount', tax_amount)
 
             # DIAN 1.7.-2020: FAS05
-            fexml.set_element_amount_for(line,
+            fexml.set_element_amount_for(
+                line,
                 '/cac:WithholdingTaxTotal/cac:TaxSubtotal/cbc:TaxableAmount',
                 amount_of['taxable_amount'])
 
             # DIAN 1.7.-2020: FAU06
-            fexml.set_element_amount_for(line,
+            fexml.set_element_amount_for(
+                line,
                 '/cac:WithholdingTaxTotal/cac:TaxSubtotal/cbc:TaxAmount',
                 amount_of['tax_amount'])
 
             # DIAN 1.7.-2020: FAS07
             if percent_for[cod_impuesto]:
-                line.set_element('/cac:WithholdingTaxTotal/cac:TaxSubtotal/cbc:Percent',
+                line.set_element(
+                    '/cac:WithholdingTaxTotal/cac:TaxSubtotal/cbc:Percent',
                     percent_for[cod_impuesto])
 
-
             if percent_for[cod_impuesto]:
-                line.set_element('/cac:WithholdingTaxTotal/cac:TaxSubtotal/cac:TaxCategory/cbc:Percent',
+                line.set_element(
+                    '/cac:WithholdingTaxTotal/cac:TaxSubtotal/cac:TaxCategory'
+                    '/cbc:Percent',
                     percent_for[cod_impuesto])
 
-            line.set_element('/cac:WithholdingTaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:ID',
+            line.set_element(
+                '/cac:WithholdingTaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac'
+                ':TaxScheme/cbc:ID',
                 cod_impuesto)
-            line.set_element('/cac:WithholdingTaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:Name',
+            line.set_element(
+                '/cac:WithholdingTaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac'
+                ':TaxScheme/cbc:Name',
                 'ReteRenta')
 
     # abstract method
@@ -552,50 +744,87 @@ class DIANInvoiceXML(fe.FeXML):
                                      invoice_line.tax_amount)
 
         # DIAN 1.7.-2020: FAX05
-        fexml.set_element_amount_for(line,
+        fexml.set_element_amount_for(
+            line,
             './cac:TaxTotal/cac:TaxSubtotal/cbc:TaxableAmount',
             invoice_line.taxable_amount)
         for subtotal in invoice_line.tax.subtotals:
-            line.set_element('./cac:TaxTotal/cac:TaxSubtotal/cbc:TaxAmount', subtotal.tax_amount, currencyID='COP')
+            line.set_element(
+                './cac:TaxTotal/cac:TaxSubtotal/cbc:TaxAmount',
+                subtotal.tax_amount,
+                currencyID='COP')
 
             if subtotal.percent is not None:
-                line.set_element('./cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cbc:Percent', '%0.2f' % round(subtotal.percent, 2))
+                line.set_element(
+                    './cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cbc'
+                    ':Percent',
+                    '%0.2f' %
+                    round(
+                        subtotal.percent,
+                        2))
 
             if subtotal.scheme is not None:
                 # DIAN 1.7.-2020: FAX15
-                line.set_element('./cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:ID', subtotal.scheme.code)
-                line.set_element('./cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:Name', subtotal.scheme.name)
-
+                line.set_element(
+                    './cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac'
+                    ':TaxScheme/cbc:ID',
+                    subtotal.scheme.code)
+                line.set_element(
+                    './cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac'
+                    ':TaxScheme/cbc:Name',
+                    subtotal.scheme.name)
 
     def set_invoice_line_withholding(fexml, line, invoice_line):
         fexml.set_element_amount_for(line,
                                      './cac:WithholdingTaxTotal/cbc:TaxAmount',
                                      invoice_line.withholding_amount)
         # DIAN 1.7.-2020: FAX05
-        fexml.set_element_amount_for(line,
+        fexml.set_element_amount_for(
+            line,
             './cac:WithholdingTaxTotal/cac:TaxSubtotal/cbc:TaxableAmount',
             invoice_line.withholding_taxable_amount)
 
         for subtotal in invoice_line.withholding.subtotals:
-            line.set_element('./cac:WithholdingTaxTotal/cac:TaxSubtotal/cbc:TaxAmount', subtotal.tax_amount, currencyID='COP')
+            line.set_element(
+                './cac:WithholdingTaxTotal/cac:TaxSubtotal/cbc:TaxAmount',
+                subtotal.tax_amount,
+                currencyID='COP')
 
             if subtotal.percent is not None:
-                line.set_element('./cac:WithholdingTaxTotal/cac:TaxSubtotal/cac:TaxCategory/cbc:Percent', '%0.2f' % round(subtotal.percent, 2))
+                line.set_element(
+                    './cac:WithholdingTaxTotal/cac:TaxSubtotal/cac'
+                    ':TaxCategory/cbc:Percent',
+                    '%0.2f' %
+                    round(
+                        subtotal.percent,
+                        2))
 
             if subtotal.scheme is not None:
                 # DIAN 1.7.-2020: FAX15
-                line.set_element('./cac:WithholdingTaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:ID', subtotal.scheme.code)
-                line.set_element('./cac:WithholdingTaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:Name', subtotal.scheme.name)
-
+                line.set_element(
+                    './cac:WithholdingTaxTotal/cac:TaxSubtotal/cac'
+                    ':TaxCategory/cac:TaxScheme/cbc:ID',
+                    subtotal.scheme.code)
+                line.set_element(
+                    './cac:WithholdingTaxTotal/cac:TaxSubtotal/cac'
+                    ':TaxCategory/cac:TaxScheme/cbc:Name',
+                    subtotal.scheme.name)
 
     def set_invoice_lines(fexml, invoice):
         next_append = False
         for index, invoice_line in enumerate(invoice.invoice_lines):
-            line = fexml.fragment('./cac:%sLine' % (fexml.tag_document()), append=next_append)
+            line = fexml.fragment(
+                './cac:%sLine' %
+                (fexml.tag_document()),
+                append=next_append)
             next_append = True
 
             line.set_element('./cbc:ID', index + 1)
-            line.set_element('./cbc:%sQuantity' % (fexml.tag_document_concilied()), invoice_line.quantity, unitCode = 'NAR')
+            line.set_element(
+                './cbc:%sQuantity' %
+                (fexml.tag_document_concilied()),
+                invoice_line.quantity,
+                unitCode='NAR')
             fexml.set_element_amount_for(line,
                                          './cbc:LineExtensionAmount',
                                          invoice_line.total_amount)
@@ -603,18 +832,26 @@ class DIANInvoiceXML(fe.FeXML):
             if not isinstance(invoice_line.tax, TaxTotalOmit):
                 fexml.set_invoice_line_tax(line, invoice_line)
 
-            if not isinstance(invoice_line.withholding, WithholdingTaxTotalOmit):
+            if not isinstance(
+                    invoice_line.withholding,
+                    WithholdingTaxTotalOmit):
                 fexml.set_invoice_line_withholding(line, invoice_line)
 
-            line.set_element('./cac:Item/cbc:Description', invoice_line.item.description)
+            line.set_element(
+                './cac:Item/cbc:Description',
+                invoice_line.item.description)
 
-            line.set_element('./cac:Item/cac:StandardItemIdentification/cbc:ID',
+            line.set_element(
+                './cac:Item/cac:StandardItemIdentification/cbc:ID',
                 invoice_line.item.id,
                 schemeID=invoice_line.item.scheme_id,
                 schemeName=invoice_line.item.scheme_name,
                 schemeAgencyID=invoice_line.item.scheme_agency_id)
 
-            line.set_element('./cac:Price/cbc:PriceAmount', invoice_line.price.amount, currencyID=invoice_line.price.amount.currency.code)
+            line.set_element(
+                './cac:Price/cbc:PriceAmount',
+                invoice_line.price.amount,
+                currencyID=invoice_line.price.amount.currency.code)
             # DIAN 1.7.-2020: FBB04
             line.set_element('./cac:Price/cbc:BaseQuantity',
                              invoice_line.price.quantity,
@@ -622,7 +859,8 @@ class DIANInvoiceXML(fe.FeXML):
 
             for idx, charge in enumerate(invoice_line.allowance_charge):
                 next_append_charge = idx > 0
-                fexml.append_allowance_charge(line, index + 1, charge, append=next_append_charge)
+                fexml.append_allowance_charge(
+                    line, index + 1, charge, append=next_append_charge)
 
     def set_allowance_charge(fexml, invoice):
         for idx, charge in enumerate(invoice.invoice_allowance_charge):
@@ -656,18 +894,28 @@ class DIANInvoiceXML(fe.FeXML):
 
         fexml.placeholder_for('./ext:UBLExtensions')
         fexml.set_element('./cbc:UBLVersionID', 'UBL 2.1')
-        fexml.set_element('./cbc:CustomizationID', invoice.invoice_operation_type)
+        fexml.set_element(
+            './cbc:CustomizationID',
+            invoice.invoice_operation_type)
         fexml.placeholder_for('./cbc:ProfileID')
         fexml.placeholder_for('./cbc:ProfileExecutionID')
         fexml.set_element('./cbc:ID', invoice.invoice_ident)
         fexml.placeholder_for('./cbc:UUID')
-        fexml.set_element('./cbc:IssueDate', invoice.invoice_issue.strftime('%Y-%m-%d'))
+        fexml.set_element(
+            './cbc:IssueDate',
+            invoice.invoice_issue.strftime('%Y-%m-%d'))
         # DIAN 1.7.-2020: FAD10
-        fexml.set_element('./cbc:IssueTime', invoice.invoice_issue.strftime('%H:%M:%S-05:00'))
-        fexml.set_element('./cbc:%sTypeCode' % (fexml.tag_document()),
+        fexml.set_element(
+            './cbc:IssueTime',
+            invoice.invoice_issue.strftime('%H:%M:%S-05:00'))
+        fexml.set_element(
+            './cbc:%sTypeCode' %
+            (fexml.tag_document()),
             invoice.invoice_type_code,
             listAgencyID='195',
-                        listAgencyName='No matching global declaration available for the validation root',
+            listAgencyName=(
+                'No matching global declaration available for the '
+                'validation root'),
             listURI='http://www.dian.gov.co')
         fexml.set_element('./cbc:DocumentCurrencyCode', 'COP')
         fexml.set_element('./cbc:LineCountNumeric', len(invoice.invoice_lines))

facho/fe/form_xml/support_document.py

@@ -14,36 +14,45 @@ __all__ = ['DIANSupportDocumentXML']
 class DIANSupportDocumentXML(fe.FeXML):
     """
     DianSupportDocumentXML mapea objeto form.Invoice a XML segun
-    lo indicado para él Documento soporte en adquisiciones efectuadas con sujetos no obligados a expedir factura de venta o documento equivalente.
+    lo indicado para él Documento soporte en adquisiciones efectuadas con
+    sujetos no obligados a expedir factura de venta o documento equivalente.
     """
 
     def __init__(self, invoice, tag_document='Invoice'):
-        super().__init__(tag_document, 'http://www.dian.gov.co/contratos/facturaelectronica/v1')
+        super().__init__(
+            tag_document,
+            'http://www.dian.gov.co/contratos/facturaelectronica/v1')
 
         # DIAN 1.1.-2021: DSAB03
         # DIAN 1.1.-2021: NSAB03
         self.placeholder_for(
-            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:InvoiceControl')
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:InvoiceControl')
 
         # DIAN 1.1.-2021: DSAB13
         # DIAN 1.1.-2021: NSAB13
         self.placeholder_for(
-            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:InvoiceSource')
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:InvoiceSource')
 
         # DIAN 1.1.-2021: DSAB18
         # DIAN 1.1.-2021: NSAB18
         self.placeholder_for(
-            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:SoftwareProvider')
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:SoftwareProvider')
 
         # DIAN 1.1.-2021: DSAB27
         # DIAN 1.1.-2021: NSAB27
         self.placeholder_for(
-            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:SoftwareSecurityCode')
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts'
+            ':DianExtensions/sts:SoftwareSecurityCode')
 
         # DIAN 1.1.-2021: DSAB30 DSAB31
         # DIAN 1.1.-2021: NSAB30 NSAB31
         self.placeholder_for(
-            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:AuthorizationProvider/sts:AuthorizationProviderID')
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/'
+            'sts:DianExtensions/sts:AuthorizationProvider/'
+            'sts:AuthorizationProviderID')
 
         # ZE02 se requiere existencia para firmar
         # DIAN 1.1.-2021: DSAA02 DSAB01
@@ -52,7 +61,7 @@ class DIANSupportDocumentXML(fe.FeXML):
             './ext:UBLExtensions/ext:UBLExtension', append=True)
         # DIAN 1.1.-2021: DSAB02
         # DIAN 1.1.-2021: NSAB02
-        extcontent = ublextension.find_or_create_element(
+        ublextension.find_or_create_element(
             '/ext:UBLExtension/ext:ExtensionContent')
         self.attach_invoice(invoice)
 
@@ -70,51 +79,61 @@ class DIANSupportDocumentXML(fe.FeXML):
         # DIAN 1.1.-2021: DSAJ07 DSAJ08
         # DIAN 1.1.-2021: NSAJ07 NSAJ08
         fexml.placeholder_for(
-            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address')
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address')
 
         # DIAN 1.1.-2021: DSAJ09
         # DIAN 1.1.-2021: NSAJ09
         fexml.set_element(
-            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:ID',
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:ID',
             invoice.invoice_supplier.address.city.code)
 
         # DIAN 1.1.-2021: DSAJ10
         # DIAN 1.1.-2021: NSAJ10
         fexml.set_element(
-            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:CityName',
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:CityName',
             invoice.invoice_supplier.address.city.name)
 
         # DIAN 1.1.-2021: DSAJ73
         # DIAN 1.1.-2021: NSAJ73
         fexml.set_element(
-            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:PostalZone',
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:PostalZone',
             invoice.invoice_supplier.address.postalzone.code)
 
         # DIAN 1.1.-2021: DSAJ11
         # DIAN 1.1.-2021: NSAJ11
         fexml.set_element(
-            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:CountrySubentity',
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:CountrySubentity',
             invoice.invoice_supplier.address.countrysubentity.name)
 
         # DIAN 1.1.-2021: DSAJ12
         # DIAN 1.1.-2021: NSAJ12
         fexml.set_element(
-            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cbc:CountrySubentityCode',
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cbc:CountrySubentityCode',
             invoice.invoice_supplier.address.countrysubentity.code)
         # DIAN 1.1.-2021: NSAJ13 NSAJ14
         fexml.set_element(
-            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cac:AddressLine/cbc:Line',
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cac:AddressLine/cbc:Line',
             invoice.invoice_supplier.address.street)
 
         # DIAN 1.1.-2021: DSAJ15 DSAJ16
         # DIAN 1.1.-2021: NSAJ15 NSAJ16
         fexml.set_element(
-            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cac:Country/cbc:IdentificationCode',
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cac:Country/cbc:IdentificationCode',
             invoice.invoice_supplier.address.country.code)
 
         # DIAN 1.1.-2021: DSAJ17
         # DIAN 1.1.-2021: NSAJ17
-        fexml.set_element('./cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac:Address/cac:Country/cbc:Name',
+        fexml.set_element(
+            './cac:AccountingSupplierParty/cac:Party/cac:PhysicalLocation/cac'
+            ':Address/cac:Country/cbc:Name',
             invoice.invoice_supplier.address.country.name,
             # DIAN 1.1.-2021: DSAJ18
             # # DIAN 1.1.-2021: NSAJ18
@@ -134,13 +153,15 @@ class DIANSupportDocumentXML(fe.FeXML):
         # DIAN 1.1.-2021: DSAJ20
         # DIAN 1.1.-2021: NSAJ20
         fexml.set_element(
-            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cbc:RegistrationName',
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cbc'
+            ':RegistrationName',
             invoice.invoice_supplier.legal_name)
 
         # DIAN 1.1.-2021: DSAJ21
         # DIAN 1.1.-2021: NSAJ21
         fexml.set_element(
-            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cbc:CompanyID',
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cbc'
+            ':CompanyID',
             invoice.invoice_supplier.ident,
             # DIAN 1.1.-2021: DSAJ22 DSAJ23 DSAJ24 DSAJ25
             # DIAN 1.1.-2021: NSAJ22 NSAJ23 NSAJ24 NSAJ25
@@ -149,25 +170,29 @@ class DIANSupportDocumentXML(fe.FeXML):
         # DIAN 1.1.-2021: DSAJ26
         # DIAN 1.1.-2021: NSAJ26
         fexml.set_element(
-            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cbc:TaxLevelCode',
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cbc'
+            ':TaxLevelCode',
             invoice.invoice_supplier.responsability_code,
             listName=invoice.invoice_supplier.responsability_regime_code)
 
         # DIAN 1.1.-2021: DSAJ39
         # DIAN 1.1.-2021: NSAJ39
         fexml.placeholder_for(
-            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:TaxScheme')
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':TaxScheme')
 
         # DIAN 1.1.-2021: DSAJ40
         # DIAN 1.1.-2021: NSAJ40
         fexml.set_element(
-            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:TaxScheme/cbc:ID',
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':TaxScheme/cbc:ID',
             invoice.invoice_customer.tax_scheme.code)
 
         # DIAN 1.1.-2021: DSAJ41
         # DIAN 1.1.-2021: NSAJ41
         fexml.set_element(
-            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac:TaxScheme/cbc:Name',
+            './cac:AccountingSupplierParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':TaxScheme/cbc:Name',
             invoice.invoice_customer.tax_scheme.name)
 
     def set_customer(fexml, invoice):
@@ -193,7 +218,8 @@ class DIANSupportDocumentXML(fe.FeXML):
         # DIAN 1.1.-2021: DSAK20
         # DIAN 1.1.-2021: NSAK20
         fexml.set_element(
-            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cbc:RegistrationName',
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cbc'
+            ':RegistrationName',
             invoice.invoice_customer.legal_name)
 
         customer_company_id_attrs = fe.SCHEME_AGENCY_ATTRS.copy()
@@ -205,7 +231,8 @@ class DIANSupportDocumentXML(fe.FeXML):
         # DIAN 1.1.-2021: DSAK21
         # DIAN 1.1.-2021: NSAK21
         fexml.set_element(
-            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cbc:CompanyID',
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cbc'
+            ':CompanyID',
             invoice.invoice_customer.ident,
             # DIAN 1.1.-2021: DSAK22 DSAK23 DSAK24 DSAK25
             # DIAN 1.1.-2021: NSAK22 NSAK23 NSAK24 NSAK25
@@ -214,24 +241,28 @@ class DIANSupportDocumentXML(fe.FeXML):
         # DIAN 1.1.-2021: DSAK26
         # DIAN 1.1.-2021: NSAK26
         fexml.set_element(
-            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cbc:TaxLevelCode',
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cbc'
+            ':TaxLevelCode',
             invoice.invoice_customer.responsability_code)
 
         # DIAN 1.1.-2021: DSAK39
         # DIAN 1.1.-2021: NSAK39
         fexml.placeholder_for(
-            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:TaxScheme')
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':TaxScheme')
 
         # DIAN 1.1.-2021: DSAK40
         # DIAN 1.1.-2021: NSAK40
         fexml.set_element(
-            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:TaxScheme/cbc:ID',
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':TaxScheme/cbc:ID',
             invoice.invoice_customer.tax_scheme.code)
 
         # DIAN 1.1.-2021: DSAK41
         # DIAN 1.1.-2021: NSAK41
         fexml.set_element(
-            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac:TaxScheme/cbc:Name',
+            './cac:AccountingCustomerParty/cac:Party/cac:PartyTaxScheme/cac'
+            ':TaxScheme/cbc:Name',
             invoice.invoice_customer.tax_scheme.name)
 
     def set_payment_mean(fexml, invoice):
@@ -328,7 +359,8 @@ class DIANSupportDocumentXML(fe.FeXML):
             reference.uuid,
             schemeName=schemeName)
         fexml.set_element(
-            './cac:BillingReference/cac:InvoiceDocumentReference/cbc:IssueDate',
+            './cac:BillingReference/cac:InvoiceDocumentReference/'
+            'cbc:IssueDate',
             reference.date.strftime("%Y-%m-%d"))
 
     def set_billing_reference(fexml, invoice):
@@ -419,14 +451,17 @@ class DIANSupportDocumentXML(fe.FeXML):
 
             if percent_for[cod_impuesto]:
                 line.set_element(
-                    '/cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cbc:Percent',
+                    '/cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/'
+                    'cbc:Percent',
                     percent_for[cod_impuesto])
 
             line.set_element(
-                '/cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:ID',
+                '/cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme'
+                '/cbc:ID',
                 cod_impuesto)
             line.set_element(
-                '/cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:Name', 'IVA')
+                '/cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme'
+                '/cbc:Name', 'IVA')
 
     # abstract method
 
@@ -455,7 +490,8 @@ class DIANSupportDocumentXML(fe.FeXML):
 
             if subtotal.percent is not None:
                 line.set_element(
-                    './cac:WithholdingTaxTotal/cac:TaxSubtotal/cac:TaxCategory/cbc:Percent',
+                    './cac:WithholdingTaxTotal/cac:TaxSubtotal/cac'
+                    ':TaxCategory/cbc:Percent',
                     '%0.2f' %
                     round(
                         subtotal.percent,
@@ -464,10 +500,12 @@ class DIANSupportDocumentXML(fe.FeXML):
             if subtotal.scheme is not None:
                 # DIAN 1.7.-2020: FAX15
                 line.set_element(
-                    './cac:WithholdingTaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:ID',
+                    './cac:WithholdingTaxTotal/cac:TaxSubtotal/cac'
+                    ':TaxCategory/cac:TaxScheme/cbc:ID',
                     subtotal.scheme.code)
                 line.set_element(
-                    './cac:WithholdingTaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:Name',
+                    './cac:WithholdingTaxTotal/cac:TaxSubtotal/cac'
+                    ':TaxCategory/cac:TaxScheme/cbc:Name',
                     subtotal.scheme.name)
 
     def set_invoice_line_tax(fexml, line, invoice_line):
@@ -488,7 +526,8 @@ class DIANSupportDocumentXML(fe.FeXML):
 
             if subtotal.percent is not None:
                 line.set_element(
-                    './cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cbc:Percent',
+                    './cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cbc'
+                    ':Percent',
                     '%0.2f' %
                     round(
                         subtotal.percent,
@@ -497,10 +536,12 @@ class DIANSupportDocumentXML(fe.FeXML):
             if subtotal.scheme is not None:
                 # DIAN 1.7.-2020: FAX15
                 line.set_element(
-                    './cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:ID',
+                    './cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac'
+                    ':TaxScheme/cbc:ID',
                     subtotal.scheme.code)
                 line.set_element(
-                    './cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac:TaxScheme/cbc:Name',
+                    './cac:TaxTotal/cac:TaxSubtotal/cac:TaxCategory/cac'
+                    ':TaxScheme/cbc:Name',
                     subtotal.scheme.name)
 
     def set_invoice_lines(fexml, invoice):
@@ -615,7 +656,9 @@ class DIANSupportDocumentXML(fe.FeXML):
             (fexml.tag_document()),
             invoice.invoice_type_code,
             listAgencyID='195',
-            listAgencyName='No matching global declaration available for the validation root',
+            listAgencyName=(
+                'No matching global declaration available for the '
+                'validation root'),
             listURI='http://www.dian.gov.co')
         fexml.set_element('./cbc:LineCountNumeric', len(invoice.invoice_lines))
         fexml.set_element(

facho/fe/nomina/__init__.py

@@ -8,21 +8,91 @@
 from dataclasses import dataclass
 from datetime import datetime
 import hashlib
-import typing
+
 
 from .. import fe
-from .. import form
 from ..data.dian import codelist
 
-from .devengado import *
-from .deduccion import *
-from .trabajador import *
-from .empleador import *
-from .pago import *
-from .lugar import Lugar
-
 from .amount import Amount
-from .exception import *
+from .deduccion import (
+    Deduccion,
+    DeduccionFondoPension,
+    DeduccionSalud,
+)
+from .devengado import (
+    Devengado,
+    DevengadoBasico,
+    DevengadoHoraExtra,
+    DevengadoHorasExtrasDiarias,
+    DevengadoHorasExtrasDiariasDominicalesYFestivos,
+    DevengadoHorasExtrasNocturnas,
+    DevengadoHorasExtrasNocturnasDominicalesYFestivos,
+    DevengadoHorasRecargoDiariasDominicalesYFestivos,
+    DevengadoHorasRecargoNocturno,
+    DevengadoHorasRecargoNocturnoDominicalesYFestivos,
+    DevengadoTransporte,
+)
+from .empleador import Empleador
+from .exception import DIANNominaIndividualError
+from .lugar import Lugar
+from .pago import (
+    FormaPago,
+    MetodoPago,
+    Pago,
+)
+from .trabajador import (
+    LugarTrabajo,
+    SubTipoTrabajador,
+    TipoContrato,
+    TipoDocumento,
+    TipoTrabajador,
+    Trabajador,
+)
+
+__all__ = [
+    'Amount',
+    'Deduccion',
+    'DeduccionFondoPension',
+    'DeduccionSalud',
+    'Devengado',
+    'DevengadoBasico',
+    'DevengadoHoraExtra',
+    'DevengadoHorasExtrasDiarias',
+    'DevengadoHorasExtrasDiariasDominicalesYFestivos',
+    'DevengadoHorasExtrasNocturnas',
+    'DevengadoHorasExtrasNocturnasDominicalesYFestivos',
+    'DevengadoHorasRecargoDiariasDominicalesYFestivos',
+    'DevengadoHorasRecargoNocturno',
+    'DevengadoHorasRecargoNocturnoDominicalesYFestivos',
+    'DevengadoTransporte',
+    'DIANNominaIndividual',
+    'DIANNominaIndividualDeAjuste',
+    'DIANNominaXML',
+    'DIANNominaIndividualError',
+    'DianXMLExtensionSigner',
+    'Empleador',
+    'Fecha',
+    'FechaPago',
+    'FormaPago',
+    'InformacionGeneral',
+    'Lugar',
+    'LugarTrabajo',
+    'Metadata',
+    'MetodoPago',
+    'Novedad',
+    'NumeroSecuencia',
+    'Pago',
+    'Periodo',
+    'PeriodoNomina',
+    'Proveedor',
+    'SubTipoTrabajador',
+    'TipoContrato',
+    'TipoDocumento',
+    'TipoMoneda',
+    'TipoTrabajador',
+    'Trabajador',
+]
+
 
 class Fecha:
     def __init__(self, fecha):
@@ -46,6 +116,7 @@ class Fecha:
     def __str__(self):
         return self.value
 
+
 class FechaPago(Fecha):
     def apply(self, fragment):
         fragment.set_element('./FechaPago', self.value)
@@ -84,19 +155,21 @@ class NumeroSecuencia:
                                 # NIE012
                                 Numero=numero)
 
+
 @dataclass
 class Periodo:
-    fecha_ingreso: typing.Union[str, Fecha]
-    fecha_liquidacion_inicio: typing.Union[str, Fecha]
-    fecha_liquidacion_fin: typing.Union[str, Fecha]
-    fecha_generacion: typing.Union[str, Fecha]
+    fecha_ingreso: str | Fecha
+    fecha_liquidacion_inicio: str | Fecha
+    fecha_liquidacion_fin: str | Fecha
+    fecha_generacion: str | Fecha
 
     tiempo_laborado: int = 1
-    fecha_retiro: typing.Union[str, Fecha] = None
+    fecha_retiro: str | Fecha | None = None
 
     def __post_init__(self):
         self.fecha_ingreso = Fecha.cast(self.fecha_ingreso)
-        self.fecha_liquidacion_inicio = Fecha.cast(self.fecha_liquidacion_inicio)
+        self.fecha_liquidacion_inicio = Fecha.cast(
+            self.fecha_liquidacion_inicio)
         self.fecha_liquidacion_fin = Fecha.cast(self.fecha_liquidacion_fin)
         self.fecha_retiro = Fecha.cast(self.fecha_retiro, optional=True)
 
@@ -107,7 +180,8 @@ class Periodo:
                                 # NIE003
                                 FechaRetiro=self.fecha_retiro,
                                 # NIE004
-                                FechaLiquidacionInicio=self.fecha_liquidacion_inicio,
+                                FechaLiquidacionInicio=(
+                                    self.fecha_liquidacion_inicio),
                                 # NIE005
                                 FechaLiquidacionFin=self.fecha_liquidacion_fin,
                                 # NIE006
@@ -115,6 +189,7 @@ class Periodo:
                                 # NIE008
                                 FechaGen=self.fecha_generacion)
 
+
 @dataclass
 class Proveedor:
     razon_social: str
@@ -141,11 +216,16 @@ class Proveedor:
         cune_xpath = scopexml.xpath_from_root('/InformacionGeneral')
         cune = fexml.get_element_attribute(cune_xpath, 'CUNE')
 
-        ambiente = fexml.get_element_attribute(scopexml.xpath_from_root('/InformacionGeneral'), 'Ambiente')
-        codigo_qr = f"https://catalogo-vpfe.dian.gov.co/document/searchqr?documentkey={cune}"
+        ambiente = fexml.get_element_attribute(
+            scopexml.xpath_from_root('/InformacionGeneral'), 'Ambiente')
+        codigo_qr = (
+            f"https://catalogo-vpfe.dian.gov.co/document/"
+            f"searchqr?documentkey={cune}")
 
         if InformacionGeneral.AMBIENTE_PRUEBAS == ambiente:
-            codigo_qr = f"https://catalogo-vpfe-hab.dian.gov.co/document/searchqr?documentkey={cune}"
+            codigo_qr = (
+                f"https://catalogo-vpfe-hab.dian.gov.co/document/"
+                f"searchqr?documentkey={cune}")
         elif ambiente is None:
             raise RuntimeError('fail to get InformacionGeneral/@Ambiente')
 
@@ -153,13 +233,15 @@ class Proveedor:
 
         # NIE020
         software_code = self._software_security_code(fexml, scopexml)
-        fexml.set_attributes(scopexml.xpath_from_root('/ProveedorXML'), SoftwareSC=software_code)
+        fexml.set_attributes(
+            scopexml.xpath_from_root('/ProveedorXML'),
+            SoftwareSC=software_code)
 
     def _software_security_code(self, fexml, scopexml):
 
-
         # 8.2
-        numero = fexml.get_element_attribute(scopexml.xpath_from_root('/NumeroSecuenciaXML'), 'Numero')
+        numero = fexml.get_element_attribute(
+            scopexml.xpath_from_root('/NumeroSecuenciaXML'), 'Numero')
         if numero is None:
             raise RuntimeError('fallo obtener NumeroSequenciaXML/@Numero')
 
@@ -173,6 +255,7 @@ class Proveedor:
         h.update(code.encode('utf-8'))
         return h.hexdigest()
 
+
 @dataclass
 class Metadata:
     novedad: Novedad
@@ -181,18 +264,33 @@ class Metadata:
     lugar_generacion: Lugar
     proveedor: Proveedor
 
-    def apply(self, novedad, numero_secuencia_xml, lugar_generacion_xml, proveedor_xml):
+    def apply(
+            self,
+            novedad,
+            numero_secuencia_xml,
+            lugar_generacion_xml,
+            proveedor_xml):
         if novedad:
             self.novedad.apply(novedad)
         self.secuencia.apply(numero_secuencia_xml)
-        self.lugar_generacion.apply(lugar_generacion_xml, './LugarGeneracionXML')
+        self.lugar_generacion.apply(
+            lugar_generacion_xml,
+            './LugarGeneracionXML')
         self.proveedor.apply(proveedor_xml)
 
-    def post_apply(self, fexml, scopexml, novedad, numero_secuencia_xml, lugar_generacion_xml, proveedor_xml):
+    def post_apply(
+            self,
+            fexml,
+            scopexml,
+            novedad,
+            numero_secuencia_xml,
+            lugar_generacion_xml,
+            proveedor_xml):
         self.proveedor.post_apply(fexml, scopexml, proveedor_xml)
         if novedad:
             self.novedad.post_apply(fexml, scopexml, proveedor_xml)
 
+
 @dataclass
 class PeriodoNomina:
     code: str
@@ -203,6 +301,7 @@ class PeriodoNomina:
             raise ValueError("code [%s] not found" % (self.code))
         self.name = codelist.PeriodoNomina[self.code]['name']
 
+
 @dataclass
 class TipoMoneda:
     code: str
@@ -213,6 +312,7 @@ class TipoMoneda:
             raise ValueError("code [%s] not found" % (self.code))
         self.name = codelist.TipoMoneda[self.code]['name']
 
+
 @dataclass
 class InformacionGeneral:
     @dataclass
@@ -259,7 +359,7 @@ class InformacionGeneral:
         def __str__(self):
             self.valor
 
-    fecha_generacion: typing.Union[str, Fecha]
+    fecha_generacion: str | Fecha
     hora_generacion: str
     periodo_nomina: PeriodoNomina
     tipo_moneda: TipoMoneda
@@ -328,17 +428,28 @@ class InformacionGeneral:
             CUNE=cune_hash
         )
 
+
 class DianXMLExtensionSigner(fe.DianXMLExtensionSigner):
 
     def __init__(self, pkcs12_path, passphrase=None, localpolicy=True):
-        super().__init__(pkcs12_path, passphrase=passphrase, localpolicy=localpolicy)
+        super().__init__(
+            pkcs12_path,
+            passphrase=passphrase,
+            localpolicy=localpolicy)
 
     def _element_extension_content(self, fachoxml):
-        return fachoxml.builder.xpath(fachoxml.root, './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent')
+        return fachoxml.builder.xpath(
+            fachoxml.root,
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent')
 
 
 class DIANNominaXML:
-    def __init__(self, tag_document, xpath_ajuste=None, schemaLocation=None, namespace_ajuste=None):
+    def __init__(
+            self,
+            tag_document,
+            xpath_ajuste=None,
+            schemaLocation=None,
+            namespace_ajuste=None):
         self.informacion_general_version = None
 
         self.tag_document = tag_document
@@ -346,21 +457,26 @@ class DIANNominaXML:
         if namespace_ajuste:
             self.fexml = fe.FeXML(tag_document, namespace_ajuste)
         else:
-            self.fexml = fe.FeXML(tag_document, 'dian:gov:co:facturaelectronica:NominaIndividual')
+            self.fexml = fe.FeXML(
+                tag_document,
+                'dian:gov:co:facturaelectronica:NominaIndividual')
 
         self.fexml.root.set("SchemaLocation", "")
         self.fexml.root.set("schemaLocation", schemaLocation)
 
         # layout, la dian requiere que los elementos
         # esten ordenados segun el anexo tecnico
-        self.fexml.placeholder_for('./ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent')
+        self.fexml.placeholder_for(
+            './ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent')
         self.fexml.placeholder_for('./TipoNota', optional=True)
 
         self.root_fragment = self.fexml
         if xpath_ajuste is not None:
             self.root_fragment = self.fexml.fragment(xpath_ajuste)
-        self.root_fragment.placeholder_for('./ReemplazandoPredecesor', optional=True)
-        self.root_fragment.placeholder_for('./EliminandoPredecesor', optional=True)
+        self.root_fragment.placeholder_for(
+            './ReemplazandoPredecesor', optional=True)
+        self.root_fragment.placeholder_for(
+            './EliminandoPredecesor', optional=True)
         if not namespace_ajuste:
             self.root_fragment.placeholder_for('./Novedad', optional=False)
         self.root_fragment.placeholder_for('./Periodo')
@@ -374,16 +490,20 @@ class DIANNominaXML:
         self.root_fragment.placeholder_for('./Pago')
         self.root_fragment.placeholder_for('./FechasPagos')
         self.root_fragment.placeholder_for('./Devengados/Basico')
-        self.root_fragment.placeholder_for('./Devengados/Transporte', optional=True)
+        self.root_fragment.placeholder_for(
+            './Devengados/Transporte', optional=True)
         if not namespace_ajuste:
             self.novedad = self.root_fragment.fragment('./Novedad')
         else:
             self.novedad = None
-        self.informacion_general_xml = self.root_fragment.fragment('./InformacionGeneral')
+        self.informacion_general_xml = self.root_fragment.fragment(
+            './InformacionGeneral')
         self.periodo_xml = self.root_fragment.fragment('./Periodo')
         self.fecha_pagos_xml = self.root_fragment.fragment('./FechasPagos')
-        self.numero_secuencia_xml = self.root_fragment.fragment('./NumeroSecuenciaXML')
-        self.lugar_generacion_xml = self.root_fragment.fragment('./LugarGeneracionXML')
+        self.numero_secuencia_xml = self.root_fragment.fragment(
+            './NumeroSecuenciaXML')
+        self.lugar_generacion_xml = self.root_fragment.fragment(
+            './LugarGeneracionXML')
         self.proveedor_xml = self.root_fragment.fragment('./ProveedorXML')
         self.empleador = self.root_fragment.fragment('./Empleador')
         self.trabajador = self.root_fragment.fragment('./Trabajador')
@@ -399,13 +519,19 @@ class DIANNominaXML:
             raise ValueError('se espera tipo Metadata')
         self.metadata = metadata
 
-        self.metadata.apply(self.novedad, self.numero_secuencia_xml, self.lugar_generacion_xml, self.proveedor_xml)
+        self.metadata.apply(
+            self.novedad,
+            self.numero_secuencia_xml,
+            self.lugar_generacion_xml,
+            self.proveedor_xml)
 
     def asignar_informacion_general(self, general):
         if not isinstance(general, InformacionGeneral):
             raise ValueError('se espera tipo InformacionGeneral')
         self.informacion_general = general
-        self.informacion_general.apply(self.informacion_general_xml, self.informacion_general_version)
+        self.informacion_general.apply(
+            self.informacion_general_xml,
+            self.informacion_general_version)
 
     def asignar_periodo(self, periodo):
         if not isinstance(periodo, Periodo):
@@ -498,10 +624,13 @@ class DIANNominaXML:
     def informacion_general(self):
         xpath = self.root_fragment.xpath_from_root('/InformacionGeneral')
         return {
-            'cune': self.fexml.get_element_attribute(cune_xpath, 'CUNE'),
-            'fecha_generacion': self.fexml.get_element_attribute(xpath, 'FechaGen'),
-            'numero': self.fexml.get_element_attribute(self.root_fragment('/NumeroSecuenciaXML', 'Numero'))
-        }
+            'cune': self.fexml.get_element_attribute(
+                xpath, 'CUNE'),
+            'fecha_generacion': self.fexml.get_element_attribute(
+                xpath, 'FechaGen'),
+            'numero': self.fexml.get_element_attribute(
+                self.root_fragment(
+                    '/NumeroSecuenciaXML', 'Numero'))}
 
     def toFachoXML(self):
         self._devengados_total()
@@ -512,26 +641,38 @@ class DIANNominaXML:
             # TODO(bit4bit) acoplamiento temporal
             # es importante el orden de ejecucion
 
-            self.informacion_general.post_apply(self.fexml, self.root_fragment, self.informacion_general_xml)
+            self.informacion_general.post_apply(
+                self.fexml, self.root_fragment, self.informacion_general_xml)
 
         if self.metadata is not None:
-            self.metadata.post_apply(self.fexml, self.root_fragment, self.novedad, self.numero_secuencia_xml, self.lugar_generacion_xml, self.proveedor_xml)
+            self.metadata.post_apply(
+                self.fexml,
+                self.root_fragment,
+                self.novedad,
+                self.numero_secuencia_xml,
+                self.lugar_generacion_xml,
+                self.proveedor_xml)
 
         return self.fexml
 
     def _comprobante_total(self):
-        devengados_total = self.root_fragment.get_element_text_or_attribute('./DevengadosTotal', '0.0')
-        deducciones_total = self.root_fragment.get_element_text_or_attribute('./DeduccionesTotal', '0.0')
+        devengados_total = self.root_fragment.get_element_text_or_attribute(
+            './DevengadosTotal', '0.0')
+        deducciones_total = self.root_fragment.get_element_text_or_attribute(
+            './DeduccionesTotal', '0.0')
 
-        comprobante_total = Amount(devengados_total) - Amount(deducciones_total)
+        comprobante_total = Amount(devengados_total) - \
+            Amount(deducciones_total)
 
-        self.root_fragment.set_element('./ComprobanteTotal', str(round(comprobante_total, 2)))
+        self.root_fragment.set_element(
+            './ComprobanteTotal', str(round(comprobante_total, 2)))
 
     def _deducciones_total(self):
         xpaths = [
-            self.root_fragment.xpath_from_root('/Deducciones/Salud/@Deduccion'),
-            self.root_fragment.xpath_from_root('/Deducciones/FondoPension/@Deduccion')
-        ]
+            self.root_fragment.xpath_from_root(
+                '/Deducciones/Salud/@Deduccion'),
+            self.root_fragment.xpath_from_root(
+                '/Deducciones/FondoPension/@Deduccion')]
         deducciones = map(lambda valor: Amount(valor),
                           self._values_of_xpaths(xpaths))
 
@@ -540,15 +681,19 @@ class DIANNominaXML:
         for deduccion in deducciones:
             deducciones_total += deduccion
 
-        self.root_fragment.set_element('./DeduccionesTotal', str(round(deducciones_total, 2)))
+        self.root_fragment.set_element(
+            './DeduccionesTotal', str(round(deducciones_total, 2)))
 
     def _devengados_total(self):
         xpaths = [
-            self.root_fragment.xpath_from_root('/Devengados/Basico/@SueldoTrabajado'),
-            self.root_fragment.xpath_from_root('/Devengados/Transporte/@AuxilioTransporte'),
-            self.root_fragment.xpath_from_root('/Devengados/Transporte/@ViaticoManuAlojS'),
-            self.root_fragment.xpath_from_root('/Devengados/Transporte/@ViaticoManuAlojNS')
-        ]
+            self.root_fragment.xpath_from_root(
+                '/Devengados/Basico/@SueldoTrabajado'),
+            self.root_fragment.xpath_from_root(
+                '/Devengados/Transporte/@AuxilioTransporte'),
+            self.root_fragment.xpath_from_root(
+                '/Devengados/Transporte/@ViaticoManuAlojS'),
+            self.root_fragment.xpath_from_root(
+                '/Devengados/Transporte/@ViaticoManuAlojNS')]
         devengados = map(lambda valor: Amount(valor),
                          self._values_of_xpaths(xpaths))
 
@@ -558,10 +703,13 @@ class DIANNominaXML:
         # TODO(bit4bit) nque valor va redondeado?
         # NIE186
         self.root_fragment.set_element('./Redondeo', str(round(0, 2)))
-        self.root_fragment.set_element('./DevengadosTotal', str(round(devengados_total,2)))
+        self.root_fragment.set_element(
+            './DevengadosTotal', str(round(devengados_total, 2)))
 
     def _values_of_xpaths(self, xpaths):
-        xpaths_values_of_values = map(lambda val: self.fexml.get_element_text_or_attribute(val, multiple=True), xpaths)
+        xpaths_values_of_values = map(
+            lambda val: self.fexml.get_element_text_or_attribute(
+                val, multiple=True), xpaths)
         xpaths_values = []
         # toda esta carreta para hacer un aplano de lista
         for xpath_values in xpaths_values_of_values:
@@ -573,15 +721,22 @@ class DIANNominaXML:
 
         return filter(lambda val: val is not None, xpaths_values)
 
+
 class DIANNominaIndividual(DIANNominaXML):
 
     def __init__(self):
-        schema = "dian:gov:co:facturaelectronica:NominaIndividual NominaIndividualElectronicaXSD.xsd"
+        schema = (
+            "dian:gov:co:facturaelectronica:NominaIndividual"
+            " NominaIndividualElectronicaXSD.xsd"
+        )
 
         super().__init__('NominaIndividual', schemaLocation=schema)
-        self.informacion_general_version = 'V1.0: Documento Soporte de Pago de Nómina Electrónica'
+        self.informacion_general_version = (
+            'V1.0: Documento Soporte de Pago de Nómina Electrónica')
 
 # TODO(bit4bit) confirmar que no tienen en comun con NominaIndividual
+
+
 class DIANNominaIndividualDeAjuste(DIANNominaXML):
 
     class Reemplazar(DIANNominaXML):
@@ -594,7 +749,8 @@ class DIANNominaIndividualDeAjuste(DIANNominaXML):
             def apply(self, fragment):
                 # NIAE214
                 fragment.set_element('./TipoNota', '1')
-                fragment.set_element('./Reemplazar/ReemplazandoPredecesor', None,
+                fragment.set_element(
+                    './Reemplazar/ReemplazandoPredecesor', None,
                     # NIAE090
                     NumeroPred=self.numero,
                     # NIAE191
@@ -604,18 +760,28 @@ class DIANNominaIndividualDeAjuste(DIANNominaXML):
                                      )
 
         def __init__(self):
-            schema = "dian:gov:co:facturaelectronica:NominaIndividualDeAjuste NominaIndividualDeAjusteElectronicaXSD.xsd"
+            schema = (
+                "dian:gov:co:facturaelectronica:NominaIndividualDeAjuste"
+                " NominaIndividualDeAjusteElectronicaXSD.xsd"
+            )
 
-            super().__init__('NominaIndividualDeAjuste', './Reemplazar', schemaLocation=schema, namespace_ajuste='dian:gov:co:facturaelectronica:NominaIndividualDeAjuste')
+            super().__init__(
+                'NominaIndividualDeAjuste',
+                './Reemplazar',
+                schemaLocation=schema,
+                namespace_ajuste=(
+                    'dian:gov:co:facturaelectronica:'
+                    'NominaIndividualDeAjuste'))
 
-            self.informacion_general_version = 'V1.0: Nota de Ajuste de Documento Soporte de Pago de Nómina Electrónica'
+            self.informacion_general_version = (
+                'V1.0: Nota de Ajuste de Documento Soporte de Pago de '
+                'Nómina Electrónica')
 
         def asignar_predecesor(self, predecesor):
             if not isinstance(predecesor, self.Predecesor):
                 raise ValueError("se espera tipo Predecesor")
             predecesor.apply(self.fexml)
 
-        
     class Eliminar(DIANNominaXML):
 
         @dataclass
@@ -636,10 +802,21 @@ class DIANNominaIndividualDeAjuste(DIANNominaXML):
                                      )
 
         def __init__(self):
-            schema = "dian:gov:co:facturaelectronica:NominaIndividualDeAjuste NominaIndividualDeAjusteElectronicaXSD.xsd"            
-            super().__init__('NominaIndividualDeAjuste', './Eliminar', schemaLocation=schema, namespace_ajuste='dian:gov:co:facturaelectronica:NominaIndividualDeAjuste')
+            schema = (
+                "dian:gov:co:facturaelectronica:NominaIndividualDeAjuste"
+                " NominaIndividualDeAjusteElectronicaXSD.xsd"
+            )
+            super().__init__(
+                'NominaIndividualDeAjuste',
+                './Eliminar',
+                schemaLocation=schema,
+                namespace_ajuste=(
+                    'dian:gov:co:facturaelectronica:'
+                    'NominaIndividualDeAjuste'))
 
-            self.informacion_general_version = "V1.0: Nota de Ajuste de Documento Soporte de Pago de Nómina Electrónica"
+            self.informacion_general_version = (
+                "V1.0: Nota de Ajuste de Documento Soporte de Pago de "
+                "Nómina Electrónica")
 
         def asignar_predecesor(self, predecesor):
             if not isinstance(predecesor, self.Predecesor):
@@ -648,4 +825,3 @@ class DIANNominaIndividualDeAjuste(DIANNominaXML):
 
     def __init__(self):
         super().__init__('NominaIndividualDeAjuste')
-

facho/fe/nomina/amount.py

@@ -1,4 +1,5 @@
 from .. import form
 
+
 class Amount(form.Amount):
     pass

facho/fe/nomina/deduccion/__init__.py

@@ -2,9 +2,9 @@
 # al crear objetos de valor
 # se debe exportar en __all__
 
-from .deduccion import *
-from .salud import *
-from .fondo_pension import *
+from .deduccion import Deduccion
+from .salud import DeduccionSalud
+from .fondo_pension import DeduccionFondoPension
 
 __all__ = [
     'Deduccion',

facho/fe/nomina/deduccion/fondo_pension.py

@@ -3,6 +3,7 @@ from dataclasses import dataclass
 from ..amount import Amount
 from .deduccion import Deduccion
 
+
 @dataclass
 class DeduccionFondoPension(Deduccion):
     porcentaje: Amount

facho/fe/nomina/deduccion/salud.py

@@ -3,6 +3,7 @@ from dataclasses import dataclass
 from ..amount import Amount
 from .deduccion import Deduccion
 
+
 @dataclass
 class DeduccionSalud(Deduccion):
     porcentaje: Amount
@@ -16,4 +17,3 @@ class DeduccionSalud(Deduccion):
                              #  NIE163
                              Deduccion=self.deduccion
                              )
-

facho/fe/nomina/departamento.py

@@ -1,4 +1,5 @@
 from .. import form
 
+
 class Departamento(form.CountrySubentity):
     pass

facho/fe/nomina/devengado/__init__.py

@@ -1,8 +1,17 @@
 
-from .basico import *
-from .transporte import *
-from .devengado import *
-from .horas_extras import *
+from .basico import DevengadoBasico
+from .transporte import DevengadoTransporte
+from .devengado import Devengado
+from .horas_extras import (
+    DevengadoHoraExtra,
+    DevengadoHorasExtrasDiarias,
+    DevengadoHorasExtrasNocturnas,
+    DevengadoHorasRecargoNocturno,
+    DevengadoHorasExtrasDiariasDominicalesYFestivos,
+    DevengadoHorasRecargoDiariasDominicalesYFestivos,
+    DevengadoHorasExtrasNocturnasDominicalesYFestivos,
+    DevengadoHorasRecargoNocturnoDominicalesYFestivos,
+)
 
 __all__ = [
     'Devengado',

facho/fe/nomina/devengado/horas_extras.py

@@ -1,5 +1,5 @@
 from dataclasses import dataclass
-from typing import List
+
 
 from ..amount import Amount
 from .devengado import Devengado
@@ -30,16 +30,17 @@ class DevengadoHoraExtra:
 
 @dataclass
 class DevengadoHorasExtrasDiarias(Devengado):
-    horas_extras: List[DevengadoHoraExtra]
+    horas_extras: list[DevengadoHoraExtra]
 
     def apply(self, fragment):
         hora_extra_xml = fragment.fragment('./HEDs')
         for hora_extra in self.horas_extras:
             hora_extra.apply('./HED', hora_extra_xml)
 
+
 @dataclass
 class DevengadoHorasExtrasNocturnas(Devengado):
-    horas_extras: List[DevengadoHoraExtra]
+    horas_extras: list[DevengadoHoraExtra]
 
     def apply(self, fragment):
         hora_extra_xml = fragment.fragment('./HENs')
@@ -49,43 +50,47 @@ class DevengadoHorasExtrasNocturnas(Devengado):
 
 @dataclass
 class DevengadoHorasRecargoNocturno(Devengado):
-    horas_extras: List[DevengadoHoraExtra]
+    horas_extras: list[DevengadoHoraExtra]
 
     def apply(self, fragment):
         hora_extra_xml = fragment.fragment('./HRNs')
         for hora_extra in self.horas_extras:
             hora_extra.apply('./HRN', hora_extra_xml)
 
+
 @dataclass
 class DevengadoHorasExtrasDiariasDominicalesYFestivos(Devengado):
-    horas_extras: List[DevengadoHoraExtra]
+    horas_extras: list[DevengadoHoraExtra]
 
     def apply(self, fragment):
         hora_extra_xml = fragment.fragment('./HEDDFs')
         for hora_extra in self.horas_extras:
             hora_extra.apply('./HEDDF', hora_extra_xml)
 
+
 @dataclass
 class DevengadoHorasRecargoDiariasDominicalesYFestivos(Devengado):
-    horas_extras: List[DevengadoHoraExtra]
+    horas_extras: list[DevengadoHoraExtra]
 
     def apply(self, fragment):
         hora_extra_xml = fragment.fragment('./HRDDFs')
         for hora_extra in self.horas_extras:
             hora_extra.apply('./HRDDF', hora_extra_xml)
 
+
 @dataclass
 class DevengadoHorasExtrasNocturnasDominicalesYFestivos(Devengado):
-    horas_extras: List[DevengadoHoraExtra]
+    horas_extras: list[DevengadoHoraExtra]
 
     def apply(self, fragment):
         hora_extra_xml = fragment.fragment('./HENDFs')
         for hora_extra in self.horas_extras:
             hora_extra.apply('./HENDF', hora_extra_xml)
 
+
 @dataclass
 class DevengadoHorasRecargoNocturnoDominicalesYFestivos(Devengado):
-    horas_extras: List[DevengadoHoraExtra]
+    horas_extras: list[DevengadoHoraExtra]
 
     def apply(self, fragment):
         hora_extra_xml = fragment.fragment('./HRNDFs')

facho/fe/nomina/devengado/transporte.py

@@ -3,6 +3,7 @@ from dataclasses import dataclass
 from ..amount import Amount
 from .devengado import Devengado
 
+
 @dataclass
 class DevengadoTransporte(Devengado):
     auxilio_transporte: Amount = None
@@ -17,5 +18,6 @@ class DevengadoTransporte(Devengado):
                              # NIE072
                              ViaticoManuAlojS=self.viatico_manutencion,
                              # NIE073
-                             ViaticoManuAlojNS = self.viatico_manutencion_no_salarial
+                             ViaticoManuAlojNS=(
+                                 self.viatico_manutencion_no_salarial)
                              )

facho/fe/nomina/empleador/__init__.py

@@ -4,6 +4,7 @@ from ..pais import Pais
 from ..departamento import Departamento
 from ..municipio import Municipio
 
+
 @dataclass
 class Empleador:
     razon_social: str
@@ -31,5 +32,3 @@ class Empleador:
 
                                 RazonSocial=self.razon_social
                                 )
-
-

facho/fe/nomina/hablitacion.py

@@ -3,6 +3,7 @@ import datetime
 
 from facho import fe
 
+
 class Habilitacion:
 
     @dataclass
@@ -16,18 +17,18 @@ class Habilitacion:
         self.metadata = metadata
 
     def generar(self, zipname, fecha):
-        nominas = []
-        dianzip = fe.DianZIP(open(zipname, 'w'))
+        fe.DianZIP(open(zipname, 'w'))
 
         fechabase = datetime.datetime.now()
         consecutivo = 0
         for _ in range(1, 11):
             consecutivo += 1
             fechabase += datetime.timedelta(days=1)
-            nomina = self._crear_nomina_individual()
+            self._crear_nomina_individual()
 
             # pag 96
-            nombre = "nie%010d%s%08x.xml" % (int(self.nit), fecha.strftime('%s'), consecutivo)
+            "nie%010d%s%08x.xml" % (
+                int(self.nit), fecha.strftime('%s'), consecutivo)
 
     def _crear_nomina_individual_reemplazar(self, nomina, fechabase):
         metadata = self.metadata
@@ -36,10 +37,15 @@ class Habilitacion:
 
         nomina_ajuste = fe.nomina.DIANNominaIndividualDeAjuste.Reemplazar()
         self._poblar_nomina(nomina_ajuste, metadata, fecha, prefijo='R')
-        informacion_general = nomina.informacion_general()
+        nomina.informacion_general()
 
-        
-    def _poblar_nomina(self, nomina, metadata, fecha, prefijo='N', consecutivo='0001'):
+    def _poblar_nomina(
+            self,
+            nomina,
+            metadata,
+            fecha,
+            prefijo='N',
+            consecutivo='0001'):
         nomina.asignar_fecha_pago(fecha)
 
         nomina.asignar_metadata(fe.nomina.Metadata(
@@ -152,4 +158,3 @@ class Habilitacion:
 
         nomina = fe.nomina.DIANNominaIndividual()
         self._poblar_nomina(nomina, metadata, fecha)
-                

facho/fe/nomina/lugar.py

@@ -5,6 +5,7 @@ from .departamento import Departamento
 from .municipio import Municipio
 from facho.fe.data.dian import codelist
 
+
 @dataclass
 class Lugar:
     pais: Pais

facho/fe/nomina/municipio.py

@@ -1,5 +1,5 @@
 from .. import form
 
+
 class Municipio(form.City):
     pass
-

facho/fe/nomina/pago/__init__.py

@@ -3,6 +3,7 @@ from dataclasses import dataclass
 from .forma_pago import FormaPago
 from .metodo_pago import MetodoPago
 
+
 @dataclass
 class Pago:
     forma: FormaPago

facho/fe/nomina/pago/forma_pago.py

@@ -2,6 +2,7 @@ from dataclasses import dataclass
 
 from facho.fe.data.dian import codelist
 
+
 @dataclass
 class FormaPago:
     code: str

facho/fe/nomina/pago/metodo_pago.py

@@ -2,6 +2,7 @@ from dataclasses import dataclass
 
 from facho.fe.data.dian import codelist
 
+
 @dataclass
 class MetodoPago:
     code: str
@@ -11,4 +12,3 @@ class MetodoPago:
         if self.code not in codelist.MediosPago:
             raise ValueError("code [%s] not found" % (self.code))
         self.name = codelist.MediosPago[self.code]['name']
-

facho/fe/nomina/pais.py

@@ -1,4 +1,5 @@
 from .. import form
 
+
 class Pais(form.Country):
     pass

facho/fe/nomina/trabajador/__init__.py

@@ -2,12 +2,11 @@ from dataclasses import dataclass, field
 
 from ..amount import Amount
 
-from .tipo_contrato import *
-from .tipo_documento import *
-from .lugar_trabajo import *
-from .tipo_trabajador import *
-from .sub_tipo_trabajador import *
-
+from .tipo_contrato import TipoContrato
+from .tipo_documento import TipoDocumento
+from .lugar_trabajo import LugarTrabajo
+from .tipo_trabajador import TipoTrabajador
+from .sub_tipo_trabajador import SubTipoTrabajador
 
 
 @dataclass
@@ -29,10 +28,13 @@ class Trabajador:
 
     codigo_trabajador: str = None
     otros_nombres: str = None
-    sub_tipo: SubTipoTrabajador = field(default_factory=lambda: SubTipoTrabajador(code='00'))
+    sub_tipo: SubTipoTrabajador = field(
+        default_factory=lambda: SubTipoTrabajador(code="00")
+    )
 
     def apply(self, fragment):
-        fragment.set_attributes('./Trabajador',
+        fragment.set_attributes(
+            "./Trabajador",
             # NIE041
             TipoTrabajador=self.tipo.code,
             # NIE042
@@ -53,13 +55,12 @@ class Trabajador:
             OtrosNombres=self.otros_nombres,
             # NIE050
             LugarTrabajoPais=self.lugar_trabajo.pais.code,
-
             # NIE051
-                                LugarTrabajoDepartamentoEstado = self.lugar_trabajo.departamento.code,
-
+            LugarTrabajoDepartamentoEstado=(
+                self.lugar_trabajo.departamento.code
+            ),
             # NIE052
             LugarTrabajoMunicipioCiudad=self.lugar_trabajo.municipio.code,
-
             # NIE053
             LugarTrabajoDireccion=self.lugar_trabajo.direccion,
             # NIE056
@@ -69,5 +70,5 @@ class Trabajador:
             # NIE062
             Sueldo=str(self.sueldo),
             # NIE063
-                                CodigoTrabajador = self.codigo_trabajador
+            CodigoTrabajador=self.codigo_trabajador,
         )

facho/fe/nomina/trabajador/lugar_trabajo.py

@@ -1,10 +1,9 @@
 from dataclasses import dataclass
-
-from . import *
 from ..pais import Pais
 from ..departamento import Departamento
 from ..municipio import Municipio
 
+
 @dataclass
 class LugarTrabajo:
     pais: Pais

facho/fe/nomina/trabajador/sub_tipo_trabajador.py

@@ -2,12 +2,13 @@ from dataclasses import dataclass
 
 from facho.fe.data.dian import codelist
 
+
 @dataclass
 class SubTipoTrabajador:
     code: str
-    name: str = ''
+    name: str = ""
 
     def __post_init__(self):
         if self.code not in codelist.SubTipoTrabajador:
             raise ValueError("code [%s] not found" % (self.code))
-        self.name = codelist.SubTipoTrabajador[self.code]['name']
+        self.name = codelist.SubTipoTrabajador[self.code]["name"]

facho/fe/nomina/trabajador/tipo_contrato.py

@@ -2,14 +2,13 @@ from dataclasses import dataclass
 
 from facho.fe.data.dian import codelist
 
+
 @dataclass
 class TipoContrato:
     code: str
-    name: str = ''
+    name: str = ""
 
     def __post_init__(self):
         if self.code not in codelist.TipoContrato:
             raise ValueError("code [%s] not found" % (self.code))
-        self.name = codelist.TipoContrato[self.code]['name']
-
-
+        self.name = codelist.TipoContrato[self.code]["name"]

facho/fe/nomina/trabajador/tipo_documento.py

@@ -2,12 +2,13 @@ from dataclasses import dataclass
 
 from facho.fe.data.dian import codelist
 
+
 @dataclass
 class TipoDocumento:
     code: str
-    name: str = ''
+    name: str = ""
 
     def __post_init__(self):
         if self.code not in codelist.TipoIdFiscal:
             raise ValueError("code [%s] not found" % (self.code))
-        self.name = codelist.TipoIdFiscal[self.code]['name']
+        self.name = codelist.TipoIdFiscal[self.code]["name"]

facho/fe/nomina/trabajador/tipo_trabajador.py

@@ -2,12 +2,13 @@ from dataclasses import dataclass
 
 from facho.fe.data.dian import codelist
 
+
 @dataclass
 class TipoTrabajador:
     code: str
-    name: str = ''
+    name: str = ""
 
     def __post_init__(self):
         if self.code not in codelist.TipoTrabajador:
             raise ValueError("code [%s] not found" % (self.code))
-        self.name = codelist.TipoTrabajador[self.code]['name']
+        self.name = codelist.TipoTrabajador[self.code]["name"]

pyproject.toml

@@ -0,0 +1,79 @@
+[build-system]
+requires = ["setuptools>=68", "wheel"]
+build-backend = "setuptools.build_meta"
+
+[project]
+name = "facho"
+version = "0.1.0"
+description = "Facturacion Electronica Colombia"
+readme = "README.rst"
+license = { text = "GPL-3.0-or-later" }
+authors = [
+    { name = "Jovany Leandro G.C", email = "bit4bit@riseup.net" },
+]
+keywords = ["facho"]
+classifiers = [
+    "Development Status :: 2 - Pre-Alpha",
+    "Intended Audience :: Developers",
+    "License :: OSI Approved :: GNU General Public License v3 (GPLv3)",
+    "Natural Language :: English",
+    "Programming Language :: Python :: 3",
+    "Programming Language :: Python :: 3.10",
+    "Programming Language :: Python :: 3.11",
+    "Programming Language :: Python :: 3.12",
+    "Programming Language :: Python :: 3.13",
+]
+requires-python = ">=3.10"
+dependencies = [
+    "Click>=8.1.7",
+    "zeep>=4.2.1",
+    "lxml>=5.2.2",
+    "cryptography>=41.0.0",
+    "pyOpenSSL>=23.2.0",
+    "xmlsig>=0.1.9",
+    "xades>=1.0.0",
+    "xmlsec>=1.3.12",
+    "python-dateutil>=2.9.0",
+    # usamos esta dependencia en runtime
+    # para forzar uso de policy_id de archivo local
+    "mock>=5.1.0",
+    "xmlschema>=3.0.0",
+]
+
+[project.scripts]
+facho = "facho.cli:main"
+
+[project.urls]
+Repository = "https://github.com/bit4bit/facho"
+
+[tool.poetry.group.dev.dependencies]
+build = ">=1.2.2"
+coverage = ">=7.8.0"
+flake8 = ">=7.0.0"
+pytest = ">=9.1.1,<10.0.0"
+tox = ">=4.23.2"
+
+
+[tool.setuptools]
+packages = { find = { exclude = ["tests*"] } }
+
+[tool.setuptools.package-data]
+"*" = ["*.gc", "*.xsd", "politicadefirmav2.pdf"]
+
+[tool.pytest.ini_options]
+testpaths = ["tests"]
+
+[tool.bumpversion]
+current_version = "0.1.0"
+commit = true
+tag = true
+
+[[tool.bumpversion.files]]
+filename = "pyproject.toml"
+search = 'version = "{current_version}"'
+replace = 'version = "{new_version}"'
+
+[[tool.bumpversion.files]]
+filename = "facho/__init__.py"
+search = "__version__ = '{current_version}'"
+replace = "__version__ = '{new_version}'"

requirements_dev.txt

@@ -1,17 +0,0 @@
-attrs==22.1.0
-distlib==0.3.6
-filelock==3.8.0
-iniconfig==1.1.1
-packaging==21.3
-platformdirs==2.5.2
-pluggy==1.0.0
-py==1.11.0
-pyparsing==3.0.9
-pytest==7.1.3
-semantic-version==2.10.0
-setuptools-rust==1.5.2
-six==1.16.0
-tomli==2.0.1
-tox==3.26.0
-typing_extensions==4.4.0
-virtualenv==20.16.5

setup.cfg

@@ -1,25 +0,0 @@
-[bumpversion]
-current_version = 0.1.0
-commit = True
-tag = True
-
-[bumpversion:file:setup.py]
-search = version='{current_version}'
-replace = version='{new_version}'
-
-[bumpversion:file:facho/__init__.py]
-search = __version__ = '{current_version}'
-replace = __version__ = '{new_version}'
-
-[bdist_wheel]
-universal = 1
-
-[flake8]
-exclude = docs
-
-[aliases]
-# Define setup.py command aliases here
-test = pytest
-
-[tool:pytest]
-addopts = --ignore=setup.py

setup.py

@@ -1,89 +0,0 @@
-#!/usr/bin/env python
-# -*- coding: utf-8 -*-
-# This file is part of facho.  The COPYRIGHT file at the top level of
-# this repository contains the full copyright notices and license terms.
-
-"""The setup script."""
-
-from setuptools import setup, find_packages
-
-with open('README.rst') as readme_file:
-    readme = readme_file.read()
-
-with open('HISTORY.rst') as history_file:
-    history = history_file.read()
-
-requirements = ['Click>=8.1.7',
-                'zeep==4.2.1',
-                'lxml==5.2.2',
-                'cryptography==3.3.2',
-                'pyOpenSSL==20.0.1',
-                'xmlsig==0.1.7',
-                'xades==1.0.0',
-                'xmlsec==1.3.14',
-                'python-dateutil==2.9.0.post0',
-                # usamos esta dependencia en runtime
-                # para forzar uso de policy_id de archivo local
-                'mock>=5.1.0',
-                'xmlschema>=3.0.0']
-
-"""
-Listado de Versiones Anteriores
-requirements = ['Click>=6.0',
-                'zeep==4.0.0',
-                'lxml==4.6.3',
-                'cryptography==3.3.2',
-                'pyOpenSSL==20.0.1',
-                'xmlsig==0.1.7',
-                'xades==0.2.2',
-                'xmlsec==1.3.12',
-                # usamos esta dependencia en runtime
-                # para forzar uso de policy_id de archivo local
-                'mock>=2.0.0',
-                'xmlschema>=1.8']
-
-"""
-
-setup_requirements = ['pytest-runner', ]
-
-test_requirements = ['pytest', ]
-
-setup(
-    author="Jovany Leandro G.C",
-    author_email='bit4bit@riseup.net',
-    classifiers=[
-        'Development Status :: 2 - Pre-Alpha',
-        'Intended Audience :: Developers',
-        'License :: OSI Approved :: GNU General Public License v3 (GPLv3)',
-        'Natural Language :: English',
-        'Programming Language :: Python :: 3',
-        'Programming Language :: Python :: 3.9',
-        'Programming Language :: Python :: 3.10',
-        'Programming Language :: Python :: 3.11',
-        'Programming Language :: Python :: 3.12',
-    ],
-    description="Facturacion Electronica Colombia",
-    entry_points={
-        'console_scripts': [
-            'facho=facho.cli:main',
-        ],
-    },
-    install_requires=requirements,
-    license="GNU General Public License v3",
-    long_description=readme + '\n\n' + history,
-    long_description_content_type='text/x-rst',
-    include_package_data=True,
-    package_data = {
-        # If any package contains *.txt or *.rst files, include them:
-        '': ['*.gc', '*.xsd', 'politicadefirmav2.pdf']
-    },
-    keywords='facho',
-    name='facho',
-    packages=find_packages(exclude=("tests",)),
-    setup_requires=setup_requirements,
-    test_suite='tests',
-    tests_require=test_requirements,
-    url='https://github.com/bit4bit/facho',
-    version='0.2.0',
-    zip_safe=False,
-)

tests/fixtures.py

@@ -2,6 +2,7 @@ import pytest
 import facho.fe.form as form
 from datetime import datetime
 
+
 @pytest.fixture
 def simple_debit_note_without_lines():
     inv = form.DebitNote(form.InvoiceDocumentReference(
@@ -36,14 +37,22 @@ def simple_debit_note_without_lines():
     ))
     return inv
 
+
 @pytest.fixture
 def simple_credit_note_without_lines():
-    inv = form.CreditNote(form.InvoiceDocumentReference('1234', 'xx', datetime.now()))
+    inv = form.CreditNote(
+        form.InvoiceDocumentReference(
+            '1234', 'xx', datetime.now()))
     inv.set_period(datetime.now(), datetime.now())
     inv.set_issue(datetime.now())
     inv.set_ident('ABC123')
     inv.set_operation_type('20')
-    inv.set_payment_mean(form.PaymentMean(form.PaymentMean.DEBIT, '41', datetime.now(), '1234'))
+    inv.set_payment_mean(
+        form.PaymentMean(
+            form.PaymentMean.DEBIT,
+            '41',
+            datetime.now(),
+            '1234'))
     inv.set_supplier(form.Party(
         name='facho-supplier',
         ident=form.PartyIdentification('123', '', '31'),
@@ -68,6 +77,7 @@ def simple_credit_note_without_lines():
     ))
     return inv
 
+
 @pytest.fixture
 def simple_invoice_without_lines():
     inv = form.NationalSalesInvoice()
@@ -75,7 +85,12 @@ def simple_invoice_without_lines():
     inv.set_issue(datetime.now())
     inv.set_ident('ABC123')
     inv.set_operation_type('10')
-    inv.set_payment_mean(form.PaymentMean(form.PaymentMean.DEBIT, '41', datetime.now(), '1234'))
+    inv.set_payment_mean(
+        form.PaymentMean(
+            form.PaymentMean.DEBIT,
+            '41',
+            datetime.now(),
+            '1234'))
     inv.set_supplier(form.Party(
         name='facho-supplier',
         ident=form.PartyIdentification('123', '', '31'),
@@ -108,7 +123,12 @@ def simple_invoice():
     inv.set_issue(datetime.now())
     inv.set_ident('ABC123')
     inv.set_operation_type('10')
-    inv.set_payment_mean(form.PaymentMean(form.PaymentMean.DEBIT, '41', datetime.now(), ' 1234'))
+    inv.set_payment_mean(
+        form.PaymentMean(
+            form.PaymentMean.DEBIT,
+            '41',
+            datetime.now(),
+            ' 1234'))
     inv.set_supplier(form.Party(
         name='facho-supplier',
         ident=form.PartyIdentification('123', '', '31'),

tests/test_amount.py

@@ -14,6 +14,7 @@ def test_amount_positive():
     with pytest.raises(ValueError):
         form.Amount(-1.0)
 
+
 def test_amount_equals():
     price1 = form.Amount(110.0)
     price2 = form.Amount(100 + 10.0)
@@ -22,6 +23,7 @@ def test_amount_equals():
     assert price1 == form.Amount(10) * form.Amount(10) + form.Amount(10)
     assert form.Amount(110) == (form.Amount(1.10) * form.Amount(100))
 
+
 def test_round_half_even():
     # https://www.w3.org/TR/xpath-functions-31/#func-round-half-to-even
     assert form.Amount(0.5).round(0).float() == 0.0
@@ -30,16 +32,20 @@ def test_round_half_even():
     assert form.Amount(3.567812e+3).round(2).float() == 3567.81e0
     assert form.Amount(4.7564e-3).round(2).float() == 0.0e0
 
+
 def test_round():
     # Entre 0 y 5 Mantener el dígito menos significativo
     assert form.Amount(1.133).round(2) == form.Amount(1.13)
     # Entre 6 y 9 Incrementar el dígito menos significativo
     assert form.Amount(1.166).round(2) == form.Amount(1.17)
-    # 5, y el segundo dígito siguiente al dígito menos significativo es cero o par Mantener el dígito menos significativo
+    # 5, y el segundo dígito siguiente al dígito menos significativo es cero o
+    # par Mantener el dígito menos significativo
     assert str(form.Amount(1.1542).round(2)) == str(form.Amount(1.15))
-    # 5, y el segundo dígito siguiente al dígito menos significativo es impar Incrementar el dígito menos significativo
+    # 5, y el segundo dígito siguiente al dígito menos significativo es impar
+    # Incrementar el dígito menos significativo
     assert str(form.Amount(1.1563).round(2)) == str(form.Amount(1.16))
 
+
 def test_amount_truncate():
     assert form.Amount(1.1569).truncate_as_string(2) == '1.15'
     assert form.Amount(587.0700).truncate_as_string(2) == '587.07'
@@ -49,5 +55,6 @@ def test_amount_truncate():
     assert form.Amount(10000.02245).truncate_as_string(2) == '10000.02'
     assert form.Amount(10000.02357).truncate_as_string(2) == '10000.02'
 
+
 def test_amount_format():
     assert str(round(form.Amount(1.1569), 2)) == '1.16'

tests/test_client_dian.py

@@ -4,12 +4,7 @@
 # this repository contains the full copyright notices and license terms.
 
 from facho.fe.client import dian
-from facho import facho
 
-import pytest
-
-
-from datetime import datetime
 
 class FakeDianClient(dian.DianClient):
     def __init__(self, username, password, resp):

tests/test_facho.py

@@ -20,18 +20,19 @@ def test_facho_xml():
     invoice.text = 'Test'
     assert xml.tostring() == '<root><Invoice>Test</Invoice></root>'
 
-    invoice_line = xml.find_or_create_element('/root/Invoice/Line')
+    xml.find_or_create_element('/root/Invoice/Line')
     assert xml.tostring() == '<root><Invoice>Test<Line/></Invoice></root>'
 
 
 def test_facho_xml_with_attr():
     xml = facho.FachoXML('root')
-    invoice = xml.find_or_create_element('/root/Invoice[id=123]')
+    xml.find_or_create_element('/root/Invoice[id=123]')
     assert xml.tostring() == '<root><Invoice id="123"/></root>'
 
+
 def test_facho_xml_idempotent():
     xml = facho.FachoXML('root')
-    invoice = xml.find_or_create_element('/root/Invoice')
+    xml.find_or_create_element('/root/Invoice')
     assert xml.tostring() == '<root><Invoice/></root>'
 
     xml.find_or_create_element('/root/Invoice')
@@ -46,6 +47,7 @@ def test_facho_xml_idempotent():
     xml.find_or_create_element('/root/Invoice/Line')
     assert xml.tostring() == '<root><Invoice><Line/></Invoice></root>'
 
+
 def test_facho_xml_aliases():
     xml = facho.FachoXML('root')
     xml.register_alias_xpath('Invoice', '/root/Invoice')
@@ -54,31 +56,42 @@ def test_facho_xml_aliases():
     invoice.text = 'Test'
     assert xml.tostring() == '<root><Invoice>Test</Invoice></root>'
 
+
 def test_facho_xmlns():
     xml = facho.FachoXML('root', nsmap={
         'ext': 'https://ext',
         'sts': 'https://sts',
     })
 
-    invoiceAuthorization = xml.find_or_create_element('/root/ext:UBLExtensions/ext:UBLExtension/'
+    invoiceAuthorization = xml.find_or_create_element(
+        '/root/ext:UBLExtensions/ext:UBLExtension/'
         'ext:ExtensionContent/sts:DianExtensions/'
         'sts:InvoiceControl/sts:InvoiceAuthorization')
-    assert xml.tostring().strip() == '<root xmlns:ext="https://ext" xmlns:sts="https://sts"><ext:UBLExtensions>'\
-        '<ext:UBLExtension>'\
-        '<ext:ExtensionContent>'\
-        '<sts:DianExtensions>'\
-        '<sts:InvoiceControl>'\
-        '<sts:InvoiceAuthorization/>'\
-        '</sts:InvoiceControl></sts:DianExtensions></ext:ExtensionContent></ext:UBLExtension></ext:UBLExtensions></root>'
+    assert xml.tostring().strip() == (
+        '<root xmlns:ext="https://ext" xmlns:sts="https://sts">'
+        '<ext:UBLExtensions>'
+        '<ext:UBLExtension>'
+        '<ext:ExtensionContent>'
+        '<sts:DianExtensions>'
+        '<sts:InvoiceControl>'
+        '<sts:InvoiceAuthorization/>'
+        '</sts:InvoiceControl></sts:DianExtensions>'
+        '</ext:ExtensionContent></ext:UBLExtension>'
+        '</ext:UBLExtensions></root>')
 
     invoiceAuthorization.text = '123456789'
-    assert xml.tostring().strip() == '<root xmlns:ext="https://ext" xmlns:sts="https://sts"><ext:UBLExtensions>'\
-        '<ext:UBLExtension>'\
-        '<ext:ExtensionContent>'\
-        '<sts:DianExtensions>'\
-        '<sts:InvoiceControl>'\
-        '<sts:InvoiceAuthorization>123456789</sts:InvoiceAuthorization>'\
-        '</sts:InvoiceControl></sts:DianExtensions></ext:ExtensionContent></ext:UBLExtension></ext:UBLExtensions></root>'
+    assert xml.tostring().strip() == (
+        '<root xmlns:ext="https://ext" xmlns:sts="https://sts">'
+        '<ext:UBLExtensions>'
+        '<ext:UBLExtension>'
+        '<ext:ExtensionContent>'
+        '<sts:DianExtensions>'
+        '<sts:InvoiceControl>'
+        '<sts:InvoiceAuthorization>123456789</sts:InvoiceAuthorization>'
+        '</sts:InvoiceControl></sts:DianExtensions>'
+        '</ext:ExtensionContent></ext:UBLExtension>'
+        '</ext:UBLExtensions></root>')
+
 
 def test_facho_xmlns_idempotent():
     xml = facho.FachoXML('root', nsmap={
@@ -87,16 +100,22 @@ def test_facho_xmlns_idempotent():
     })
 
     xml.find_or_create_element('/root/ext:Extension/sts:Sotoros')
-    assert xml.tostring() == '<root xmlns:ext="https://ext" xmlns:sts="https://sts"><ext:Extension><sts:Sotoros/></ext:Extension></root>'
+    assert xml.tostring() == (
+        '<root xmlns:ext="https://ext" xmlns:sts="https://sts">'
+        '<ext:Extension><sts:Sotoros/></ext:Extension></root>')
 
     xml.find_or_create_element('/root/ext:Extension/sts:Sotoros')
-    assert xml.tostring() == '<root xmlns:ext="https://ext" xmlns:sts="https://sts"><ext:Extension><sts:Sotoros/></ext:Extension></root>'
+    assert xml.tostring() == (
+        '<root xmlns:ext="https://ext" xmlns:sts="https://sts">'
+        '<ext:Extension><sts:Sotoros/></ext:Extension></root>')
+
 
 def test_facho_xml_set_element_with_format():
     xml = facho.FachoXML('root')
-    invoice = xml.set_element('/root/Invoice', 1, format_='%02d')
+    xml.set_element('/root/Invoice', 1, format_='%02d')
     assert xml.tostring() == '<root><Invoice>01</Invoice></root>'
 
+
 def test_facho_xml_fragment():
     xml = facho.FachoXML('root')
     invoice = xml.fragment('/root/Invoice')
@@ -116,7 +135,10 @@ def test_facho_xml_fragments():
     line = xml.fragment('/Invoice/Line', append=True)
     line.set_element('/Line/Id', 3)
 
-    assert xml.tostring() == '<Invoice><Line><Id>1</Id></Line><Line><Id>2</Id></Line><Line><Id>3</Id></Line></Invoice>'
+    assert xml.tostring() == (
+        '<Invoice><Line><Id>1</Id></Line><Line><Id>2</Id></Line>'
+        '<Line><Id>3</Id></Line></Invoice>')
+
 
 def test_facho_xml_nested_fragments():
     xml = facho.FachoXML('Invoice')
@@ -128,7 +150,11 @@ def test_facho_xml_nested_fragments():
 
     party.set_element('/Party/LastName', 'test')
 
-    assert xml.tostring() == '<Invoice><Party><Name>test</Name><Address><Line>line 1</Line></Address><LastName>test</LastName></Party></Invoice>'
+    assert xml.tostring() == (
+        '<Invoice><Party><Name>test</Name>'
+        '<Address><Line>line 1</Line></Address>'
+        '<LastName>test</LastName></Party></Invoice>')
+
 
 def test_facho_xml_get_element_text_of_fragment():
     xml = facho.FachoXML('root')
@@ -137,6 +163,7 @@ def test_facho_xml_get_element_text_of_fragment():
 
     assert invoice.get_element_text('/Invoice/Id') == '1'
 
+
 def test_facho_xml_get_element_text():
     xml = facho.FachoXML('Invoice')
     xml.set_element('/Invoice/ID', 'ABC123')
@@ -147,6 +174,7 @@ def test_facho_xml_get_element_text():
     line.set_element('/Line/Quantity', 5)
     assert line.get_element_text('/Line/Quantity', format_=int) == 5
 
+
 def test_facho_xml_get_element_text_next_child():
     xml = facho.FachoXML('Invoice')
     xml.set_element('/Invoice/ID', 'ABC123')
@@ -166,25 +194,32 @@ def test_facho_xml_set_element_relative():
 
     assert xml.get_element_text('/Invoice/ID') == 'ABC123'
 
+
 def test_facho_xml_set_element_relative_with_namespace():
-    xml = facho.FachoXML('{%s}Invoice' % ('http://www.dian.gov.co/contratos/facturaelectronica/v1'),
-                         nsmap={'fe': 'http://www.dian.gov.co/contratos/facturaelectronica/v1'})
+    xml = facho.FachoXML(
+        '{%s}Invoice' %
+        ('http://www.dian.gov.co/contratos/facturaelectronica/v1'),
+        nsmap={
+            'fe': 'http://www.dian.gov.co/contratos/facturaelectronica/v1'})
     xml.set_element('./ID', 'ABC123')
 
     assert xml.get_element_text('/fe:Invoice/ID') == 'ABC123'
 
+
 def test_facho_xml_fragment_relative():
     xml = facho.FachoXML('root')
     invoice = xml.fragment('./Invoice')
     invoice.set_element('./Id', 1)
     assert xml.tostring() == '<root><Invoice><Id>1</Id></Invoice></root>'
 
+
 def test_facho_xml_get_element_fragment_relative():
     xml = facho.FachoXML('root')
     invoice = xml.fragment('./Invoice')
     invoice.set_element('./Id', 1)
     assert invoice.get_element_text('./Id') == '1'
 
+
 def test_facho_xml_replacement_for():
     xml = facho.FachoXML('root')
     xml.placeholder_for('./child/type')
@@ -192,37 +227,47 @@ def test_facho_xml_replacement_for():
                         './child/code', 'test')
     assert xml.tostring() == '<root><child><code>test</code></child></root>'
 
+
 def test_facho_xml_set_element_content_invalid_validation():
     xml = facho.FachoXML('root')
 
-    with pytest.raises(facho.FachoValueInvalid) as e:
+    with pytest.raises(facho.FachoValueInvalid):
         xml.set_element_validator('./Id', lambda text, attrs: text == 'mero')
         xml.set_element('./Id', 'bad')
 
+
 def test_facho_xml_set_element_content_valid_validation():
     xml = facho.FachoXML('root')
 
     xml.set_element_validator('./Id', lambda text, attrs: text == 'mero')
     xml.set_element('./Id', 'mero')
 
+
 def test_facho_xml_set_element_attribute_invalid_validation():
     xml = facho.FachoXML('root')
 
-    with pytest.raises(facho.FachoValueInvalid) as e:
-        xml.set_element_validator('./Id', lambda text, attrs: attrs['code'] == 'ABC')
+    with pytest.raises(facho.FachoValueInvalid):
+        xml.set_element_validator(
+            './Id', lambda text, attrs: attrs['code'] == 'ABC')
         xml.set_element('./Id', 'mero', code='CBA')
 
+
 def test_facho_xml_set_element_attribute_valid_validation():
     xml = facho.FachoXML('root')
 
-    xml.set_element_validator('./Id', lambda text, attrs: attrs['code'] == 'ABC')
+    xml.set_element_validator(
+        './Id',
+        lambda text,
+        attrs: attrs['code'] == 'ABC')
     xml.set_element('./Id', 'mero', code='ABC')
 
+
 def test_facho_xml_get_element_attribute():
     xml = facho.FachoXML('root')
     xml.set_element('./Id', 'mero', code='ABC')
     assert xml.get_element_attribute('/root/Id', 'code') == 'ABC'
 
+
 def test_facho_xml_keep_orden_slibing():
     xml = facho.FachoXML('root')
     xml.find_or_create_element('./A')
@@ -233,6 +278,7 @@ def test_facho_xml_keep_orden_slibing():
 
     assert xml.tostring() == '<root><A/><A/><B/><B/><C/></root>'
 
+
 def test_facho_xml_placeholder_optional():
     xml = facho.FachoXML('root')
     xml.placeholder_for('./A')
@@ -241,6 +287,7 @@ def test_facho_xml_placeholder_optional():
 
     assert xml.tostring() == '<root><A/><C/></root>'
 
+
 def test_facho_xml_placeholder_append_to_optional():
     xml = facho.FachoXML('root')
     xml.placeholder_for('./A')
@@ -250,6 +297,7 @@ def test_facho_xml_placeholder_append_to_optional():
     xml.find_or_create_element('./B')
     assert xml.tostring() == '<root><A/><B/><C/></root>'
 
+
 def test_facho_xml_placeholder_set_element_to_optional():
     xml = facho.FachoXML('root')
     xml.placeholder_for('./A')
@@ -259,6 +307,7 @@ def test_facho_xml_placeholder_set_element_to_optional():
     xml.set_element('./B', '2')
     assert xml.tostring() == '<root><A/><B>2</B><C/></root>'
 
+
 def test_facho_xml_placeholder_set_element_to_optional_with_append():
     xml = facho.FachoXML('root')
     xml.placeholder_for('./A')
@@ -294,6 +343,7 @@ def test_facho_xml_set_attributes_not_set_optional():
         xml.get_element_attribute('/root/A', 'value1')
     assert xml.get_element_attribute('/root/A', 'value2') == '2'
 
+
 def test_facho_xml_placeholder_with_fragment():
     xml = facho.FachoXML('root')
     xml.placeholder_for('./A')
@@ -306,7 +356,9 @@ def test_facho_xml_placeholder_with_fragment():
 
     AA = xml.fragment('./AA/Child', append=True)
 
-    assert xml.tostring() == '<root><A/><AA><Child><B/><B/></Child><Child/></AA><AAA/></root>'
+    assert xml.tostring() == (
+        '<root><A/><AA><Child><B/><B/></Child><Child/></AA><AAA/></root>')
+
 
 def test_facho_xml_create_on_first_append():
     xml = facho.FachoXML('root')
@@ -314,6 +366,7 @@ def test_facho_xml_create_on_first_append():
     xml.find_or_create_element('./A', append=True)
     assert xml.tostring() == '<root><A/></root>'
 
+
 def test_facho_xml_create_on_first_append_multiple_appends():
     xml = facho.FachoXML('root')
 
@@ -324,6 +377,7 @@ def test_facho_xml_create_on_first_append_multiple_appends():
     xml.find_or_create_element('./C', append=True)
     assert xml.tostring() == '<root><B/><A/><A/><A/><C/></root>'
 
+
 def test_facho_xml_fragment_create_on_first_append():
     xml = facho.FachoXML('root')
 
@@ -333,6 +387,7 @@ def test_facho_xml_fragment_create_on_first_append():
     A.find_or_create_element('./C')
     assert xml.tostring() == '<root><A><B/></A><A><C/></A></root>'
 
+
 def test_facho_xml_placeholder_optional_and_fragment():
     xml = facho.FachoXML('root')
 
@@ -346,6 +401,7 @@ def test_facho_xml_placeholder_optional_and_fragment():
 
     assert xml.tostring() == '<root><A><AA><B/><C/></AA></A></root>'
 
+
 def test_facho_xml_placeholder_optional_and_set_attributes():
     xml = facho.FachoXML('root')
     xml.placeholder_for('./A')
@@ -354,6 +410,7 @@ def test_facho_xml_placeholder_optional_and_set_attributes():
     assert xml.get_element_attribute('/root/A', 'prueba') == 'OK'
     assert xml.tostring() == '<root><A prueba="OK"/></root>'
 
+
 def test_facho_xml_placeholder_optional_and_fragment_with_set_element():
     xml = facho.FachoXML('root')
 
@@ -365,17 +422,19 @@ def test_facho_xml_placeholder_optional_and_fragment_with_set_element():
     assert xml.tostring() == '<root><A><AA prueba="OK"/></A></root>'
     assert xml.get_element_attribute('/root/A/AA', 'prueba') == 'OK'
 
+
 def test_facho_xml_exist_element():
     xml = facho.FachoXML('root')
 
     xml.placeholder_for('./A')
-    assert xml.exist_element('/root/A') == False
+    assert xml.exist_element('/root/A') is False
     assert xml.tostring() == '<root><A/></root>'
 
     xml.find_or_create_element('./A')
-    assert xml.exist_element('/root/A') == True
+    assert xml.exist_element('/root/A')
     assert xml.tostring() == '<root><A/></root>'
 
+
 def test_facho_xml_query_element_text_or_attribute():
     xml = facho.FachoXML('root')
 
@@ -384,6 +443,7 @@ def test_facho_xml_query_element_text_or_attribute():
     assert xml.get_element_text_or_attribute('/root/A') == 'contenido'
     assert xml.get_element_text_or_attribute('/root/A/@clave') == 'valor'
 
+
 def test_facho_xml_query_element_text_or_attribute_from_fragment():
     xml = facho.FachoXML('root')
 
@@ -392,6 +452,7 @@ def test_facho_xml_query_element_text_or_attribute_from_fragment():
 
     assert invoice.get_element_text_or_attribute('/Invoice/A') == 'contenido'
 
+
 def test_facho_xml_build_xml_absolute():
     xml = facho.FachoXML('root')
 
@@ -400,18 +461,23 @@ def test_facho_xml_build_xml_absolute():
 
 
 def test_facho_xml_build_xml_absolute_namespace():
-    xml = facho.FachoXML('{%s}root' % ('http://www.dian.gov.co/contratos/facturaelectronica/v1'),
-                         nsmap={'fe': 'http://www.dian.gov.co/contratos/facturaelectronica/v1'})
+    xml = facho.FachoXML(
+        '{%s}root' %
+        ('http://www.dian.gov.co/contratos/facturaelectronica/v1'),
+        nsmap={
+            'fe': 'http://www.dian.gov.co/contratos/facturaelectronica/v1'})
 
     xpath = xml.xpath_from_root('/A')
     assert xpath == '/fe:root/A'
 
 
 def test_facho_xml_build_xml_absolute_namespace_from_fragment():
-    xml = facho.FachoXML('{%s}root' % ('http://www.dian.gov.co/contratos/facturaelectronica/v1'),
-                         nsmap={'fe': 'http://www.dian.gov.co/contratos/facturaelectronica/v1'})
+    xml = facho.FachoXML(
+        '{%s}root' %
+        ('http://www.dian.gov.co/contratos/facturaelectronica/v1'),
+        nsmap={
+            'fe': 'http://www.dian.gov.co/contratos/facturaelectronica/v1'})
     invoice = xml.fragment('/root/Invoice')
 
     xpath = invoice.xpath_from_root('/A')
     assert xpath == '/fe:root/Invoice/A'
-

tests/test_fe.py

@@ -4,16 +4,14 @@
 # this repository contains the full copyright notices and license terms.
 from datetime import datetime
 
-import pytest
-
 from facho import fe
 
-import helpers
-
 
 def test_xmlsigned_build(monkeypatch):
-    # openssl req -x509 -sha256 -nodes -subj "/CN=test" -days 1 -newkey rsa:2048 -keyout example.key -out example.pem
-    # openssl pkcs12 -export -out example.p12 -inkey example.key -in example.pem
+    # openssl req -x509 -sha256 -nodes -subj "/CN=test" -days 1
+    # -newkey rsa:2048 -keyout example.key -out example.pem
+    # openssl pkcs12 -export -out example.p12 -inkey example.key -in
+    # example.pem
     signer = fe.DianXMLExtensionSigner('./tests/example.p12')
 
     xml = fe.FeXML('Invoice',
@@ -21,68 +19,90 @@ def test_xmlsigned_build(monkeypatch):
 
     signer.sign_xml_element(xml.root)
 
-    elem = xml.find_or_create_element('/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/ds:Signature')
+    elem = xml.find_or_create_element(
+        '/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent'
+        '/ds:Signature')
 
     assert elem is not None
     # assert elem.findall('ds:SignedInfo', fe.NAMESPACES) is not None
 
 
 def test_xmlsigned_with_passphrase_build(monkeypatch):
-    #openssl req -x509 -sha256 -nodes -subj "/CN=test" -days 1 -newkey rsa:2048 -keyout example.key -out example.pem
-    #openssl pkcs12 -export -out example.p12 -inkey example.key -in example.pem
-    signer = fe.DianXMLExtensionSigner('./tests/example-with-passphrase.p12', 'test')
+    # openssl req -x509 -sha256 -nodes -subj "/CN=test" -days 1
+    # -newkey rsa:2048 -keyout example.key -out example.pem
+    # openssl pkcs12 -export -out example.p12 -inkey example.key -in
+    # example.pem
+    signer = fe.DianXMLExtensionSigner(
+        './tests/example-with-passphrase.p12', 'test')
 
     xml = fe.FeXML('Invoice',
                    'http://www.dian.gov.co/contratos/facturaelectronica/v1')
 
     signer.sign_xml_element(xml.root)
 
-    elem = xml.find_or_create_element('/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/ds:Signature')
+    elem = xml.find_or_create_element(
+        '/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent'
+        '/ds:Signature')
 
     assert elem is not None
     # assert elem.findall('ds:SignedInfo', fe.NAMESPACES) is not None
 
 
 def test_dian_extension_software_security_code():
-    security_code = fe.DianXMLExtensionSoftwareSecurityCode('idsoftware', '1234', '1')
+    security_code = fe.DianXMLExtensionSoftwareSecurityCode(
+        'idsoftware', '1234', '1')
     xml = fe.FeXML('Invoice',
                    'http://www.dian.gov.co/contratos/facturaelectronica/v1')
     xml.add_extension(security_code)
-    content = xml.get_element_text('/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:SoftwareSecurityCode')
+    content = xml.get_element_text(
+        '/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent'
+        '/sts:DianExtensions/sts:SoftwareSecurityCode')
     assert content is not None
 
+
 def test_dian_extension_invoice_authorization():
     invoice_authorization = '18762002346472'
-    inv_auth_ext = fe.DianXMLExtensionInvoiceAuthorization(invoice_authorization,
-                                                           datetime(2017, 2, 23),
-                                                           datetime(2019, 8, 23),
-                                                           'MD', 100001, 174999)
+    inv_auth_ext = fe.DianXMLExtensionInvoiceAuthorization(
+        invoice_authorization, datetime(
+            2017, 2, 23), datetime(
+            2019, 8, 23), 'MD', 100001, 174999)
     xml = fe.FeXML('Invoice',
                    'http://www.dian.gov.co/contratos/facturaelectronica/v1')
     xml.add_extension(inv_auth_ext)
-    auth = xml.get_element_text('/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:InvoiceControl/sts:InvoiceAuthorization')
+    auth = xml.get_element_text(
+        '/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent'
+        '/sts:DianExtensions/sts:InvoiceControl/sts:InvoiceAuthorization')
     assert auth == invoice_authorization
 
+
 def test_dian_extension_software_provider():
     nit = '123456789'
     id_software = 'ABCDASDF123'
-    software_provider_extension = fe.DianXMLExtensionSoftwareProvider(nit, '', id_software)
+    software_provider_extension = fe.DianXMLExtensionSoftwareProvider(
+        nit, '', id_software)
 
     xml = fe.FeXML('Invoice',
                    'http://www.dian.gov.co/contratos/facturaelectronica/v1')
     xml.add_extension(software_provider_extension)
 
-    give_nit = xml.get_element_text('/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:SoftwareProvider/sts:ProviderID')
+    give_nit = xml.get_element_text(
+        '/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent'
+        '/sts:DianExtensions/sts:SoftwareProvider/sts:ProviderID')
     assert nit == give_nit
 
+
 def test_dian_extension_authorization_provider():
     auth_provider_extension = fe.DianXMLExtensionAuthorizationProvider()
     xml = fe.FeXML('Invoice',
                    'http://www.dian.gov.co/contratos/facturaelectronica/v1')
     xml.add_extension(auth_provider_extension)
-    dian_nit = xml.get_element_text('/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/sts:DianExtensions/sts:AuthorizationProvider/sts:AuthorizationProviderID')
+    dian_nit = xml.get_element_text(
+        '/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/'
+        'ext:ExtensionContent/sts:DianExtensions/sts:AuthorizationProvider/'
+        'sts:AuthorizationProviderID')
     assert dian_nit == '800197268'
 
+
 def test_dian_invoice_without_namespace_in_root():
     xml = fe.FeXML('Invoice',
                    'http://www.dian.gov.co/contratos/facturaelectronica/v1')
@@ -93,9 +113,13 @@ def test_dian_invoice_without_namespace_in_root():
 def test_xml_sign_dian(monkeypatch):
     xml = fe.FeXML('Invoice',
                    'http://www.dian.gov.co/contratos/facturaelectronica/v1')
-    xml.find_or_create_element('/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent')
-    ublextension = xml.fragment('/fe:Invoice/ext:UBLExtensions/ext:UBLExtension', append=True)
-    extcontent = ublextension.find_or_create_element('/ext:UBLExtension/ext:ExtensionContent')
+    xml.find_or_create_element(
+        '/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent')
+    ublextension = xml.fragment(
+        '/fe:Invoice/ext:UBLExtensions/ext:UBLExtension',
+        append=True)
+    ublextension.find_or_create_element(
+        '/ext:UBLExtension/ext:ExtensionContent')
 
     xmlstring = xml.tostring()
     print(xmlstring)
@@ -103,12 +127,17 @@ def test_xml_sign_dian(monkeypatch):
     xmlsigned = signer.sign_xml_string(xmlstring)
     assert "Signature" in xmlsigned
 
+
 def test_xml_sign_dian_using_bytes(monkeypatch):
     xml = fe.FeXML('Invoice',
                    'http://www.dian.gov.co/contratos/facturaelectronica/v1')
-    xml.find_or_create_element('/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent')
-    ublextension = xml.fragment('/fe:Invoice/ext:UBLExtensions/ext:UBLExtension', append=True)
-    extcontent = ublextension.find_or_create_element('/ext:UBLExtension/ext:ExtensionContent')
+    xml.find_or_create_element(
+        '/fe:Invoice/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent')
+    ublextension = xml.fragment(
+        '/fe:Invoice/ext:UBLExtensions/ext:UBLExtension',
+        append=True)
+    ublextension.find_or_create_element(
+        '/ext:UBLExtension/ext:ExtensionContent')
 
     xmlstring = xml.tostring()
     p12_data = open('./tests/example.p12', 'rb').read()
@@ -117,6 +146,7 @@ def test_xml_sign_dian_using_bytes(monkeypatch):
     xmlsigned = signer.sign_xml_string(xmlstring)
     assert "Signature" in xmlsigned
 
+
 def test_xml_signature_timestamp(monkeypatch):
     xml = fe.FeXML(
         'Invoice',

tests/test_fe_form.py

@@ -41,11 +41,13 @@ def test_invoicesimple_build(simple_invoice):
     xml = DIANInvoiceXML(simple_invoice)
 
     supplier_name = xml.get_element_text(
-        '/fe:Invoice/cac:AccountingSupplierParty/cac:Party/cac:PartyName/cbc:Name')
+        '/fe:Invoice/cac:AccountingSupplierParty/cac:Party/cac:PartyName/cbc'
+        ':Name')
     assert supplier_name == simple_invoice.invoice_supplier.name
 
     customer_name = xml.get_element_text(
-        '/fe:Invoice/cac:AccountingCustomerParty/cac:Party/cac:PartyName/cbc:Name')
+        '/fe:Invoice/cac:AccountingCustomerParty/cac:Party/cac:PartyName/cbc'
+        ':Name')
     assert customer_name == simple_invoice.invoice_customer.name
 
 
@@ -66,7 +68,8 @@ def test_invoicesimple_xml_signed(monkeypatch, simple_invoice):
     xml.add_extension(signer)
 
     elem = xml.get_element(
-        '/fe:Invoice/ext:UBLExtensions/ext:UBLExtension[2]/ext:ExtensionContent/ds:Signature')
+        '/fe:Invoice/ext:UBLExtensions/ext:UBLExtension[2]/ext'
+        ':ExtensionContent/ds:Signature')
     assert elem.text is not None
 
 
@@ -75,14 +78,16 @@ def test_invoicesimple_zip(simple_invoice):
 
     zipdata = io.BytesIO()
     with fe.DianZIP(zipdata) as dianzip:
-        name_invoice = dianzip.add_invoice_xml(simple_invoice.invoice_ident, str(xml_invoice))
+        name_invoice = dianzip.add_invoice_xml(
+            simple_invoice.invoice_ident, str(xml_invoice))
 
     # el zip ademas de archivar debe comprimir los archivos
     # de lo contrario la DIAN lo rechaza
     with zipfile.ZipFile(zipdata) as dianzip:
         dianzip.testzip()
         for zipinfo in dianzip.infolist():
-            assert zipinfo.compress_type == zipfile.ZIP_DEFLATED, "se espera el zip comprimido"
+            assert zipinfo.compress_type == zipfile.ZIP_DEFLATED, (
+                "se espera el zip comprimido")
 
     with zipfile.ZipFile(zipdata) as dianzip:
         xml_data = dianzip.open(name_invoice).read().decode('utf-8')
@@ -122,7 +127,8 @@ def test_invoice_invoice_type_code(simple_invoice):
 def test_invoice_totals(simple_invoice_without_lines):
     simple_invoice = simple_invoice_without_lines
     simple_invoice.invoice_ident = '323200000129'
-    simple_invoice.invoice_issue = datetime.strptime('2019-01-16 10:53:10-05:00', '%Y-%m-%d %H:%M:%S%z')
+    simple_invoice.invoice_issue = datetime.strptime(
+        '2019-01-16 10:53:10-05:00', '%Y-%m-%d %H:%M:%S%z')
     simple_invoice.invoice_supplier.ident = '700085371'
     simple_invoice.invoice_customer.ident = '800199436'
     simple_invoice.add_invoice_line(form.InvoiceLine(
@@ -222,7 +228,8 @@ def test_invoice_cufe(simple_invoice_without_lines):
     assert formatVars[12] == '800199436', "NumAdq"
 
     # ClTec
-    assert formatVars[13] == '693ff6f2a553c3646a063436fd4dd9ded0311471', "ClTec"
+    assert formatVars[13] == (
+        '693ff6f2a553c3646a063436fd4dd9ded0311471'), "ClTec"
 
     # TipoAmbiente
     assert formatVars[14] == '1', "TipoAmbiente"
@@ -323,8 +330,13 @@ def test_debit_note_cude(simple_debit_note_without_lines):
     assert build_vars['TipoAmb'] == 2
 
     cude_composicion = "".join(cude_extension.formatVars())
-    assert cude_composicion == 'ND10012019-01-1810:58:00-05:0030000.00010.00042400.00030.0032400.0090019726410254102102012'
+    assert cude_composicion == (
+        'ND10012019-01-1810:58:00-05:0030000.00010.00042400.00030.0032400.009'
+        '001'
+        '9726410254102102012')
 
     xml_invoice.add_extension(cude_extension)
     cude = xml_invoice.get_element_text('/fe:DebitNote/cbc:UUID')
-    assert cude == '3fa73a86d57d9341c536afde1f85c4efd9d4591c2c22bce4dfb0e6b0d2e83b8f047a8bde7098292e9d2493e60d1c31da'
+    assert cude == (
+        '3fa73a86d57d9341c536afde1f85c4efd9d4591c2c22bce4dfb0e6b0d2e83b8f047a8'
+        'bde7098292e9d2493e60d1c31da')

tests/test_form.py

@@ -121,7 +121,9 @@ def test_FAU14():
 
     wants = form.Amount(119.0 + 19.0 - 50.0)
 
-    assert inv.invoice_legal_monetary_total.payable_amount == wants, "got %s want %s" % (inv.invoice_legal_monetary_total.payable_amount, wants)
+    assert inv.invoice_legal_monetary_total.payable_amount == wants, (
+        "got %s want %s" % (
+            inv.invoice_legal_monetary_total.payable_amount, wants))
 
 
 def test_invalid_tipo_operacion_nota_debito():

tests/test_form_xml.py

@@ -6,10 +6,7 @@
 """Tests for `facho` package."""
 
 import pytest
-# from datetime import datetime
-import copy
 
-from facho.fe import form
 from facho.fe import form_xml
 # from fixtures import *
 

tests/test_nomina.py

@@ -28,8 +28,12 @@
 #     ))
 
 #     xml = nomina.toFachoXML()
-#     assert xml.get_element_attribute('/nomina:NominaIndividual/Devengados/Basico', 'DiasTrabajados') == '30'
-#     assert xml.get_element_attribute('/nomina:NominaIndividual/Devengados/Basico', 'SueldoTrabajado') == '1000000.00'
+#      assert
+# xml.get_element_attribute('/nomina:NominaIndividual/Devengados/Basico',
+# 'DiasTrabajados') == '30'
+#      assert
+# xml.get_element_attribute('/nomina:NominaIndividual/Devengados/Basico',
+# 'SueldoTrabajado') == '1000000.00'
 
 # def test_adicionar_devengado_transporte():
 #     nomina = fe.nomina.DIANNominaIndividual()
@@ -40,7 +44,9 @@
 
 #     xml = nomina.toFachoXML()
 
-#     assert xml.get_element_attribute('/nomina:NominaIndividual/Devengados/Transporte', 'AuxilioTransporte') == '2000000.0'
+#      assert
+# xml.get_element_attribute('/nomina:NominaIndividual/Devengados/Transporte',
+# 'AuxilioTransporte') == '2000000.0'
 
 # def test_adicionar_devengado_comprobante_total():
 #     nomina = fe.nomina.DIANNominaIndividual()
@@ -58,7 +64,9 @@
 
 #     xml = nomina.toFachoXML()
 
-#     assert xml.get_element_text('/nomina:NominaIndividual/ComprobanteTotal') == '1000000.00'
+#      assert
+# xml.get_element_text('/nomina:NominaIndividual/ComprobanteTotal') ==
+# '1000000.00'
 
 # def test_adicionar_devengado_comprobante_total_cero():
 #     nomina = fe.nomina.DIANNominaIndividual()
@@ -75,7 +83,8 @@
 
 #     xml = nomina.toFachoXML()
 
-#     assert xml.get_element_text('/nomina:NominaIndividual/ComprobanteTotal') == '0.00'
+#      assert
+# xml.get_element_text('/nomina:NominaIndividual/ComprobanteTotal') == '0.00'
 
 # def test_adicionar_devengado_transporte_muchos():
 #     nomina = fe.nomina.DIANNominaIndividual()
@@ -90,7 +99,8 @@
 
 #     xml = nomina.toFachoXML()
 #     print(xml)
-#     assert xml.get_element_text('/nomina:NominaIndividual/DevengadosTotal') == '5000000.00'
+#      assert xml.get_element_text('/nomina:NominaIndividual/DevengadosTotal')
+# == '5000000.00'
 
 # def test_adicionar_deduccion_salud():
 #     nomina = fe.nomina.DIANNominaIndividual()
@@ -107,7 +117,9 @@
 
 #     xml = nomina.toFachoXML()
 #     print(xml)
-#     assert xml.get_element_text('/nomina:NominaIndividual/DeduccionesTotal') == '1000.00'
+#      assert
+# xml.get_element_text('/nomina:NominaIndividual/DeduccionesTotal') ==
+# '1000.00'
 
 # def test_nomina_obligatorios_segun_anexo_tecnico():
 #     nomina = fe.nomina.DIANNominaIndividual()
@@ -213,26 +225,62 @@
 #     ))
 
 #     xml = nomina.toFachoXML()
-#     expected_cune = 'b8f9b6c24de07ffd92ea5467433a3b69357cfaffa7c19722db94b2e0eca41d057085a54f484b5da15ff585e773b0b0ab'
-#     assert xml.get_element_attribute('/nomina:NominaIndividual/InformacionGeneral', 'CUNE') == expected_cune
-#     assert xml.get_element_attribute('/nomina:NominaIndividual/InformacionGeneral', 'TipoXML') == '102'
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/NumeroSecuenciaXML/@Numero') == 'N00001'
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/NumeroSecuenciaXML/@Consecutivo') == '00001'
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/LugarGeneracionXML/@Pais') == 'CO'
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/LugarGeneracionXML/@DepartamentoEstado') == '05'
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/LugarGeneracionXML/@MunicipioCiudad') == '05001'
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/ProveedorXML/@NIT') == '999999'
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/ProveedorXML/@DV') == '2'
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/ProveedorXML/@SoftwareID') == 'xx'
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/ProveedorXML/@SoftwareSC') is not None
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/CodigoQR') == f"https://catalogo-vpfe.dian.gov.co/document/searchqr?documentkey={expected_cune}"
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/Empleador/@NIT') == '700085371'
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/Trabajador/@NumeroDocumento') == '800199436'
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/Novedad') == 'True'
-#     assert xml.get_element_text_or_attribute('/nomina:NominaIndividual/Novedad/@CUNENov') == 'N0111'
+#      expected_cune =
+# 'b8f9b6c24de07ffd92ea5467433a3b69357cfaffa7c19722db94b2e0eca41d057085a54f484
+# b5da15ff585e773b0b0ab'
+#      assert
+# xml.get_element_attribute('/nomina:NominaIndividual/InformacionGeneral',
+# 'CUNE') == expected_cune
+#      assert
+# xml.get_element_attribute('/nomina:NominaIndividual/InformacionGeneral',
+# 'TipoXML') == '102'
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/NumeroSecuenciaX
+# ML/@Numero') == 'N00001'
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/NumeroSecuenciaX
+# ML/@Consecutivo') == '00001'
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/LugarGeneracionX
+# ML/@Pais') == 'CO'
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/LugarGeneracionX
+# ML/@DepartamentoEstado') == '05'
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/LugarGeneracionX
+# ML/@MunicipioCiudad') == '05001'
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/ProveedorXML/@NI
+# T') == '999999'
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/ProveedorXML/@DV
+# ') == '2'
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/ProveedorXML/@So
+# ftwareID') == 'xx'
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/ProveedorXML/@So
+# ftwareSC') is not None
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/CodigoQR') ==
+# f"https://catalogo-vpfe.dian.gov.co/document/searchqr?documentkey={expected_
+# cune}"
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/Empleador/@NIT')
+# == '700085371'
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/Trabajador/@Nume
+# roDocumento') == '800199436'
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/Novedad') ==
+# 'True'
+#      assert
+# xml.get_element_text_or_attribute('/nomina:NominaIndividual/Novedad/@CUNENov
+# ') == 'N0111'
 
 #     # confirmar el namespace
-#     assert 'xmlns="dian:gov:co:facturaelectronica:NominaIndividual"' in xml.tostring()
+#      assert 'xmlns="dian:gov:co:facturaelectronica:NominaIndividual"' in
+# xml.tostring()
 
 # def test_asignar_pago():
 #     nomina = fe.nomina.DIANNominaIndividual()
@@ -249,11 +297,11 @@
 #     xml.add_extension(signer)
 
 #     print(xml.tostring())
-#     elem = xml.get_element('/nomina:NominaIndividual/ext:UBLExtensions/ext:UBLExtension/ext:ExtensionContent/ds:Signature')
+#      elem =
+# xml.get_element('/nomina:NominaIndividual/ext:UBLExtensions/ext:UBLExtension
+# /ext:ExtensionContent/ds:Signature')
 #     assert elem is not None
 
-
-
 # def test_nomina_devengado_horas_extras_nocturnas():
 #     nomina = fe.nomina.DIANNominaIndividual()
 
@@ -277,7 +325,9 @@
 #     ))
 
 #     xml = nomina.toFachoXML()
-#     extras = xml.get_element('/nomina:NominaIndividual/Devengados/HENs/HEN', multiple=True)
+#      extras =
+# xml.get_element('/nomina:NominaIndividual/Devengados/HENs/HEN',
+# multiple=True)
 #     assert extras[0].get('HoraInicio') == '2021-11-30T19:09:55'
 #     assert extras[0].get('HoraFin') == '2021-11-30T20:09:55'
 #     assert extras[0].get('Cantidad') == '1'
@@ -312,7 +362,9 @@
 #     ))
 
 #     xml = nomina.toFachoXML()
-#     extras = xml.get_element('/nomina:NominaIndividual/Devengados/HRNs/HRN', multiple=True)
+#      extras =
+# xml.get_element('/nomina:NominaIndividual/Devengados/HRNs/HRN',
+# multiple=True)
 #     assert extras[0].get('HoraInicio') == '2021-11-30T19:09:55'
 #     assert extras[0].get('HoraFin') == '2021-11-30T20:09:55'
 #     assert extras[0].get('Cantidad') == '1'
@@ -327,7 +379,9 @@
 # def test_nomina_devengado_horas_extras_diarias_dominicales_y_festivos():
 #     nomina = fe.nomina.DIANNominaIndividual()
 
-#     nomina.adicionar_devengado(fe.nomina.DevengadoHorasExtrasDiariasDominicalesYFestivos(
+#
+# nomina.adicionar_devengado(fe.nomina.DevengadoHorasExtrasDiariasDominicalesY
+# Festivos(
 #         horas_extras=[
 #             fe.nomina.DevengadoHoraExtra(
 #                 hora_inicio='2021-11-30T19:09:55',
@@ -347,7 +401,9 @@
 #     ))
 
 #     xml = nomina.toFachoXML()
-#     extras = xml.get_element('/nomina:NominaIndividual/Devengados/HEDDFs/HEDDF', multiple=True)
+#      extras =
+# xml.get_element('/nomina:NominaIndividual/Devengados/HEDDFs/HEDDF',
+# multiple=True)
 #     assert extras[0].get('HoraInicio') == '2021-11-30T19:09:55'
 #     assert extras[0].get('HoraFin') == '2021-11-30T20:09:55'
 #     assert extras[0].get('Cantidad') == '1'
@@ -362,7 +418,9 @@
 # def test_nomina_devengado_horas_recargo_diarias_dominicales_y_festivos():
 #     nomina = fe.nomina.DIANNominaIndividual()
 
-#     nomina.adicionar_devengado(fe.nomina.DevengadoHorasRecargoDiariasDominicalesYFestivos(
+#
+# nomina.adicionar_devengado(fe.nomina.DevengadoHorasRecargoDiariasDominicales
+# YFestivos(
 #         horas_extras=[
 #             fe.nomina.DevengadoHoraExtra(
 #                 hora_inicio='2021-11-30T19:09:55',
@@ -382,7 +440,9 @@
 #     ))
 
 #     xml = nomina.toFachoXML()
-#     extras = xml.get_element('/nomina:NominaIndividual/Devengados/HRDDFs/HRDDF', multiple=True)
+#      extras =
+# xml.get_element('/nomina:NominaIndividual/Devengados/HRDDFs/HRDDF',
+# multiple=True)
 #     assert extras[0].get('HoraInicio') == '2021-11-30T19:09:55'
 #     assert extras[0].get('HoraFin') == '2021-11-30T20:09:55'
 #     assert extras[0].get('Cantidad') == '1'
@@ -398,7 +458,9 @@
 # def test_nomina_devengado_horas_extras_nocturnas_dominicales_y_festivos():
 #     nomina = fe.nomina.DIANNominaIndividual()
 
-#     nomina.adicionar_devengado(fe.nomina.DevengadoHorasExtrasNocturnasDominicalesYFestivos(
+#
+# nomina.adicionar_devengado(fe.nomina.DevengadoHorasExtrasNocturnasDominicale
+# sYFestivos(
 #         horas_extras=[
 #             fe.nomina.DevengadoHoraExtra(
 #                 hora_inicio='2021-11-30T19:09:55',
@@ -418,7 +480,9 @@
 #     ))
 
 #     xml = nomina.toFachoXML()
-#     extras = xml.get_element('/nomina:NominaIndividual/Devengados/HENDFs/HENDF', multiple=True)
+#      extras =
+# xml.get_element('/nomina:NominaIndividual/Devengados/HENDFs/HENDF',
+# multiple=True)
 #     assert extras[0].get('HoraInicio') == '2021-11-30T19:09:55'
 #     assert extras[0].get('HoraFin') == '2021-11-30T20:09:55'
 #     assert extras[0].get('Cantidad') == '1'
@@ -433,7 +497,9 @@
 # def test_nomina_devengado_horas_recargo_nocturno_dominicales_y_festivos():
 #     nomina = fe.nomina.DIANNominaIndividual()
 
-#     nomina.adicionar_devengado(fe.nomina.DevengadoHorasRecargoNocturnoDominicalesYFestivos(
+#
+# nomina.adicionar_devengado(fe.nomina.DevengadoHorasRecargoNocturnoDominicale
+# sYFestivos(
 #         horas_extras=[
 #             fe.nomina.DevengadoHoraExtra(
 #                 hora_inicio='2021-11-30T19:09:55',
@@ -453,7 +519,9 @@
 #     ))
 
 #     xml = nomina.toFachoXML()
-#     extras = xml.get_element('/nomina:NominaIndividual/Devengados/HRNDFs/HRNDF', multiple=True)
+#      extras =
+# xml.get_element('/nomina:NominaIndividual/Devengados/HRNDFs/HRNDF',
+# multiple=True)
 #     assert extras[0].get('HoraInicio') == '2021-11-30T19:09:55'
 #     assert extras[0].get('HoraFin') == '2021-11-30T20:09:55'
 #     assert extras[0].get('Cantidad') == '1'

tests/test_nomina_ajuste.py

@@ -101,7 +101,9 @@
 
 #     xml = nomina.toFachoXML()
 
-#     assert xml.get_element_attribute('/nominaajuste:NominaIndividualDeAjuste/Reemplazar/InformacionGeneral', 'TipoXML') == '103'
+#      assert
+# xml.get_element_attribute('/nominaajuste:NominaIndividualDeAjuste/Reemplazar
+# /InformacionGeneral', 'TipoXML') == '103'
 
 
 # def test_adicionar_reemplazar_devengado_comprobante_total():
@@ -119,13 +121,17 @@
 
 #     xml = nomina.toFachoXML()
 
-#     assert xml.get_element_text('/nominaajuste:NominaIndividualDeAjuste/Reemplazar/ComprobanteTotal') == '1000000.00'
+#      assert
+# xml.get_element_text('/nominaajuste:NominaIndividualDeAjuste/Reemplazar/Comp
+# robanteTotal') == '1000000.00'
 
 
 # def test_adicionar_reemplazar_asignar_predecesor():
 #     nomina = fe.nomina.DIANNominaIndividualDeAjuste.Reemplazar()
 
-#     nomina.asignar_predecesor(fe.nomina.DIANNominaIndividualDeAjuste.Reemplazar.Predecesor(
+#
+# nomina.asignar_predecesor(fe.nomina.DIANNominaIndividualDeAjuste.Reemplazar.
+# Predecesor(
 #         numero = '123456',
 #         cune = 'ABC123456',
 #         fecha_generacion = '2021-11-16'
@@ -133,15 +139,23 @@
 
 #     xml = nomina.toFachoXML()
 #     print(xml.tostring())
-#     assert xml.get_element_text_or_attribute('/nominaajuste:NominaIndividualDeAjuste/Reemplazar/ReemplazandoPredecesor/@NumeroPred') == '123456'
-#     assert xml.get_element_text_or_attribute('/nominaajuste:NominaIndividualDeAjuste/Reemplazar/ReemplazandoPredecesor/@CUNEPred') == 'ABC123456'
-#     assert xml.get_element_text_or_attribute('/nominaajuste:NominaIndividualDeAjuste/Reemplazar/ReemplazandoPredecesor/@FechaGenPred') == '2021-11-16'
+#      assert
+# xml.get_element_text_or_attribute('/nominaajuste:NominaIndividualDeAjuste/Re
+# emplazar/ReemplazandoPredecesor/@NumeroPred') == '123456'
+#      assert
+# xml.get_element_text_or_attribute('/nominaajuste:NominaIndividualDeAjuste/Re
+# emplazar/ReemplazandoPredecesor/@CUNEPred') == 'ABC123456'
+#      assert
+# xml.get_element_text_or_attribute('/nominaajuste:NominaIndividualDeAjuste/Re
+# emplazar/ReemplazandoPredecesor/@FechaGenPred') == '2021-11-16'
 
 
 # def test_adicionar_reemplazar_eliminar_predecesor_opcional():
 #     nomina = fe.nomina.DIANNominaIndividualDeAjuste.Reemplazar()
 
-#     nomina.asignar_predecesor(fe.nomina.DIANNominaIndividualDeAjuste.Reemplazar.Predecesor(
+#
+# nomina.asignar_predecesor(fe.nomina.DIANNominaIndividualDeAjuste.Reemplazar.
+# Predecesor(
 #         numero = '123456',
 #         cune = 'ABC123456',
 #         fecha_generacion = '2021-11-16'
@@ -150,13 +164,19 @@
 #     xml = nomina.toFachoXML()
 #     print(xml.tostring())
 
-#     assert xml.get_element('/nominaajuste:NominaIndividualDeAjuste/Reemplazar/ReemplazandoPredecesor') is not None
-#     assert xml.get_element('/nominaajuste:NominaIndividualDeAjuste/Eliminar/EliminandoPredecesor') is None
+#      assert
+# xml.get_element('/nominaajuste:NominaIndividualDeAjuste/Reemplazar/Reemplaza
+# ndoPredecesor') is not None
+#      assert
+# xml.get_element('/nominaajuste:NominaIndividualDeAjuste/Eliminar/EliminandoP
+# redecesor') is None
 
 # def test_adicionar_eliminar_reemplazar_predecesor_opcional():
 #     nomina = fe.nomina.DIANNominaIndividualDeAjuste.Eliminar()
 
-#     nomina.asignar_predecesor(fe.nomina.DIANNominaIndividualDeAjuste.Eliminar.Predecesor(
+#
+# nomina.asignar_predecesor(fe.nomina.DIANNominaIndividualDeAjuste.Eliminar.Pr
+# edecesor(
 #         numero = '123456',
 #         cune = 'ABC123456',
 #         fecha_generacion = '2021-11-16'
@@ -164,8 +184,12 @@
 
 #     xml = nomina.toFachoXML()
 #     print(xml.tostring())
-#     assert xml.get_element('/nominaajuste:NominaIndividualDeAjuste/Eliminar/EliminandoPredecesor') is not None
-#     assert xml.get_element('/nominaajuste:NominaIndividualDeAjuste/Reemplazar/ReemplazandoPredecesor') is None
+#      assert
+# xml.get_element('/nominaajuste:NominaIndividualDeAjuste/Eliminar/EliminandoP
+# redecesor') is not None
+#      assert
+# xml.get_element('/nominaajuste:NominaIndividualDeAjuste/Reemplazar/Reemplaza
+# ndoPredecesor') is None
 
 # def test_adicionar_eliminar_devengado_comprobante_total():
 #     nomina = fe.nomina.DIANNominaIndividualDeAjuste.Eliminar()
@@ -182,12 +206,16 @@
 
 #     xml = nomina.toFachoXML()
 
-#     assert xml.get_element_text('/nominaajuste:NominaIndividualDeAjuste/Eliminar/ComprobanteTotal') == '1000000.00'
+#      assert
+# xml.get_element_text('/nominaajuste:NominaIndividualDeAjuste/Eliminar/Compro
+# banteTotal') == '1000000.00'
 
 # def test_adicionar_eliminar_asignar_predecesor():
 #     nomina = fe.nomina.DIANNominaIndividualDeAjuste.Eliminar()
 
-#     nomina.asignar_predecesor(fe.nomina.DIANNominaIndividualDeAjuste.Eliminar.Predecesor(
+#
+# nomina.asignar_predecesor(fe.nomina.DIANNominaIndividualDeAjuste.Eliminar.Pr
+# edecesor(
 #         numero = '123456',
 #         cune = 'ABC123456',
 #         fecha_generacion = '2021-11-16'
@@ -195,9 +223,15 @@
 
 #     xml = nomina.toFachoXML()
 #     print(xml.tostring())
-#     assert xml.get_element_text_or_attribute('/nominaajuste:NominaIndividualDeAjuste/Eliminar/EliminandoPredecesor/@NumeroPred') == '123456'
-#     assert xml.get_element_text_or_attribute('/nominaajuste:NominaIndividualDeAjuste/Eliminar/EliminandoPredecesor/@CUNEPred') == 'ABC123456'
-#     assert xml.get_element_text_or_attribute('/nominaajuste:NominaIndividualDeAjuste/Eliminar/EliminandoPredecesor/@FechaGenPred') == '2021-11-16'
+#      assert
+# xml.get_element_text_or_attribute('/nominaajuste:NominaIndividualDeAjuste/El
+# iminar/EliminandoPredecesor/@NumeroPred') == '123456'
+#      assert
+# xml.get_element_text_or_attribute('/nominaajuste:NominaIndividualDeAjuste/El
+# iminar/EliminandoPredecesor/@CUNEPred') == 'ABC123456'
+#      assert
+# xml.get_element_text_or_attribute('/nominaajuste:NominaIndividualDeAjuste/El
+# iminar/EliminandoPredecesor/@FechaGenPred') == '2021-11-16'
 
 # def test_nomina_devengado_horas_extras_diarias():
 #     nomina = fe.nomina.DIANNominaIndividual()

tox.ini

@@ -1,26 +1,18 @@
 [tox]
-envlist = py39, py310, py311, py312, flake8
+envlist = py310, py311, py312, py313, flake8
 
-[travis]
-python =
-    3.9: py39
-    3.10: py310
-    3.11: py311
-    3.12: py312
+[flake8]
+exclude = docs, build, venv, .tox, .git, .pytest_cache, *.egg-info
 
 [testenv:flake8]
-basepython = python
-deps = flake8
-commands = flake8 facho
+basepython = python3.13
+deps = flake8>=7.0.0
+commands = flake8 facho tests examples
 
 [testenv]
 setenv =
     PYTHONPATH = {toxinidir}
-deps =
-    -r{toxinidir}/requirements_dev.txt
-; If you want to make tox run the tests with the same versions, create a
-; requirements.txt with the pinned versions and uncomment the following line:
-;     -r{toxinidir}/requirements.txt
+deps = pytest>=9.1.1,<10.0.0
 commands =
     pip install -U pip
     py.test --basetemp={envtmpdir}