refactoring signature-related tests into their own file, adding new test for patch method

Author: maxkahan

tests/test_client.py

@@ -3,90 +3,6 @@
 from vonage.errors import InvalidAuthenticationTypeError
 
 
-def test_check_signature(dummy_data):
-    params = {
-        "a": "1",
-        "b": "2",
-        "timestamp": "1461605396",
-        "sig": "6af838ef94998832dbfc29020b564830",
-    }
-
-    client = vonage.Client(
-        key=dummy_data.api_key, secret=dummy_data.api_secret, signature_secret="secret"
-    )
-
-    assert client.check_signature(params)
-
-
-def test_signature(client, dummy_data):
-    params = {"a": "1", "b": "2", "timestamp": "1461605396"}
-    client = vonage.Client(
-        key=dummy_data.api_key, secret=dummy_data.api_secret, signature_secret="secret"
-    )
-    assert client.signature(params) == "6af838ef94998832dbfc29020b564830"
-
-
-def test_signature_adds_timestamp(dummy_data):
-    params = {"a=7": "1", "b": "2 & 5"}
-
-    client = vonage.Client(
-        key=dummy_data.api_key, secret=dummy_data.api_secret, signature_secret="secret"
-    )
-
-    client.signature(params)
-    assert params["timestamp"] is not None
-
-
-def test_signature_md5(dummy_data):
-    params = {"a": "1", "b": "2", "timestamp": "1461605396"}
-    client = vonage.Client(
-        key=dummy_data.api_key,
-        secret=dummy_data.api_secret,
-        signature_secret=dummy_data.signature_secret,
-        signature_method="md5",
-    )
-    assert client.signature(params) == "c15c21ced558c93a226c305f58f902f2"
-
-
-def test_signature_sha1(dummy_data):
-    params = {"a": "1", "b": "2", "timestamp": "1461605396"}
-    client = vonage.Client(
-        key=dummy_data.api_key,
-        secret=dummy_data.api_secret,
-        signature_secret=dummy_data.signature_secret,
-        signature_method="sha1",
-    )
-    assert client.signature(params) == "3e19a4e6880fdc2c1426bfd0587c98b9532f0210"
-
-
-def test_signature_sha256(dummy_data):
-    params = {"a": "1", "b": "2", "timestamp": "1461605396"}
-    client = vonage.Client(
-        key=dummy_data.api_key,
-        secret=dummy_data.api_secret,
-        signature_secret=dummy_data.signature_secret,
-        signature_method="sha256",
-    )
-    assert (
-        client.signature(params)
-        == "a321e824b9b816be7c3f28859a31749a098713d39f613c80d455bbaffae1cd24"
-    )
-
-
-def test_signature_sha512(dummy_data):
-    params = {"a": "1", "b": "2", "timestamp": "1461605396"}
-    client = vonage.Client(
-        key=dummy_data.api_key,
-        secret=dummy_data.api_secret,
-        signature_secret=dummy_data.signature_secret,
-        signature_method="sha512",
-    )
-    assert (
-        client.signature(params)
-        == "812a18f76680fa0fe1b8bd9ee1625466ceb1bd96242e4d050d2cfd9a7b40166c63ed26ec9702168781b6edcf1633db8ff95af9341701004eec3fcf9550572ee8"
-    )
-
-
 def test_client_doesnt_require_api_key(dummy_data):
     client = vonage.Client(application_id="myid", private_key=dummy_data.private_key)
     assert client is not None
@@ -117,4 +33,3 @@ def test_timeout_is_set_on_client_calls(dummy_data):
 
     assert len(responses.calls) == 1
     assert responses.calls[0].request.req_kwargs["timeout"] == 1
-

tests/test_rest_calls.py

@@ -80,3 +80,20 @@ def test_delete_with_header_auth(client, dummy_data):
     assert isinstance(response, dict)
     assert request_user_agent() == dummy_data.user_agent
     assert_basic_auth()
+
+
+@responses.activate
+def test_patch(client, dummy_data):
+    stub(responses.PATCH, "https://api.nexmo.com/v1/applications")
+    host = "api.nexmo.com"
+    request_uri = "/v1/applications"
+    params = {"aaa": "xxx", "bbb": "yyy"}
+    response = client.patch(host, request_uri, params=params, auth_type='jwt')
+    assert request_headers()['Content-Type'] == 'application/json'
+    assert re.search(b'^Bearer ', request_headers()['Authorization']) is not None
+    assert isinstance(response, dict)
+    assert request_user_agent() == dummy_data.user_agent
+    assert b"aaa" in request_body()
+    assert b"xxx" in request_body()
+    assert b"bbb" in request_body()
+    assert b"yyy" in request_body()