Skip to content
Snippets Groups Projects
Commit 136e20815c5e authored by Pierre-Yves David's avatar Pierre-Yves David :octopus:
Browse files

benchmark: rename the `strip` to `partial_key`

The strip is an implementation detail. The partial_key seems a clearer name.
(note: I am not sure about the "partial" part, but I could not find any
better).
parent 74ab07f91995
No related branches found
No related tags found
No related merge requests found
......@@ -293,8 +293,8 @@
if self.rev:
cmd.extend(['-r', self.rev])
if expected_return_code is None:
expected_return_code = 1 if self.strip == "same" else 0
expected_return_code = 1 if self.partial_key == "same" else 0
self.hg(*cmd, expected_return_code=expected_return_code)
@params_as_kwargs
def setup(self, repo, repo_type, strip, revset, **kwargs):
......@@ -297,8 +297,8 @@
self.hg(*cmd, expected_return_code=expected_return_code)
@params_as_kwargs
def setup(self, repo, repo_type, strip, revset, **kwargs):
self.strip = strip
self.partial_key = strip
super(BaseExchangeTimeSuite, self).setup(repo, **kwargs)
self._setup_repo_type(repo_type)
self._setup_revset(revset)
......@@ -302,7 +302,7 @@
super(BaseExchangeTimeSuite, self).setup(repo, **kwargs)
self._setup_repo_type(repo_type)
self._setup_revset(revset)
repo_suffix = urllib.quote_plus(self.strip)
repo_suffix = urllib.quote_plus(self.partial_key)
# We need to use the repo name here because the repo doesn't contains
# the hash
self.clone_path = os.path.join(REPOS_DIR, 'partial-references', '{}-partial-{}'.format(
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment