Reenable ensureTestNamingConvention test

trinodb · Feb 20, 2024 · 3456be5 · 3456be5
1 parent 96057bf
commit 3456be5
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/testing/trino-testing/src/main/java/io/trino/testing/AbstractTestQueryFramework.java b/testing/trino-testing/src/main/java/io/trino/testing/AbstractTestQueryFramework.java
@@ -54,12 +54,12 @@
 import io.trino.sql.tree.ExplainType;
 import io.trino.testing.QueryRunner.MaterializedResultWithPlan;
 import io.trino.testing.TestingAccessControlManager.TestingPrivilege;
-import io.trino.testng.services.ReportBadTestAnnotations;
 import io.trino.util.AutoCloseableCloser;
 import org.assertj.core.api.AssertProvider;
 import org.intellij.lang.annotations.Language;
 import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.TestInstance;
 import org.junit.jupiter.api.parallel.Execution;
 
@@ -274,8 +274,7 @@ private static String createQueryDebuggingSummary(BasicQueryInfo basicQueryInfo,
         }
     }
 
-    // TODO @Test - Temporarily disabled to avoid test classes running twice. Re-enable once all tests migrated to JUnit.
-    @ReportBadTestAnnotations.Suppress
+    @Test
     public void ensureTestNamingConvention()
     {
         // Enforce a naming convention to make code navigation easier.