Fix JUnit issue with Python tests (#148)

* update pom with junit engine Signed-off-by: Hugo Queinnec <[email protected]> * update all python tests Signed-off-by: Hugo Queinnec <[email protected]> --------- Signed-off-by: Hugo Queinnec <[email protected]>
IBM · Sep 19, 2024 · 4aaf8b9 · 4aaf8b9
1 parent 27059a2
commit 4aaf8b9
Show file tree

Hide file tree

Showing 44 changed files with 138 additions and 137 deletions.
diff --git a/pom.xml b/pom.xml
@@ -120,6 +120,12 @@
  <version>5.11.0</version>
  <scope>test</scope>
  </dependency>
+ <dependency>
+ <groupId>org.junit.jupiter</groupId>
+ <artifactId>junit-jupiter-engine</artifactId>
+ <version>5.11.0</version>
+ <scope>test</scope>
+ </dependency> 
  <dependency>
  <groupId>org.junit.platform</groupId>
  <artifactId>junit-platform-launcher</artifactId>

diff --git a/python/src/test/java/com/ibm/plugin/rules/detection/aead/PycaAESGCMTest.java b/python/src/test/java/com/ibm/plugin/rules/detection/aead/PycaAESGCMTest.java
@@ -40,17 +40,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaAESGCMTest extends TestBase {
+class PycaAESGCMTest extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/aead/PycaAESGCMTestFile.py", this);
  }

diff --git a/python/src/test/java/com/ibm/plugin/rules/detection/aead/PycaChaCha20Poly1305Test.java b/python/src/test/java/com/ibm/plugin/rules/detection/aead/PycaChaCha20Poly1305Test.java
@@ -36,20 +36,19 @@
 import com.ibm.mapper.model.functionality.Encrypt;
 import com.ibm.mapper.model.functionality.KeyGeneration;
 import com.ibm.plugin.TestBase;
-import com.ibm.plugin.utils.GenerateAssertsHelper;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaChaCha20Poly1305Test extends TestBase {
+class PycaChaCha20Poly1305Test extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/aead/PycaChaCha20Poly1305TestFile.py", this);
  }
@@ -83,8 +82,6 @@ public void asserts(
  assertThat(encryptValue).isInstanceOf(CipherAction.class);
  assertThat(encryptValue.asString()).isEqualTo("DECRYPT");
 
- GenerateAssertsHelper.generate(detectionStore, nodes);
-
  /*
  * Translation
  */

diff --git a/python/src/test/java/com/ibm/plugin/rules/detection/asymmetric/DSA/PycaDSANumbersTest.java b/python/src/test/java/com/ibm/plugin/rules/detection/asymmetric/DSA/PycaDSANumbersTest.java
@@ -35,17 +35,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaDSANumbersTest extends TestBase {
+class PycaDSANumbersTest extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/DSA/PycaDSANumbersTestFile.py", this);
  }

diff --git a/python/src/test/java/com/ibm/plugin/rules/detection/asymmetric/DSA/PycaDSASignTest.java b/python/src/test/java/com/ibm/plugin/rules/detection/asymmetric/DSA/PycaDSASignTest.java
@@ -42,17 +42,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaDSASignTest extends TestBase {
+class PycaDSASignTest extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/DSA/PycaDSASignTestFile.py", this);
  }

diff --git a/...om/ibm/plugin/rules/detection/asymmetric/DiffieHellman/PycaDiffieHellmanGenerateTest.java b/...om/ibm/plugin/rules/detection/asymmetric/DiffieHellman/PycaDiffieHellmanGenerateTest.java
@@ -33,7 +33,7 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
@@ -45,7 +45,7 @@ public final class PycaDiffieHellmanGenerateTest extends TestBase {
  // The key size does not yet appear because
  // of the TraceSymbol problem documented on the Github issue
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/DiffieHellman/PycaDiffieHellmanGenerateTestFile.py",
  this);

diff --git a/...com/ibm/plugin/rules/detection/asymmetric/DiffieHellman/PycaDiffieHellmanNumbersTest.java b/...com/ibm/plugin/rules/detection/asymmetric/DiffieHellman/PycaDiffieHellmanNumbersTest.java
@@ -33,17 +33,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaDiffieHellmanNumbersTest extends TestBase {
+class PycaDiffieHellmanNumbersTest extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/DiffieHellman/PycaDiffieHellmanNumbersTestFile.py",
  this);

diff --git a/.../com/ibm/plugin/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveDeriveTest.java b/.../com/ibm/plugin/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveDeriveTest.java
@@ -34,17 +34,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaEllipticCurveDeriveTest extends TestBase {
+class PycaEllipticCurveDeriveTest extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveDeriveTestFile.py",
  this);

diff --git a/...ibm/plugin/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveKeyExchangeTest.java b/...ibm/plugin/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveKeyExchangeTest.java
@@ -45,17 +45,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaEllipticCurveKeyExchangeTest extends TestBase {
+class PycaEllipticCurveKeyExchangeTest extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveKeyExchangeTestFile.py",
  this);

diff --git a/...com/ibm/plugin/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveNumbersTest.java b/...com/ibm/plugin/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveNumbersTest.java
@@ -25,18 +25,18 @@
 import java.util.List;
 import javax.annotation.Nonnull;
 import org.junit.Ignore;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaEllipticCurveNumbersTest extends TestBase {
+class PycaEllipticCurveNumbersTest extends TestBase {
 
  @Ignore("In this testcase the name of a var is resolved, but not teh actual value.")
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveNumbersTestFile.py",
  this);

diff --git a/...a/com/ibm/plugin/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveSign2Test.java b/...a/com/ibm/plugin/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveSign2Test.java
@@ -38,17 +38,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaEllipticCurveSign2Test extends TestBase {
+class PycaEllipticCurveSign2Test extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveSign2TestFile.py",
  this);

diff --git a/...va/com/ibm/plugin/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveSignTest.java b/...va/com/ibm/plugin/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveSignTest.java
@@ -42,17 +42,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaEllipticCurveSignTest extends TestBase {
+class PycaEllipticCurveSignTest extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveSignTestFile.py",
  this);

diff --git a/.../com/ibm/plugin/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveVerifyTest.java b/.../com/ibm/plugin/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveVerifyTest.java
@@ -24,22 +24,22 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Ignore;
-import org.junit.Test;
+import org.junit.jupiter.api.Disabled;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaEllipticCurveVerifyTest extends TestBase {
+class PycaEllipticCurveVerifyTest extends TestBase {
 
  // junit4
- @Ignore(
+ @Disabled(
  "Reenable once we have an approach to detect `verify` (either make it an entry\n"
  + "point, or better handle file imports for depending detection rule)")
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/EllipticCurve/PycaEllipticCurveVerifyTestFile.py",
  this);

diff --git a/python/src/test/java/com/ibm/plugin/rules/detection/asymmetric/RSA/PycaRSADecryptTest.java b/python/src/test/java/com/ibm/plugin/rules/detection/asymmetric/RSA/PycaRSADecryptTest.java
@@ -46,17 +46,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaRSADecryptTest extends TestBase {
+class PycaRSADecryptTest extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/RSA/PycaRSADecryptTestFile.py", this);
  }

diff --git a/python/src/test/java/com/ibm/plugin/rules/detection/asymmetric/RSA/PycaRSANumbersTest.java b/python/src/test/java/com/ibm/plugin/rules/detection/asymmetric/RSA/PycaRSANumbersTest.java
@@ -35,17 +35,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaRSANumbersTest extends TestBase {
+class PycaRSANumbersTest extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/RSA/PycaRSANumbersTestFile.py", this);
  }

diff --git a/python/src/test/java/com/ibm/plugin/rules/detection/asymmetric/RSA/PycaRSASign1Test.java b/python/src/test/java/com/ibm/plugin/rules/detection/asymmetric/RSA/PycaRSASign1Test.java
@@ -44,17 +44,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaRSASign1Test extends TestBase {
+class PycaRSASign1Test extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/RSA/PycaRSASign1TestFile.py", this);
  }

diff --git a/python/src/test/java/com/ibm/plugin/rules/detection/asymmetric/RSA/PycaRSASign2Test.java b/python/src/test/java/com/ibm/plugin/rules/detection/asymmetric/RSA/PycaRSASign2Test.java
@@ -43,17 +43,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaRSASign2Test extends TestBase {
+class PycaRSASign2Test extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/asymmetric/RSA/PycaRSASign2TestFile.py", this);
  }

diff --git a/python/src/test/java/com/ibm/plugin/rules/detection/fernet/PycaFernetDecryptTest.java b/python/src/test/java/com/ibm/plugin/rules/detection/fernet/PycaFernetDecryptTest.java
@@ -46,17 +46,17 @@
 import com.ibm.plugin.TestBase;
 import java.util.List;
 import javax.annotation.Nonnull;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.sonar.plugins.python.api.PythonCheck;
 import org.sonar.plugins.python.api.PythonVisitorContext;
 import org.sonar.plugins.python.api.symbols.Symbol;
 import org.sonar.plugins.python.api.tree.Tree;
 import org.sonar.python.checks.utils.PythonCheckVerifier;
 
-public class PycaFernetDecryptTest extends TestBase {
+class PycaFernetDecryptTest extends TestBase {
 
  @Test
- public void test() {
+ void test() {
  PythonCheckVerifier.verify(
  "src/test/files/rules/detection/fernet/PycaFernetDecryptTestFile.py", this);
  }