[PW-2394] Remove the app_name (#110)

* Remove the app_name

* Update Adyen/httpclient.py

Co-authored-by: Alessio Zampatti <alessio.zampatti@adyen.com>

* Update Adyen/httpclient.py

Co-authored-by: Alessio Zampatti <alessio.zampatti@adyen.com>

* Update Adyen/httpclient.py

Co-authored-by: Alessio Zampatti <alessio.zampatti@adyen.com>

* remove app_name from test

Co-authored-by: Alessio Zampatti <alessio.zampatti@adyen.com>

Author: AlexandrosMor

Adyen/client.py

@@ -71,7 +71,7 @@ def __init__(self, username=None, password=None, xapikey=None,
                  store_payout_username=None, store_payout_password=None,
                  platform="test", merchant_account=None,
                  merchant_specific_url=None, skin_code=None,
-                 hmac=None, app_name="",
+                 hmac=None,
                  http_force=None, live_endpoint_prefix=None):
         self.username = username
         self.password = password
@@ -86,7 +86,6 @@ def __init__(self, username=None, password=None, xapikey=None,
         self.merchant_account = merchant_account
         self.skin_code = skin_code
         self.psp_list = []
-        self.app_name = app_name
         self.LIB_VERSION = settings.LIB_VERSION
         self.USER_AGENT_SUFFIX = settings.LIB_NAME + "/"
         self.http_init = False
@@ -221,8 +220,7 @@ def call_api(self, request_data, service, action, idempotency=False,
                 succesful.
         """
         if not self.http_init:
-            self.http_client = HTTPClient(self.app_name,
-                                          self.USER_AGENT_SUFFIX,
+            self.http_client = HTTPClient(self.USER_AGENT_SUFFIX,
                                           self.LIB_VERSION,
                                           self.http_force)
             self.http_init = True
@@ -360,8 +358,7 @@ def call_hpp(self, message, action, hmac_key="", **kwargs):
                 :param hmac_key:
         """
         if not self.http_init:
-            self.http_client = HTTPClient(self.app_name,
-                                          self.USER_AGENT_SUFFIX,
+            self.http_client = HTTPClient(self.USER_AGENT_SUFFIX,
                                           self.LIB_VERSION,
                                           self.http_force)
             self.http_init = True
@@ -425,8 +422,7 @@ def call_checkout_api(self, request_data, action, **kwargs):
             action (str): The specific action of the API service to be called
         """
         if not self.http_init:
-            self.http_client = HTTPClient(self.app_name,
-                                          self.USER_AGENT_SUFFIX,
+            self.http_client = HTTPClient(self.USER_AGENT_SUFFIX,
                                           self.LIB_VERSION,
                                           self.http_force)
             self.http_init = True
@@ -497,8 +493,7 @@ def call_checkout_api(self, request_data, action, **kwargs):
     def hpp_payment(self, request_data, action, hmac_key="", **kwargs):
 
         if not self.http_init:
-            self.http_client = HTTPClient(self.app_name,
-                                          self.USER_AGENT_SUFFIX,
+            self.http_client = HTTPClient(self.USER_AGENT_SUFFIX,
                                           self.LIB_VERSION,
                                           self.http_force)
             self.http_init = True

Adyen/httpclient.py

@@ -35,12 +35,9 @@
 
 
 class HTTPClient(object):
-    def __init__(self, app_name, user_agent_suffix,
-                 lib_version, force_request=None):
+    def __init__(self, user_agent_suffix, lib_version, force_request=None):
         # Check if requests already available, default to urllib
-        self.user_agent = app_name + " " + user_agent_suffix + lib_version
-        # In case the app_name is empty
-        self.user_agent = self.user_agent.strip()
+        self.user_agent = user_agent_suffix + lib_version
         if not force_request:
             if requests:
                 self.request = self._requests_post

Adyen/services.py

@@ -7,7 +7,7 @@
 
 class AdyenBase(object):
     def __setattr__(self, attr, value):
-        client_attr = ["username", "password", "platform", "app_name"]
+        client_attr = ["username", "password", "platform"]
         if attr in client_attr:
             if value:
                 self.client[attr] = value

README.md

@@ -47,7 +47,6 @@ ady.payment.client.hmac = "HMAC key for skin code"
 ady.payment.client.platform = "test" # Environment to use the library in.
 ady.payment.client.merchant_account = "merchant account name from CA"
 ady.payment.client.password = "webservice user password"
-ady.payment.client.app_name = "your app name"
 ```
 
 ## Documentation

test/CheckoutTest.py

@@ -14,7 +14,6 @@ class TestCheckout(unittest.TestCase):
     test = BaseTest(adyen)
     client.xapikey = "YourXapikey"
     client.platform = "test"
-    client.app_name = "appname"
 
     def test_payment_methods_success_mocked(self):
         request = {'merchantAccount': "YourMerchantAccount"}

test/CheckoutUtilityTest.py

@@ -15,7 +15,6 @@ class TestCheckoutUtility(unittest.TestCase):
     test = BaseTest(ady)
     client.xapikey = "YourXapikey"
     client.platform = "test"
-    client.app_name = "appname"
 
     def test_origin_keys_success_mocked(self):
         request = {

test/DetermineEndpointTest.py

@@ -15,7 +15,6 @@ class TestDetermineUrl(unittest.TestCase):
     client = adyen.client
     test = BaseTest(adyen)
     client.xapikey = "YourXapikey"
-    client.app_name = "appname"
 
     def test_checkout_api_url_custom(self):
         self.client.live_endpoint_prefix = "1797a841fbb37ca7-AdyenDemo"
@@ -50,6 +49,7 @@ def test_payments_invalid_platform(self):
 
         self.client.platform = "live"
         self.client.live_endpoint_prefix = None
+
         try:
             self.adyen.checkout.payments(request)
         except AdyenEndpointInvalidFormat as error:

test/DirectoryLookupTest.py

@@ -20,7 +20,6 @@ class TestDirectoryLookup(unittest.TestCase):
     client.platform = "test"
     client.hmac = "DFB1EB5485895CFA84146406857104A" \
                   "BB4CBCABDC8AAF103A624C8F6A3EAAB00"
-    client.app_name = "appname"
 
     def test_get_post_parameters(self):
         request = {

test/ModificationTest.py

@@ -14,7 +14,6 @@ class TestModifications(unittest.TestCase):
     client.username = "YourWSUser"
     client.password = "YourWSPassword"
     client.platform = "test"
-    client.app_name = "appname"
 
     def test_capture_success(self):
         request = {}

test/PaymentTest.py

@@ -15,7 +15,6 @@ class TestPayments(unittest.TestCase):
     client.username = "YourWSUser"
     client.password = "YourWSPassword"
     client.platform = "test"
-    client.app_name = "appname"
     client.xapikey = ""
 
     def test_authorise_success_mocked(self):
@@ -208,7 +207,6 @@ class TestPaymentsWithXapiKey(unittest.TestCase):
     client = adyen.client
     test = BaseTest(adyen)
     client.platform = "test"
-    client.app_name = "appname"
     client.username = "YourWSUser"
     client.password = "YourWSPassword"
     client.xapikey = ""