This is an automated email from the ASF dual-hosted git repository.
apkhmv pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/ignite-3.git
The following commit(s) were added to refs/heads/main by this push:
new 79f9d07ee6 IGNITE-23371 Disable route filters in tests (#4511)
79f9d07ee6 is described below
commit 79f9d07ee6d8c05eb425d2f129c67dfb46e1ad28
Author: Vadim Pakhnushev <[email protected]>
AuthorDate: Mon Oct 7 11:10:42 2024 +0300
IGNITE-23371 Disable route filters in tests (#4511)
---
.../ClusterConfigurationControllerTest.java | 27 +-------------------
.../NodeConfigurationControllerTest.java | 29 +++-------------------
.../SystemDisasterRecoveryControllerTest.java | 29 +++-------------------
3 files changed, 7 insertions(+), 78 deletions(-)
diff --git
a/modules/rest/src/test/java/org/apache/ignite/internal/rest/configuration/ClusterConfigurationControllerTest.java
b/modules/rest/src/test/java/org/apache/ignite/internal/rest/configuration/ClusterConfigurationControllerTest.java
index 55e1a74fb8..e51eb1211e 100644
---
a/modules/rest/src/test/java/org/apache/ignite/internal/rest/configuration/ClusterConfigurationControllerTest.java
+++
b/modules/rest/src/test/java/org/apache/ignite/internal/rest/configuration/ClusterConfigurationControllerTest.java
@@ -18,7 +18,6 @@
package org.apache.ignite.internal.rest.configuration;
import io.micronaut.context.annotation.Bean;
-import io.micronaut.context.annotation.Factory;
import io.micronaut.context.annotation.Property;
import io.micronaut.context.annotation.Replaces;
import io.micronaut.http.client.HttpClient;
@@ -29,25 +28,14 @@ import jakarta.inject.Named;
import org.apache.ignite.internal.configuration.ConfigurationRegistry;
import
org.apache.ignite.internal.configuration.presentation.ConfigurationPresentation;
import org.apache.ignite.internal.configuration.presentation.HoconPresentation;
-import
org.apache.ignite.internal.configuration.testframework.ConfigurationExtension;
-import
org.apache.ignite.internal.configuration.testframework.InjectConfiguration;
-import org.apache.ignite.internal.rest.RestManager;
-import org.apache.ignite.internal.rest.RestManagerFactory;
-import
org.apache.ignite.internal.security.authentication.AuthenticationManager;
-import
org.apache.ignite.internal.security.authentication.AuthenticationManagerImpl;
-import org.apache.ignite.internal.security.configuration.SecurityConfiguration;
-import org.junit.jupiter.api.extension.ExtendWith;
/**
* Functional test for {@link ClusterConfigurationController}.
*/
@MicronautTest
-@ExtendWith(ConfigurationExtension.class)
@Property(name = "ignite.endpoints.filter-non-initialized", value = "false")
+@Property(name = "micronaut.security.enabled", value = "false")
class ClusterConfigurationControllerTest extends
ConfigurationControllerBaseTest {
- @InjectConfiguration
- SecurityConfiguration securityConfiguration;
-
@Inject
@Client("/management/v1/configuration/cluster/")
HttpClient client;
@@ -66,17 +54,4 @@ class ClusterConfigurationControllerTest extends
ConfigurationControllerBaseTest
public ConfigurationPresentation<String>
cfgPresentation(ConfigurationRegistry configurationRegistry) {
return new HoconPresentation(configurationRegistry);
}
-
- @Factory
- @Bean
- @Replaces(RestManagerFactory.class)
- public RestManagerFactory restManagerProvider() {
- return new RestManagerFactory(new RestManager());
- }
-
- @Bean
- @Factory
- AuthenticationManager authenticationManager() {
- return new AuthenticationManagerImpl(securityConfiguration, ign -> {});
- }
}
diff --git
a/modules/rest/src/test/java/org/apache/ignite/internal/rest/configuration/NodeConfigurationControllerTest.java
b/modules/rest/src/test/java/org/apache/ignite/internal/rest/configuration/NodeConfigurationControllerTest.java
index 96eed68795..942c3cf83d 100644
---
a/modules/rest/src/test/java/org/apache/ignite/internal/rest/configuration/NodeConfigurationControllerTest.java
+++
b/modules/rest/src/test/java/org/apache/ignite/internal/rest/configuration/NodeConfigurationControllerTest.java
@@ -18,7 +18,7 @@
package org.apache.ignite.internal.rest.configuration;
import io.micronaut.context.annotation.Bean;
-import io.micronaut.context.annotation.Factory;
+import io.micronaut.context.annotation.Property;
import io.micronaut.context.annotation.Replaces;
import io.micronaut.http.client.HttpClient;
import io.micronaut.http.client.annotation.Client;
@@ -28,24 +28,14 @@ import jakarta.inject.Named;
import org.apache.ignite.internal.configuration.ConfigurationRegistry;
import
org.apache.ignite.internal.configuration.presentation.ConfigurationPresentation;
import org.apache.ignite.internal.configuration.presentation.HoconPresentation;
-import
org.apache.ignite.internal.configuration.testframework.ConfigurationExtension;
-import
org.apache.ignite.internal.configuration.testframework.InjectConfiguration;
-import org.apache.ignite.internal.rest.RestManager;
-import org.apache.ignite.internal.rest.RestManagerFactory;
-import
org.apache.ignite.internal.security.authentication.AuthenticationManager;
-import
org.apache.ignite.internal.security.authentication.AuthenticationManagerImpl;
-import org.apache.ignite.internal.security.configuration.SecurityConfiguration;
-import org.junit.jupiter.api.extension.ExtendWith;
/**
* Functional test for {@link NodeConfigurationController}.
*/
@MicronautTest
-@ExtendWith(ConfigurationExtension.class)
+@Property(name = "ignite.endpoints.filter-non-initialized", value = "false")
+@Property(name = "micronaut.security.enabled", value = "false")
class NodeConfigurationControllerTest extends ConfigurationControllerBaseTest {
- @InjectConfiguration
- SecurityConfiguration securityConfiguration;
-
@Inject
@Client("/management/v1/configuration/node/")
HttpClient client;
@@ -64,17 +54,4 @@ class NodeConfigurationControllerTest extends
ConfigurationControllerBaseTest {
public ConfigurationPresentation<String>
cfgPresentation(ConfigurationRegistry configurationRegistry) {
return new HoconPresentation(configurationRegistry);
}
-
- @Factory
- @Bean
- @Replaces(RestManagerFactory.class)
- public RestManagerFactory restManagerProvider() {
- return new RestManagerFactory(new RestManager());
- }
-
- @Bean
- @Factory
- AuthenticationManager authenticationManager() {
- return new AuthenticationManagerImpl(securityConfiguration, ign -> {});
- }
}
diff --git
a/modules/rest/src/test/java/org/apache/ignite/internal/rest/recovery/system/SystemDisasterRecoveryControllerTest.java
b/modules/rest/src/test/java/org/apache/ignite/internal/rest/recovery/system/SystemDisasterRecoveryControllerTest.java
index c444d55153..dcdb5f5a70 100644
---
a/modules/rest/src/test/java/org/apache/ignite/internal/rest/recovery/system/SystemDisasterRecoveryControllerTest.java
+++
b/modules/rest/src/test/java/org/apache/ignite/internal/rest/recovery/system/SystemDisasterRecoveryControllerTest.java
@@ -34,7 +34,7 @@ import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when;
import io.micronaut.context.annotation.Bean;
-import io.micronaut.context.annotation.Factory;
+import io.micronaut.context.annotation.Property;
import io.micronaut.context.annotation.Replaces;
import io.micronaut.http.HttpRequest;
import io.micronaut.http.HttpResponse;
@@ -47,32 +47,22 @@ import java.util.List;
import java.util.Optional;
import java.util.UUID;
import org.apache.ignite.internal.cluster.management.ClusterState;
-import
org.apache.ignite.internal.configuration.testframework.ConfigurationExtension;
-import
org.apache.ignite.internal.configuration.testframework.InjectConfiguration;
import
org.apache.ignite.internal.disaster.system.SystemDisasterRecoveryManager;
import
org.apache.ignite.internal.disaster.system.exception.ClusterResetException;
import org.apache.ignite.internal.disaster.system.exception.MigrateException;
-import org.apache.ignite.internal.rest.RestManager;
-import org.apache.ignite.internal.rest.RestManagerFactory;
import org.apache.ignite.internal.rest.api.Problem;
import org.apache.ignite.internal.rest.api.recovery.system.MigrateRequest;
import org.apache.ignite.internal.rest.api.recovery.system.ResetClusterRequest;
-import
org.apache.ignite.internal.security.authentication.AuthenticationManager;
-import
org.apache.ignite.internal.security.authentication.AuthenticationManagerImpl;
-import org.apache.ignite.internal.security.configuration.SecurityConfiguration;
import org.apache.ignite.internal.testframework.BaseIgniteAbstractTest;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.extension.ExtendWith;
import org.mockito.ArgumentCaptor;
import org.mockito.Mockito;
@MicronautTest
-@ExtendWith(ConfigurationExtension.class)
+@Property(name = "ignite.endpoints.filter-non-initialized", value = "false")
+@Property(name = "micronaut.security.enabled", value = "false")
class SystemDisasterRecoveryControllerTest extends BaseIgniteAbstractTest {
- @InjectConfiguration
- private SecurityConfiguration securityConfiguration;
-
@Inject
@Client("/management/v1/recovery/cluster/")
private HttpClient client;
@@ -84,19 +74,6 @@ class SystemDisasterRecoveryControllerTest extends
BaseIgniteAbstractTest {
Mockito.reset(systemDisasterRecoveryManager);
}
- @Factory
- @Bean
- @Replaces(RestManagerFactory.class)
- RestManagerFactory restManagerProvider() {
- return new RestManagerFactory(new RestManager());
- }
-
- @Bean
- @Factory
- AuthenticationManager authenticationManager() {
- return new AuthenticationManagerImpl(securityConfiguration, ign -> {});
- }
-
@Bean
@Replaces(SystemDisasterRecoveryFactory.class)
SystemDisasterRecoveryFactory systemDisasterRecoveryFactory() {