From 44e6ad87340d50f48daf53b6a61138377d0d0d10 Mon Sep 17 00:00:00 2001 From: James Tocknell Date: Tue, 22 Aug 2017 08:46:30 +1000 Subject: [PATCH] bpo-23835: Enforce that configparser defaults are strings (#2558) * Enforce that configparser defaults are strings * Update test_configparser.py --- Lib/configparser.py | 5 ++--- Lib/test/test_configparser.py | 9 +++++++++ 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/Lib/configparser.py b/Lib/configparser.py index ea971f39338..360924008dc 100644 --- a/Lib/configparser.py +++ b/Lib/configparser.py @@ -610,9 +610,6 @@ class RawConfigParser(MutableMapping): self._converters = ConverterMapping(self) self._proxies = self._dict() self._proxies[default_section] = SectionProxy(self, default_section) - if defaults: - for key, value in defaults.items(): - self._defaults[self.optionxform(key)] = value self._delimiters = tuple(delimiters) if delimiters == ('=', ':'): self._optcre = self.OPTCRE_NV if allow_no_value else self.OPTCRE @@ -637,6 +634,8 @@ class RawConfigParser(MutableMapping): self._interpolation = Interpolation() if converters is not _UNSET: self._converters.update(converters) + if defaults: + self.read_dict({default_section: defaults}) def defaults(self): return self._defaults diff --git a/Lib/test/test_configparser.py b/Lib/test/test_configparser.py index 72c3f19fb41..be22fa40310 100644 --- a/Lib/test/test_configparser.py +++ b/Lib/test/test_configparser.py @@ -855,6 +855,15 @@ boolean {0[0]} NO self.assertEqual(cf.get('DEFAULT', 'test'), 'test') self.assertEqual(cf['DEFAULT']['test'], 'test') + def test_defaults_keyword(self): + # test that bpo-23835 is fixed + cf = self.newconfig(defaults={1: 2.4}) + self.assertEqual(cf[self.default_section]['1'], '2.4') + self.assertAlmostEqual(cf[self.default_section].getfloat('1'), 2.4) + cf = self.newconfig(defaults={"A": 5.2}) + self.assertEqual(cf[self.default_section]['a'], '5.2') + self.assertAlmostEqual(cf[self.default_section].getfloat('a'), 5.2) + class StrictTestCase(BasicTestCase, unittest.TestCase): config_class = configparser.RawConfigParser