Third pass done.

Author: BatuhanSA

autograder/api/config.py

@@ -110,7 +110,7 @@ def get_local_config_path(local_config_root_cutoff = None):
 
     If no configuration file is found, returns None.
     The cutoff limits config search depth.
-    This helps to prevent detection of config files in higher directories during testing.
+    This helps to prevent detection of a config file in higher directories during testing.
     """
 
     # The case where DEFAULT_CONFIG_FILENAME file in current directory.
@@ -224,7 +224,7 @@ def _load_config_file(config_path, config, sources, source_label):
     with open(config_path, 'r') as file:
         for (key, value) in json.load(file).items():
             config[key] = value
-            sources[key] = f"{source_label}::{os.path.abspath(config_path)}"
+            sources[key] = f"{source_label}{CONFIG_TYPE_DELIMITER}{os.path.abspath(config_path)}"
 
 def _get_ancestor_config_file_path(
         current_directory,
@@ -238,7 +238,7 @@ def _get_ancestor_config_file_path(
     """
 
     current_directory = os.path.abspath(current_directory)
-    if local_config_root_cutoff is not None:
+    if (local_config_root_cutoff is not None):
         local_config_root_cutoff = os.path.abspath(local_config_root_cutoff)
 
     for _ in range(DEPTH_LIMIT):

autograder/cli/config/list.py

@@ -5,7 +5,7 @@
 DESCRIPTION = "List your current configuration options."
 
 def run(args):
-    config, sources = autograder.api.config.get_tiered_config(
+    (config, sources) = autograder.api.config.get_tiered_config(
         cli_arguments = args,
         skip_keys = [
             'show_origin', 'verbose',

tests/base.py

@@ -8,7 +8,7 @@ class BaseTest(unittest.TestCase):
     A base class for unit tests.
     """
 
-    # TestCase variable to show full diffs in assertion errors
+    # TestCase variable to show full diffs in assertion errors.
     maxDiff = None
 
     def assertListEqual(self, a, b):

tests/test_config.py

@@ -216,8 +216,8 @@ def _evaluate_test_config(
         """
 
         temp_dir = autograder.util.dir.get_temp_dir(prefix = 'autograder-test-config-')
-        global_config = os.path.join(temp_dir)
 
+        global_config = os.path.join(temp_dir)
         if (global_config_path is not None):
             global_config = os.path.join(temp_dir, global_config_path)
 
@@ -235,7 +235,7 @@ def _evaluate_test_config(
         os.chdir(initial_work_directory)
 
         try:
-            actual_configs, actual_sources = autograder.api.config.get_tiered_config(
+            (actual_configs, actual_sources) = autograder.api.config.get_tiered_config(
                 cli_arguments = cli_args,
                 global_config_path = global_config,
                 local_config_root_cutoff = temp_dir,