Set workspace name for http_file, http_jar, and maven_jar

Fixes #1166.

While I was here, I also:

* Renamed the implementations of Decompressor to show that they are
Decompressors, not SkyFunctions (they used to be, it's confusing to have them
still have the -Function suffix).
* Added a unit test and moved the testing target to the rule/repository
subdirectory, since it was a good chance to break off a tiny chunk of the
monolithic BUILD file in lib/.

--
MOS_MIGRATED_REVID=121306165

1 files changed, 1 insertions, 2 deletions

diff --git a/src/test/java/com/google/devtools/build/lib/rules/repository/RepositoryFunctionTest.java b/src/test/java/com/google/devtools/build/lib/rules/repository/RepositoryFunctionTest.java
index cbc65dd104..3cc0a84d24 100644
--- a/src/test/java/com/google/devtools/build/lib/rules/repository/RepositoryFunctionTest.java
+++ b/src/test/java/com/google/devtools/build/lib/rules/repository/RepositoryFunctionTest.java
@@ -91,8 +91,7 @@ public class RepositoryFunctionTest extends BuildViewTestCase {
         "    name = 'abc',",
         "    path = '/a/b/c',",
         ")");
-    TestingRepositoryFunction repositoryFunction = new TestingRepositoryFunction();
-    repositoryFunction.createWorkspaceFile(rootDirectory, rule);
+    RepositoryFunction.createWorkspaceFile(rootDirectory, rule.getTargetKind(), rule.getName());
     String workspaceContent = new String(
         FileSystemUtils.readContentAsLatin1(rootDirectory.getRelative("WORKSPACE")));
     assertThat(workspaceContent).contains("workspace(name = \"abc\")");