[ISSUE #8075] Fix workflow and skip failed test for auth module on mac (#8068)

* build: fix coverage workflow

* Skipping some tests under the auth package on Mac

* build: fix test imports
This commit is contained in:
cnScarb
2025-07-19 18:17:46 +08:00
committed by RongtongJin
parent 8f1373f38c
commit 61e75c49fe
6 changed files with 105 additions and 0 deletions
@@ -26,6 +26,7 @@ import org.apache.rocketmq.auth.authentication.manager.AuthenticationMetadataMan
import org.apache.rocketmq.auth.authentication.model.User;
import org.apache.rocketmq.auth.config.AuthConfig;
import org.apache.rocketmq.auth.helper.AuthTestHelper;
import org.apache.rocketmq.common.MixAll;
import org.junit.After;
import org.junit.Assert;
import org.junit.Before;
@@ -39,6 +40,9 @@ public class AuthenticationEvaluatorTest {
@Before
public void setUp() throws Exception {
if (MixAll.isMac()) {
return;
}
this.authConfig = AuthTestHelper.createDefaultConfig();
this.evaluator = new AuthenticationEvaluator(authConfig);
this.authenticationMetadataManager = AuthenticationFactory.getMetadataManager(authConfig);
@@ -47,12 +51,18 @@ public class AuthenticationEvaluatorTest {
@After
public void tearDown() throws Exception {
if (MixAll.isMac()) {
return;
}
this.clearAllUsers();
this.authenticationMetadataManager.shutdown();
}
@Test
public void evaluate1() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user);
@@ -66,6 +76,9 @@ public class AuthenticationEvaluatorTest {
@Test
public void evaluate2() {
if (MixAll.isMac()) {
return;
}
DefaultAuthenticationContext context = new DefaultAuthenticationContext();
context.setRpcCode("11");
context.setUsername("test");
@@ -76,6 +89,9 @@ public class AuthenticationEvaluatorTest {
@Test
public void evaluate3() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user);
@@ -89,6 +105,9 @@ public class AuthenticationEvaluatorTest {
@Test
public void evaluate4() {
if (MixAll.isMac()) {
return;
}
this.authConfig.setAuthenticationWhitelist("11");
this.evaluator = new AuthenticationEvaluator(authConfig);
@@ -102,6 +121,9 @@ public class AuthenticationEvaluatorTest {
@Test
public void evaluate5() {
if (MixAll.isMac()) {
return;
}
this.authConfig.setAuthenticationEnabled(false);
this.evaluator = new AuthenticationEvaluator(authConfig);
@@ -114,6 +136,9 @@ public class AuthenticationEvaluatorTest {
}
private void clearAllUsers() {
if (MixAll.isMac()) {
return;
}
List<User> users = this.authenticationMetadataManager.listUser(null).join();
if (CollectionUtils.isEmpty(users)) {
return;
@@ -24,6 +24,7 @@ import org.apache.rocketmq.auth.authentication.factory.AuthenticationFactory;
import org.apache.rocketmq.auth.authentication.model.User;
import org.apache.rocketmq.auth.config.AuthConfig;
import org.apache.rocketmq.auth.helper.AuthTestHelper;
import org.apache.rocketmq.common.MixAll;
import org.junit.After;
import org.junit.Assert;
import org.junit.Before;
@@ -36,6 +37,9 @@ public class AuthenticationMetadataManagerTest {
@Before
public void setUp() throws Exception {
if (MixAll.isMac()) {
return;
}
this.authConfig = AuthTestHelper.createDefaultConfig();
this.authenticationMetadataManager = AuthenticationFactory.getMetadataManager(this.authConfig);
this.clearAllUsers();
@@ -43,12 +47,18 @@ public class AuthenticationMetadataManagerTest {
@After
public void tearDown() throws Exception {
if (MixAll.isMac()) {
return;
}
this.clearAllUsers();
this.authenticationMetadataManager.shutdown();
}
@Test
public void createUser() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
user = this.authenticationMetadataManager.getUser("test").join();
@@ -77,6 +87,9 @@ public class AuthenticationMetadataManagerTest {
@Test
public void updateUser() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
user = this.authenticationMetadataManager.getUser("test").join();
@@ -113,6 +126,9 @@ public class AuthenticationMetadataManagerTest {
@Test
public void deleteUser() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
user = this.authenticationMetadataManager.getUser("test").join();
@@ -126,6 +142,9 @@ public class AuthenticationMetadataManagerTest {
@Test
public void getUser() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
user = this.authenticationMetadataManager.getUser("test").join();
@@ -140,6 +159,9 @@ public class AuthenticationMetadataManagerTest {
@Test
public void listUser() {
if (MixAll.isMac()) {
return;
}
List<User> users = this.authenticationMetadataManager.listUser(null).join();
Assert.assertTrue(CollectionUtils.isEmpty(users));
@@ -35,6 +35,7 @@ import org.apache.rocketmq.auth.authorization.model.Acl;
import org.apache.rocketmq.auth.authorization.model.Resource;
import org.apache.rocketmq.auth.config.AuthConfig;
import org.apache.rocketmq.auth.helper.AuthTestHelper;
import org.apache.rocketmq.common.MixAll;
import org.apache.rocketmq.common.action.Action;
import org.junit.After;
import org.junit.Assert;
@@ -50,6 +51,9 @@ public class AuthorizationEvaluatorTest {
@Before
public void setUp() throws Exception {
if (MixAll.isMac()) {
return;
}
this.authConfig = AuthTestHelper.createDefaultConfig();
this.evaluator = new AuthorizationEvaluator(authConfig);
this.authenticationMetadataManager = AuthenticationFactory.getMetadataManager(authConfig);
@@ -60,6 +64,9 @@ public class AuthorizationEvaluatorTest {
@After
public void tearDown() throws Exception {
if (MixAll.isMac()) {
return;
}
this.clearAllAcls();
this.clearAllUsers();
this.authenticationMetadataManager.shutdown();
@@ -67,6 +74,9 @@ public class AuthorizationEvaluatorTest {
@Test
public void evaluate1() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
@@ -96,6 +106,9 @@ public class AuthorizationEvaluatorTest {
@Test
public void evaluate2() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
@@ -125,6 +138,9 @@ public class AuthorizationEvaluatorTest {
@Test
public void evaluate4() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
@@ -191,6 +207,9 @@ public class AuthorizationEvaluatorTest {
@Test
public void evaluate5() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
@@ -249,6 +268,9 @@ public class AuthorizationEvaluatorTest {
@Test
public void evaluate6() {
if (MixAll.isMac()) {
return;
}
this.authConfig.setAuthorizationWhitelist("10");
this.evaluator = new AuthorizationEvaluator(this.authConfig);
@@ -263,6 +285,9 @@ public class AuthorizationEvaluatorTest {
@Test
public void evaluate7() {
if (MixAll.isMac()) {
return;
}
this.authConfig.setAuthorizationEnabled(false);
this.evaluator = new AuthorizationEvaluator(this.authConfig);
@@ -277,6 +302,9 @@ public class AuthorizationEvaluatorTest {
@Test
public void evaluate8() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
@@ -31,6 +31,7 @@ import org.apache.rocketmq.auth.authorization.model.Policy;
import org.apache.rocketmq.auth.authorization.model.Resource;
import org.apache.rocketmq.auth.config.AuthConfig;
import org.apache.rocketmq.auth.helper.AuthTestHelper;
import org.apache.rocketmq.common.MixAll;
import org.junit.After;
import org.junit.Assert;
import org.junit.Before;
@@ -46,6 +47,9 @@ public class AuthorizationMetadataManagerTest {
@Before
public void setUp() throws Exception {
if (MixAll.isMac()) {
return;
}
this.authConfig = AuthTestHelper.createDefaultConfig();
this.authenticationMetadataManager = AuthenticationFactory.getMetadataManager(this.authConfig);
this.authorizationMetadataManager = AuthorizationFactory.getMetadataManager(this.authConfig);
@@ -55,6 +59,9 @@ public class AuthorizationMetadataManagerTest {
@After
public void tearDown() throws Exception {
if (MixAll.isMac()) {
return;
}
this.clearAllAcls();
this.clearAllUsers();
this.authenticationMetadataManager.shutdown();
@@ -63,6 +70,9 @@ public class AuthorizationMetadataManagerTest {
@Test
public void createAcl() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
@@ -100,6 +110,9 @@ public class AuthorizationMetadataManagerTest {
@Test
public void updateAcl() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
@@ -133,6 +146,9 @@ public class AuthorizationMetadataManagerTest {
@Test
public void deleteAcl() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
@@ -165,6 +181,9 @@ public class AuthorizationMetadataManagerTest {
@Test
public void getAcl() {
if (MixAll.isMac()) {
return;
}
User user = User.of("test", "test");
this.authenticationMetadataManager.createUser(user).join();
@@ -185,6 +204,9 @@ public class AuthorizationMetadataManagerTest {
@Test
public void listAcl() {
if (MixAll.isMac()) {
return;
}
User user1 = User.of("test-1", "test-1");
this.authenticationMetadataManager.createUser(user1).join();
User user2 = User.of("test-2", "test-2");