Rollup merge of #143814 - lolbinarycat:htmldocck-negative-err, r=fmease

htmldocck: better error messages for some negative directives

Previously it was saying "did not match pattern" even when the error was that it did match the pattern, and it wasn't supposed to.
This commit is contained in:
León Orell Valerian Liehr 2025-07-13 07:21:23 +02:00 committed by GitHub
commit f8bbc91f76
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -564,10 +564,14 @@ def check_command(c, cache):
# hasraw/matchesraw <path> <pat> = string test
elif len(c.args) == 2 and "raw" in c.cmd:
cerr = "`PATTERN` did not match"
if c.negated:
cerr = "`PATTERN` unexpectedly matched"
ret = check_string(cache.get_file(c.args[0]), c.args[1], regexp)
# has/matches <path> <pat> <match> = XML tree test
elif len(c.args) == 3 and "raw" not in c.cmd:
cerr = "`XPATH PATTERN` did not match"
if c.negated:
cerr = "`XPATH PATTERN` unexpectedly matched"
ret = get_nb_matching_elements(cache, c, regexp, True) != 0
else:
raise InvalidCheck("Invalid number of {} arguments".format(c.cmd))