refactor: extract _format_run_info from ConsoleEmitter for testability

Follows the same pattern as the _format_summary extraction (c79694c).
The run info line building logic (timeout, command count, max iterations)
is now a standalone function that can be unit tested without a Console
instance, with 10 focused tests covering all combinations.

Co-authored-by: Ralphify <noreply@ralphify.co>

Author: 2 people

src/ralphify/_console_emitter.py

@@ -64,6 +64,25 @@ def _format_summary(
     return f"{_plural(total, 'iteration')} {_ICON_DASH} {detail}"
 
 
+def _format_run_info(
+    timeout: float | None, command_count: int, max_iterations: int | None
+) -> str:
+    """Build a plain-text run info string from configuration values.
+
+    Returns an empty string when no information is available.  Used by
+    :meth:`ConsoleEmitter._on_run_started` to show the config summary
+    beneath the "Running:" header.
+    """
+    parts: list[str] = []
+    if timeout is not None and timeout > 0:
+        parts.append(f"timeout {format_duration(timeout)}")
+    if command_count > 0:
+        parts.append(_plural(command_count, "command"))
+    if max_iterations is not None:
+        parts.append(f"max {_plural(max_iterations, 'iteration')}")
+    return " · ".join(parts)
+
+
 class _IterationSpinner:
     """Rich renderable that shows a spinner with elapsed time."""
 
@@ -113,19 +132,9 @@ def _on_run_started(self, data: RunStartedData) -> None:
         self._console.print(
             f"\n[bold {_brand.PURPLE}]▶ Running:[/] [bold]{escape_markup(ralph_name)}[/]"
         )
-
-        info_parts: list[str] = []
-        timeout = data["timeout"]
-        if timeout is not None and timeout > 0:
-            info_parts.append(f"timeout {format_duration(timeout)}")
-        command_count = data["commands"]
-        if command_count > 0:
-            info_parts.append(_plural(command_count, "command"))
-        max_iter = data["max_iterations"]
-        if max_iter is not None:
-            info_parts.append(f"max {_plural(max_iter, 'iteration')}")
-        if info_parts:
-            self._console.print(f"  [dim]{' · '.join(info_parts)}[/]")
+        info = _format_run_info(data["timeout"], data["commands"], data["max_iterations"])
+        if info:
+            self._console.print(f"  [dim]{info}[/]")
 
     def _start_live(self) -> None:
         spinner = _IterationSpinner()

tests/test_console_emitter.py

@@ -3,7 +3,12 @@
 import pytest
 from rich.console import Console
 
-from ralphify._console_emitter import ConsoleEmitter, _IterationSpinner, _format_summary
+from ralphify._console_emitter import (
+    ConsoleEmitter,
+    _IterationSpinner,
+    _format_run_info,
+    _format_summary,
+)
 from ralphify._events import Event, EventType
 
 
@@ -505,6 +510,43 @@ def test_all_failed(self):
         assert "3 failed" in result
 
 
+class TestFormatRunInfo:
+    def test_empty_when_no_config(self):
+        assert _format_run_info(timeout=0, command_count=0, max_iterations=None) == ""
+        assert _format_run_info(timeout=None, command_count=0, max_iterations=None) == ""
+
+    def test_timeout_only(self):
+        result = _format_run_info(timeout=120, command_count=0, max_iterations=None)
+        assert result == "timeout 2m 0s"
+
+    def test_commands_only(self):
+        assert _format_run_info(timeout=0, command_count=3, max_iterations=None) == "3 commands"
+
+    def test_singular_command(self):
+        assert _format_run_info(timeout=0, command_count=1, max_iterations=None) == "1 command"
+
+    def test_max_iterations_only(self):
+        assert _format_run_info(timeout=0, command_count=0, max_iterations=5) == "max 5 iterations"
+
+    def test_singular_iteration(self):
+        assert _format_run_info(timeout=0, command_count=0, max_iterations=1) == "max 1 iteration"
+
+    def test_all_fields(self):
+        result = _format_run_info(timeout=60, command_count=2, max_iterations=3)
+        assert "timeout 1m 0s" in result
+        assert "2 commands" in result
+        assert "max 3 iterations" in result
+        assert " · " in result
+
+    def test_zero_timeout_excluded(self):
+        result = _format_run_info(timeout=0, command_count=2, max_iterations=None)
+        assert "timeout" not in result
+
+    def test_negative_timeout_excluded(self):
+        result = _format_run_info(timeout=-1, command_count=2, max_iterations=None)
+        assert "timeout" not in result
+
+
 class TestIterationSpinner:
     def test_renders_elapsed_time(self):
         spinner = _IterationSpinner()