aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/test/java/com/google/devtools/build/android/ideinfo/PackageParserTest.java
diff options
context:
space:
mode:
Diffstat (limited to 'src/test/java/com/google/devtools/build/android/ideinfo/PackageParserTest.java')
-rw-r--r--src/test/java/com/google/devtools/build/android/ideinfo/PackageParserTest.java219
1 files changed, 178 insertions, 41 deletions
diff --git a/src/test/java/com/google/devtools/build/android/ideinfo/PackageParserTest.java b/src/test/java/com/google/devtools/build/android/ideinfo/PackageParserTest.java
index a02ac30baf..3f466d6d79 100644
--- a/src/test/java/com/google/devtools/build/android/ideinfo/PackageParserTest.java
+++ b/src/test/java/com/google/devtools/build/android/ideinfo/PackageParserTest.java
@@ -17,9 +17,11 @@ package com.google.devtools.build.android.ideinfo;
import static com.google.common.truth.Truth.assertThat;
import static org.junit.Assert.fail;
+import com.google.common.base.Joiner;
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.Lists;
import com.google.common.collect.Maps;
+import com.google.devtools.build.lib.ideinfo.androidstudio.PackageManifestOuterClass.ArtifactLocation;
import com.google.protobuf.MessageLite;
import org.junit.Before;
@@ -35,6 +37,7 @@ import java.io.InputStreamReader;
import java.io.StringWriter;
import java.io.UnsupportedEncodingException;
import java.nio.charset.StandardCharsets;
+import java.nio.file.InvalidPathException;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.util.List;
@@ -50,12 +53,16 @@ public class PackageParserTest {
private static class MockPackageParserIoProvider extends PackageParserIoProvider {
private final Map<Path, InputStream> sources = Maps.newHashMap();
+ private final List<ArtifactLocation> sourceLocations = Lists.newArrayList();
private StringWriter writer = new StringWriter();
- public MockPackageParserIoProvider addSource(String filePath, String javaSrc) {
+ public MockPackageParserIoProvider addSource(ArtifactLocation source, String javaSrc) {
try {
- sources.put(Paths.get(filePath), new ByteArrayInputStream(javaSrc.getBytes("UTF-8")));
- } catch (UnsupportedEncodingException e) {
+ Path path = Paths.get(source.getRootExecutionPathFragment(), source.getRelativePath());
+ sources.put(path, new ByteArrayInputStream(javaSrc.getBytes("UTF-8")));
+ sourceLocations.add(source);
+
+ } catch (UnsupportedEncodingException | InvalidPathException e) {
fail(e.getMessage());
}
return this;
@@ -63,11 +70,12 @@ public class PackageParserTest {
public void reset() {
sources.clear();
+ sourceLocations.clear();
writer = new StringWriter();
}
- public List<Path> getPaths() {
- return Lists.newArrayList(sources.keySet());
+ public List<ArtifactLocation> getSourceLocations() {
+ return Lists.newArrayList(sourceLocations);
}
@Nonnull
@@ -83,6 +91,21 @@ public class PackageParserTest {
}
}
+ private static final ArtifactLocation DUMMY_SOURCE_ARTIFACT =
+ ArtifactLocation.newBuilder()
+ .setRootPath("/usr/local/google/code")
+ .setRelativePath("java/com/google/Foo.java")
+ .setIsSource(true)
+ .build();
+
+ private static final ArtifactLocation DUMMY_DERIVED_ARTIFACT =
+ ArtifactLocation.newBuilder()
+ .setRootPath("/usr/local/_tmp/code/bin")
+ .setRootExecutionPathFragment("bin")
+ .setRelativePath("java/com/google/Bla.java")
+ .setIsSource(false)
+ .build();
+
private MockPackageParserIoProvider mockIoProvider;
private PackageParser parser;
@@ -92,9 +115,9 @@ public class PackageParserTest {
parser = new PackageParser(mockIoProvider);
}
- private Map<Path, String> parsePackageStrings() throws Exception {
- List<Path> paths = mockIoProvider.getPaths();
- return parser.parsePackageStrings(paths, paths);
+ private Map<ArtifactLocation, String> parsePackageStrings() throws Exception {
+ List<ArtifactLocation> sources = mockIoProvider.getSourceLocations();
+ return parser.parsePackageStrings(sources);
}
@Test
@@ -102,91 +125,205 @@ public class PackageParserTest {
String[] args = new String[] {
"--output_manifest",
"/tmp/out.manifest",
- "--sources_absolute_paths",
- "/path/test1.java:/path/test2.java",
- "--sources_execution_paths",
- "/path/test1.java:/path/test2.java"
- };
+ "--sources",
+ Joiner.on(":").join(
+ ",java/com/google/Foo.java,/usr/local/google/code",
+ "bin,java/com/google/Bla.java,/usr/local/_tmp/code/bin"
+ )};
PackageParser.PackageParserOptions options = PackageParser.parseArgs(args);
assertThat(options.outputManifest.toString()).isEqualTo("/tmp/out.manifest");
- assertThat(options.sourcesAbsolutePaths).hasSize(2);
- assertThat(options.sourcesExecutionPaths).hasSize(2);
- assertThat(options.sourcesAbsolutePaths.get(0).toString()).isEqualTo("/path/test1.java");
- assertThat(options.sourcesAbsolutePaths.get(1).toString()).isEqualTo("/path/test2.java");
+ assertThat(options.sources).hasSize(2);
+ assertThat(options.sources.get(0)).isEqualTo(
+ ArtifactLocation.newBuilder()
+ .setRootPath("/usr/local/google/code")
+ .setRelativePath("java/com/google/Foo.java")
+ .setIsSource(true)
+ .build());
+ assertThat(options.sources.get(1)).isEqualTo(
+ ArtifactLocation.newBuilder()
+ .setRootPath("/usr/local/_tmp/code/bin")
+ .setRootExecutionPathFragment("bin")
+ .setRelativePath("java/com/google/Bla.java")
+ .setIsSource(false)
+ .build());
}
@Test
public void testReadNoSources() throws Exception {
- Map<Path, String> map = parsePackageStrings();
+ Map<ArtifactLocation, String> map = parsePackageStrings();
assertThat(map).isEmpty();
}
@Test
public void testSingleRead() throws Exception {
mockIoProvider
- .addSource("java/com/google/Bla.java",
- "package com.test;\n public class Bla {}\"");
- Map<Path, String> map = parsePackageStrings();
+ .addSource(
+ DUMMY_SOURCE_ARTIFACT,
+ "package com.google;\n public class Bla {}\"");
+ Map<ArtifactLocation, String> map = parsePackageStrings();
assertThat(map).hasSize(1);
- assertThat(map).containsEntry(Paths.get("java/com/google/Bla.java"), "com.test");
+ assertThat(map).containsEntry(
+ DUMMY_SOURCE_ARTIFACT,
+ "com.google");
}
@Test
public void testMultiRead() throws Exception {
mockIoProvider
- .addSource("java/com/google/Bla.java",
+ .addSource(
+ DUMMY_SOURCE_ARTIFACT,
"package com.test;\n public class Bla {}\"")
- .addSource("java/com/other/Foo.java",
+ .addSource(
+ DUMMY_DERIVED_ARTIFACT,
"package com.other;\n public class Foo {}\"");
- Map<Path, String> map = parsePackageStrings();
+ Map<ArtifactLocation, String> map = parsePackageStrings();
assertThat(map).hasSize(2);
- assertThat(map).containsEntry(Paths.get("java/com/google/Bla.java"), "com.test");
- assertThat(map).containsEntry(Paths.get("java/com/other/Foo.java"), "com.other");
+ assertThat(map).containsEntry(
+ DUMMY_SOURCE_ARTIFACT,
+ "com.test");
+ assertThat(map).containsEntry(
+ DUMMY_DERIVED_ARTIFACT,
+ "com.other");
}
@Test
public void testReadSomeInvalid() throws Exception {
mockIoProvider
- .addSource("java/com/google/Bla.java",
+ .addSource(
+ DUMMY_SOURCE_ARTIFACT,
"package %com.test;\n public class Bla {}\"")
- .addSource("java/com/other/Foo.java",
+ .addSource(
+ DUMMY_DERIVED_ARTIFACT,
"package com.other;\n public class Foo {}\"");
- Map<Path, String> map = parsePackageStrings();
+ Map<ArtifactLocation, String> map = parsePackageStrings();
assertThat(map).hasSize(1);
- assertThat(map).containsEntry(Paths.get("java/com/other/Foo.java"), "com.other");
+ assertThat(map).containsEntry(
+ DUMMY_DERIVED_ARTIFACT,
+ "com.other");
}
@Test
public void testReadAllInvalid() throws Exception {
mockIoProvider
- .addSource("java/com/google/Bla.java",
+ .addSource(
+ DUMMY_SOURCE_ARTIFACT,
"#package com.test;\n public class Bla {}\"")
- .addSource("java/com/other/Foo.java",
+ .addSource(
+ DUMMY_DERIVED_ARTIFACT,
"package com.other\n public class Foo {}\"");
- Map<Path, String> map = parsePackageStrings();
+ Map<ArtifactLocation, String> map = parsePackageStrings();
assertThat(map).isEmpty();
}
@Test
public void testWriteEmptyMap() throws Exception {
parser.writeManifest(
- Maps.<Path, String> newHashMap(), Paths.get("/java/com/google/test.manifest"));
+ Maps.<ArtifactLocation, String> newHashMap(),
+ Paths.get("/java/com/google/test.manifest"),
+ false);
assertThat(mockIoProvider.writer.toString()).isEmpty();
}
@Test
public void testWriteMap() throws Exception {
- Map<Path, String> map = ImmutableMap.of(
- Paths.get("/java/com/google/Bla.java"), "com.google",
- Paths.get("/java/com/other/Foo.java"), "com.other"
+ Map<ArtifactLocation, String> map = ImmutableMap.of(
+ DUMMY_SOURCE_ARTIFACT,
+ "com.google",
+ DUMMY_DERIVED_ARTIFACT,
+ "com.other"
);
- parser.writeManifest(map, Paths.get("/java/com/google/test.manifest"));
+ parser.writeManifest(map, Paths.get("/java/com/google/test.manifest"), false);
String writtenString = mockIoProvider.writer.toString();
- assertThat(writtenString).contains("absolute_path: \"/java/com/google/Bla.java\"");
+ assertThat(writtenString).contains(String.format(
+ "root_path: \"%s\"",
+ DUMMY_SOURCE_ARTIFACT.getRootPath()));
+ assertThat(writtenString).contains(String.format(
+ "relative_path: \"%s\"",
+ DUMMY_SOURCE_ARTIFACT.getRelativePath()));
assertThat(writtenString).contains("package_string: \"com.google\"");
- assertThat(writtenString).contains("absolute_path: \"/java/com/other/Foo.java\"");
+
+ assertThat(writtenString).contains(String.format(
+ "root_path: \"%s\"",
+ DUMMY_DERIVED_ARTIFACT.getRootPath()));
+ assertThat(writtenString).contains(String.format(
+ "root_execution_path_fragment: \"%s\"",
+ DUMMY_DERIVED_ARTIFACT.getRootExecutionPathFragment()));
+ assertThat(writtenString).contains(String.format(
+ "relative_path: \"%s\"",
+ DUMMY_DERIVED_ARTIFACT.getRelativePath()));
assertThat(writtenString).contains("package_string: \"com.other\"");
}
+ @Test
+ public void testHandlesOldFormat() throws Exception {
+ String[] args = new String[] {
+ "--output_manifest",
+ "/tmp/out.manifest",
+ "--sources_absolute_paths",
+ "/usr/local/code/java/com/google/Foo.java:/usr/local/code/java/com/google/Bla.java",
+ "--sources_execution_paths",
+ "java/com/google/Foo.java:java/com/google/Bla.java"
+ };
+ PackageParser.PackageParserOptions options = PackageParser.parseArgs(args);
+ assertThat(options.outputManifest.toString()).isEqualTo("/tmp/out.manifest");
+ assertThat(options.sourcesAbsolutePaths.get(0).toString())
+ .isEqualTo("/usr/local/code/java/com/google/Foo.java");
+ assertThat(options.sourcesAbsolutePaths.get(1).toString())
+ .isEqualTo("/usr/local/code/java/com/google/Bla.java");
+
+ PackageParser.convertFromOldFormat(options);
+ assertThat(options.sources).hasSize(2);
+ assertThat(options.sources.get(0)).isEqualTo(
+ ArtifactLocation.newBuilder()
+ .setRootPath("/usr/local/code/")
+ .setRelativePath("java/com/google/Foo.java")
+ .build());
+
+ mockIoProvider
+ .addSource(options.sources.get(0),
+ "package com.test;\n public class Bla {}\"")
+ .addSource(options.sources.get(1),
+ "package com.other;\n public class Foo {}\"");
+
+ Map<ArtifactLocation, String> map = parser.parsePackageStrings(options.sources);
+ assertThat(map).hasSize(2);
+ assertThat(map).containsEntry(options.sources.get(0), "com.test");
+ assertThat(map).containsEntry(options.sources.get(1), "com.other");
+
+ parser.writeManifest(map, options.outputManifest, true);
+
+ String writtenString = mockIoProvider.writer.toString();
+ assertThat(writtenString).doesNotContain("location");
+ assertThat(writtenString).contains(
+ "absolute_path: \"/usr/local/code/java/com/google/Foo.java\"");
+ assertThat(writtenString).contains(
+ "absolute_path: \"/usr/local/code/java/com/google/Bla.java\"");
+ }
+
+ @Test
+ public void testHandlesFutureFormat() throws Exception {
+ String[] args = new String[] {
+ "--output_manifest",
+ "/tmp/out.manifest",
+ "--sources",
+ ",java/com/google/Foo.java:"
+ + "bin/out,java/com/google/Bla.java",
+ };
+ PackageParser.PackageParserOptions options = PackageParser.parseArgs(args);
+ assertThat(options.outputManifest.toString()).isEqualTo("/tmp/out.manifest");
+ assertThat(options.sources).hasSize(2);
+ assertThat(options.sources.get(0)).isEqualTo(
+ ArtifactLocation.newBuilder()
+ .setRelativePath("java/com/google/Foo.java")
+ .setIsSource(true)
+ .build());
+ assertThat(options.sources.get(1)).isEqualTo(
+ ArtifactLocation.newBuilder()
+ .setRootExecutionPathFragment("bin/out")
+ .setRelativePath("java/com/google/Bla.java")
+ .setIsSource(false)
+ .build());
+ }
+
}
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-01 01:28:47 +0000