fixed the hmac signature verification

Author: E-wave112

setup.py

@@ -12,8 +12,9 @@
 
 setup(
     name="pythepeer",
-    version="1.0.2",
-    description="python sdk for interacting with thepeer payment processing infrastructure",
+    version="1.0.3",
+    description="official python sdk for interacting with thepeer payment processing \
+    infrastructure",
     author="Osagie Iyayi",
     packages=["thepeer"],
     author_email="[email protected]",

thepeer/main.py

@@ -1,5 +1,5 @@
-import hashlib
 import hmac
+import hashlib
 import json  # type: ignore
 import os
 import sys
@@ -31,9 +31,8 @@ def validate_signature(self, data, signature):
         Returns:
             _type_: boolean if the signature is valid or not
         """
-        return (
-            signature == hmac.new(self.secret.encode(), data.encode(), hashlib.sha1()).hexdigest()
-        )
+        SHA1 = hashlib.sha1
+        return signature == hmac.new(self.secret.encode(), data.encode(), SHA1).hexdigest()
 
     def index_user(self, name, identifier, email):
 

thepeer/tests/thepeer_test.py

@@ -5,6 +5,15 @@
 
 
 class ThePeerInitMethods(unittest.TestCase):
+    def test_validate_signature(self):
+        self.assertEqual.__self__.maxDiff = None
+        self.assertEqual(
+            thepeer_test_suites.validate_signature(
+                "I love the peer", "27154e4751dc49d1b40281b18deecd4fd2392e43"
+            ),
+            True,
+        )
+
     def test_index_user(self):
         self.assertEqual.__self__.maxDiff = None
         self.assertEqual(
@@ -32,7 +41,7 @@ def test_view_user(self):
         )
 
     def test_all_users(self):
-
+        self.assertEqual.__self__.maxDiff = None
         self.assertEqual(
             thepeer_test_suites.all_users(),
             {
@@ -63,13 +72,64 @@ def test_all_users(self):
         )
 
     def test_user_links(self):
+        self.assertEqual.__self__.maxDiff = None
         self.assertEqual(
             thepeer_test_suites.get_user_links("3bbb0fbf-82fa-48a0-80eb-d2c0338fe7dd"),
             {"links": []},
         )
 
     def test_single_link(self):
+        self.assertEqual.__self__.maxDiff = None
         self.assertEqual(
             thepeer_test_suites.get_single_link("3bbb0fbf-82fa-48a0-80eb-d2c0338fe7dd"),
             {"message": "link not found"},
         )
+
+    def test_charge_link(self):
+        self.assertEqual.__self__.maxDiff = None
+        self.assertEqual(
+            thepeer_test_suites.charge_link(
+                "3bbb0fbf-82fa-48a0-80eb-d2c0338fe7dd", 2000, "try thepeer"
+            ),
+            {
+                "message": "The given data was invalid.",
+                "errors": {"amount": ["The amount must be at least 10000."]},
+            },
+        )
+
+    def test_charge_link_2(self):
+        self.assertEqual.__self__.maxDiff = None
+        self.assertEqual(
+            thepeer_test_suites.charge_link(
+                "3bbb0fbf-82fa-48a0-80eb-d2c0338fe7dd", 10000, "try thepeer"
+            ),
+            {"message": "link not found"},
+        )
+
+    def test_authorize_direct_charge(self):
+        self.assertEqual.__self__.maxDiff = None
+        self.assertEqual(
+            thepeer_test_suites.authorize_direct_charge(
+                "3bbb0fbf-82fa-48a0-80eb-d2c0338fe7dd", "charge"
+            ),
+            {
+                "message": "The given data was invalid.",
+                "errors": {"event": ["The selected event is invalid."]},
+            },
+        )
+
+    def test_transaction_detail(self):
+        self.assertEqual.__self__.maxDiff = None
+        self.assertEqual(
+            thepeer_test_suites.get_transaction_detail("3bbb0fbf-82fa-48a0-80eb-d2c0338fe7dd"),
+            {"message": "transaction not found"},
+        )
+
+    def test_refund_transaction(self):
+        self.assertEqual.__self__.maxDiff = None
+        self.assertEqual(
+            thepeer_test_suites.refund_transaction(
+                "3bbb0fbf-82fa-48a0-80eb-d2c0338fe7dd", "possibly fraudulent"
+            ),
+            {"message": "this feature is only available on live mode"},
+        )