fix: replace print/silent exceptions with loguru logger and skip generator tests without API key (#176)

Author: jc200808

openjudge/utils/prompt_format_checker.py

@@ -33,6 +33,8 @@
 from pathlib import Path
 from typing import Dict, List, Optional, Tuple
 
+from loguru import logger
+
 
 class Language(Enum):
     """Language enum for prompt validation."""
@@ -453,7 +455,7 @@ def extract_prompts_from_file(self, file_path: str) -> Dict[str, str]:
         try:
             tree = ast.parse(content)
         except SyntaxError as e:
-            print(f"  ⚠️ Syntax error in {file_path}: {e}")
+            logger.warning(f"Syntax error in {file_path}: {e}")
             return prompts
 
         for node in ast.walk(tree):
@@ -473,7 +475,7 @@ def extract_prompts_from_file(self, file_path: str) -> Dict[str, str]:
                                     if extracted:
                                         prompts[var_name] = extracted
                             except Exception:
-                                pass
+                                logger.debug(f"Failed to extract prompt from variable {var_name} in {file_path}")
 
         return prompts
 

tests/generator/test_iterative_rubric.py

@@ -33,6 +33,7 @@
 """
 
 import asyncio
+import os
 from copy import deepcopy
 
 import pytest
@@ -48,6 +49,15 @@
 from openjudge.models.openai_chat_model import OpenAIChatModel
 from openjudge.models.schema.prompt_template import LanguageEnum
 
+OPENAI_API_KEY = os.getenv("OPENAI_API_KEY")
+OPENAI_BASE_URL = os.getenv("OPENAI_BASE_URL")
+RUN_INTEGRATION_TESTS = bool(OPENAI_API_KEY and OPENAI_BASE_URL)
+
+pytestmark = pytest.mark.skipif(
+    not RUN_INTEGRATION_TESTS,
+    reason="Requires OPENAI_API_KEY and OPENAI_BASE_URL environment variables",
+)
+
 # pylint: disable=line-too-long
 
 # =============================================================================

tests/generator/test_simple_rubric.py

@@ -30,6 +30,7 @@
 """
 
 import asyncio
+import os
 
 import pytest
 from loguru import logger
@@ -44,6 +45,15 @@
 from openjudge.models.openai_chat_model import OpenAIChatModel
 from openjudge.models.schema.prompt_template import LanguageEnum
 
+OPENAI_API_KEY = os.getenv("OPENAI_API_KEY")
+OPENAI_BASE_URL = os.getenv("OPENAI_BASE_URL")
+RUN_INTEGRATION_TESTS = bool(OPENAI_API_KEY and OPENAI_BASE_URL)
+
+pytestmark = pytest.mark.skipif(
+    not RUN_INTEGRATION_TESTS,
+    reason="Requires OPENAI_API_KEY and OPENAI_BASE_URL environment variables",
+)
+
 # =============================================================================
 # Test Data
 # =============================================================================