Updated unit tests and comments.

Author: kmvanbrunt

cmd2/cmd2.py

@@ -4043,6 +4043,14 @@ def complete_help_subcommands(
         return completer.complete_subcommand_help(text, line, begidx, endidx, arg_tokens['subcommands'])
 
     def _build_command_info(self) -> tuple[dict[str, list[str]], list[str], list[str], list[str]]:
+        """Categorizes and sorts visible commands and help topics for display.
+
+        :return: tuple containing:
+                  - dictionary mapping category names to lists of command names
+                  - list of documented command names
+                  - list of undocumented command names
+                  - list of help topic names that are not also commands
+        """
         # Get a sorted list of help topics
         help_topics = sorted(self.get_help_topics(), key=self.default_sort_key)
 
@@ -4268,7 +4276,8 @@ def columnize(self, str_list: list[str] | None, display_width: int = 80) -> None
             # The output is wider than display_width. Print 1 column with each string on its own row.
             nrows = len(str_list)
             ncols = 1
-            colwidths = [1]
+            max_width = max(su.str_width(s) for s in str_list)
+            colwidths = [max_width]
         for row in range(nrows):
             texts = []
             for col in range(ncols):

tests/test_cmd2.py

@@ -1245,7 +1245,10 @@ class HelpApp(cmd2.Cmd):
 
     def __init__(self, *args, **kwargs) -> None:
         super().__init__(*args, **kwargs)
-        self.doc_leader = "I now present you a list of help topics."
+        self.doc_leader = "I now present you with a list of help topics."
+        self.doc_header = "My very custom doc header."
+        self.misc_header = "Various topics found here."
+        self.undoc_header = "Why did no one document these?"
 
     def do_squat(self, arg) -> None:
         """This docstring help will never be shown because the help_squat method overrides it."""
@@ -1360,6 +1363,16 @@ def test_columnize_empty_list(capsys) -> None:
     assert "<empty>" in out
 
 
+def test_columnize_too_wide(capsys) -> None:
+    help_app = HelpApp()
+    commands = ["kind_of_long_string", "a_slightly_longer_string"]
+    help_app.columnize(commands, display_width=10)
+    out, err = capsys.readouterr()
+
+    expected = "kind_of_long_string     \na_slightly_longer_string\n"
+    assert expected == out
+
+
 class HelpCategoriesApp(cmd2.Cmd):
     """Class for testing custom help_* methods which override docstring help."""