jan/masque
1
1
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Add NoReturn __contains__ with a more descriptive error message

Browse Source
This commit is contained in:
jan 2023-10-12 01:32:33 -07:00
parent f2e2de1d2e
commit 2d88c00cd7
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
masque/ports.py
View File

@ -1,4 +1,4 @@
from typing import Iterable, KeysView, ValuesView, overload, Self, Mapping from typing import Iterable, KeysView, ValuesView, overload, Self, Mapping, NoReturn
import warnings import warnings
import traceback import traceback
import logging import logging
@ -138,6 +138,9 @@ class PortList(metaclass=ABCMeta):
else: else:
return {k: self.ports[k] for k in key} return {k: self.ports[k] for k in key}
def __contains__(self, key: str) -> NoReturn:
raise NotImplementedError('PortsList.__contains__ is left unimplemented. Use `key in container.ports` instead.')
# NOTE: Didn't add keys(), items(), values(), __contains__(), etc. # NOTE: Didn't add keys(), items(), values(), __contains__(), etc.
# because it's weird on stuff like Pattern that contains other lists # because it's weird on stuff like Pattern that contains other lists
# and because you can just grab .ports and use that instead # and because you can just grab .ports and use that instead