danvics/ytdl-sub
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

more tests fixed

Browse source
This commit is contained in:
Jesse Bannon 2023-12-08 14:17:56 -08:00
parent b63ee65396
commit ec605d1b1b
2 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/ytdl_sub/script/script.py
View file

@ -519,7 +519,7 @@ class ScriptBuilder:
for missing_metadata in self._missing_metadata: for missing_metadata in self._missing_metadata:
for name, missing in missing_metadata.items(): for name, missing in missing_metadata.items():
raise ScriptBuilderMissingDefinitions( 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) return self._build(variables=self._variables, functions=self._functions)

5
tests/unit/script/test_script_builder.py
View file

@ -18,12 +18,12 @@ class TestScriptBuilder:
script = ScriptBuilder( script = ScriptBuilder(
{ {
"entry": "{ {} }", "entry": "{ {} }",
"title": "{%map_get(entry, 'title')}", "title": "{%map_get(entry, 'title', '')}",
"resolved_override": "{override} mom", "resolved_override": "{override} mom",
} }
) )
assert script.partial_build().resolve() assert script.partial_build().resolve(unresolvable={"entry"})
with pytest.raises( with pytest.raises(
ScriptBuilderMissingDefinitions, ScriptBuilderMissingDefinitions,
@ -32,6 +32,7 @@ class TestScriptBuilder:
script.build() script.build()
script.add({"override": "hi"}) script.add({"override": "hi"})
script.add_resolved({"entry": entry_map})
script.build() script.build()