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

fix tests

Browse source
This commit is contained in:
Jesse Bannon 2023-12-18 15:37:53 -08:00
parent b3e6550c0e
commit cf92178c82
2 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/ytdl_sub/plugins/regex.py
View file

@ -231,7 +231,7 @@ class RegexOptions(OptionsDictValidator):
return False return False
for capture_group_default in regex_options.capture_group_defaults or []: for capture_group_default in regex_options.capture_group_defaults or []:
parsed_default = parse(capture_group_default.format_string) parsed_default = parse(capture_group_default.format_string)
if parsed_default.variables.issubset(unresolved_variables): if parsed_default.variables and parsed_default.variables.issubset(unresolved_variables):
return False return False
return True return True

7
tests/e2e/plugins/test_regex.py
View file

@ -313,7 +313,8 @@ class TestRegex:
with pytest.raises( with pytest.raises(
ValidationException, ValidationException,
match=re.escape( match=re.escape(
"cannot use 'uid' as a capture group name because it is a defined variable" "Cannot use the variable name uid because it exists as a built-in "
"ytdl-sub variable name."
), ),
): ):
_ = Subscription.from_dict( _ = Subscription.from_dict(
@ -333,8 +334,8 @@ class TestRegex:
with pytest.raises( with pytest.raises(
ValidationException, ValidationException,
match=re.escape( match=re.escape(
"cannot use 'contains_regex_default' as a capture group name because it is a " "Override variable with name contains_regex_default cannot be used since it is "
"defined variable" "added by a plugin."
), ),
): ):
_ = Subscription.from_dict( _ = Subscription.from_dict(