Add patch to fix tests in parallel.

This commit is contained in:
Elliott Sales de Andrade 2018-03-06 23:18:20 -05:00
parent dbaa8e4d88
commit a3e355459f
2 changed files with 74 additions and 0 deletions

View File

@ -0,0 +1,72 @@
From 458d669a2893d1aa4028a8ce9a44cdabfe3ec6e9 Mon Sep 17 00:00:00 2001
From: Elliott Sales de Andrade <quantum.analyst@gmail.com>
Date: Tue, 6 Mar 2018 19:03:07 -0500
Subject: [PATCH] Use unique repo name for tests that require credentials.
Otherwise, when running tests in parallel, they might conflict.
Fixes #2899.
Signed-off-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
---
test/cmd/lfstest-gitserver.go | 2 +-
test/test-credentials.sh | 8 ++++----
test/test-extra-header.sh | 2 +-
3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/test/cmd/lfstest-gitserver.go b/test/cmd/lfstest-gitserver.go
index 87a1d4ee..97732995 100644
--- a/test/cmd/lfstest-gitserver.go
+++ b/test/cmd/lfstest-gitserver.go
@@ -1246,7 +1246,7 @@ func locksHandler(w http.ResponseWriter, r *http.Request, repo string) {
}
func missingRequiredCreds(w http.ResponseWriter, r *http.Request, repo string) bool {
- if repo != "requirecreds" {
+ if !strings.HasPrefix(repo, "requirecreds") {
return false
}
diff --git a/test/test-credentials.sh b/test/test-credentials.sh
index 0ebf93a2..3efbe6b5 100755
--- a/test/test-credentials.sh
+++ b/test/test-credentials.sh
@@ -294,9 +294,9 @@ begin_test "credentials from lfs.url"
(
set -e
- reponame="requirecreds"
+ reponame="requirecreds-lfsurl"
setup_remote_repo "$reponame"
- clone_repo "$reponame" requirecreds-lfsurl
+ clone_repo "$reponame" "$reponame"
git lfs track "*.dat"
echo "push a" > a.dat
@@ -335,9 +335,9 @@ begin_test "credentials from remote.origin.url"
(
set -e
- reponame="requirecreds"
+ reponame="requirecreds-remoteurl"
setup_remote_repo "$reponame"
- clone_repo "$reponame" requirecreds-remoteurl
+ clone_repo "$reponame" "$reponame"
git lfs track "*.dat"
echo "push b" > b.dat
diff --git a/test/test-extra-header.sh b/test/test-extra-header.sh
index 823a7d62..1dbcfe30 100755
--- a/test/test-extra-header.sh
+++ b/test/test-extra-header.sh
@@ -30,7 +30,7 @@ begin_test "http.<url>.extraHeader with authorization"
(
set -e
- reponame="requirecreds"
+ reponame="requirecreds-extraHeader"
setup_remote_repo "$reponame"
clone_repo "$reponame" "$reponame"
--
2.14.3

View File

@ -39,6 +39,8 @@ Summary: Git extension for versioning large files
License: MIT License: MIT
URL: https://git-lfs.github.io/ URL: https://git-lfs.github.io/
Source0: https://%{provider_prefix}/archive/v%{version}/%{name}-%{version}.tar.gz Source0: https://%{provider_prefix}/archive/v%{version}/%{name}-%{version}.tar.gz
# https://github.com/git-lfs/git-lfs/pull/2901
Patch000: 0001-Use-unique-repo-name-for-tests-that-require-credenti.patch
ExclusiveArch: %{go_arches} ExclusiveArch: %{go_arches}
BuildRequires: %{?go_compiler:compiler(go-compiler)}%{!?go_compiler:golang} BuildRequires: %{?go_compiler:compiler(go-compiler)}%{!?go_compiler:golang}