Se han corregido algunas instancias donde el nombre y URL del proyecto eran incorrectos. Se han corregido varios tests que utilizaban urlencode. Se corrigió documentación de tests. Se corrigió el manejo de errores en ApiClient, y se corrigió el manejo de AssertException en run.py. Se reestructuró ligeramente la carpeta docs: Ahora todo lo relacionado a Empezando está en su carpeta getting-started, y todo lo relacionado a desarrollo está en su carpeta dev.

Author: SantiagoFiol

Makefile

@@ -29,4 +29,4 @@ docs:
 	sphinx-apidoc -o docs contafi && sphinx-build -b html docs docs/_build/html
 
 clean:
-	rm -rf dist bhexpress.egg-info bhexpress/__pycache__ bhexpress/*.pyc
+	rm -rf dist contafi.egg-info contafi/__pycache__ contafi/*.pyc

contafi/api_client/__init__.py

@@ -318,10 +318,11 @@ def __check_and_return_response(self, response):
                     ) or 'Error desconocido.'
                 except json.decoder.JSONDecodeError:
                     message = 'Error al decodificar los datos en JSON: %(response)s' % {
-                        'response': response.text
+                        'response': response.reason
                     }
-                raise ApiException('Error HTTP: %(message)s' % {
-                    'message': message
+                raise ApiException('%(message)s (%(code)s)' % {
+                    'message': message,
+                    'code': response.status_code
                 })
         return response
 

contafi/api_client/client/bhe.py

@@ -58,10 +58,9 @@ def listar(self, filtros = {}):
             dict
         '''
         url = '/bhe/boletas'
-        query = {}
 
         if len(filtros) > 0:
-            query_string = urlencode(query)
+            query_string = urlencode(filtros)
             url += '?%(query)s' % {'url': url, 'query': query_string}
 
         response = self.client.get(url)
@@ -116,10 +115,9 @@ def pdf(self, emisor, numero, filtros = {}):
         url = '/bhe/pdf/%(emisor)s/%(numero)s' % {
             'emisor': emisor, 'numero': numero
         }
-        query = {}
 
         if len(filtros) > 0:
-            query_string = urlencode(query)
+            query_string = urlencode(filtros)
             url += '?%(query)s' % {'url': url, 'query': query_string}
 
         response = self.client.get(url)

contafi/api_client/client/bte.py

@@ -77,10 +77,9 @@ def listar(self, filtros = {}):
             dict
         '''
         url = '/bte/boletas'
-        query = {}
 
         if len(filtros) > 0:
-            query_string = urlencode(query)
+            query_string = urlencode(filtros)
             url += '?%(query)s' % {'url': url, 'query': query_string}
 
         response = self.client.get(url)

contafi/api_client/client/facturacion.py

@@ -80,10 +80,9 @@ def listarVentas(self, filtros = {}):
             dict
         '''
         url = '/dte/ventas'
-        query = {}
 
         if len(filtros) > 0:
-            query_string = urlencode(query)
+            query_string = urlencode(filtros)
             url += '?%(query)s' % {'url': url, 'query': query_string}
 
         response = self.client.get(url)
@@ -108,11 +107,12 @@ def listarCompras(self, estado, filtros):
         :rtype:
             dict
         '''
-        url = '/dte/compras?estado=%(estado)s'
-        query = {}
+        url = '/dte/compras?estado=%(estado)s' % {
+            'estado': estado
+        }
 
         if len(filtros) > 0:
-            query_string = urlencode(query)
+            query_string = urlencode(filtros)
             url += '&%(query)s' % {'url': url, 'query': query_string}
 
         response = self.client.get(url)

contafi/api_client/client/remuneraciones.py

@@ -56,7 +56,7 @@ def listarRemuneraciones(self, periodo = None):
         '''
         url = '/remuneraciones'
 
-        if periodo:
+        if periodo is not None:
             url += '?periodo=%(periodo)s' % {'periodo': periodo}
 
         response = self.client.get(url)

docs/dev.index.rst docs/dev/index.rstdocs/dev.index.rst renamed to docs/dev/index.rst

@@ -4,4 +4,4 @@ Desarrollo
 .. toctree::
     :maxdepth: 1
 
-    dev.unittest
+    unit-test

docs/dev.unittest.rst docs/dev/unit-test.rstdocs/dev.unittest.rst renamed to docs/dev/unit-test.rst

@@ -59,4 +59,4 @@ El siguiente es un ejemplo básico de cómo emitir una BTE utilizando el cliente
     print("\nEmitir BTE ejemplo: ", response, "\n")
 
 .. seealso::
-    Para saber más sobre los parámetros posibles y el cómo consumir las API, referirse a la `documentación de BHExpress. <https://developers.bhexpress.cl/>`_
+    Para saber más sobre los parámetros posibles y el cómo consumir las API, referirse a la `documentación de ContaFi. <https://developers.contafi.cl/>`_

docs/gettingstarted.example.rst docs/getting-started/example.rstdocs/gettingstarted.example.rst renamed to docs/getting-started/example.rst

@@ -0,0 +1,10 @@
+Empezando
+=========
+
+.. toctree::
+    :maxdepth: 1
+
+    legal
+    install
+    setup
+    example