Skip to content

Commit

Permalink
Remove unused attribute and parameter in CallSpec2
Browse files Browse the repository at this point in the history
  • Loading branch information
nicoddemus committed Jun 19, 2018
1 parent 2925f30 commit 93e55ad
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
1 change: 1 addition & 0 deletions changelog/3598.trivial.rst
@@ -0,0 +1 @@
Internal refactoring: removed unused ``CallSpec2tox ._globalid_args`` attribute and ``metafunc`` parameter from ``CallSpec2.copy()``.
6 changes: 2 additions & 4 deletions src/_pytest/python.py
Expand Up @@ -735,13 +735,12 @@ def __init__(self, metafunc):
self._idlist = []
self.params = {}
self._globalid = NOTSET
self._globalid_args = set()
self._globalparam = NOTSET
self._arg2scopenum = {} # used for sorting parametrized resources
self.marks = []
self.indices = {}

def copy(self, metafunc):
def copy(self):
cs = CallSpec2(self.metafunc)
cs.funcargs.update(self.funcargs)
cs.params.update(self.params)
Expand All @@ -750,7 +749,6 @@ def copy(self, metafunc):
cs._arg2scopenum.update(self._arg2scopenum)
cs._idlist = list(self._idlist)
cs._globalid = self._globalid
cs._globalid_args = self._globalid_args
cs._globalparam = self._globalparam
return cs

Expand Down Expand Up @@ -933,7 +931,7 @@ def parametrize(self, argnames, argvalues, indirect=False, ids=None, scope=None)
param.values, argnames
)
)
newcallspec = callspec.copy(self)
newcallspec = callspec.copy()
newcallspec.setmulti2(
valtypes,
argnames,
Expand Down

0 comments on commit 93e55ad

Please sign in to comment.