Added unit tests and updated change log.

Author: kmvanbrunt

CHANGELOG.md

@@ -15,9 +15,12 @@
     - Added `Cmd.macro_arg_complete()` which tab completes arguments to a macro. Its default
       behavior is to perform path completion, but it can be overridden as needed.
 
-    - Bug Fixes
-        - No longer redirecting `sys.stdout` if it's a different stream than `self.stdout`. This
-          fixes issue where we overwrote an application's `sys.stdout` while redirecting.
+    - All print methods (`poutput()`, `perror()`, `ppaged()`, etc.) have the ability to print Rich
+      objects.
+
+- Bug Fixes
+    - No longer redirecting `sys.stdout` if it's a different stream than `self.stdout`. This fixes
+      issue where we overwrote an application's `sys.stdout` while redirecting.
 
 ## 2.7.0 (June 30, 2025)
 

cmd2/cmd2.py

@@ -1503,7 +1503,7 @@ def ppaged(
         can_block = not (self._redirecting or self.in_pyscript() or self.in_script())
 
         # Check if we are outputting to a pager.
-        if functional_terminal and can_block:  # pragma: no cover
+        if functional_terminal and can_block:
             prepared_objects = rich_utils.prepare_objects_for_rich_print(*objects)
 
             # Chopping overrides soft_wrap

cmd2/rich_utils.py

@@ -156,11 +156,7 @@ def __init__(self, file: IO[str]) -> None:
 
     def on_broken_pipe(self) -> None:
         """Override which raises BrokenPipeError instead of SystemExit."""
-        import contextlib
-
-        with contextlib.suppress(SystemExit):
-            super().on_broken_pipe()
-
+        self.quiet = True
         raise BrokenPipeError
 
 

tests/test_cmd2.py

@@ -2058,6 +2058,19 @@ def test_poutput_ansi_terminal(outsim_app) -> None:
     assert out == expected
 
 
+def test_broken_pipe_error(outsim_app, monkeypatch, capsys):
+    write_mock = mock.MagicMock()
+    write_mock.side_effect = BrokenPipeError
+    monkeypatch.setattr("cmd2.utils.StdSim.write", write_mock)
+
+    outsim_app.broken_pipe_warning = "The pipe broke"
+    outsim_app.poutput("My test string")
+
+    out, err = capsys.readouterr()
+    assert not out
+    assert outsim_app.broken_pipe_warning in err
+
+
 # These are invalid names for aliases and macros
 invalid_command_name = [
     '""',  # Blank name
@@ -2519,8 +2532,36 @@ def test_pexcept_not_exception(base_app, capsys) -> None:
     assert err.startswith("\x1b[91mEXCEPTION of type 'bool' occurred with message: False")
 
 
-def test_ppaged(outsim_app) -> None:
-    """ppaged() will just call poutput() since a pager won't run while testing."""
+def test_ppaged_with_pager(outsim_app, monkeypatch) -> None:
+    """Force ppaged() to run the pager by mocking an actual terminal state."""
+
+    # Make it look like we're in a terminal
+    stdin_mock = mock.MagicMock()
+    stdin_mock.isatty.return_value = True
+    monkeypatch.setattr(outsim_app, "stdin", stdin_mock)
+
+    stdout_mock = mock.MagicMock()
+    stdout_mock.isatty.return_value = True
+    monkeypatch.setattr(outsim_app, "stdout", stdout_mock)
+
+    if not sys.platform.startswith('win'):
+        orig_term = os.environ.get("TERM")
+        os.environ.setdefault('TERM', 'simulated')
+
+    # This will force ppaged to call Popen to run a pager
+    popen_mock = mock.MagicMock(name='Popen')
+    monkeypatch.setattr("subprocess.Popen", popen_mock)
+    outsim_app.ppaged("Test", chop=True)
+
+    # Restore TERM
+    if not sys.platform.startswith('win'):
+        os.environ['TERM'] = orig_term
+
+    popen_mock.assert_called_once()
+
+
+def test_ppaged_no_pager(outsim_app) -> None:
+    """Since we're not in a fully-functional terminal, ppaged() will just call poutput()."""
     msg = 'testing...'
     end = '\n'
     outsim_app.ppaged(msg)