Added requirements,travis and improved code quality

Author: ppatidar2021

.travis.yml

@@ -0,0 +1,10 @@
+language: python
+python:
+  - "3.7"
+  - "3.8"
+  - "3.9"
+  - "3.9-dev"
+  - "nightly"  # nightly build
+# command to install dependencies
+install:
+  - pip install -r requirements.txt

docs/GUIDE.md GUIDE.mddocs/GUIDE.md renamed to GUIDE.md

@@ -0,0 +1,3 @@
+ecdsa==0.17.0
+requests==2.27.1
+

requirements.txt

@@ -2,6 +2,7 @@
 Setup file
 """
 import setuptools
+
 setuptools.setup(
     name="bitpay",
     package_dir={"": "src"},
@@ -16,16 +17,16 @@
     python_requires=">=3.7",
     install_requires=["requests", "ecdsa"],
     classifiers=[
-        'Programming Language :: Python :: 3.7, 3.8, 3.9, 3.10',
+        "Programming Language :: Python :: 3.7, 3.8, 3.9, 3.10",
         "Development Status :: 5 - Production/Stable",
-        'Intended Audience :: Developers',
-        'Natural Language :: English',
+        "Intended Audience :: Developers",
+        "Natural Language :: English",
         "Environment :: Web Environment",
         "Intended Audience :: Developers",
         "License :: OSI Approved :: MIT License",
         "Operating System :: OS Independent",
         "Topic :: Software Development :: Libraries :: Python Modules",
-        "Topic :: Office/Business :: Financial"
+        "Topic :: Office/Business :: Financial",
     ],
     long_description="""\
         Python Library for integrating with BitPay
@@ -40,5 +41,5 @@
         
         This version requires only Python 3.8.
         """,
-    long_description_content_type='text/markdown'
+    long_description_content_type="text/markdown",
 )

setup.py

@@ -11,7 +11,7 @@
 from src.bitpay.exceptions.bitpay_exception import BitPayException
 
 # Will be set to Test otherwise
-private_key_name = 'private_key.pem'  # Add here the name for your Private key
+private_key_name = "private_key.pem"  # Add here the name for your Private key
 private_key_path = os.path.join(os.path.abspath(os.curdir), private_key_name)
 plain_private_key = None
 proxy = None
@@ -25,12 +25,12 @@ def select_env():
     global environment
     try:
         print("Select target environment: ")
-        target_environment = input('Press T for testing or P for production: \n')
+        target_environment = input("Press T for testing or P for production: \n")
 
-        if target_environment.lower() == 't':
-            environment = 'Test'
-        elif target_environment.lower() == 'p':
-            environment = 'Prod'
+        if target_environment.lower() == "t":
+            environment = "Test"
+        elif target_environment.lower() == "p":
+            environment = "Prod"
         else:
             select_env()
 
@@ -42,16 +42,18 @@ def select_env():
 
 def set_environment(env):
     global api_url
-    if env == 'Test':
-        api_url = 'https://test.bitpay.com'
+    if env == "Test":
+        api_url = "https://test.bitpay.com"
     else:
-        api_url = 'https://bitpay.com'
+        api_url = "https://bitpay.com"
 
 
 def select_create_key():
     try:
-        input_value = input('Press enter to generate a brand new key or enter your private key location:')
-        if input_value == '':
+        input_value = input(
+            "Press enter to generate a brand new key or enter your private key location:"
+        )
+        if input_value == "":
             create_new_key()
         else:
             load_key()
@@ -71,15 +73,17 @@ def store_key(private_key):
     global plain_private_key, private_key_path
     try:
         print("Select the way you want to store your private key:")
-        input_value = input("Press F for storing in a pem file or T for plain text in your config file: ")
+        input_value = input(
+            "Press F for storing in a pem file or T for plain text in your config file: "
+        )
 
-        if input_value.lower() == 'f':
+        if input_value.lower() == "f":
             with open(str(private_key_path), "wb") as f:
                 f.write(private_key.encode())
             plain_private_key = None
             print("Private key saved at path:", private_key_path)
             select_tokens(private_key)
-        elif input_value.lower() == 't':
+        elif input_value.lower() == "t":
             plain_private_key = private_key
             private_key_path = None
             print("Saving private key... \n")
@@ -93,8 +97,8 @@ def store_key(private_key):
 def select_tokens(private_key):
     try:
         print("Select the tokens that you would like to request:")
-        input_value = input('Press M for merchant, P for payout, or B for both: \n')
-        if input_value.lower() in ['m', 'p', 'b']:
+        input_value = input("Press M for merchant, P for payout, or B for both: \n")
+        if input_value.lower() in ["m", "p", "b"]:
             print("Requesting Tokens... \n")
             request_tokens(input_value.lower(), private_key)
         else:
@@ -112,27 +116,37 @@ def request_tokens(token_type, private_key):
         url = "%s/tokens" % api_url
         headers = {"content-type": "application/json", "X-accept-version": "2.0.0"}
 
-        if token_type in ['m', 'b']:
+        if token_type in ["m", "b"]:
             print("Requesting Merchant token... \n")
-            facade = 'merchant'
-            payload = {'id': sin, 'facade': facade}
+            facade = "merchant"
+            payload = {"id": sin, "facade": facade}
 
-            response = requests.post(url, verify=True, data=json.dumps(payload), headers=headers)
+            response = requests.post(
+                url, verify=True, data=json.dumps(payload), headers=headers
+            )
             if response.ok:
-                merchant_token = response.json()['data'][0]['token']
-                print("Merchant Token: ", response.json()['data'][0]['token'])
-                print("Merchant Token Pairing Code: ", response.json()['data'][0]['pairingCode'] + "\n")
-
-        if token_type in ['p', 'b']:
+                merchant_token = response.json()["data"][0]["token"]
+                print("Merchant Token: ", response.json()["data"][0]["token"])
+                print(
+                    "Merchant Token Pairing Code: ",
+                    response.json()["data"][0]["pairingCode"] + "\n",
+                )
+
+        if token_type in ["p", "b"]:
             print("Requesting Payout token... \n")
-            facade = 'payout'
-            payload = {'id': sin, 'facade': facade}
+            facade = "payout"
+            payload = {"id": sin, "facade": facade}
 
-            response = requests.post(url, verify=True, data=json.dumps(payload), headers=headers)
+            response = requests.post(
+                url, verify=True, data=json.dumps(payload), headers=headers
+            )
             if response.ok:
-                payout_token = response.json()['data'][0]['token']
-                print("Payout Token: ", response.json()['data'][0]['token'])
-                print("Payout Token Pairing Code: ", response.json()['data'][0]['pairingCode'] + "\n")
+                payout_token = response.json()["data"][0]["token"]
+                print("Payout Token: ", response.json()["data"][0]["token"])
+                print(
+                    "Payout Token Pairing Code: ",
+                    response.json()["data"][0]["pairingCode"] + "\n",
+                )
 
         update_config_file()
     except BitPayException as exe:
@@ -150,23 +164,30 @@ def update_config_file():
                         "PrivateKey": plain_private_key,
                         "ApiTokens": {
                             "merchant": merchant_token,
-                            "payout": payout_token
+                            "payout": payout_token,
                         },
-                        "proxy": proxy
+                        "proxy": proxy,
                     }
-                }
+                },
             }
         }
 
-        with open(os.path.abspath("bitpay.config.json"), 'w') as outfile:
+        with open(os.path.abspath("bitpay.config.json"), "w") as outfile:
             json.dump(config, outfile)
-            print('Generated configuration file at path: ', os.path.abspath("bitpay.config.json"))
-
-        print('Configuration generated successfully! \n')
-        print("\r\nPlease, copy the above pairing code/s and approve on your BitPay Account at the following link:\r\n")
+            print(
+                "Generated configuration file at path: ",
+                os.path.abspath("bitpay.config.json"),
+            )
+
+        print("Configuration generated successfully! \n")
+        print(
+            "\r\nPlease, copy the above pairing code/s and approve on your BitPay Account at the following link:\r\n"
+        )
         print(f"{api_url}/dashboard/merchant/api-tokens\r\n")
-        print("\r\nOnce you have this Pairing Code/s approved you can move the generated files to a secure location "
-              "and start using the Client.\r\n")
+        print(
+            "\r\nOnce you have this Pairing Code/s approved you can move the generated files to a secure location "
+            "and start using the Client.\r\n"
+        )
     except BitPayException as exe:
         print(exe)
 
@@ -176,6 +197,5 @@ def load_key():
     pass
 
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     select_env()
-

setup/bitpay_setup.py

@@ -17,8 +17,8 @@ def __init__(self, **kwargs):
         for key, value in kwargs.items():
             try:
                 getattr(self, "set_%s" % change_camel_case_to_snake_case(key))(value)
-            except AttributeError as exe:
-                print(exe)
+            except AttributeError:
+                pass
 
     def get_name(self):
         """

src/bitpay/models/Rate/rate.py

@@ -17,13 +17,35 @@ class Rates:
     __rates = None
 
     def __init__(self, rates, b_p, **kwargs):
-        self.__bp = b_p
-        self.__rates = rates
+        self.set_bp(b_p)
+        self.set_rates(rates)
+
         for key, value in kwargs.items():
             try:
                 getattr(self, "set_%s" % change_camel_case_to_snake_case(key))(value)
-            except AttributeError as exe:
-                print(exe)
+            except AttributeError:
+                pass
+
+    def get_bp(self):
+        """
+        Get method for the bp
+        :return: bp
+        """
+        return self.__bp
+
+    def set_bp(self, b_p):
+        """
+        Set method for the b_p
+        :param b_p: b_p
+        """
+        self.__bp = b_p
+
+    def set_rates(self, rates):
+        """
+        Set method for the rates
+        :param rates: rates
+        """
+        self.__rates = rates
 
     def get_rates(self):
         rates = []

src/bitpay/models/Rate/rates.py

@@ -25,8 +25,7 @@ class Bill:
     __city = None
     __state = None
     __zip = None
-    # TODO: Country should not have default value
-    __country = "US"
+    __country = None
     __cc = None
     __delivered = None
     __phone = None
@@ -40,9 +39,9 @@ class Bill:
     __merchant = None
 
     def __init__(self, number=None, currency=None, email=None, **kwargs):
-        self.__number = number
-        self.__currency = currency
-        self.__email = email
+        self.set_number(number)
+        self.set_currency(currency)
+        self.set_email(email)
 
         for key, value in kwargs.items():
             try:
@@ -59,8 +58,8 @@ def __init__(self, number=None, currency=None, email=None, **kwargs):
                     else:
                         value = klass(**value)
                 getattr(self, "set_%s" % change_camel_case_to_snake_case(key))(value)
-            except AttributeError as exe:
-                print(exe)
+            except AttributeError:
+                pass
 
     def get_currency(self):
         """

src/bitpay/models/bill/bill.py

@@ -18,8 +18,8 @@ def __init__(self, **kwargs):
         for key, value in kwargs.items():
             try:
                 getattr(self, "set_%s" % change_camel_case_to_snake_case(key))(value)
-            except AttributeError as exe:
-                print(exe)
+            except AttributeError:
+                pass
 
     def get_id(self):
         """

src/bitpay/models/bill/item.py

@@ -214,15 +214,15 @@ class Currency:
     def is_valid(cls, value):
         try:
             return hasattr(Currency(), value)
-        except Exception as exe:
+        except Exception:
             return False
 
     def __init__(self, **kwargs):
         for key, value in kwargs.items():
             try:
                 getattr(self, "set_%s" % change_camel_case_to_snake_case(key))(value)
-            except AttributeError as e:
-                print(e)
+            except AttributeError:
+                pass
 
     def get_code(self):
         """