From e7798bbd9bf0a69b84a3f0d24b16efe476df3336 Mon Sep 17 00:00:00 2001 From: CJ Harries Date: Fri, 16 Mar 2018 20:03:39 -0500 Subject: [PATCH 1/2] Return entry, a ConfigEntry from ConfigIterator.__next__ --- pygit2/config.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pygit2/config.py b/pygit2/config.py index ce8cdb97..fd9ab496 100644 --- a/pygit2/config.py +++ b/pygit2/config.py @@ -63,7 +63,7 @@ def next(self): def __next__(self): entry = self._next_entry() - return entry.name + return entry class ConfigMultivarIterator(ConfigIterator): From 52de16ac23702d72c77b8a06567ed08701cbf6f6 Mon Sep 17 00:00:00 2001 From: CJ Harries Date: Fri, 16 Mar 2018 20:15:08 -0500 Subject: [PATCH 2/2] Update test_iterator to consume ConfigEntr[ies] --- test/test_config.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/test/test_config.py b/test/test_config.py index 03be486e..5678c699 100644 --- a/test/test_config.py +++ b/test/test_config.py @@ -173,8 +173,9 @@ def test_iterator(self): config = self.repo.config lst = {} - for name in config: - lst[name] = config[name] + for entry in config: + self.assertGreater(entry.level, -1) + lst[entry.name] = entry.value_string self.assertTrue('core.bare' in lst) self.assertTrue(lst['core.bare'])