diff --git a/libmat2/exiftool.py b/libmat2/exiftool.py index a70b162d4034221423f20fba30bc03e1c279e350..2d0484166df5d7d185aa1edc4234780a519cfcbe 100644 --- a/libmat2/exiftool.py +++ b/libmat2/exiftool.py @@ -30,7 +30,7 @@ class ExiftoolParser(abstract.AbstractParser): out = subprocess.run([_get_exiftool_path(), '-json', self.filename], check=True, stdout=subprocess.PIPE).stdout - except subprocess.CalledProcessError: + except subprocess.CalledProcessError: # pragma: no cover raise ValueError meta = json.loads(out.decode('utf-8'))[0] for key in self.meta_allowlist: diff --git a/libmat2/web.py b/libmat2/web.py index 18ef04f37f88ee22b369398d471c1f501d23e181..b77020092c879d1574639a2549a40c690f04c571 100644 --- a/libmat2/web.py +++ b/libmat2/web.py @@ -96,15 +96,6 @@ class _HTMLParser(parser.HTMLParser): self.tag_required_blocklist = required_blocklisted_tags self.tag_blocklist = blocklisted_tags - def error(self, message): - """ Amusingly, Python's documentation doesn't mention that this - function needs to be implemented in subclasses of the parent class - of parser.HTMLParser. This was found by fuzzing, - triggering the following exception: - NotImplementedError: subclasses of ParserBase must override error() - """ - raise ValueError(message) - def handle_starttag(self, tag: str, attrs: List[Tuple[str, Optional[str]]]): # Ignore the type, because mypy is too stupid to infer # that get_starttag_text() can't return None.