Higher level on retry logs

dan/datasets/extract/utils.py

@@ -33,7 +33,7 @@ TRIM_RETURN_REGEX = re.compile(r"[\r\n]+")
 
 
 def _retry_log(retry_state, *args, **kwargs):
-    logger.debug(
+    logger.warning(
         f"Request to {retry_state.args[0]} failed ({repr(retry_state.outcome.exception())}), "
         f"retrying in {retry_state.idle_for} seconds"
     )

tests/test_extract.py

@@ -741,7 +741,7 @@ def test_download_image_error(iiif_url, caplog, capsys):
     assert captured.out == "deadbeef: Image URL must be HTTP(S)\n"
 
 
-def test_download_image_error_try_max(responses):
+def test_download_image_error_try_max(responses, caplog):
     # An image's URL
     url = (
         "https://blabla.com/iiif/2/image_path.jpg/231,699,2789,3659/full/0/default.jpg"
@@ -774,6 +774,12 @@ def test_download_image_error_try_max(responses):
         fixed_url
     ]
 
+    # Check error log
+    assert len(caplog.record_tuples) == 2
+
+    # We should only have WARNING levels
+    assert set(level for _, level, _ in caplog.record_tuples) == {logging.WARNING}
+
 
 @pytest.mark.parametrize("allow_empty", (True, False))
 def test_empty_transcription(allow_empty, mock_database):