lint fixes

Author: shriyanshagnihotri

.github/workflows/main-test.yml

@@ -22,6 +22,7 @@ jobs:
       AGENTS_LLM_CONFIG_FILE_REF_KEY: ${{ vars.AGENTS_LLM_CONFIG_FILE_REF_KEY }}
       AGENTS_LLM_CONFIG_FILE: ${{ vars.AGENTS_LLM_CONFIG_FILE }}
       GEO_API_KEY: ${{ secrets.GEO_API_KEY }}
+      PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION: python 
       ENABLE_TELEMETRY: 0
       AUTO_MODE: 1
     steps:

tests/test_certificate_errors.py

@@ -1,38 +1,37 @@
-import pytest
 import os
-from unittest.mock import patch, AsyncMock
+import pytest
+from typing import Generator
 from testzeus_hercules.config import SingletonConfigManager, set_global_conf, get_global_conf
-from testzeus_hercules.core.playwright_manager import PlaywrightManager
 
 
 class TestCertificateErrors:
     """Basic test suite for IGNORE_CERTIFICATE_ERRORS functionality."""
 
     @pytest.fixture(autouse=True)
-    def setup_teardown(self):
+    def setup_teardown(self) -> Generator[None, None, None]:
         """Setup and teardown for each test."""
         os.environ["IS_TEST_ENV"] = "true"
 
         SingletonConfigManager.reset_instance()
         yield
         SingletonConfigManager.reset_instance()
 
-    def test_should_ignore_certificate_errors_default_false(self):
+    def test_should_ignore_certificate_errors_default_false(self) -> None:
         """Test that should_ignore_certificate_errors returns False by default."""
         if "IGNORE_CERTIFICATE_ERRORS" in os.environ:
             del os.environ["IGNORE_CERTIFICATE_ERRORS"]
 
         config = set_global_conf({"IGNORE_CERTIFICATE_ERRORS": "false", "LLM_MODEL_NAME": "test-model", "LLM_MODEL_API_KEY": "test-key"}, ignore_env=True)
         assert config.should_ignore_certificate_errors() is False
 
-    def test_should_ignore_certificate_errors_true(self):
+    def test_should_ignore_certificate_errors_true(self) -> None:
         """Test that should_ignore_certificate_errors returns True when set to 'true'."""
         os.environ["IGNORE_CERTIFICATE_ERRORS"] = "true"
 
         config = set_global_conf({"IGNORE_CERTIFICATE_ERRORS": "true", "LLM_MODEL_NAME": "test-model", "LLM_MODEL_API_KEY": "test-key"}, ignore_env=True)
         assert config.should_ignore_certificate_errors() is True
 
-    def test_should_ignore_certificate_errors_case_insensitive(self):
+    def test_should_ignore_certificate_errors_case_insensitive(self) -> None:
         """Test that should_ignore_certificate_errors is case insensitive."""
         os.environ["IGNORE_CERTIFICATE_ERRORS"] = "TRUE"
         config = set_global_conf({"IGNORE_CERTIFICATE_ERRORS": "TRUE", "LLM_MODEL_NAME": "test-model", "LLM_MODEL_API_KEY": "test-key"}, ignore_env=True)
@@ -42,15 +41,15 @@ def test_should_ignore_certificate_errors_case_insensitive(self):
         config = set_global_conf({"IGNORE_CERTIFICATE_ERRORS": "True", "LLM_MODEL_NAME": "test-model", "LLM_MODEL_API_KEY": "test-key"}, ignore_env=True)
         assert config.should_ignore_certificate_errors() is True
 
-    def test_should_ignore_certificate_errors_missing_key(self):
+    def test_should_ignore_certificate_errors_missing_key(self) -> None:
         """Test that should_ignore_certificate_errors returns False when key is missing."""
         if "IGNORE_CERTIFICATE_ERRORS" in os.environ:
             del os.environ["IGNORE_CERTIFICATE_ERRORS"]
 
         config = set_global_conf({"LLM_MODEL_NAME": "test-model", "LLM_MODEL_API_KEY": "test-key"}, ignore_env=True)
         assert config.should_ignore_certificate_errors() is False
 
-    def test_environment_variable_override(self):
+    def test_environment_variable_override(self) -> None:
         """Test that environment variable IGNORE_CERTIFICATE_ERRORS overrides config."""
         os.environ["IGNORE_CERTIFICATE_ERRORS"] = "true"
 
@@ -59,7 +58,7 @@ def test_environment_variable_override(self):
 
         assert config.should_ignore_certificate_errors() is True
 
-    def test_playwright_manager_with_certificate_errors_enabled(self):
+    def test_playwright_manager_with_certificate_errors_enabled(self) -> None:
         """Test that PlaywrightManager includes certificate error flags when enabled."""
         os.environ["IGNORE_CERTIFICATE_ERRORS"] = "true"
 
@@ -79,7 +78,7 @@ def test_playwright_manager_with_certificate_errors_enabled(self):
         config = get_global_conf()
         assert config.should_ignore_certificate_errors() is True
 
-    def test_playwright_manager_with_certificate_errors_disabled(self):
+    def test_playwright_manager_with_certificate_errors_disabled(self) -> None:
         """Test that PlaywrightManager does NOT include certificate error flags when disabled."""
         if "IGNORE_CERTIFICATE_ERRORS" in os.environ:
             del os.environ["IGNORE_CERTIFICATE_ERRORS"]