Root cause of intermittent failures may have been introduced in 9ad5d2a due to schema generator adding log handlers. Address by reverting previous workaround strategy (bf0000c, 12fab1a), updating client _get_logger method to store loggers by name, and using it instead of an ad hoc implementation in the schema generator.

Author: softwareengineerprogrammer

src/geophires_x_client/common.py

@@ -1,20 +1,20 @@
 import logging
 import sys
 
-_geophires_x_client_logger = None
+_geophires_x_client_loggers_by_name = {}
 
 
 def _get_logger(logger_name=None):
-    global _geophires_x_client_logger
-    if _geophires_x_client_logger is None:
+    if logger_name is None:
+        logger_name = __name__
+
+    global _geophires_x_client_loggers_by_name
+    if logger_name not in _geophires_x_client_loggers_by_name:
         sh = logging.StreamHandler(sys.stdout)
         sh.setLevel(logging.INFO)
         sh.setFormatter(logging.Formatter(fmt='[%(asctime)s][%(levelname)s] %(message)s', datefmt='%Y-%m-%d %H:%M:%S'))
 
-        if logger_name is None:
-            logger_name = __name__
-
-        _geophires_x_client_logger = logging.getLogger(logger_name)
-        _geophires_x_client_logger.addHandler(sh)
+        _geophires_x_client_loggers_by_name[logger_name] = logging.getLogger(logger_name)
+        _geophires_x_client_loggers_by_name[logger_name].addHandler(sh)
 
-    return _geophires_x_client_logger
+    return _geophires_x_client_loggers_by_name[logger_name]

src/geophires_x_schema_generator/__init__.py

@@ -1,5 +1,4 @@
 import json
-import logging
 import os
 import sys
 from pathlib import Path
@@ -28,9 +27,13 @@
 from geophires_x.SUTRAWellBores import SUTRAWellBores
 from geophires_x.TDPReservoir import TDPReservoir
 from geophires_x.TOUGH2Reservoir import TOUGH2Reservoir
-from geophires_x_client import GeophiresXResult
+
+# noinspection PyProtectedMember
+from geophires_x_client import GeophiresXResult, _get_logger
 from hip_ra_x.hip_ra_x import HIP_RA_X
 
+_log = _get_logger()
+
 
 class GeophiresXSchemaGenerator:
     def __init__(self):
@@ -421,20 +424,3 @@ def get_input_schema_reference(self) -> str:
 
     def get_output_schema_reference(self) -> str:
         return None
-
-
-def _get_logger(logger_name=None):
-    sh = logging.StreamHandler(sys.stdout)
-    sh.setLevel(logging.INFO)
-    sh.setFormatter(logging.Formatter(fmt='[%(asctime)s][%(levelname)s] %(message)s', datefmt='%Y-%m-%d %H:%M:%S'))
-
-    if logger_name is None:
-        logger_name = __name__
-
-    _l = logging.getLogger(logger_name)
-    _l.addHandler(sh)
-
-    return _l
-
-
-_log = _get_logger()

tests/test_base_test_case.py

@@ -1,6 +1,3 @@
-import os
-import platform
-import sys
 import unittest
 
 from tests.base_test_case import BaseTestCase
@@ -38,26 +35,11 @@ def test_assertAlmostEqualWithinPercentage_bad_arguments(self):
             with self.assertRaises(AssertionError):
                 self.assertAlmostEqualWithinPercentage([1, 2, 3], [1.1, 2.2, 3.3], percent=10.5)
 
-            try:
-                self.assertHasLogRecordWithMessage(
-                    logs,
-                    'Got 2 lists, you probably meant to call:\n\t'
-                    'self.assertListAlmostEqual([1, 2, 3], [1.1, 2.2, 3.3], msg=None, percent=10.5)',
-                )
-            except AssertionError as ae:
-                if (
-                    'CI' in os.environ
-                    and (platform.system() in ['Darwin', 'Linux'])
-                    and (sys.version_info.major, sys.version_info.minor) == (3, 11)
-                ):
-                    # Intermittent failures observed in GitHub Actions py311 macos and ubuntu beginning on 2025-07-23.
-                    # Examples:
-                    # - https://github.com/softwareengineerprogrammer/GEOPHIRES/actions/runs/16476574734/job/46579711905
-                    # - https://github.com/softwareengineerprogrammer/GEOPHIRES/actions/runs/16477002832/job/46581220253
-                    # TODO to investigate and resolve
-                    self.skipTest(f'Skipping test due to platform-specific intermittent failure: {ae!s}')
-                else:
-                    raise ae
+            self.assertHasLogRecordWithMessage(
+                logs,
+                'Got 2 lists, you probably meant to call:\n\t'
+                'self.assertListAlmostEqual([1, 2, 3], [1.1, 2.2, 3.3], msg=None, percent=10.5)',
+            )
 
 
 if __name__ == '__main__':