Automated rollback of commit 0a63dadf4eafbae1730f88644b0c537fb1c16d5c.

*** Reason for rollback ***

This has broken rules_docker: https://ci.bazel.io/blue/organizations/jenkins/Global%2Frules_docker/detail/rules_docker/243/pipeline/

*** Original change description ***

Properly escape GitHub repository download URL for git_repository w/ sha256

Addresses #3661

Closes #3770.

PiperOrigin-RevId: 173525982

1 files changed, 0 insertions, 10 deletions

diff --git a/src/test/java/com/google/devtools/build/lib/bazel/repository/GitClonerTest.java b/src/test/java/com/google/devtools/build/lib/bazel/repository/GitClonerTest.java
index 738058c421..10b668b171 100644
--- a/src/test/java/com/google/devtools/build/lib/bazel/repository/GitClonerTest.java
+++ b/src/test/java/com/google/devtools/build/lib/bazel/repository/GitClonerTest.java
@@ -19,11 +19,9 @@ import static org.mockito.Matchers.any;
 import static org.mockito.Matchers.anyListOf;
 import static org.mockito.Matchers.anyMapOf;
 import static org.mockito.Matchers.eq;
-import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
 import com.google.common.base.Optional;
-import com.google.common.collect.ImmutableList;
 import com.google.common.collect.Maps;
 import com.google.devtools.build.lib.analysis.util.BuildViewTestCase;
 import com.google.devtools.build.lib.bazel.repository.cache.RepositoryCache;
@@ -84,14 +82,6 @@ public class GitClonerTest extends BuildViewTestCase {
 
     HttpDownloadValue value = GitCloner.clone(
         rule, outputDirectory, eventHandler, clientEnvironment, downloader);
-    verify(downloader)
-        .download(
-            eq(ImmutableList.of(new URL("https://github.com/foo/bar/archive/1.2.3.tar.gz"))),
-            any(String.class),
-            eq(Optional.of("tar.gz")),
-            eq(outputDirectory),
-            any(ExtendedEventHandler.class),
-            anyMapOf(String.class, String.class));
     assertThat(value).isNotNull();
     assertThat(value.getPath()).isEqualTo(outputDirectory);
   }