[ARTEMIS-2582] EmbeddedActiveMQ.stop() should check for null

• Type: Bug
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 2.10.1
• Fix Version/s: 2.21.0
• Component/s: Broker
• Labels: easy
• Assignee: Unassigned
• Reporter: SL
• Created: 26/Dec/19 19:14
• Updated: 22/Mar/22 18:57
• Resolved: 07/Feb/22 14:58

Description

https://github.com/apache/activemq-artemis/blob/master/artemis-server/src/main/java/org/apache/activemq/artemis/core/server/embedded/EmbeddedActiveMQ.java

when calling EmbeddedActiveMQ.stop() method, there is no guarantee that the init was called or successful (responsability of the caller)
the method should check that activeMQServer is not null

   public EmbeddedActiveMQ stop() throws Exception {
+      if (activeMQServer != null) {
           activeMQServer.stop();
+      }
      return this;
   }

use case : after methods in unit tests.

Issue Links

• links to: GitHub Pull Request #3811