summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorRan Benita <ran@unusedvar.com>2020-10-06 20:19:52 +0300
committerRan Benita <ran@unusedvar.com>2021-01-01 23:23:08 +0200
commit2ff88098a7340142ed2c6d2c090b8c9fac001a5e (patch)
tree245277f533f4d7af61747a8fed1233706209c04c /src
parent8ee6d0a8666f91c9c537afacbe3c61f54e342f28 (diff)
downloadpytest-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.py7
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