Adds optional code coverage reporting to run_tests.sh.

[python_utils.git] / arper.py

diff --git a/arper.py b/arper.py
index 2171e773088076aefc6f7d43c80fff59abb9b46f..a4e8d3e1196a1f683a24a9d43f4ec48fc8890253 100644 (file)
--- a/arper.py
+++ b/arper.py
@@ -6,6 +6,7 @@ import datetime
 import logging
 import os
 from typing import Any, Optional
+import warnings
 
 from overrides import overrides
 
@@ -48,7 +49,7 @@ cfg.add_argument(
 @persistent.persistent_autoloaded_singleton()
 class Arper(persistent.Persistent):
     def __init__(
-            self, cached_state: Optional[BiDict[str, str]] = None
+            self, cached_state: Optional[BiDict] = None
     ) -> None:
         self.state = BiDict()
         if cached_state is not None:
@@ -126,11 +127,10 @@ class Arper(persistent.Persistent):
             if len(cached_state) > config.config['arper_min_entries_to_be_valid']:
                 return cls(cached_state)
             else:
-                logger.warning(
-                    f'{cache_file} sucks, only {len(cached_state)} entries.  Deleting it.'
-                )
+                msg = f'{cache_file} is invalid: only {len(cached_state)} entries.  Deleting it.'
+                logger.warning(msg)
+                warnings.warn(msg, stacklevel=2)
                 os.remove(cache_file)
-
         logger.debug('No usable saved state found')
         return None