Load test resources from classpath (#1823)

mwiede · eddumelendez · web-flow · commit 7ce35f42b813 · 2023-03-08T18:02:33.000-06:00
Co-authored-by: Eddú Meléndez &lt;eddu.melendez@gmail.com&gt;
diff --git a/docker-java/src/test/java/com/github/dockerjava/cmd/CopyArchiveFromContainerCmdIT.java b/docker-java/src/test/java/com/github/dockerjava/cmd/CopyArchiveFromContainerCmdIT.java
@@ -66,7 +66,7 @@ public void copyFromContainerBinaryFile() throws Exception {
         assertThat(container.getId(), not(isEmptyOrNullString()));
 
         Path temp = Files.createTempFile("", ".tar.gz");
-        Path binaryFile = Paths.get("src/test/resources/testCopyFromArchive/binary.dat");
+        Path binaryFile = Paths.get(ClassLoader.getSystemResource("testCopyFromArchive/binary.dat").toURI());
         CompressArchiveUtil.tar(binaryFile, temp, true, false);
 
         try (InputStream uploadStream = Files.newInputStream(temp)) {
diff --git a/docker-java/src/test/java/com/github/dockerjava/cmd/CopyArchiveToContainerCmdIT.java b/docker-java/src/test/java/com/github/dockerjava/cmd/CopyArchiveToContainerCmdIT.java
@@ -33,7 +33,7 @@ public class CopyArchiveToContainerCmdIT extends CmdIT {
     public void copyFileToContainer() throws Exception {
         CreateContainerResponse container = prepareContainerForCopy("1");
         Path temp = Files.createTempFile("", ".tar.gz");
-        CompressArchiveUtil.tar(Paths.get("src/test/resources/testReadFile"), temp, true, false);
+        CompressArchiveUtil.tar(Paths.get(ClassLoader.getSystemResource("testReadFile").toURI()), temp, true, false);
         try (InputStream uploadStream = Files.newInputStream(temp)) {
             dockerRule.getClient()
                     .copyArchiveToContainerCmd(container.getId())
@@ -47,16 +47,16 @@ public void copyFileToContainer() throws Exception {
     public void copyStreamToContainer() throws Exception {
         CreateContainerResponse container = prepareContainerForCopy("2");
         dockerRule.getClient().copyArchiveToContainerCmd(container.getId())
-                .withHostResource("src/test/resources/testReadFile")
+                .withHostResource(Paths.get(ClassLoader.getSystemResource("testReadFile").toURI()).toString())
                 .exec();
         assertFileCopied(container);
     }
 
     @Test
     public void copyStreamToContainerTwice() throws Exception {
         CreateContainerResponse container = prepareContainerForCopy("rerun");
-        CopyArchiveToContainerCmd copyArchiveToContainerCmd=dockerRule.getClient().copyArchiveToContainerCmd(container.getId())
-                .withHostResource("src/test/resources/testReadFile");
+        CopyArchiveToContainerCmd copyArchiveToContainerCmd = dockerRule.getClient().copyArchiveToContainerCmd(container.getId())
+                .withHostResource(Paths.get(ClassLoader.getSystemResource("testReadFile").toURI()).toString());
         copyArchiveToContainerCmd.exec();
         assertFileCopied(container);
         //run again to make sure no DockerClientException
@@ -82,9 +82,9 @@ private void assertFileCopied(CreateContainerResponse container) throws IOExcept
     }
 
     @Test(expected = NotFoundException.class)
-    public void copyToNonExistingContainer() {
-
-        dockerRule.getClient().copyArchiveToContainerCmd("non-existing").withHostResource("src/test/resources/testReadFile").exec();
+    public void copyToNonExistingContainer() throws Exception {
+        dockerRule.getClient().copyArchiveToContainerCmd("non-existing")
+                .withHostResource(Paths.get(ClassLoader.getSystemResource("testReadFile").toURI()).toString()).exec();
     }
 
     @Test
@@ -113,7 +113,7 @@ public void copyDirWithLastAddedTarEntryEmptyDir() throws Exception{
         // cleanup dir
         FileUtils.deleteDirectory(localDir.toFile());
     }
-    
+
     @Test
     public void copyFileWithExecutePermission() throws Exception {
         // create script file, add permission to execute
diff --git a/docker-java/src/test/java/com/github/dockerjava/utils/TestResources.java b/docker-java/src/test/java/com/github/dockerjava/utils/TestResources.java
@@ -1,5 +1,6 @@
 package com.github.dockerjava.utils;
 
+import java.net.URISyntaxException;
 import java.nio.file.Path;
 import java.nio.file.Paths;
 
@@ -8,7 +9,7 @@ public class TestResources {
     private TestResources() {
     }
 
-    public static Path getApiImagesLoadTestTarball() {
-        return Paths.get("src/test/resources/api/images/load/image.tar");
+    public static Path getApiImagesLoadTestTarball() throws URISyntaxException {
+        return Paths.get(ClassLoader.getSystemResource("api/images/load/image.tar").toURI());
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,6 @@`
`1`	`1`	`package com.github.dockerjava.utils;`
`2`	`2`
	`3`	`+import java.net.URISyntaxException;`
`3`	`4`	`import java.nio.file.Path;`
`4`	`5`	`import java.nio.file.Paths;`
`5`	`6`
`@@ -8,7 +9,7 @@ public class TestResources {`
`8`	`9`	`private TestResources() {`
`9`	`10`	`}`
`10`	`11`
`11`		`- public static Path getApiImagesLoadTestTarball() {`
`12`		`- return Paths.get("src/test/resources/api/images/load/image.tar");`
	`12`	`+ public static Path getApiImagesLoadTestTarball() throws URISyntaxException {`
	`13`	`+ return Paths.get(ClassLoader.getSystemResource("api/images/load/image.tar").toURI());`
`13`	`14`	`}`
`14`	`15`	`}`