[ 
https://issues.apache.org/jira/browse/ARTEMIS-3808?focusedWorklogId=769830&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-769830
 ]

ASF GitHub Bot logged work on ARTEMIS-3808:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 12/May/22 19:05
            Start Date: 12/May/22 19:05
    Worklog Time Spent: 10m 
      Work Description: jbertram commented on code in PR #4061:
URL: https://github.com/apache/activemq-artemis/pull/4061#discussion_r871719618


##########
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/management/ActiveMQServerControlTest.java:
##########
@@ -4290,6 +4292,80 @@ public boolean isStarted() {
       }
    }
 
+   @Test
+   public void testManualStopStartEmbeddedWebServer() throws Exception {
+      testStartStopEmbeddedWebServer(true);
+   }
+
+   @Test
+   public void testRestartEmbeddedWebServer() throws Exception {
+      testStartStopEmbeddedWebServer(false);
+   }
+
+   private void testStartStopEmbeddedWebServer(boolean manual) throws 
Exception {
+      class Fake implements ServiceComponent, WebServerComponentMarker {
+         boolean started = false;
+         long startTime = 0;
+         long stopTime = 0;
+
+         @Override
+         public void start() throws Exception {
+            if (started) {
+               return;
+            }
+            started = true;
+            startTime = System.currentTimeMillis();
+         }
+
+         @Override
+         public void stop() throws Exception {
+         }
+
+         @Override
+         public void stop(boolean shutdown) throws Exception {
+            if (!shutdown) {
+               throw new RuntimeException("shutdown flag must be true");
+            }
+            started = false;
+            stopTime = System.currentTimeMillis();
+         }
+
+         @Override
+         public boolean isStarted() {
+            return started;
+         }
+
+         public long getStartTime() {
+            return startTime;
+         }
+
+         public long getStopTime() {
+            return startTime;
+         }
+      }
+      Fake fake = new Fake();
+      server.addExternalComponent(fake, true);
+      Assert.assertTrue(fake.isStarted());
+
+      ActiveMQServerControl serverControl = createManagementControl();
+      if (manual) {
+         serverControl.stopEmbeddedWebServer();
+         Assert.assertFalse(fake.isStarted());
+         serverControl.startEmbeddedWebServer();
+         Assert.assertTrue(fake.isStarted());
+      } else {
+         // this is technically async so need to do some gymnastics to test 
effectively
+         long time = System.currentTimeMillis();
+         Thread.sleep(5);
+         Assert.assertTrue(time > fake.getStartTime());
+         Assert.assertTrue(time > fake.getStopTime());
+         serverControl.restartEmbeddedWebServer();
+         Wait.assertTrue(() -> time < fake.getStopTime(), 5000, 100);
+         Wait.assertTrue(() -> serverControl.isEmbeddedWebServerStarted(), 
5000, 100);

Review Comment:
   Should be resolved now.





Issue Time Tracking
-------------------

    Worklog Id:     (was: 769830)
    Time Spent: 4.5h  (was: 4h 20m)

> Support starting/stopping the embedded web server via mangement
> ---------------------------------------------------------------
>
>                 Key: ARTEMIS-3808
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-3808
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>            Reporter: Justin Bertram
>            Assignee: Justin Bertram
>            Priority: Major
>          Time Spent: 4.5h
>  Remaining Estimate: 0h
>
> It would be useful to be able to cycle the embedded web server if, for 
> example, one needed to renew the SSL certificates.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to