test: fix testDictLoggerConfigViaJson

The path to the log file in the config did not exist. Change to use
the tracker home.


Also add a test where the log file directory does not exist. This
reports the full path, so have to edit the full path in the error
message before comparison.

Author: rouilj

test/test_config.py

@@ -20,6 +20,7 @@
 import logging
 import os
 import pytest
+import re
 import shutil
 import sys
 import unittest
@@ -1072,14 +1073,14 @@ def testDictLoggerConfigViaJson(self):
                   "level": "INFO",
                   "formatter": "http",
                   "class": "logging.FileHandler",
-                  "filename": "demo/access.log"
+                  "filename": "_test_instance/access.log"
                 },
                 # logging for roundup.* loggers
                 "roundup": {
                   "level": "DEBUG",
                   "formatter": "standard",
                   "class": "logging.FileHandler",
-                  "filename": "demo/roundup.log"
+                  "filename": "_test_instance/roundup.log"
                 },
                 # print to stdout - fall through for other logging
                 "default": {
@@ -1241,3 +1242,29 @@ def testDictLoggerConfigViaJson(self):
              "'MANGO'\n")
 
         )
+
+        # broken invalid output directory
+        test_config = config1.replace(
+            ' "_test_instance/access.log"',
+            ' "not_a_test_instance/access.log"')
+        with open(log_config_filename, "w") as log_config_file:
+            log_config_file.write(test_config)
+
+        # file is made relative to tracker dir.
+        self.db.config["LOGGING_CONFIG"] = '_test_log_config.json'
+        with self.assertRaises(configuration.LoggingConfigError) as cm:
+            config = self.db.config.init_logging()
+
+        # error includes full path which is different on different
+        # CI and dev platforms. So munge the path using re.sub.
+        self.assertEqual(
+            re.sub("directory: \'/.*not_a", 'directory: not_a' ,
+                   cm.exception.args[0]),
+            ("Error loading logging dict from "
+             "_test_instance/_test_log_config.json.\n"
+             "ValueError: Unable to configure handler 'access'\n"
+             "[Errno 2] No such file or directory: "
+             "not_a_test_instance/access.log'\n"
+            )
+        )
+