Track all config source for repr

Author: Agent-Hellboy

Lib/configparser.py

@@ -663,6 +663,7 @@ def __init__(self, defaults=None, dict_type=_default_dict,
             full=tuple(comment_prefixes or ()),
             inline=tuple(inline_comment_prefixes or ()),
         )
+        self._loaded_sources = []
         self._strict = strict
         self._allow_no_value = allow_no_value
         self._empty_lines_in_values = empty_lines_in_values
@@ -682,7 +683,7 @@ def __init__(self, defaults=None, dict_type=_default_dict,
         if defaults:
             self._read_defaults(defaults)
         self._allow_unnamed_section = allow_unnamed_section
-        self._loaded_files = []
+
 
     def defaults(self):
         return self._defaults
@@ -746,12 +747,13 @@ def read(self, filenames, encoding=None):
             try:
                 with open(filename, encoding=encoding) as fp:
                     self._read(fp, filename)
+                    self._loaded_sources.append(filename)
             except OSError:
                 continue
             if isinstance(filename, os.PathLike):
                 filename = os.fspath(filename)
             read_ok.append(filename)
-        self._loaded_files.extend(read_ok)
+
         return read_ok
 
     def read_file(self, f, source=None):
@@ -768,11 +770,13 @@ def read_file(self, f, source=None):
             except AttributeError:
                 source = '<???>'
         self._read(f, source)
+        self._loaded_sources.append(source)
 
     def read_string(self, string, source='<string>'):
         """Read configuration from a given string."""
         sfile = io.StringIO(string)
         self.read_file(sfile, source)
+        self._loaded_sources.append(source)
 
     def read_dict(self, dictionary, source='<dict>'):
         """Read configuration from a dictionary.
@@ -804,6 +808,8 @@ def read_dict(self, dictionary, source='<dict>'):
                     raise DuplicateOptionError(section, key, source)
                 elements_added.add((section, key))
                 self.set(section, key, value)
+        self._loaded_sources.append(source)
+
 
     def get(self, section, option, *, raw=False, vars=None, fallback=_UNSET):
         """Get an option value for a given section.
@@ -1061,12 +1067,15 @@ def __repr__(self):
             "interpolation": type(self._interpolation).__name__,
         }
         init_params = {k: v for k, v in init_params.items() if v is not None}
-
         state_summary = {
-            "loaded_files": self._loaded_files if hasattr(self, '_loaded_files') else "(no files loaded)",
-            "sections": len(self._sections),
+            "loaded_sources": self._loaded_sources,
+            "sections_count": len(self._sections),
+            "sections": list(self._sections.keys())[:5],  # Limit to 5 section names for readability
         }
 
+        if len(self._sections) > 5:
+            state_summary["sections_truncated"] = f"...and {len(self._sections) - 5} more"
+
         return (f"<{self.__class__.__name__}("
                 f"params={init_params}, "
                 f"state={state_summary})>")

Lib/test/test_configparser.py

@@ -983,12 +983,19 @@ def test_set_nonstring_types(self):
     def test_str_and_repr(self):
         self.maxDiff = None
         cf = self.config_class(allow_no_value=True, delimiters=('=',), strict=True)
-
-        cf.add_section("sect")
-        cf.set("sect", "option1", "foo")
-        cf.set("sect", "option2", "bar")
-
-        expected_str = "{'sect': {'option1': 'foo', 'option2': 'bar'}}"
+        cf.add_section("sect1")
+        cf.add_section("sect2")
+        cf.add_section("sect3")
+        cf.add_section("sect4")
+        cf.add_section("sect5")
+        cf.add_section("sect6")  # Added more than 5 sections to trigger truncation
+        cf.set("sect1", "option1", "foo")
+        cf.set("sect2", "option2", "bar")
+
+        expected_str = (
+            "{'sect1': {'option1': 'foo'}, 'sect2': {'option2': 'bar'}, 'sect3': {}, "
+            "'sect4': {}, 'sect5': {}, 'sect6': {}}"
+        )
         self.assertEqual(str(cf), expected_str)
 
         dict_type = type(cf._dict).__name__
@@ -998,10 +1005,13 @@ def test_str_and_repr(self):
             f"params={{'dict_type': '{dict_type}', 'allow_no_value': True, "
             "'delimiters': ('=',), 'strict': True, 'default_section': 'DEFAULT', "
             "'interpolation': 'BasicInterpolation'}, "
-            "state={'loaded_files': [], 'sections': 1})>"
+            "state={'loaded_sources': [], 'sections_count': 6, "
+            "'sections': ['sect1', 'sect2', 'sect3', 'sect4', 'sect5'], "
+            "'sections_truncated': '...and 1 more'})>"
         )
         self.assertEqual(repr(cf), expected_repr)
 
+
     def test_add_section_default(self):
         cf = self.newconfig()
         self.assertRaises(ValueError, cf.add_section, self.default_section)