Merge pull request #367 from mollie/342-testmode

Pass True instead of 'true' for testmode when posting using oauth_client

Author: Thomasvdh92

mollie/api/client.py

@@ -201,7 +201,7 @@ def _get_testmode(self, data, params, http_method):
             if http_method == "GET":
                 params["testmode"] = params.get("testmode") or "true"
             elif not data or "testmode" not in data:
-                data["testmode"] = params.get("testmode") or "true"
+                data["testmode"] = params.get("testmode") or True
 
                 # Delete from the params since it's not a valid parameter when the request is not GET
                 params.pop("testmode", None)

tests/test_api_client.py

@@ -536,7 +536,7 @@ def test_update_customer_bad_request(client, response):
     "testmode,params,http_method,expected",
     [
         (True, {}, "GET", ({}, {"testmode": "true"})),
-        (True, {}, "POST", ({"testmode": "true"}, {})),
+        (True, {}, "POST", ({"testmode": True}, {})),
         (False, {"testmode": "true"}, "GET", ({}, {"testmode": "true"})),
         (False, {"testmode": "true"}, "POST", ({"testmode": "true"}, {})),
         (False, {"invalid": "something"}, "POST", ({}, {"invalid": "something"})),