[ https://issues.apache.org/jira/browse/ARTEMIS-2582?focusedWorklogId=669851&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-669851 ]
ASF GitHub Bot logged work on ARTEMIS-2582: ------------------------------------------- Author: ASF GitHub Bot Created on: 26/Oct/21 00:14 Start Date: 26/Oct/21 00:14 Worklog Time Spent: 10m Work Description: nbrendah commented on a change in pull request #3811: URL: https://github.com/apache/activemq-artemis/pull/3811#discussion_r736052471 ########## File path: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/embedded/EmbeddedActiveMQ.java ########## @@ -139,7 +139,9 @@ protected void initStart() throws Exception { } public EmbeddedActiveMQ stop() throws Exception { - activeMQServer.stop(); + if (activeMQServer != null) { + activeMQServer.stop(); + } return this; Review comment: Thanks @jbertram for the great explanation. I think this is good as is. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 669851) Time Spent: 2h 20m (was: 2h 10m) > EmbeddedActiveMQ.stop() should check for null > --------------------------------------------- > > Key: ARTEMIS-2582 > URL: https://issues.apache.org/jira/browse/ARTEMIS-2582 > Project: ActiveMQ Artemis > Issue Type: Bug > Components: Broker > Affects Versions: 2.10.1 > Reporter: SL > Priority: Trivial > Labels: easy > Time Spent: 2h 20m > Remaining Estimate: 0h > > 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 > {code:java} > public EmbeddedActiveMQ stop() throws Exception { > + if (activeMQServer != null) { > activeMQServer.stop(); > + } > return this; > } > {code} > use case : after methods in unit tests. -- This message was sent by Atlassian Jira (v8.3.4#803005)