improvement in test control

arieslab · Oct 16, 2023 · 793ad5c · 793ad5c
1 parent 54dae99
commit 793ad5c
Show file tree

Hide file tree

Showing 20 changed files with 130 additions and 82 deletions.
diff --git a/pom.xml b/pom.xml
@@ -36,13 +36,30 @@
             <artifactId>slf4j-simple</artifactId>
             <version>${slf4j.version}</version>
         </dependency>
+<!--        <dependency>-->
+<!--            <groupId>junit</groupId>-->
+<!--            <artifactId>junit</artifactId>-->
+<!--            <version>${junit.version}</version>-->
+<!--            <scope>test</scope>-->
+<!--        </dependency>-->
+
+        <!-- https://mvnrepository.com/artifact/org.junit.jupiter/junit-jupiter-api -->
         <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
-            <version>${junit.version}</version>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter-api</artifactId>
+            <version>5.10.0</version>
             <scope>test</scope>
         </dependency>
 
+        <!-- https://mvnrepository.com/artifact/org.junit.jupiter/junit-jupiter-migrationsupport -->
+        <dependency>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter-migrationsupport</artifactId>
+            <version>5.10.0</version>
+            <scope>test</scope>
+        </dependency>
+
+
         <dependency>
             <groupId>com.github.javaparser</groupId>
             <artifactId>javaparser-core</artifactId>
@@ -139,6 +156,18 @@
                     </descriptorRefs>
                 </configuration>
             </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-surefire-plugin</artifactId>
+                <version>3.1.2</version>
+                <dependencies>
+                    <dependency>
+                        <groupId>org.junit.jupiter</groupId>
+                        <artifactId>junit-jupiter-engine</artifactId>
+                        <version>5.10.0</version>
+                    </dependency>
+                </dependencies>
+            </plugin>
         </plugins>
     </build>
 

diff --git a/src/test/java/br/ufba/jnose/test/AssertionRouletteTest.java b/src/test/java/br/ufba/jnose/test/AssertionRouletteTest.java
@@ -1,22 +1,22 @@
 package br.ufba.jnose.test;
 
-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;
+
 
 import java.io.File;
 import java.io.FileInputStream;
 import java.util.ArrayList;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.*;
 
 import com.github.javaparser.JavaParser;
 import com.github.javaparser.ast.CompilationUnit;
 
 import br.ufba.jnose.core.testsmelldetector.testsmell.SmellyElement;
 import br.ufba.jnose.core.testsmelldetector.testsmell.smell.AssertionRoulette;
 
-
+@DisplayName("Assertion Roulette Test")
 public class AssertionRouletteTest {
 
 	public AssertionRoulette assertionTest;
@@ -25,13 +25,14 @@ public class AssertionRouletteTest {
 	SmellyElement smellyElementList;
 
 
-	@Before
+	@BeforeEach
 	public void setUp() throws Exception {
 		assertionTest = new AssertionRoulette();
 		fileInputStream = new FileInputStream(new File("src/test/java/br/ufba/jnose/test/fixtures/AssertionRouletteFixture.java"));
 	}
 
 	@Test
+	@DisplayName("should get number of tests")
 	public void should_get_number_of_tests() {
 		try{ 
 			CompilationUnit compilationUnit = JavaParser.parse(fileInputStream);
@@ -46,6 +47,7 @@ public void should_get_number_of_tests() {
 	}
 
 	@Test
+	@DisplayName("should get test smells informations")
 	public void should_get_test_smells_informations() {
 		try{ 
 			CompilationUnit compilationUnit = JavaParser.parse(fileInputStream);

diff --git a/src/test/java/br/ufba/jnose/test/ConditionalTestLogicTest.java b/src/test/java/br/ufba/jnose/test/ConditionalTestLogicTest.java
@@ -1,13 +1,15 @@
 package br.ufba.jnose.test;
 
-import static org.junit.Assert.*;
+import static org.junit.jupiter.api.Assertions.*;
 
 import java.io.File;
 import java.io.FileInputStream;
 import java.util.ArrayList;
 
 import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 import com.github.javaparser.JavaParser;
 import com.github.javaparser.ast.CompilationUnit;
@@ -22,7 +24,7 @@ public class ConditionalTestLogicTest {
 	CompilationUnit compilationUnit;
 	SmellyElement smellyElementList;
 
-	@Before
+	@BeforeEach
 	public void setUp() throws Exception {
 		conditionalTest = new ConditionalTestLogic();
 		fileInputStream = new FileInputStream(new File("src/test/java/br/ufba/jnose/test/fixtures/ConditionalFixture.java"));

diff --git a/src/test/java/br/ufba/jnose/test/ConstructorInitializationTest.java b/src/test/java/br/ufba/jnose/test/ConstructorInitializationTest.java
@@ -1,13 +1,14 @@
 package br.ufba.jnose.test;
 
-import static org.junit.Assert.*;
+import static org.junit.jupiter.api.Assertions.*;
 
 import java.io.File;
 import java.io.FileInputStream;
 import java.util.ArrayList;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 import com.github.javaparser.JavaParser;
 import com.github.javaparser.ast.CompilationUnit;
@@ -22,7 +23,7 @@ public class ConstructorInitializationTest {
 	CompilationUnit compilationUnit;
 	SmellyElement smellyElementList;
 
-	@Before
+	@BeforeEach
 	public void setUp() throws Exception {
 		constructorTest = new ConstructorInitialization();
 		fileInputStream = new FileInputStream(new File("src/test/java/br/ufba/jnose/test/fixtures/ConstructorFixture.java"));

diff --git a/src/test/java/br/ufba/jnose/test/DuplicateAssertTest.java b/src/test/java/br/ufba/jnose/test/DuplicateAssertTest.java
@@ -1,13 +1,14 @@
 package br.ufba.jnose.test;
 
-import static org.junit.Assert.*;
+import static org.junit.jupiter.api.Assertions.*;
 
 import java.io.File;
 import java.io.FileInputStream;
 import java.util.ArrayList;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 import com.github.javaparser.JavaParser;
 import com.github.javaparser.ast.CompilationUnit;
@@ -22,7 +23,7 @@ public class DuplicateAssertTest {
 	CompilationUnit compilationUnit;
 	SmellyElement smellyElementList;
 
-	@Before
+	@BeforeEach
 	public void setUp() throws Exception {
 		duplicateTest = new DuplicateAssert();
 		fileInputStream = new FileInputStream(new File("src/test/java/br/ufba/jnose/test/fixtures/DuplicateAssertFixture.java"));

diff --git a/src/test/java/br/ufba/jnose/test/EagerTestTest.java b/src/test/java/br/ufba/jnose/test/EagerTestTest.java
@@ -1,13 +1,14 @@
 package br.ufba.jnose.test;
 
-import static org.junit.Assert.*;
+import static org.junit.jupiter.api.Assertions.*;
 
 import java.io.File;
 import java.io.FileInputStream;
 import java.util.ArrayList;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 import com.github.javaparser.JavaParser;
 import com.github.javaparser.ast.CompilationUnit;
@@ -23,7 +24,7 @@ public class EagerTestTest {
 	CompilationUnit compilationUnit;
 	SmellyElement smellyElementList;
 
-	@Before
+	@BeforeEach
 	public void setUp() throws Exception {
 		eagerTest = new EagerTest();
 		fileInputStream = new FileInputStream(new File("src/test/java/br/ufba/jnose/test/fixtures/EagerFixture.java"));

diff --git a/src/test/java/br/ufba/jnose/test/EmptyTestTest.java b/src/test/java/br/ufba/jnose/test/EmptyTestTest.java
@@ -6,15 +6,15 @@
 import br.ufba.jnose.core.testsmelldetector.testsmell.SmellyElement;
 import br.ufba.jnose.core.testsmelldetector.testsmell.smell.EmptyTest;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.*;
 
 import java.io.File;
 import java.io.FileInputStream;
 import java.util.ArrayList;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 public class EmptyTestTest {
 
@@ -23,7 +23,7 @@ public class EmptyTestTest {
 	CompilationUnit compilationUnit;
 	SmellyElement smellyElementList;
 
-	@Before
+	@BeforeEach
 	public void setUp() throws Exception {
 		emptytest = new EmptyTest();
 		fileInputStream = new FileInputStream(new File("src/test/java/br/ufba/jnose/test/fixtures/EmptyFixture.java"));

diff --git a/src/test/java/br/ufba/jnose/test/ExceptionCatchingThrowingTest.java b/src/test/java/br/ufba/jnose/test/ExceptionCatchingThrowingTest.java
@@ -1,13 +1,14 @@
 package br.ufba.jnose.test;
 
-import static org.junit.Assert.*;
+import static org.junit.jupiter.api.Assertions.*;
 
 import java.io.File;
 import java.io.FileInputStream;
 import java.util.ArrayList;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 import com.github.javaparser.JavaParser;
 import com.github.javaparser.ast.CompilationUnit;
@@ -22,7 +23,7 @@ public class ExceptionCatchingThrowingTest {
 	CompilationUnit compilationUnit;
 	SmellyElement smellyElementList;
 
-	@Before
+	@BeforeEach
 	public void setUp() throws Exception {
 		exceptionTest = new ExceptionCatchingThrowing();
 		fileInputStream = new FileInputStream(new File("src/test/java/br/ufba/jnose/test/fixtures/ExceptionFixture.java"));

diff --git a/src/test/java/br/ufba/jnose/test/GeneralFixtureTest.java b/src/test/java/br/ufba/jnose/test/GeneralFixtureTest.java
@@ -1,13 +1,14 @@
 package br.ufba.jnose.test;
 
-import static org.junit.Assert.*;
+import static org.junit.jupiter.api.Assertions.*;
 
 import java.io.File;
 import java.io.FileInputStream;
 import java.util.ArrayList;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 import com.github.javaparser.JavaParser;
 import com.github.javaparser.ast.CompilationUnit;
@@ -22,7 +23,7 @@ public class GeneralFixtureTest {
 	CompilationUnit compilationUnit;
 	SmellyElement smellyElementList;
 
-	@Before
+	@BeforeEach
 	public void setUp() throws Exception {
 		generalTest = new GeneralFixture();
 		fileInputStream = new FileInputStream(new File("src/test/java/br/ufba/jnose/test/fixtures/GeneralFixtureFixture.java"));

diff --git a/src/test/java/br/ufba/jnose/test/IgnoredTestTest.java b/src/test/java/br/ufba/jnose/test/IgnoredTestTest.java
@@ -1,13 +1,14 @@
 package br.ufba.jnose.test;
 
-import static org.junit.Assert.*;
+import static org.junit.jupiter.api.Assertions.*;
 
 import java.io.File;
 import java.io.FileInputStream;
 import java.util.ArrayList;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 import com.github.javaparser.JavaParser;
 import com.github.javaparser.ast.CompilationUnit;
@@ -22,7 +23,7 @@ public class IgnoredTestTest {
 	CompilationUnit compilationUnit;
 	SmellyElement smellyElementList;
 
-	@Before
+	@BeforeEach
 	public void setUp() throws Exception {
 		ignoredTest = new IgnoredTest();
 		fileInputStream = new FileInputStream(new File("src/test/java/br/ufba/jnose/test/fixtures/IgnoredFixture.java"));

diff --git a/src/test/java/br/ufba/jnose/test/LazyTestTest.java b/src/test/java/br/ufba/jnose/test/LazyTestTest.java
@@ -1,14 +1,14 @@
 package br.ufba.jnose.test;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.*;
 
 import java.io.File;
 import java.io.FileInputStream;
 import java.util.ArrayList;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 import com.github.javaparser.JavaParser;
 import com.github.javaparser.ast.CompilationUnit;
@@ -24,7 +24,7 @@ public class LazyTestTest {
 	CompilationUnit compilationUnit;
 	SmellyElement smellyElementList;
 
-	@Before
+	@BeforeEach
 	public void setUp() throws Exception {
 		lazyTest = new LazyTest();
 		fileInputStream = new FileInputStream(new File("src/test/java/br/ufba/jnose/test/fixtures/LazyFixture.java"));

diff --git a/src/test/java/br/ufba/jnose/test/MagicNumberTestTest.java b/src/test/java/br/ufba/jnose/test/MagicNumberTestTest.java
@@ -1,13 +1,14 @@
 package br.ufba.jnose.test;
 
-import static org.junit.Assert.*;
+import static org.junit.jupiter.api.Assertions.*;
 
 import java.io.File;
 import java.io.FileInputStream;
 import java.util.ArrayList;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 import com.github.javaparser.JavaParser;
 import com.github.javaparser.ast.CompilationUnit;
@@ -22,7 +23,7 @@ public class MagicNumberTestTest {
 	CompilationUnit compilationUnit;
 	SmellyElement smellyElementList;
 
-	@Before
+	@BeforeEach
 	public void setUp() throws Exception {
 		magicTest = new MagicNumberTest();
 		fileInputStream = new FileInputStream(new File("src/test/java/br/ufba/jnose/test/fixtures/MagicNumberFixture.java"));