test fix - run test container as non-root.

k-wall · k-wall · commit daf854379ce9 · 2022-09-28T15:08:16.000+01:00
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
@@ -16,6 +16,8 @@
 import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.util.concurrent.TimeUnit;
+import java.util.regex.Matcher;
+import java.util.regex.Pattern;
 
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.Matchers.containsString;
@@ -158,10 +160,11 @@ public void copyFileWithUIDGID() throws Exception {
         Path without = Files.createFile(Files.createTempDirectory("copyFileWithUIDGID").resolve("uidgid.without"));
         Files.write(without, "with".getBytes());
 
-        String containerCmd = "while [ ! -f /home/uidgid.with ]; do true; done && stat -c %n:%u /home/uidgid.with /home/uidgid.without";
+        String containerCmd = "while [ ! -f /home/uidgid.with ]; do true; done && echo uid=$(id -u) && stat -c %n:%u /home/uidgid.with /home/uidgid.without";
         CreateContainerResponse container = dockerRule.getClient().createContainerCmd("busybox")
                 .withName("copyFileWithUIDGID")
                 .withCmd("/bin/sh", "-c", containerCmd)
+                .withUser("sync")
                 .exec();
         // start the container
         dockerRule.getClient().startContainerCmd(container.getId()).exec();
@@ -192,11 +195,18 @@ public void copyFileWithUIDGID() throws Exception {
             .exec(loggingCallback);
 
         loggingCallback.awaitCompletion(3, TimeUnit.SECONDS);
-        assertThat(loggingCallback.toString(), containsString("/home/uidgid.with:0"));
+        String containerOutput = loggingCallback.toString();
+
+        Matcher uidMatcher = Pattern.compile("uid=(\\d+)").matcher(containerOutput);
+        assertThat(String.format("cannot read effective uid on container from '%s'", containerOutput), uidMatcher.find(), equalTo(true));
+        assertThat(String.format("cannot read effective uid on container from '%s'", containerOutput), uidMatcher.groupCount(), equalTo(1));
+        Long containerEffectiveUid = Long.parseLong(uidMatcher.group(1));
+
+        assertThat(containerOutput, containsString(String.format("/home/uidgid.with:%d", containerEffectiveUid)));
 
         Long hostUid = getHostUidIfPossible();
         assumeThat("could not get the uid on host platform", hostUid, notNullValue(Long.class));
-        assertThat(loggingCallback.toString(), containsString(String.format("/home/uidgid.without:%d", hostUid)));
+        assertThat(containerOutput, containsString(String.format("/home/uidgid.without:%d", hostUid)));
     }
 
     private static Long getHostUidIfPossible() {
