Issue #21159: Improve message in configparser.InterpolationMissingOpt… · python/cpython@f7a9267 · GitHub
Skip to content

Commit f7a9267

Browse files
committed
Issue #21159: Improve message in configparser.InterpolationMissingOptionError.
Patch from Łukasz Langa.
2 parents 5e8d47f + ac37ba0 commit f7a9267

3 files changed

Lines changed: 21 additions & 18 deletions

File tree

Lib/configparser.py

Lines changed: 14 additions & 15 deletions

Lib/test/test_configparser.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -847,7 +847,8 @@ def test_interpolation(self):
847847
"something with lots of interpolation (10 steps)")
848848
e = self.get_error(cf, configparser.InterpolationDepthError, "Foo", "bar11")
849849
if self.interpolation == configparser._UNSET:
850-
self.assertEqual(e.args, ("bar11", "Foo", "%(with1)s"))
850+
self.assertEqual(e.args, ("bar11", "Foo",
851+
"something %(with11)s lots of interpolation (11 steps)"))
851852
elif isinstance(self.interpolation, configparser.LegacyInterpolation):
852853
self.assertEqual(e.args, ("bar11", "Foo",
853854
"something %(with11)s lots of interpolation (11 steps)"))
@@ -861,7 +862,7 @@ def test_interpolation_missing_value(self):
861862
self.assertEqual(e.option, "name")
862863
if self.interpolation == configparser._UNSET:
863864
self.assertEqual(e.args, ('name', 'Interpolation Error',
864-
'', 'reference'))
865+
'%(reference)s', 'reference'))
865866
elif isinstance(self.interpolation, configparser.LegacyInterpolation):
866867
self.assertEqual(e.args, ('name', 'Interpolation Error',
867868
'%(reference)s', 'reference'))
@@ -1177,7 +1178,7 @@ def test_strange_options(self):
11771178
with self.assertRaises(exception_class) as cm:
11781179
cf['interpolated']['$trying']
11791180
self.assertEqual(cm.exception.reference, 'dollars:${sick')
1180-
self.assertEqual(cm.exception.args[2], '}') #rawval
1181+
self.assertEqual(cm.exception.args[2], '${dollars:${sick}}') #rawval
11811182

11821183
def test_case_sensitivity_basic(self):
11831184
ini = textwrap.dedent("""

Misc/NEWS

Lines changed: 3 additions & 0 deletions

0 commit comments

Comments
 (0)