Revert "[ISSUE apache#7135] Temporarily ignoring plainAccessValidator…

… test (apache#7135)" This reverts commit 6bc2c84.
caigy · Aug 10, 2023 · e055deb · e055deb
1 parent de5a970
commit e055deb
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 11 deletions.
diff --git a/acl/src/test/java/org/apache/rocketmq/acl/plain/PlainAccessControlFlowTest.java b/acl/src/test/java/org/apache/rocketmq/acl/plain/PlainAccessControlFlowTest.java
@@ -31,7 +31,6 @@
 import org.apache.rocketmq.remoting.protocol.header.SendMessageRequestHeader;
 import org.apache.rocketmq.remoting.protocol.header.SendMessageRequestHeaderV2;
 import org.junit.Assert;
-import org.junit.Ignore;
 import org.junit.Test;
 
 import java.io.File;
@@ -44,17 +43,13 @@
 import java.util.LinkedList;
 import java.util.List;
 
-
 /**
  * <p> In this class, we'll test the following scenarios, each containing several consecutive operations on ACL,
  * <p> like updating and deleting ACL, changing config files and checking validations.
  * <p> Case 1: Only conf/plain_acl.yml exists;
  * <p> Case 2: Only conf/acl/plain_acl.yml exists;
  * <p> Case 3: Both conf/plain_acl.yml and conf/acl/plain_acl.yml exists.
  */
-
-// Ignore this test case as it is currently unable to pass on ubuntu workflow
-@Ignore
 public class PlainAccessControlFlowTest {
     public static final String DEFAULT_TOPIC = "topic-acl";
 

diff --git a/acl/src/test/java/org/apache/rocketmq/acl/plain/PlainAccessValidatorTest.java b/acl/src/test/java/org/apache/rocketmq/acl/plain/PlainAccessValidatorTest.java
@@ -56,11 +56,8 @@
 import org.junit.After;
 import org.junit.Assert;
 import org.junit.Before;
-import org.junit.Ignore;
 import org.junit.Test;
 
-// Ignore this test case as it is currently unable to pass on ubuntu workflow
-@Ignore
 public class PlainAccessValidatorTest {
 
     private PlainAccessValidator plainAccessValidator;

diff --git a/acl/src/test/java/org/apache/rocketmq/acl/plain/PlainPermissionManagerTest.java b/acl/src/test/java/org/apache/rocketmq/acl/plain/PlainPermissionManagerTest.java
@@ -29,7 +29,6 @@
 import org.assertj.core.util.Lists;
 import org.junit.Assert;
 import org.junit.Before;
-import org.junit.Ignore;
 import org.junit.Test;
 
 import java.io.File;
@@ -42,8 +41,6 @@
 import java.util.Map;
 import java.util.Set;
 
-// Ignore this test case as it is currently unable to pass on ubuntu workflow
-@Ignore
 public class PlainPermissionManagerTest {
 
     PlainPermissionManager plainPermissionManager;