Skip to content
Snippets Groups Projects
Commit 039b164ab761 authored by Philippe Pepiot's avatar Philippe Pepiot
Browse files

Use *args, **kwargs instead of enumerate all params

This will ease adding new params later without needing to change each benchmark
method.
parent 5c01204e039b
No related branches found
No related tags found
No related merge requests found
......@@ -173,6 +173,6 @@
class ExchangeTimeSuite(BaseExchangeTimeSuite):
def time_incoming(self, repo_name, repo_type, strip, revset):
def time_incoming(self, *args, **kwargs):
self.run(self.clone_path, 'incoming', self.repo_path)
......@@ -177,6 +177,6 @@
self.run(self.clone_path, 'incoming', self.repo_path)
def time_outgoing(self, repo_name, repo_type, strip, revset):
def time_outgoing(self, *args, **kwargs):
self.run(self.repo_path, 'outgoing', self.clone_path)
......@@ -187,7 +187,7 @@
['same', 'last(all(), 10)', 'last(all(), 100)', 'last(all(), 1000)'],
[None]]
def time_debugdiscovery(self, repo_name, repo_type, strip, revset):
def time_debugdiscovery(self, *args, **kwargs):
self.run(self.clone_path, 'debugdiscovery', self.repo_path,
expected_return_code=0)
......@@ -200,8 +200,8 @@
repeat = 20
warmup_time = 0
def setup(self, repo_name, repo_type, strip, revset):
super(PushPullTimeSuite, self).setup(repo_name, repo_type, strip, revset)
def setup(self, *args, **kwargs):
super(PushPullTimeSuite, self).setup(*args, **kwargs)
tmpdir = os.path.join(REPOS_DIR, '.tmp')
try:
os.makedirs(tmpdir)
......@@ -218,6 +218,6 @@
# impact performances metrics
self._single_execute(['sync'])
def time_push(self, repo_name, repo_type, strip, revset):
def time_push(self, *args, **kwargs):
self.run(self.repo_path, ['push', '-f'], self.tmp_clone_path)
......@@ -222,6 +222,6 @@
self.run(self.repo_path, ['push', '-f'], self.tmp_clone_path)
def time_pull(self, repo_name, repo_type, strip, revset):
def time_pull(self, *args, **kwargs):
self.run(self.tmp_clone_path, 'pull', self.repo_path,
expected_return_code=0)
......@@ -232,8 +232,8 @@
timeout = 300
def setup(self, repo_name, repo_type, strip, revset):
super(CloneTimeSuite, self).setup(repo_name, repo_type, strip, revset)
def setup(self, *args, **kwargs):
super(CloneTimeSuite, self).setup(*args, **kwargs)
tmpdir = os.path.join(REPOS_DIR, '.tmp')
try:
os.makedirs(tmpdir)
......@@ -244,6 +244,6 @@
os.path.basename(self.clone_path)))
shutil.rmtree(self.tmp_clone_path, ignore_errors=True)
def teardown(self, repo_name, repo_type, strip, revset):
def teardown(self, *args, **kwargs):
shutil.rmtree(self.tmp_clone_path, ignore_errors=True)
......@@ -248,6 +248,6 @@
shutil.rmtree(self.tmp_clone_path, ignore_errors=True)
def time_clone(self, repo_name, repo_type, strip, revset):
def time_clone(self, *args, **kwargs):
cmd = [self.hgpath, 'clone', '--pull']
cmd.extend(self._remote_path_cmd(self.clone_path))
cmd.append(self.tmp_clone_path)
......
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