diff options
author | Ran Benita <ran@unusedvar.com> | 2020-10-06 20:19:52 +0300 |
---|---|---|
committer | Ran Benita <ran@unusedvar.com> | 2021-01-01 23:23:08 +0200 |
commit | 2ff88098a7340142ed2c6d2c090b8c9fac001a5e (patch) | |
tree | 245277f533f4d7af61747a8fed1233706209c04c /src | |
parent | 8ee6d0a8666f91c9c537afacbe3c61f54e342f28 (diff) | |
download | pytest-2ff88098a7340142ed2c6d2c090b8c9fac001a5e.tar.gz |
python: inline a simple method
I don't think it adds much value!
Diffstat (limited to 'src')
-rw-r--r-- | src/_pytest/python.py | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/src/_pytest/python.py b/src/_pytest/python.py index 50ea60c2d..29ebd176b 100644 --- a/src/_pytest/python.py +++ b/src/_pytest/python.py @@ -922,10 +922,6 @@ class CallSpec2: cs._idlist = list(self._idlist) return cs - def _checkargnotcontained(self, arg: str) -> None: - if arg in self.params or arg in self.funcargs: - raise ValueError(f"duplicate {arg!r}") - def getparam(self, name: str) -> object: try: return self.params[name] @@ -947,7 +943,8 @@ class CallSpec2: param_index: int, ) -> None: for arg, val in zip(argnames, valset): - self._checkargnotcontained(arg) + if arg in self.params or arg in self.funcargs: + raise ValueError(f"duplicate {arg!r}") valtype_for_arg = valtypes[arg] if valtype_for_arg == "params": self.params[arg] = val |