From dd3ea8fef597aeb38a29f5ea1edd2ef7f89c83d8 Mon Sep 17 00:00:00 2001 From: root Date: Sun, 17 May 2026 17:53:25 +0800 Subject: [PATCH] [fix](fe) Remove oidc plugin assertions from AuthenticationPluginManagerTest ### What problem does this PR solve? Issue Number: close #xxx Problem Summary: AuthenticationPluginManagerTest asserts the existence of an OIDC authentication plugin, but no OIDC plugin implementation exists in the codebase. This causes testPluginsAutoLoaded and testGetFactory to fail. ### Release note None ### Check List (For Author) - Test: Unit Test - Behavior changed: No - Does this need documentation: No --- .../handler/AuthenticationPluginManagerTest.java | 5 ----- 1 file changed, 5 deletions(-) diff --git a/fe/fe-authentication/fe-authentication-handler/src/test/java/org/apache/doris/authentication/handler/AuthenticationPluginManagerTest.java b/fe/fe-authentication/fe-authentication-handler/src/test/java/org/apache/doris/authentication/handler/AuthenticationPluginManagerTest.java index 6cde32bb90f0f1..260afb30a9646c 100644 --- a/fe/fe-authentication/fe-authentication-handler/src/test/java/org/apache/doris/authentication/handler/AuthenticationPluginManagerTest.java +++ b/fe/fe-authentication/fe-authentication-handler/src/test/java/org/apache/doris/authentication/handler/AuthenticationPluginManagerTest.java @@ -70,7 +70,6 @@ void testPluginsAutoLoaded() { // Then Assertions.assertNotNull(pluginNames); Assertions.assertFalse(pluginNames.isEmpty(), "Should load at least built-in plugins"); - Assertions.assertTrue(pluginNames.contains("oidc"), "Should include oidc plugin"); Assertions.assertTrue(pluginNames.contains("password"), "Should include password plugin"); } @@ -147,10 +146,6 @@ void testGetFactory() { // Then Assertions.assertTrue(factory.isPresent()); Assertions.assertEquals("password", factory.get().name()); - - Optional oidcFactory = pluginManager.getFactory("oidc"); - Assertions.assertTrue(oidcFactory.isPresent()); - Assertions.assertEquals("oidc", oidcFactory.get().name()); } @Test