Improve Config.get() function 29/9729/4
authorDavid Garcia <david.garcia@canonical.com>
Tue, 22 Sep 2020 09:36:48 +0000 (11:36 +0200)
committergarciadav <david.garcia@canonical.com>
Wed, 23 Sep 2020 10:24:17 +0000 (12:24 +0200)
The .get is a bit confusing because apparently has the same behaviour as dict.get(),
which returns None when the key doesn't exist. But that's not the case, because Config.get in this case is
doing dict["key"], which returns a KeyError when the key does not exist

Change-Id: Ief82eb1ae2e2fea60b8be60cae8610bf8605fb6f
Signed-off-by: David Garcia <david.garcia@canonical.com>
osm_mon/core/config.py

index c4c3972..cd99ffc 100644 (file)
@@ -48,7 +48,7 @@ class Config:
     def get(self, section, field=None):
         if not field:
             return self.conf[section]
-        return self.conf[section][field]
+        return self.conf[section].get(field)
 
     def set(self, section, field, value):
         if section not in self.conf: