diff options
author | Stan Hu <stanhu@gmail.com> | 2019-07-18 19:41:21 +0300 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2019-07-18 19:41:21 +0300 |
commit | 43cbc233df451ffd0fbc93212cfde126554f2a66 (patch) | |
tree | 0ea054b1fc0ffaf25f198eaefeebff937df1adf3 /spec | |
parent | 0067ea312c4ca8511e2fbec601514cfc50bacb8f (diff) | |
parent | 79c7eae39ab2b41ce4678b9640f5b393b7a291b2 (diff) |
Merge branch 'fix-fetch-test-repos' into 'master'
Fix broken handling for outdated test repos
See merge request gitlab-org/gitlab-ce!30893
Diffstat (limited to 'spec')
-rw-r--r-- | spec/support/helpers/test_env.rb | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/spec/support/helpers/test_env.rb b/spec/support/helpers/test_env.rb index 893b10ea752..16938da50cc 100644 --- a/spec/support/helpers/test_env.rb +++ b/spec/support/helpers/test_env.rb @@ -141,14 +141,6 @@ module TestEnv FileUtils.mkdir_p(artifacts_path) end - def clean_gitlab_test_path - Dir[TMP_TEST_PATH].each do |entry| - unless test_dirs.include?(File.basename(entry)) - FileUtils.rm_rf(entry) - end - end - end - def setup_gitlab_shell component_timed_setup('GitLab Shell', install_dir: Gitlab.config.gitlab_shell.path, @@ -368,10 +360,7 @@ module TestEnv # Try to reset without fetching to avoid using the network. unless reset.call raise 'Could not fetch test seed repository.' unless system(*%W(#{Gitlab.config.git.bin_path} -C #{repo_path} fetch origin)) - - # Before we used Git clone's --mirror option, bare repos could end up - # with missing refs, clearing them and retrying should fix the issue. - clean_gitlab_test_path && init unless reset.call + raise "Could not update test seed repository, please delete #{repo_path} and try again" unless reset.call end end |