Refactor code to replace collections.Mapping with dict

Author: karlnyr

microSALT/app/app.py

@@ -1,4 +1,3 @@
-import collections
 import logging
 import json
 import os
@@ -56,7 +55,7 @@ def handle_config_entry(entry, value, logger):
         if not value.startswith("/"):
             sys.exit(-1)
         create_path(os.path.abspath(value), logger)
-    elif isinstance(value, collections.Mapping):
+    elif isinstance(value, dict):
         for sub_entry, sub_value in value.items():
             handle_config_entry(sub_entry, sub_value, logger)
 

tests/test_config.py

@@ -1,6 +1,5 @@
 #!/usr/bin/env python
 
-import collections
 import os
 import pathlib
 import pytest
@@ -33,7 +32,7 @@ def test_existence(exp_config):
       assert entry in config_level_one
 
       # level two
-      if isinstance(preset_config[entry], collections.Mapping):
+      if isinstance(preset_config[entry], dict):
         config_level_two = preset_config[entry].keys()
         for thing in exp_config[entry]:
           assert thing in config_level_two
@@ -49,7 +48,7 @@ def test_reverse_existence(exp_config):
 
       # level two
       config_level_two = exp_config[entry]
-      if isinstance(preset_config[entry], collections.Mapping):
+      if isinstance(preset_config[entry], dict):
         for thing in preset_config[entry].keys():
           if thing != '_comment':
             assert thing in config_level_two
@@ -68,7 +67,7 @@ def test_paths(exp_config):
         assert (pathlib.Path(unmade_fldr).exists())
 
       # level two
-      elif isinstance(preset_config[entry], collections.Mapping):
+      elif isinstance(preset_config[entry], dict):
         for thing in preset_config[entry].keys():
           if isinstance(preset_config[entry][thing], str) and '/' in preset_config[entry][thing] and entry not in ['database', 'genologics']:
             unmade_fldr = preset_config[entry][thing]