Commit0e5358a

authored

Merge pull request#1555 from Codym48/fix/get_values

Fix get_values() so it correctly loads section names

2 parents202e31c +fc843d3 commit0e5358aCopy full SHA for 0e5358a

File tree

+12

-0

lines changed

+12

-0

lines changed

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -796,6 +796,7 @@ def get_values(`
`796`	`796`	`:raise TypeError: in case the value could not be understood`
`797`	`797`	`Otherwise the exceptions known to the ConfigParser will be raised."""`
`798`	`798`	`try:`
	`799`	`+self.sections()`
`799`	`800`	`lst=self._sections[section].getall(option)`
`800`	`801`	`exceptException:`
`801`	`802`	`ifdefaultisnotNone:`

Lines changed: 11 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -398,6 +398,17 @@ def test_empty_config_value(self):`
`398`	`398`	`withself.assertRaises(cp.NoOptionError):`
`399`	`399`	`cr.get_value("color","ui")`
`400`	`400`
	`401`	`+deftest_get_values_works_without_requiring_any_other_calls_first(self):`
	`402`	`+file_obj=self._to_memcache(fixture_path("git_config_multiple"))`
	`403`	`+cr=GitConfigParser(file_obj,read_only=True)`
	`404`	`+self.assertEqual(cr.get_values("section0","option0"), ["value0"])`
	`405`	`+file_obj.seek(0)`
	`406`	`+cr=GitConfigParser(file_obj,read_only=True)`
	`407`	`+self.assertEqual(cr.get_values("section1","option1"), ["value1a","value1b"])`
	`408`	`+file_obj.seek(0)`
	`409`	`+cr=GitConfigParser(file_obj,read_only=True)`
	`410`	`+self.assertEqual(cr.get_values("section1","other_option1"), ["other_value1"])`
	`411`	`+`
`401`	`412`	`deftest_multiple_values(self):`
`402`	`413`	`file_obj=self._to_memcache(fixture_path("git_config_multiple"))`
`403`	`414`	`withGitConfigParser(file_obj,read_only=False)ascw:`

Comments

(0)