diff --git a/tests_with_gitaly/__init__.py b/tests_with_gitaly/__init__.py index 0c1ed47dcf2a40aa6d7f9d0fb230611be03ab269_dGVzdHNfd2l0aF9naXRhbHkvX19pbml0X18ucHk=..4250ce078c4ddcf022651aede2ab6c1f79adc7ba_dGVzdHNfd2l0aF9naXRhbHkvX19pbml0X18ucHk= 100644 --- a/tests_with_gitaly/__init__.py +++ b/tests_with_gitaly/__init__.py @@ -26,8 +26,6 @@ logger = logging.getLogger(__name__) -GITALY_BIN_REL_PATH = '_build/bin/gitaly' - def gitaly_home(): # pragma no cover """Find where Gitaly sits on the file system @@ -42,8 +40,8 @@ """ from_env = os.environ.get("GITALY_INSTALL_DIR") if from_env is not None: - install_dir = Path(from_env).resolve() + bin_dir = install_dir = Path(from_env).resolve() else: hgitaly_clone = Path(hgitaly.__file__).resolve().parent.parent hdk = hgitaly_clone.parent install_dir = hdk / 'gitaly' @@ -46,5 +44,6 @@ else: hgitaly_clone = Path(hgitaly.__file__).resolve().parent.parent hdk = hgitaly_clone.parent install_dir = hdk / 'gitaly' + bin_dir = install_dir / '_build/bin' @@ -50,5 +49,4 @@ - if install_dir.is_dir(): - for bin_path in ('_build/bin/gitaly', - 'gitaly', # Gitaly < 14.5 and Heptapod CI + if bin_dir.is_dir(): + for bin_path in ('gitaly', ): @@ -54,5 +52,5 @@ ): - gitaly = install_dir / bin_path + gitaly = bin_dir / bin_path if gitaly.is_file(): # startup costs about 0.15s. That's a lot, but it will # make sure our skip is correct and it is useful to log the @@ -62,9 +60,8 @@ assert 'Gitaly' in version logger.info("Found %s at %s. Will lauch comparison tests " "of HGitaly with Gitaly", version, gitaly) - return install_dir, bin_path - return install_dir, None + return install_dir, bin_dir return None, None @@ -67,8 +64,8 @@ return None, None -GITALY_INSTALL_DIR, GITALY_BIN_REL_PATH = gitaly_home() +GITALY_INSTALL_DIR, GITALY_BIN_DIR = gitaly_home() def gitaly_not_installed(): diff --git a/tests_with_gitaly/gitaly.py b/tests_with_gitaly/gitaly.py index 0c1ed47dcf2a40aa6d7f9d0fb230611be03ab269_dGVzdHNfd2l0aF9naXRhbHkvZ2l0YWx5LnB5..4250ce078c4ddcf022651aede2ab6c1f79adc7ba_dGVzdHNfd2l0aF9naXRhbHkvZ2l0YWx5LnB5 100644 --- a/tests_with_gitaly/gitaly.py +++ b/tests_with_gitaly/gitaly.py @@ -13,7 +13,7 @@ from hgitaly.testing.grpc import wait_server_accepts_connection from . import ( - GITALY_BIN_REL_PATH, + GITALY_BIN_DIR, GITALY_INSTALL_DIR, ) @@ -38,7 +38,7 @@ 'socket_path = "%s"' % self.gitaly_socket, # Gitaly compilation outputs its binaries at the root # of the checkout - 'bin_dir = "%s"' % GITALY_INSTALL_DIR, + 'bin_dir = "%s"' % GITALY_BIN_DIR, '[gitlab-shell]', 'dir = "%s"' % gitlab_shell_dir, '[gitaly-ruby]', @@ -68,7 +68,7 @@ # it write to a file, let's redirect ourselves. with open(self.home_dir / 'gitaly.log', 'w') as logf: gitaly = subprocess.Popen( - [GITALY_INSTALL_DIR / GITALY_BIN_REL_PATH, self.gitaly_conf], + [GITALY_BIN_DIR / 'gitaly', self.gitaly_conf], stdout=logf, stderr=logf, env=env)