Fix configuration parameter. Improve test (#166)

checkout_sdk/oauth_sdk.py

@@ -40,8 +40,8 @@ def build(self):
                                                      scopes=self._scopes,
                                                      authorization_uri=self._authorization_uri),
                 environment=self._environment,
-                environment_subdomain=self._environment_subdomain,
-                http_client=self._http_client)
+                http_client=self._http_client,
+                environment_subdomain=self._environment_subdomain)
         else:
             configuration = CheckoutConfiguration(
                 credentials=OAuthSdkCredentials.init(http_client=self._http_client,

checkout_sdk/previous/previous_sdk.py

@@ -34,8 +34,8 @@ def build(self):
             configuration = CheckoutConfiguration(
                 credentials=PreviousKeysSdkCredentials(secret_key=self._secret_key, public_key=self._public_key),
                 environment=self._environment,
-                environment_subdomain=self._environment_subdomain,
-                http_client=self._http_client)
+                http_client=self._http_client,
+                environment_subdomain=self._environment_subdomain)
         else:
             configuration = CheckoutConfiguration(
                 credentials=PreviousKeysSdkCredentials(secret_key=self._secret_key, public_key=self._public_key),

tests/checkout_configuration_test.py

@@ -24,8 +24,11 @@ def test_should_create_configuration():
         http_client=http_client
     )
 
-    assert Environment.sandbox().base_uri == configuration.environment.base_uri
-    assert "https://api.sandbox.checkout.com/" == configuration.environment.base_uri
+    assert configuration.credentials == credentials
+    assert configuration.environment.base_uri == Environment.sandbox().base_uri
+    assert configuration.environment.base_uri == "https://api.sandbox.checkout.com/"
+    assert configuration.http_client == http_client
+    assert configuration.environment_subdomain is None
 
 
 @pytest.mark.parametrize(
@@ -53,7 +56,10 @@ def test_should_create_configuration_with_subdomain(subdomain, expected_url):
         environment_subdomain=environment_subdomain
     )
 
-    assert expected_url == configuration.environment_subdomain.base_uri
+    assert configuration.credentials == credentials
+    assert configuration.environment.base_uri == Environment.sandbox().base_uri
+    assert configuration.http_client == http_client
+    assert configuration.environment_subdomain.base_uri == expected_url
 
 
 @pytest.mark.parametrize(
@@ -81,4 +87,7 @@ def test_should_create_configuration_with_bad_subdomain(subdomain, expected_url)
         environment_subdomain=environment_subdomain
     )
 
-    assert expected_url == configuration.environment_subdomain.base_uri
+    assert configuration.credentials == credentials
+    assert configuration.environment.base_uri == Environment.sandbox().base_uri
+    assert configuration.http_client == http_client
+    assert configuration.environment_subdomain.base_uri == expected_url