Merge remote-tracking branch 'origin/master' into nightly

overhangio · Sep 5, 2023 · 6941021 · 6941021
2 parents 14dfc82 + 8eccaa6
commit 6941021
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 0 deletions.
diff --git a/changelog.d/20230905_155859_regis_fix_render_config_lists.md b/changelog.d/20230905_155859_regis_fix_render_config_lists.md
@@ -0,0 +1 @@
+- [Bugfix] Render config settings that are inside lists. (by @regisb)
diff --git a/tests/test_env.py b/tests/test_env.py
@@ -64,6 +64,7 @@ def test_render_unknown(self) -> None:
         }
         self.assertEqual("ab", env.render_unknown(config, "{{ var1 }}b"))
         self.assertEqual({"x": "ac"}, env.render_unknown(config, {"x": "{{ var1 }}c"}))
+        self.assertEqual(["x", "ac"], env.render_unknown(config, ["x", "{{ var1 }}c"]))
 
     def test_common_domain(self) -> None:
         self.assertEqual(

diff --git a/tutor/env.py b/tutor/env.py
@@ -376,6 +376,8 @@ def render_unknown(config: Config, value: t.Any) -> t.Any:
         return render_str(config, value)
     if isinstance(value, dict):
         return {k: render_unknown(config, v) for k, v in value.items()}
+    if isinstance(value, list):
+        return [render_unknown(config, v) for v in value]
     return value