Merge branch 'acccache_exc' into 'master'

AccumulatorCache: fix cache statistics if exception is thrown

See merge request atlas/athena!48336

Control/AthenaConfiguration/python/AccumulatorCache.py

@@ -110,6 +110,7 @@ class AccumulatorDecorator:
         return None
 
     def __call__(self , *args , **kwargs):
+        cacheHit = None
         try:
             t0 = time.perf_counter()
             res, cacheHit = self._callImpl(*args, **kwargs)

Control/AthenaConfiguration/python/testAccumulatorCache.py

@@ -52,6 +52,19 @@ class TestCache(unittest.TestCase):
         info = fac.getInfo()
         self.assertEqual(info["hits"] , 1)
 
+    def test_exception(self):
+        """Test cache when function throws exception."""
+
+        class MyException(BaseException):
+            pass
+
+        @AccumulatorCache
+        def throw():
+            raise MyException()
+
+        # Make sure no other exception is thrown by decorator itself
+        self.assertRaises(MyException, throw)
+
     def test_cache_limit(self):
         """Test cache limits."""