test: add and update tests for webhook and error handling

- Add tests for invalid and missing signatures in webhook requests
- Add tests for method not allowed errors on webhook endpoint
- Update integration.json structure test to match actual response
- Ensure comprehensive test coverage for error handling and webhook processing

Author: CynthiaWahome

tests/__init__.py

@@ -0,0 +1,12 @@
+import unittest
+import os
+from dotenv import load_dotenv
+
+class TestEnvironmentConfig(unittest.TestCase):
+    def setUp(self):
+        load_dotenv()
+        
+    def test_environment_variables_exist(self):
+        """Test that required environment variables are set"""
+        self.assertIsNotNone(os.getenv('MY_GITHUB_SECRET'))
+        self.assertIsNotNone(os.getenv('CHANNEL_ID'))

tests/test_config.py

@@ -0,0 +1,42 @@
+import unittest
+import json
+from app import app
+
+class TestErrorHandling(unittest.TestCase):
+    def setUp(self):
+        self.app = app.test_client()
+
+    def test_invalid_signature(self):
+        """Test webhook request with invalid signature"""
+        payload = {
+            "head_commit": {
+                "message": "test commit",
+                "author": {"name": "Test User"}
+            }
+        }
+        headers = {
+            'X-Hub-Signature-256': 'invalid_signature',
+            'Content-Type': 'application/json'
+        }
+        response = self.app.post(
+            '/github-webhook',
+            data=json.dumps(payload),
+            headers=headers
+        )
+        self.assertEqual(response.status_code, 401)
+        self.assertIn('Invalid signature', response.get_json()['error'])
+
+    def test_missing_signature(self):
+        """Test webhook request without signature header"""
+        response = self.app.post(
+            '/github-webhook',
+            data=json.dumps({}),
+            headers={'Content-Type': 'application/json'}
+        )
+        self.assertEqual(response.status_code, 401)
+        self.assertIn('No signature provided', response.get_json()['error'])
+
+    def test_method_not_allowed(self):
+        """Test GET request to webhook endpoint"""
+        response = self.app.get('/github-webhook')
+        self.assertEqual(response.status_code, 405)

tests/test_error_handling.py

@@ -0,0 +1,25 @@
+import unittest
+from app import app
+
+class TestIntegrationJSON(unittest.TestCase):
+    def setUp(self):
+        self.app = app.test_client()
+    
+    def test_integration_json_structure(self):
+        """Test integration.json returns correct structure"""
+        response = self.app.get('/integration.json')
+        data = response.get_json()
+        
+
+        self.assertEqual(response.status_code, 200)
+        self.assertIn('data', data)
+        self.assertIn('descriptions', data['data'])
+        self.assertIn('settings', data['data'])
+        self.assertIn('target_url', data['data'])
+        
+        # More specific assertions
+        descriptions = data['data']['descriptions']
+        self.assertIn('app_name', descriptions)
+        self.assertIn('app_description', descriptions)
+        self.assertIn('app_logo', descriptions)
+        self.assertIn('app_url', descriptions)

tests/test_intergration_json.py

@@ -0,0 +1,17 @@
+import unittest
+import concurrent.futures
+import requests
+
+class TestLoad(unittest.TestCase):
+    def setUp(self):
+        self.base_url = "https://github-commit-monitor-4a53c549b932.herokuapp.com"
+    
+    def test_concurrent_webhook_requests(self):
+        """Test handling multiple concurrent webhook requests"""
+        def make_request():
+            return requests.get(f"{self.base_url}/integration.json")
+        
+        with concurrent.futures.ThreadPoolExecutor(max_workers=10) as executor:
+            futures = [executor.submit(make_request) for _ in range(10)]
+            for future in concurrent.futures.as_completed(futures):
+                self.assertEqual(future.result().status_code, 200)

tests/test_load.py

@@ -0,0 +1,44 @@
+import unittest
+import json
+from app import app
+import hmac
+import hashlib
+import os
+from dotenv import load_dotenv
+
+class TestWebhook(unittest.TestCase):
+    def setUp(self):
+        self.app = app.test_client()
+        load_dotenv()
+        self.github_secret = os.getenv('MY_GITHUB_SECRET')
+        
+    def generate_signature(self, payload):
+        """Generate GitHub signature for payload"""
+        return 'sha256=' + hmac.new(
+            self.github_secret.encode(),
+            payload.encode(),
+            hashlib.sha256
+        ).hexdigest()
+        
+    def test_valid_webhook_request(self):
+        """Test successful webhook processing"""
+        payload = {
+            "head_commit": {
+                "message": "test: add new feature",
+                "author": {"name": "Test User"},
+            },
+            "repository": {"name": "test-repo"}
+        }
+        
+        headers = {
+            'X-Hub-Signature-256': self.generate_signature(json.dumps(payload)),
+            'Content-Type': 'application/json'
+        }
+        
+        response = self.app.post(
+            '/github-webhook',
+            data=json.dumps(payload),
+            headers=headers
+        )
+        
+        self.assertEqual(response.status_code, 200)