Merge pull request #2 from JoseHJBlanco/fix-toml-config

More TOML tests
JoseHJBlanco · Apr 25, 2023 · 93d5163 · 93d5163
2 parents 465c2e5 + 1c4be22
commit 93d5163
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 1 deletion.
diff --git a/test/setup_logging_test.py b/test/setup_logging_test.py
@@ -49,7 +49,7 @@ def test_section(self):
                         'formatter': 'mockformatter',
                     },
                 },
-                'mockloggers': {
+                'loggers': {
                     'mocklogger': {
                         'handlers': ('mockhandler',),
                         'level': 'INFO',
@@ -62,6 +62,11 @@ def test_section(self):
         result = self.cls._section(None)
         self.assertTrue(result)
 
+        self.cls.config = LuigiTomlParser()
+        self.cls.config.read(['./test/testconfig/luigi_logging.toml'])
+        result = self.cls._section(None)
+        self.assertTrue(result)
+
         self.cls.config = {}
         result = self.cls._section(None)
         self.assertFalse(result)

diff --git a/test/testconfig/luigi_logging.toml b/test/testconfig/luigi_logging.toml
@@ -0,0 +1,18 @@
+[logging]
+version = 1
+disable_existing_loggers = false
+
+[logging.formatters.mockformatter]
+format = "{levelname}: {message}"
+style = "{"
+
+[logging.handlers.mockhandler]
+class = "logging.StreamHandler"
+level = "INFO"
+formatter = "mockformatter"
+
+[logging.loggers.mocklogger]
+handlers = ["mockhandler"]
+level = 'INFO'
+disabled = false
+propagate = false