diff --git a/hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-workload/src/test/java/org/apache/hadoop/tools/dynamometer/workloadgenerator/TestWorkloadGenerator.java b/hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-workload/src/test/java/org/apache/hadoop/tools/dynamometer/workloadgenerator/TestWorkloadGenerator.java index f3edc24b70444..17692e0a9eaba 100644 --- a/hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-workload/src/test/java/org/apache/hadoop/tools/dynamometer/workloadgenerator/TestWorkloadGenerator.java +++ b/hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-workload/src/test/java/org/apache/hadoop/tools/dynamometer/workloadgenerator/TestWorkloadGenerator.java @@ -40,16 +40,16 @@ import org.apache.hadoop.security.authorize.AuthorizationException; import org.apache.hadoop.security.authorize.ImpersonationProvider; import org.jline.utils.Log; -import org.junit.After; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.AfterEach; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; import org.slf4j.Logger; import org.slf4j.LoggerFactory; import static org.apache.hadoop.fs.CommonConfigurationKeysPublic.HADOOP_SECURITY_IMPERSONATION_PROVIDER_CLASS; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertFalse; -import static org.junit.Assert.assertTrue; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertFalse; +import static org.junit.jupiter.api.Assertions.assertTrue; /** Tests for {@link WorkloadDriver} and related classes. */ @@ -61,7 +61,7 @@ public class TestWorkloadGenerator { private MiniDFSCluster miniCluster; private FileSystem dfs; - @Before + @BeforeEach public void setup() throws Exception { conf = new Configuration(); conf.setClass(HADOOP_SECURITY_IMPERSONATION_PROVIDER_CLASS, @@ -74,7 +74,7 @@ public void setup() throws Exception { dfs.setOwner(new Path("/tmp"), "hdfs", "hdfs"); } - @After + @AfterEach public void tearDown() throws Exception { if (miniCluster != null) { miniCluster.shutdown(); @@ -135,7 +135,7 @@ private void testAuditWorkloadWithOutput(String auditOutputPath) Job workloadJob = WorkloadDriver.getJobForSubmission(conf, dfs.getUri().toString(), workloadStartTime, AuditReplayMapper.class); boolean success = workloadJob.waitForCompletion(true); - assertTrue("workload job should succeed", success); + assertTrue(success, "workload job should succeed"); Counters counters = workloadJob.getCounters(); assertEquals(6, counters.findCounter(AuditReplayMapper.REPLAYCOUNTERS.TOTALCOMMANDS) diff --git a/hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-workload/src/test/java/org/apache/hadoop/tools/dynamometer/workloadgenerator/audit/TestAuditLogDirectParser.java b/hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-workload/src/test/java/org/apache/hadoop/tools/dynamometer/workloadgenerator/audit/TestAuditLogDirectParser.java index 007445e10fb39..353e2b8ab55bd 100644 --- a/hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-workload/src/test/java/org/apache/hadoop/tools/dynamometer/workloadgenerator/audit/TestAuditLogDirectParser.java +++ b/hadoop-tools/hadoop-dynamometer/hadoop-dynamometer-workload/src/test/java/org/apache/hadoop/tools/dynamometer/workloadgenerator/audit/TestAuditLogDirectParser.java @@ -20,10 +20,10 @@ import java.util.function.Function; import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.io.Text; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; -import static org.junit.Assert.assertEquals; +import static org.junit.jupiter.api.Assertions.assertEquals; /** Tests for {@link AuditLogDirectParser}. */ public class TestAuditLogDirectParser { @@ -32,7 +32,7 @@ public class TestAuditLogDirectParser { private AuditLogDirectParser parser; private Long sequence = 1L; - @Before + @BeforeEach public void setup() throws Exception { parser = new AuditLogDirectParser(); Configuration conf = new Configuration();