Add first tests.

Author: Natim

.pre-commit-config.yaml

@@ -29,3 +29,8 @@ repos:
       - id: flake8
         args: [--max-line-length=99]
         language_version: python3.7
+
+  - repo: https://github.com/pre-commit/mirrors-mypy
+    rev: 'v0.761'
+    hooks:
+    -   id: mypy

.travis.yml

@@ -0,0 +1,13 @@
+dist: bionic
+language: python
+cache: pip
+install: pip install tox tox-travis
+script: tox -v
+python:
+- 3.7
+- 3.8
+matrix:
+  fast_finish: true
+  include:
+  - python: 3.8
+    env: TOXENV=flake8

CHANGELOG.md

@@ -7,4 +7,3 @@
 - Handle the refund endpoint
 - Handle pagination for orders
 - Handle the send-sms API for payments.
-

MANIFEST.in

@@ -0,0 +1 @@
+include README.md CHANGELOG.md tox.ini Makefile LICENSE

Makefile

@@ -40,10 +40,13 @@ black: install-dev
 	$(VENV)/bin/black setup.py alma
 
 isort: install-dev
-	$(VENV)/bin/isort --recursive setup.py alma
+	$(VENV)/bin/isort --recursive setup.py alma tests
 
 flake8: install-dev
-	$(VENV)/bin/flake8 setup.py alma
+	$(VENV)/bin/flake8 setup.py alma tests
+
+mypy: install-dev
+	$(VENV)/bin/mypy --ignore-missing-imports --scripts-are-modules alma
 
 build-requirements:
 	$(VIRTUALENV) $(TEMPDIR)

alma/__init__.py

@@ -1,9 +1,5 @@
-# flake8: noqa
-
-import pkg_resources
-
-from . import endpoints, entities
 from .api_modes import ApiModes
 from .client import Client
+from .version import __version__
 
-__version__ = pkg_resources.get_distribution(__package__).version
+__all__ = ["Client", "ApiModes", "__version__"]

alma/client.py

@@ -1,10 +1,10 @@
 import logging
 import platform
 
-from . import __version__ as alma_version
 from . import endpoints
 from .api_modes import ApiModes
 from .context import Context
+from .version import __version__ as alma_version
 
 
 class Client:

alma/endpoints/orders.py

@@ -13,10 +13,10 @@ def update(self, order_id, data):
         return Order(response.json)
 
     def fetch_all(self, limit: int = 20, starting_after: str = None, **filters):
-        args = {"limit": limit}
+        args = {"limit": str(limit)}
 
         if starting_after:
-            args["starting_after"] = starting_after
+            args["starting_after"] = str(starting_after)
 
         if filters:
             for attribute, value in filters.items():

alma/endpoints/payments.py

@@ -19,9 +19,9 @@ def create(self, data):
         return Payment(response.json)
 
     def fetch_all(self, limit: int = 20, states: list = None, starting_after: str = None):
-        args = {"limit": limit}
+        args = {"limit": str(limit)}
         if starting_after:
-            args["starting_after"] = starting_after
+            args["starting_after"] = str(starting_after)
         if states:
             args["state"] = ",".join(states)
 
@@ -55,7 +55,7 @@ def add_orders_to(self, payment_id, order_data: Union[List[dict], dict]) -> List
                            or a list of such dicts for several
         :return: List of Order instances that were added to the payment
         """
-        if type(order_data) is dict:
+        if isinstance(order_data, dict):
             order_data = [order_data]
 
         response = (
@@ -75,7 +75,7 @@ def set_orders_for(self, payment_id, order_data: Union[List[dict], dict]) -> Lis
                            or a list of such dicts for several
         :return: List of Order instances that were added to the payment
         """
-        if type(order_data) is dict:
+        if isinstance(order_data, dict):
             order_data = [order_data]
 
         response = (

alma/entities/eligibility.py

@@ -1,4 +1,4 @@
-from . import Base
+from .base import Base
 
 
 class Eligibility(Base):