Apply review feedback

testcontainers · Apr 7, 2022 · 52a5c4b · 52a5c4b
1 parent 4f28875
commit 52a5c4b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 9 deletions.
diff --git a/core/src/main/java/org/testcontainers/containers/GenericContainer.java b/core/src/main/java/org/testcontainers/containers/GenericContainer.java
@@ -543,14 +543,12 @@ Checksum hashCopiedFiles() {
             byte[] bytes = string.getBytes();
             checksum.update(bytes, 0, bytes.length);
         };
-        copyToFileContainerPathMap.entrySet().stream().sorted(Entry.comparingByValue()).forEach(entry -> {
-            updateFromString.accept(entry.getValue());
-            entry.getKey().updateChecksum(checksum);
-        });
-        copyToTransferableContainerPathMap.entrySet().stream().sorted(Entry.comparingByValue()).forEach(entry -> {
-            updateFromString.accept(entry.getValue());
-            entry.getKey().updateChecksum(checksum);
-        });
+        Stream.of(copyToFileContainerPathMap, copyToTransferableContainerPathMap)
+            .flatMap(it -> it.entrySet().stream())
+            .sorted(Entry.comparingByValue()).forEach(entry -> {
+                updateFromString.accept(entry.getValue());
+                entry.getKey().updateChecksum(checksum);
+            });
         return checksum;
     }
 

diff --git a/core/src/main/java/org/testcontainers/images/builder/Transferable.java b/core/src/main/java/org/testcontainers/images/builder/Transferable.java
@@ -35,7 +35,6 @@ public byte[] getBytes() {
 
             @Override
             public void updateChecksum(Checksum checksum) {
-                byte[] bytes = getBytes();
                 checksum.update(bytes, 0, bytes.length);
             }