You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by vi...@apache.org on 2020/04/29 05:57:40 UTC
[incubator-hudi] branch master updated: [HUDI-814] Migrate
hudi-client tests to JUnit 5 (#1570)
This is an automated email from the ASF dual-hosted git repository.
vinoyang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-hudi.git
The following commit(s) were added to refs/heads/master by this push:
new 69b1630 [HUDI-814] Migrate hudi-client tests to JUnit 5 (#1570)
69b1630 is described below
commit 69b16309c8c46f831c8b9be42de8b2e29c74f03e
Author: Raymond Xu <27...@users.noreply.github.com>
AuthorDate: Tue Apr 28 22:57:28 2020 -0700
[HUDI-814] Migrate hudi-client tests to JUnit 5 (#1570)
---
.../hudi/common/config/TestHoodieWriteConfig.java | 8 ++--
.../bloom/TestBucketizedBloomCheckPartitioner.java | 24 +++++-----
.../hudi/index/bloom/TestKeyRangeLookupTree.java | 4 +-
.../strategy/TestHoodieCompactionStrategy.java | 54 +++++++++++-----------
4 files changed, 44 insertions(+), 46 deletions(-)
diff --git a/hudi-client/src/test/java/org/apache/hudi/common/config/TestHoodieWriteConfig.java b/hudi-client/src/test/java/org/apache/hudi/common/config/TestHoodieWriteConfig.java
index 3516a6a..a1904a5 100644
--- a/hudi-client/src/test/java/org/apache/hudi/common/config/TestHoodieWriteConfig.java
+++ b/hudi-client/src/test/java/org/apache/hudi/common/config/TestHoodieWriteConfig.java
@@ -22,7 +22,7 @@ import org.apache.hudi.config.HoodieCompactionConfig;
import org.apache.hudi.config.HoodieWriteConfig;
import org.apache.hudi.config.HoodieWriteConfig.Builder;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import java.io.ByteArrayInputStream;
import java.io.ByteArrayOutputStream;
@@ -32,7 +32,7 @@ import java.util.HashMap;
import java.util.Map;
import java.util.Properties;
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
public class TestHoodieWriteConfig {
@@ -52,8 +52,8 @@ public class TestHoodieWriteConfig {
inputStream.close();
}
HoodieWriteConfig config = builder.build();
- assertEquals(config.getMaxCommitsToKeep(), 5);
- assertEquals(config.getMinCommitsToKeep(), 2);
+ assertEquals(5, config.getMaxCommitsToKeep());
+ assertEquals(2, config.getMinCommitsToKeep());
}
private ByteArrayOutputStream saveParamsIntoOutputStream(Map<String, String> params) throws IOException {
diff --git a/hudi-client/src/test/java/org/apache/hudi/index/bloom/TestBucketizedBloomCheckPartitioner.java b/hudi-client/src/test/java/org/apache/hudi/index/bloom/TestBucketizedBloomCheckPartitioner.java
index 3ad5a99..e946450 100644
--- a/hudi-client/src/test/java/org/apache/hudi/index/bloom/TestBucketizedBloomCheckPartitioner.java
+++ b/hudi-client/src/test/java/org/apache/hudi/index/bloom/TestBucketizedBloomCheckPartitioner.java
@@ -20,7 +20,7 @@ package org.apache.hudi.index.bloom;
import org.apache.hudi.common.util.collection.Pair;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import java.util.HashMap;
import java.util.List;
@@ -28,9 +28,9 @@ import java.util.Map;
import java.util.stream.Collectors;
import java.util.stream.IntStream;
-import static org.junit.Assert.assertArrayEquals;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertArrayEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
public class TestBucketizedBloomCheckPartitioner {
@@ -45,12 +45,12 @@ public class TestBucketizedBloomCheckPartitioner {
};
BucketizedBloomCheckPartitioner p = new BucketizedBloomCheckPartitioner(4, fileToComparisons, 10);
Map<String, List<Integer>> assignments = p.getFileGroupToPartitions();
- assertEquals("f1 should have 4 buckets", 4, assignments.get("f1").size());
- assertEquals("f2 should have 4 buckets", 4, assignments.get("f2").size());
- assertEquals("f3 should have 2 buckets", 2, assignments.get("f3").size());
- assertArrayEquals("f1 spread across 3 partitions", new Integer[] {0, 0, 1, 3}, assignments.get("f1").toArray());
- assertArrayEquals("f2 spread across 3 partitions", new Integer[] {1, 2, 2, 0}, assignments.get("f2").toArray());
- assertArrayEquals("f3 spread across 2 partitions", new Integer[] {3, 1}, assignments.get("f3").toArray());
+ assertEquals(4, assignments.get("f1").size(), "f1 should have 4 buckets");
+ assertEquals(4, assignments.get("f2").size(), "f2 should have 4 buckets");
+ assertEquals(2, assignments.get("f3").size(), "f3 should have 2 buckets");
+ assertArrayEquals(new Integer[] {0, 0, 1, 3}, assignments.get("f1").toArray(), "f1 spread across 3 partitions");
+ assertArrayEquals(new Integer[] {1, 2, 2, 0}, assignments.get("f2").toArray(), "f2 spread across 3 partitions");
+ assertArrayEquals(new Integer[] {3, 1}, assignments.get("f3").toArray(), "f3 spread across 2 partitions");
}
@Test
@@ -78,7 +78,7 @@ public class TestBucketizedBloomCheckPartitioner {
}
};
BucketizedBloomCheckPartitioner p = new BucketizedBloomCheckPartitioner(10000, comparisons1, 10);
- assertEquals("num partitions must equal total buckets", 100, p.numPartitions());
+ assertEquals(100, p.numPartitions(), "num partitions must equal total buckets");
}
@Test
@@ -92,7 +92,7 @@ public class TestBucketizedBloomCheckPartitioner {
IntStream.range(0, 100000).forEach(f -> {
int partition = p.getPartition(Pair.of("f" + f, "value"));
- assertTrue("partition is out of range: " + partition, 0 <= partition && partition <= 1000);
+ assertTrue(0 <= partition && partition <= 1000, "partition is out of range: " + partition);
});
}
diff --git a/hudi-client/src/test/java/org/apache/hudi/index/bloom/TestKeyRangeLookupTree.java b/hudi-client/src/test/java/org/apache/hudi/index/bloom/TestKeyRangeLookupTree.java
index b127045..10232ca 100644
--- a/hudi-client/src/test/java/org/apache/hudi/index/bloom/TestKeyRangeLookupTree.java
+++ b/hudi-client/src/test/java/org/apache/hudi/index/bloom/TestKeyRangeLookupTree.java
@@ -18,7 +18,7 @@
package org.apache.hudi.index.bloom;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import java.util.Collections;
import java.util.HashMap;
@@ -27,7 +27,7 @@ import java.util.Map;
import java.util.Random;
import java.util.UUID;
-import static junit.framework.TestCase.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
/**
* Tests {@link KeyRangeLookupTree}.
diff --git a/hudi-client/src/test/java/org/apache/hudi/table/action/compact/strategy/TestHoodieCompactionStrategy.java b/hudi-client/src/test/java/org/apache/hudi/table/action/compact/strategy/TestHoodieCompactionStrategy.java
index e49aafa..44899c3 100644
--- a/hudi-client/src/test/java/org/apache/hudi/table/action/compact/strategy/TestHoodieCompactionStrategy.java
+++ b/hudi-client/src/test/java/org/apache/hudi/table/action/compact/strategy/TestHoodieCompactionStrategy.java
@@ -27,8 +27,7 @@ import org.apache.hudi.config.HoodieCompactionConfig;
import org.apache.hudi.config.HoodieWriteConfig;
import org.apache.hadoop.fs.Path;
-import org.junit.Assert;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import java.text.SimpleDateFormat;
import java.util.ArrayList;
@@ -42,8 +41,8 @@ import java.util.Random;
import java.util.UUID;
import java.util.stream.Collectors;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
public class TestHoodieCompactionStrategy {
@@ -63,7 +62,7 @@ public class TestHoodieCompactionStrategy {
.withCompactionConfig(HoodieCompactionConfig.newBuilder().withCompactionStrategy(strategy).build()).build();
List<HoodieCompactionOperation> operations = createCompactionOperations(writeConfig, sizesMap);
List<HoodieCompactionOperation> returned = strategy.orderAndFilter(writeConfig, operations, new ArrayList<>());
- assertEquals("UnBounded should not re-order or filter", operations, returned);
+ assertEquals(operations, returned, "UnBounded should not re-order or filter");
}
@Test
@@ -80,13 +79,13 @@ public class TestHoodieCompactionStrategy {
List<HoodieCompactionOperation> operations = createCompactionOperations(writeConfig, sizesMap);
List<HoodieCompactionOperation> returned = strategy.orderAndFilter(writeConfig, operations, new ArrayList<>());
- assertTrue("BoundedIOCompaction should have resulted in fewer compactions", returned.size() < operations.size());
- assertEquals("BoundedIOCompaction should have resulted in 2 compactions being chosen", 2, returned.size());
+ assertTrue(returned.size() < operations.size(), "BoundedIOCompaction should have resulted in fewer compactions");
+ assertEquals(2, returned.size(), "BoundedIOCompaction should have resulted in 2 compactions being chosen");
// Total size of all the log files
Long returnedSize = returned.stream().map(s -> s.getMetrics().get(BoundedIOCompactionStrategy.TOTAL_IO_MB))
.map(Double::longValue).reduce(Long::sum).orElse(0L);
- assertEquals("Should chose the first 2 compactions which should result in a total IO of 690 MB", 610,
- (long) returnedSize);
+ assertEquals(610, (long) returnedSize,
+ "Should chose the first 2 compactions which should result in a total IO of 690 MB");
}
@Test
@@ -103,14 +102,14 @@ public class TestHoodieCompactionStrategy {
List<HoodieCompactionOperation> operations = createCompactionOperations(writeConfig, sizesMap);
List<HoodieCompactionOperation> returned = strategy.orderAndFilter(writeConfig, operations, new ArrayList<>());
- assertTrue("LogFileSizeBasedCompactionStrategy should have resulted in fewer compactions",
- returned.size() < operations.size());
- assertEquals("LogFileSizeBasedCompactionStrategy should have resulted in 1 compaction", 1, returned.size());
+ assertTrue(returned.size() < operations.size(),
+ "LogFileSizeBasedCompactionStrategy should have resulted in fewer compactions");
+ assertEquals(1, returned.size(), "LogFileSizeBasedCompactionStrategy should have resulted in 1 compaction");
// Total size of all the log files
Long returnedSize = returned.stream().map(s -> s.getMetrics().get(BoundedIOCompactionStrategy.TOTAL_IO_MB))
.map(Double::longValue).reduce(Long::sum).orElse(0L);
- assertEquals("Should chose the first 2 compactions which should result in a total IO of 690 MB", 1204,
- (long) returnedSize);
+ assertEquals(1204, (long) returnedSize,
+ "Should chose the first 2 compactions which should result in a total IO of 690 MB");
}
@Test
@@ -137,14 +136,14 @@ public class TestHoodieCompactionStrategy {
List<HoodieCompactionOperation> operations = createCompactionOperations(writeConfig, sizesMap, keyToPartitionMap);
List<HoodieCompactionOperation> returned = strategy.orderAndFilter(writeConfig, operations, new ArrayList<>());
- assertTrue("DayBasedCompactionStrategy should have resulted in fewer compactions",
- returned.size() < operations.size());
- Assert.assertEquals("DayBasedCompactionStrategy should have resulted in fewer compactions", returned.size(), 2);
+ assertTrue(returned.size() < operations.size(),
+ "DayBasedCompactionStrategy should have resulted in fewer compactions");
+ assertEquals(2, returned.size(), "DayBasedCompactionStrategy should have resulted in fewer compactions");
int comparision = strategy.getComparator().compare(returned.get(returned.size() - 1).getPartitionPath(),
returned.get(0).getPartitionPath());
// Either the partition paths are sorted in descending order or they are equal
- assertTrue("DayBasedCompactionStrategy should sort partitions in descending order", comparision >= 0);
+ assertTrue(comparision >= 0, "DayBasedCompactionStrategy should sort partitions in descending order");
}
@Test
@@ -185,15 +184,15 @@ public class TestHoodieCompactionStrategy {
List<HoodieCompactionOperation> operations = createCompactionOperations(writeConfig, sizesMap, keyToPartitionMap);
List<HoodieCompactionOperation> returned = strategy.orderAndFilter(writeConfig, operations, new ArrayList<>());
- assertTrue("BoundedPartitionAwareCompactionStrategy should have resulted in fewer compactions",
- returned.size() < operations.size());
- Assert.assertEquals("BoundedPartitionAwareCompactionStrategy should have resulted in fewer compactions",
- returned.size(), 5);
+ assertTrue(returned.size() < operations.size(),
+ "BoundedPartitionAwareCompactionStrategy should have resulted in fewer compactions");
+ assertEquals(5, returned.size(),
+ "BoundedPartitionAwareCompactionStrategy should have resulted in fewer compactions");
int comparision = strategy.getComparator().compare(returned.get(returned.size() - 1).getPartitionPath(),
returned.get(0).getPartitionPath());
// Either the partition paths are sorted in descending order or they are equal
- assertTrue("BoundedPartitionAwareCompactionStrategy should sort partitions in descending order", comparision >= 0);
+ assertTrue(comparision >= 0, "BoundedPartitionAwareCompactionStrategy should sort partitions in descending order");
}
@Test
@@ -234,12 +233,11 @@ public class TestHoodieCompactionStrategy {
List<HoodieCompactionOperation> operations = createCompactionOperations(writeConfig, sizesMap, keyToPartitionMap);
List<HoodieCompactionOperation> returned = strategy.orderAndFilter(writeConfig, operations, new ArrayList<>());
- assertTrue(
+ assertTrue(returned.size() < operations.size(),
"UnBoundedPartitionAwareCompactionStrategy should not include last "
- + writeConfig.getTargetPartitionsPerDayBasedCompaction() + " partitions or later partitions from today",
- returned.size() < operations.size());
- Assert.assertEquals("BoundedPartitionAwareCompactionStrategy should have resulted in 1 compaction", returned.size(),
- 1);
+ + writeConfig.getTargetPartitionsPerDayBasedCompaction() + " partitions or later partitions from today");
+ assertEquals(1, returned.size(),
+ "BoundedPartitionAwareCompactionStrategy should have resulted in 1 compaction");
}
private List<HoodieCompactionOperation> createCompactionOperations(HoodieWriteConfig config,