diff options
author | Lars Wirzenius <liw@liw.fi> | 2017-03-04 23:37:18 +0200 |
---|---|---|
committer | Lars Wirzenius <liw@liw.fi> | 2017-03-04 23:37:18 +0200 |
commit | 93a0feadb737d8c3271f57895c6192f401befb79 (patch) | |
tree | 5ff9da942f9b43d1a79cf0fc3c3712a40de86c4d | |
parent | 8f64a94c390a887ea3b1a35a298c868024fa6117 (diff) | |
download | git.liw.fi-ruleset-tests-93a0feadb737d8c3271f57895c6192f401befb79.tar.gz |
Refactor to make use off .git_as or .git_as_checked
-rw-r--r-- | 000.yarn | 11 | ||||
-rw-r--r-- | yarnhelper.py | 9 |
2 files changed, 8 insertions, 12 deletions
@@ -330,19 +330,14 @@ groups, and respositories. repo = helper.get_next_match() url = helper.repo_ssh_url(repo) dirname = helper.local_checkout_dirname(user, repo) - env = dict(os.environ) - env['GIT_SSH_COMMAND'] = helper.env_ssh_command(user) - cliapp.runcmd(['git', 'push', '--all', 'origin'], cwd=dirname, env=env) - + helper.git_as_checked(user, ['push', '--all', 'origin'], cwd=dirname) IMPLEMENTS THEN (\S+) cannot push (\S+) user = helper.get_next_match() repo = helper.get_next_match() dirname = helper.local_checkout_dirname(user, repo) - env = dict(os.environ) - env['GIT_SSH_COMMAND'] = helper.env_ssh_command(user) - exit, out, err = cliapp.runcmd_unchecked( - ['git', 'push', '--all', 'origin'], cwd=dirname, env=env) + exit, out, err = helper.git_as(user, + ['push', '--all', 'origin'], cwd=dirname) sys.stdout.write(out) sys.stderr.write(err) helper.assertNotEqual(exit, 0) diff --git a/yarnhelper.py b/yarnhelper.py index a1a95ea..d132b88 100644 --- a/yarnhelper.py +++ b/yarnhelper.py @@ -150,17 +150,18 @@ class YarnHelper(object): def local_checkout_dirname(self, user, repo): # pragma: no cover return '{}_{}'.format(user, repo) - def git_as(self, user, args): # pragma: no cover + def git_as(self, user, args, **kwargs): # pragma: no cover server = os.environ['GITANO_SERVER'] env = dict(os.environ) env['GIT_SSH_COMMAND'] = self.env_ssh_command(user) return cliapp.runcmd_unchecked( ['git'] + args, stderr=subprocess.STDOUT, - env=env) + env=env, + **kwargs) - def git_as_checked(self, user, args): # pragma: no cover - exit, out, err = self.git_as(user, args) + def git_as_checked(self, user, args, **kwargs): # pragma: no cover + exit, out, err = self.git_as(user, args, **kwargs) self.assertEqual(exit, 0) def env_ssh_command(self, user): # pragma: no cover |