fix test failure

Author: Agent-Hellboy

Lib/configparser.py

@@ -691,6 +691,7 @@ def __init__(self, defaults=None, dict_type=_default_dict,
             self._read_defaults(defaults)
         self._allow_unnamed_section = allow_unnamed_section
 
+
     def defaults(self):
         return self._defaults
 
@@ -758,7 +759,6 @@ def read(self, filenames, encoding=None):
             if isinstance(filename, os.PathLike):
                 filename = os.fspath(filename)
             read_ok.append(filename)
-            self._loaded_sources.append(read_ok)
         return read_ok
 
     def read_file(self, f, source=None):
@@ -775,7 +775,6 @@ 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."""
@@ -812,7 +811,6 @@ 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.
@@ -1054,11 +1052,12 @@ def __iter__(self):
 
     def __str__(self):
         config_dict = {
-            section: dict(self.items(section, raw=True))
+            section: {key: value for key, value in self.items(section, raw=True)}
             for section in self.sections()
         }
         return f"<ConfigParser: {config_dict}>"
 
+
     def __repr__(self):
         params = {
             "defaults": self._defaults if self._defaults else None,
@@ -1073,8 +1072,8 @@ def __repr__(self):
         sections_count = len(self._sections)
         state = {
             "loaded_sources": self._loaded_sources,
-            "sections_count": sections_count,
-            "sections": list(self._sections)[:5],  # limit to 5 section names for readability
+            "sections_count": len(self._sections),
+            "sections": list(self._sections.keys())[:5],  # Limit to 5 section names for readability
         }
 
         if sections_count > 5:
@@ -1084,6 +1083,7 @@ def __repr__(self):
                 f"params={params}, "
                 f"state={state})>")
 
+
     def _read(self, fp, fpname):
         """Parse a sectioned configuration file.
 
@@ -1255,14 +1255,11 @@ def _convert_to_boolean(self, value):
 
     def _validate_key_contents(self, key):
         """Raises an InvalidWriteError for any keys containing
-        delimiters or that begins with the section header pattern"""
+        delimiters or that match the section header pattern"""
         if re.match(self.SECTCRE, key):
-            raise InvalidWriteError(
-                f"Cannot write key {key}; begins with section pattern")
-        for delim in self._delimiters:
-            if delim in key:
-                raise InvalidWriteError(
-                    f"Cannot write key {key}; contains delimiter {delim}")
+            raise InvalidWriteError("Cannot write keys matching section pattern")
+        if any(delim in key for delim in self._delimiters):
+            raise InvalidWriteError("Cannot write key that contains delimiters")
 
     def _validate_value_types(self, *, section="", option="", value=""):
         """Raises a TypeError for illegal non-string values.