Ensure dynaconf get returns a valid json string.

rochacbruno · rochacbruno · commit e288e422ac8a · 2022-09-23T17:22:48.000+01:00
diff --git a/dynaconf/cli.py b/dynaconf/cli.py
@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import importlib
+import json
 import os
 import pprint
 import sys
@@ -439,7 +440,10 @@ def init(ctx, fileformat, path, env, _vars, _secrets, wg, y, django):
     is_flag=True,
 )
 def get(key, default, env, unparse):
-    """Returns the raw value for a settings key"""
+    """Returns the raw value for a settings key.
+
+    If result is a dict, list or tuple it is printes as a valid json string.
+    """
     if env:
         env = env.strip()
     if key:
@@ -456,6 +460,9 @@ def get(key, default, env, unparse):
     if unparse:
         result = unparse_conf_data(result)
 
+    if isinstance(result, (dict, list, tuple)):
+        result = json.dumps(result, sort_keys=True)
+
     click.echo(result, nl=False)
 
 
diff --git a/tests/config.py b/tests/config.py
@@ -3,4 +3,8 @@
 from dynaconf import Dynaconf
 
 settingsenv = Dynaconf(environments=True)
-settings = Dynaconf(TEST_KEY="test_value", ANOTHER_KEY="another_value")
+settings = Dynaconf(
+    TEST_KEY="test_value",
+    ANOTHER_KEY="another_value",
+    DATA={"KEY": "value", "OTHERKEY": "other value"},
+)
diff --git a/tests/test_cli.py b/tests/test_cli.py
@@ -125,6 +125,18 @@ def test_get(testdir):
     assert result == "test_value"
 
 
+def test_get_json_dict(testdir):
+    """Tests get command printing json"""
+    env = env = {
+        "ROOT_PATH_FOR_DYNACONF": testdir,
+        "DYNACONF_DATA__KEY": "value",
+        "DYNACONF_DATA__OTHERKEY": "other value",
+        "INSTANCE_FOR_DYNACONF": "tests.config.settings",
+    }
+    result = run(["get", "data"], env=env)
+    assert result == '{"KEY": "value", "OTHERKEY": "other value"}'
+
+
 def test_get_lower(testdir):
     """Tests get command"""
     result = run(
