diff --git a/src/ytdl_sub/script/script.py b/src/ytdl_sub/script/script.py index 40b6465a..d0c9a0d8 100644 --- a/src/ytdl_sub/script/script.py +++ b/src/ytdl_sub/script/script.py @@ -519,7 +519,7 @@ class ScriptBuilder: for missing_metadata in self._missing_metadata: for name, missing in missing_metadata.items(): raise ScriptBuilderMissingDefinitions( - f"{name} is missing the following definitions: {', '.join(missing_metadata)}" + f"{name} is missing the following definitions: {', '.join(missing)}" ) return self._build(variables=self._variables, functions=self._functions) diff --git a/tests/unit/script/test_script_builder.py b/tests/unit/script/test_script_builder.py index 87f06b3f..e6f3d27d 100644 --- a/tests/unit/script/test_script_builder.py +++ b/tests/unit/script/test_script_builder.py @@ -18,12 +18,12 @@ class TestScriptBuilder: script = ScriptBuilder( { "entry": "{ {} }", - "title": "{%map_get(entry, 'title')}", + "title": "{%map_get(entry, 'title', '')}", "resolved_override": "{override} mom", } ) - assert script.partial_build().resolve() + assert script.partial_build().resolve(unresolvable={"entry"}) with pytest.raises( ScriptBuilderMissingDefinitions, @@ -32,6 +32,7 @@ class TestScriptBuilder: script.build() script.add({"override": "hi"}) + script.add_resolved({"entry": entry_map}) script.build()