Merge pull request #258 from nih-sparc/fix-failing-test

Fix failing test

Author: egauzens

app/main.py

@@ -1699,7 +1699,7 @@ def hubspot_webhook():
 
         base64_hashed_signature = base64.b64encode(hashed_signature).decode('utf-8')
 
-        # Validate the signature
+        # Validate the signature if we are not running a test
         if not hmac.compare_digest(base64_hashed_signature, signature_header):
             logging.error(f'Signature is invalid')
             return jsonify({"error": "Signature is invalid"}), 401

tests/test_api.py

@@ -185,8 +185,8 @@ def test_hubspot_webhook(client):
     # The timestamp must be a Unix epoch time within 5 minutes (300 seconds) of the current time when the webhook request is received.
     valid_timestamp = int(time.time())
     # Concatenate the string as HubSpot does
-    stringified_body = json.dumps(mock_body, separators=(",", ":"))
-    data_to_sign = f'{http_method}{full_url}{stringified_body}{valid_timestamp}'
+    raw_json = json.dumps(mock_body, separators=(",", ":"))
+    data_to_sign = f'{http_method}{full_url}{raw_json}{valid_timestamp}'
 
     # Generate the HMAC SHA256 signature
     signature = hmac.new(
@@ -200,7 +200,7 @@ def test_hubspot_webhook(client):
     # Send a mock POST request
     response = client.post(
         endpoint,
-        json=mock_body,
+        data=raw_json,
         headers={
             "Content-Type": "application/json",
             "X-HubSpot-Signature-Version": "v3",