IGNITE-9792 Fixed assert in case if IGNITE_MBEANS_DISABLED is true. Fixed tests.
(cherry picked from commit 78c2d3bbbd620bb7795d9f362785e073d2dec0a2)
diff --git a/modules/core/src/main/java/org/apache/ignite/internal/IgniteKernal.java b/modules/core/src/main/java/org/apache/ignite/internal/IgniteKernal.java
index 6b1c995..32e5dd8 100644
--- a/modules/core/src/main/java/org/apache/ignite/internal/IgniteKernal.java
+++ b/modules/core/src/main/java/org/apache/ignite/internal/IgniteKernal.java
@@ -1352,7 +1352,9 @@
private void validateCommon(IgniteConfiguration cfg) {
A.notNull(cfg.getNodeId(), "cfg.getNodeId()");
- A.notNull(cfg.getMBeanServer(), "cfg.getMBeanServer()");
+ if (!U.IGNITE_MBEANS_DISABLED)
+ A.notNull(cfg.getMBeanServer(), "cfg.getMBeanServer()");
+
A.notNull(cfg.getGridLogger(), "cfg.getGridLogger()");
A.notNull(cfg.getMarshaller(), "cfg.getMarshaller()");
A.notNull(cfg.getUserAttributes(), "cfg.getUserAttributes()");
diff --git a/modules/core/src/test/java/org/apache/ignite/testframework/junits/IgniteTestResources.java b/modules/core/src/test/java/org/apache/ignite/testframework/junits/IgniteTestResources.java
index 5fef8bc..59e2994 100644
--- a/modules/core/src/test/java/org/apache/ignite/testframework/junits/IgniteTestResources.java
+++ b/modules/core/src/test/java/org/apache/ignite/testframework/junits/IgniteTestResources.java
@@ -79,6 +79,13 @@
private GridResourceProcessor rsrcProc;
/**
+ * @return Default MBean server or {@code null} if {@code IGNITE_MBEANS_DISABLED} is configured.
+ */
+ @Nullable private static MBeanServer prepareMBeanServer() {
+ return U.IGNITE_MBEANS_DISABLED ? null : ManagementFactory.getPlatformMBeanServer();
+ }
+
+ /**
* @throws IgniteCheckedException If failed.
*/
public IgniteTestResources() throws IgniteCheckedException {
@@ -87,7 +94,8 @@
else
log = rootLog.getLogger(getClass());
- this.jmx = ManagementFactory.getPlatformMBeanServer();
+ this.jmx = prepareMBeanServer();
+
this.rsrcProc = new GridResourceProcessor(new GridTestKernalContext(this.log));
}
@@ -97,7 +105,7 @@
public IgniteTestResources(IgniteConfiguration cfg) throws IgniteCheckedException {
this.cfg = cfg;
this.log = rootLog.getLogger(getClass());
- this.jmx = ManagementFactory.getPlatformMBeanServer();
+ this.jmx = prepareMBeanServer();
this.rsrcProc = new GridResourceProcessor(new GridTestKernalContext(this.log, this.cfg));
}
@@ -119,7 +127,7 @@
assert log != null;
this.log = log.getLogger(getClass());
- this.jmx = ManagementFactory.getPlatformMBeanServer();
+ this.jmx = prepareMBeanServer();
this.rsrcProc = new GridResourceProcessor(new GridTestKernalContext(this.log));
}
diff --git a/modules/core/src/test/java/org/apache/ignite/testframework/junits/common/GridCommonAbstractTest.java b/modules/core/src/test/java/org/apache/ignite/testframework/junits/common/GridCommonAbstractTest.java
index f3e2b27..7e98ec7 100755
--- a/modules/core/src/test/java/org/apache/ignite/testframework/junits/common/GridCommonAbstractTest.java
+++ b/modules/core/src/test/java/org/apache/ignite/testframework/junits/common/GridCommonAbstractTest.java
@@ -481,7 +481,7 @@
}
/** {@inheritDoc} */
- @Override protected final void setUp() throws Exception {
+ @Override protected void setUp() throws Exception {
// Disable SSL hostname verifier.
HttpsURLConnection.setDefaultHostnameVerifier(new HostnameVerifier() {
@Override public boolean verify(String s, SSLSession sslSes) {
@@ -495,7 +495,7 @@
}
/** {@inheritDoc} */
- @Override protected final void tearDown() throws Exception {
+ @Override protected void tearDown() throws Exception {
getTestCounters().incrementStopped();
super.tearDown();
diff --git a/modules/core/src/test/java/org/apache/ignite/util/mbeans/GridMBeanDisableSelfTest.java b/modules/core/src/test/java/org/apache/ignite/util/mbeans/GridMBeanDisableSelfTest.java
index ac8f011..b98cd46 100644
--- a/modules/core/src/test/java/org/apache/ignite/util/mbeans/GridMBeanDisableSelfTest.java
+++ b/modules/core/src/test/java/org/apache/ignite/util/mbeans/GridMBeanDisableSelfTest.java
@@ -33,15 +33,17 @@
*/
public class GridMBeanDisableSelfTest extends GridCommonAbstractTest {
/** {@inheritDoc} */
- @Override protected void beforeTestsStarted() throws Exception {
+ @Override protected void setUp() throws Exception {
IgniteUtils.IGNITE_MBEANS_DISABLED = true;
- super.beforeTestsStarted();
+ super.setUp();
}
/** {@inheritDoc} */
- @Override protected void afterTestsStopped() throws Exception {
+ @Override protected void tearDown() throws Exception {
IgniteUtils.IGNITE_MBEANS_DISABLED = false;
+
+ super.tearDown();
}
/**
@@ -52,12 +54,12 @@
public void testCorrectMBeanInfo() throws Exception {
// Node should start and stopped with no errors.
try (final Ignite ignite = startGrid(0)) {
- final MBeanServer server = ignite.configuration().getMBeanServer();
+ final MBeanServer srv = ignite.configuration().getMBeanServer();
GridTestUtils.assertThrowsWithCause(
new Callable<Void>() {
@Override public Void call() throws Exception {
- U.registerMBean(server, ignite.name(), "dummy", "DummyMbean1", new DummyMBeanImpl(), DummyMBean.class);
+ U.registerMBean(srv, ignite.name(), "dummy", "DummyMbean1", new DummyMBeanImpl(), DummyMBean.class);
return null;
}
@@ -72,7 +74,7 @@
"DummyMbean2"
);
- U.registerMBean(server, objName, new DummyMBeanImpl(), DummyMBean.class);
+ U.registerMBean(srv, objName, new DummyMBeanImpl(), DummyMBean.class);
return null;
@@ -109,4 +111,4 @@
// No op.
}
}
-}
\ No newline at end of file
+}