Added Docs, Write-Permission & Unit test

components/inspectit-ocelot-configurationserver/src/main/java/rocks/inspectit/ocelot/rest/configuration/ConfigurationController.java

@@ -3,6 +3,7 @@
 import io.swagger.annotations.ApiOperation;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.context.ApplicationEventPublisher;
+import org.springframework.security.access.annotation.Secured;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.RestController;
 import rocks.inspectit.ocelot.file.FileChangedEvent;
@@ -14,9 +15,13 @@
 import rocks.inspectit.ocelot.agentconfiguration.AgentConfiguration;
 import rocks.inspectit.ocelot.agentconfiguration.AgentConfigurationManager;
 import rocks.inspectit.ocelot.config.model.InspectitConfig;
+import rocks.inspectit.ocelot.security.config.UserRoleConfiguration;
 
 import java.util.Map;
 
+/**
+ * Controller for endpoints related to configuration files.
+ */
 @RestController
 public class ConfigurationController extends AbstractBaseController {
 
@@ -40,6 +45,7 @@ private void fireFileChangeEvent() {
     /**
      * Reloads all configuration files present in the servers working directory.
      */
+    @Secured(UserRoleConfiguration.WRITE_ACCESS_ROLE)
     @ApiOperation(value = "Reloads all configuration files.", notes = "Reloads all configuration files present in the " +
             "servers working directory.")
     @GetMapping(value = "configuration/reload", produces = "text/plain")

components/inspectit-ocelot-configurationserver/src/test/java/rocks/inspectit/ocelot/rest/configuration/ConfigurationControllerTest.java

@@ -21,6 +21,11 @@
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
+import org.springframework.context.ApplicationEventPublisher;
+import rocks.inspectit.ocelot.file.FileChangedEvent;
+
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.verify;
 
 @ExtendWith(MockitoExtension.class)
 public class ConfigurationControllerTest {
@@ -31,6 +36,9 @@ public class ConfigurationControllerTest {
     @Mock
     AgentConfigurationManager agentConfigurationManager;
 
+    @Mock
+    ApplicationEventPublisher applicationEventPublisher;
+
     @Nested
     public class FetchConfiguration {
 
@@ -54,4 +62,15 @@ public void noConfigurationAvailable() {
             assertThat(output.getStatusCode()).isEqualTo(HttpStatus.NOT_FOUND);
         }
     }
+
+    @Nested
+    public class ReloadConfiguration {
+
+        @Test
+        public void firesFileChangedEvent() {
+            configurationController.reloadConfiguration();
+
+            verify(applicationEventPublisher).publishEvent(any(FileChangedEvent.class));
+        }
+    }
 }