Bugfix: fix __repr__ of Config

qcodes/config/config.py

@@ -355,12 +355,13 @@ def save_to_cwd(self):
         self.save_config(self.cwd_file_name)
         self.save_schema(self.schema_cwd_file_name)
 
-    def describe(self, name):
+    def describe(self, name: str) -> str:
         """
         Describe a configuration entry
 
         Args:
-            name (str): name of entry to describe
+            name: name of entry to describe in 'dotdict' notation,
+              e.g. name="user.scriptfolder"
         """
         val = self.current_config
         sch = self.current_schema["properties"]
@@ -392,9 +393,9 @@ def __getattr__(self, name):
 
     def __repr__(self):
         old = super().__repr__()
-        output = f"""Current values: \n {current_config} \n
-                     Current paths: \n {self._loaded_config_files} \n
-                     {old}"""
+        output = (f"Current values: \n {self.current_config} \n"
+                  f"Current paths: \n {self._loaded_config_files} \n"
+                  f"{old}")
         return output
 
 

qcodes/tests/test_config.py

@@ -325,3 +325,38 @@ def test_update_from_path(path_to_config_file_on_disk):
         expected_path = os.path.join(path_to_config_file_on_disk,
                                      'qcodesrc.json')
         assert cfg.current_config_path == expected_path
+
+
+def test_repr():
+    cfg = Config()
+    rep = cfg.__repr__()
+
+    expected_rep = (f"Current values: \n {cfg.current_config} \n"
+                    f"Current paths: \n {cfg._loaded_config_files} \n"
+                    f"{super(Config, cfg).__repr__()}")
+
+    assert rep == expected_rep
+
+
+def test_add_and_describe():
+    """
+    Test that a key an be added and described
+    """
+    with default_config():
+
+        key = 'newkey'
+        value ='testvalue'
+        value_type ='string'
+        description ='A test'
+        default = 'testdefault'
+
+        cfg = Config()
+        cfg.add(key=key, value=value, value_type=value_type,
+                description=description, default=default)
+
+
+        desc = cfg.describe(f'user.{key}')
+        expected_desc = (f"{description}.\nCurrent value: {value}. "
+                         f"Type: {value_type}. Default: {default}.")
+
+        assert desc == expected_desc