suppressed hash value of SHA from pooch (#695)

* suppressed hash value of SHA from pooch

* changed linting issues and hide SHA hash

Author: LakshmiSowmya04

movement/sample_data.py

@@ -6,7 +6,9 @@
 are used.
 """
 
+import logging
 import shutil
+from contextlib import contextmanager
 from pathlib import Path
 
 import pooch
@@ -32,6 +34,32 @@
 METADATA_FILE = "metadata.yaml"
 
 
+@contextmanager
+def hide_pooch_hash_logs():
+    """Hide SHA256 hash printouts from ``pooch.retrieve``.
+
+    This context manager temporarily suppresses SHA256 hash messages
+    when downloading files with Pooch.
+    """
+    logger = pooch.get_logger()
+
+    class HashFilter(logging.Filter):
+        def filter(self, record):
+            msg = record.getMessage()
+            # Suppress only hash display lines
+            return not (
+                "SHA256 hash of downloaded file" in msg
+                or "Use this value as the 'known_hash'" in msg
+            )
+
+    flt = HashFilter()
+    logger.addFilter(flt)
+    try:
+        yield
+    finally:
+        logger.removeFilter(flt)
+
+
 def _download_metadata_file(file_name: str, data_dir: Path = DATA_DIR) -> Path:
     """Download the metadata yaml file.
 
@@ -54,13 +82,15 @@ def _download_metadata_file(file_name: str, data_dir: Path = DATA_DIR) -> Path:
         Path to the downloaded file.
 
     """
-    local_file_path = pooch.retrieve(
-        url=f"{DATA_URL}/{file_name}",
-        known_hash=None,
-        path=data_dir,
-        fname=f"temp_{file_name}",
-        progressbar=False,
-    )
+    with hide_pooch_hash_logs():
+        local_file_path = pooch.retrieve(
+            url=f"{DATA_URL}/{file_name}",
+            known_hash=None,
+            path=data_dir,
+            fname=f"temp_{file_name}",
+            progressbar=False,
+        )
+
     logger.debug(
         f"Successfully downloaded sample metadata file {file_name} "
         f"from {DATA_URL} to {data_dir}"

tests/test_unit/test_sample_data.py

@@ -1,5 +1,6 @@
 """Test suite for the sample_data module."""
 
+import logging
 from pathlib import Path
 from unittest.mock import MagicMock, patch
 
@@ -14,6 +15,7 @@
     _fetch_metadata,
     fetch_dataset,
     fetch_dataset_paths,
+    hide_pooch_hash_logs,
     list_datasets,
 )
 
@@ -273,3 +275,61 @@ def test_fetch_and_unzip_exceptions(
             assert result == tmp_path / "poses" / "test"
             assert ".DS_Store" not in result_names
         assert expected_warning in caplog.records[-1].getMessage()
+
+
+@pytest.mark.parametrize(
+    "message, should_block",
+    [
+        ("SHA256 hash of downloaded file: abc123", True),
+        ("Use this value as the 'known_hash' argument", True),
+        ("Downloading file from ", False),
+    ],
+)
+def test_hide_pooch_hash_logs(message, should_block):
+    """Test that ``HashFilter`` blocks only hash-related messages."""
+    with hide_pooch_hash_logs():
+        logger = pooch.get_logger()
+        hash_filter = logger.filters[-1]
+
+        # Create a mock log record
+        record = logging.LogRecord(
+            name="pooch",
+            level=logging.INFO,
+            pathname="",
+            lineno=0,
+            msg=message,
+            args=(),
+            exc_info=None,
+        )
+
+        # The filter returns False to block, True to allow
+        message_allowed = hash_filter.filter(record)
+
+        if should_block:
+            assert not message_allowed, (
+                f"Expected message to be blocked but was allowed: '{message}'"
+            )
+        else:
+            assert message_allowed, (
+                f"Expected message to be allowed but was blocked: '{message}'"
+            )
+
+
+@pytest.mark.parametrize("raise_exception", [True, False])
+def test_hash_filter_removed_after_context(raise_exception):
+    """Test that the hash filter applied by ``hide_pooch_hash_logs``
+    is properly removed after exiting context, even when an exception occurs.
+    """
+    logger = pooch.get_logger()
+    initial_filter_count = len(logger.filters)
+
+    if raise_exception:
+        with pytest.raises(ValueError), hide_pooch_hash_logs():
+            assert len(logger.filters) == initial_filter_count + 1
+            raise ValueError("Test exception")
+    else:
+        with hide_pooch_hash_logs():
+            assert len(logger.filters) == initial_filter_count + 1
+
+    # Filter should be removed after context exits
+    assert len(logger.filters) == initial_filter_count