relink.py: Timing now shows in quiet mode.

Author: samsrabin

relink.py

@@ -19,6 +19,20 @@
 # Set up logger
 logger = logging.getLogger(__name__)
 
+# Define a custom log level that always prints
+ALWAYS = logging.CRITICAL * 2
+logging.addLevelName(ALWAYS, "ALWAYS")
+
+
+def always(self, message, *args, **kwargs):
+    """Log message that always appears regardless of log level."""
+    if self.isEnabledFor(ALWAYS):
+        # pylint: disable=protected-access
+        self._log(ALWAYS, message, args, **kwargs)
+
+
+logging.Logger.always = always
+
 
 def find_and_replace_owned_files(source_dir, target_dir, username, dry_run=False):
     """
@@ -234,7 +248,7 @@ def main():
 
     if args.timing:
         elapsed_time = time.time() - start_time
-        logger.info("Execution time: %.2f seconds", elapsed_time)
+        logger.always("Execution time: %.2f seconds", elapsed_time)
 
 
 if __name__ == "__main__":

tests/relink/test_timing.py

@@ -56,3 +56,40 @@ def test_timing_logging(tmp_path, caplog, use_timing, should_log_timing):
         assert "seconds" in caplog.text
     else:
         assert "Execution time:" not in caplog.text
+
+
+def test_timing_shows_in_quiet_mode(tmp_path, caplog):
+    """Test that timing message is shown even when --quiet flag is used."""
+    # Create real directories
+    source_dir = tmp_path / "source"
+    target_dir = tmp_path / "target"
+    source_dir.mkdir()
+    target_dir.mkdir()
+
+    # Create a file
+    source_file = source_dir / "test_file.txt"
+    target_file = target_dir / "test_file.txt"
+    source_file.write_text("source")
+    target_file.write_text("target")
+
+    # Build argv with both --timing and --quiet flags
+    test_argv = [
+        "relink.py",
+        "--source-root",
+        str(source_dir),
+        "--target-root",
+        str(target_dir),
+        "--timing",
+        "--quiet",
+    ]
+
+    with patch("sys.argv", test_argv):
+        with caplog.at_level(logging.WARNING):
+            # Call main() which includes the timing logic
+            relink.main()
+
+    # Verify timing message appears even in quiet mode
+    assert "Execution time:" in caplog.text
+    assert "seconds" in caplog.text
+    # Verify that INFO messages are suppressed
+    assert "Searching for files owned by" not in caplog.text