gridsuite · khouadrired · Feb 6, 2026
diff --git a/monitor-server/src/main/java/org/gridsuite/monitor/server/controllers/MonitorController.java b/monitor-server/src/main/java/org/gridsuite/monitor/server/controllers/MonitorController.java
@@ -16,7 +16,7 @@
 import org.gridsuite.monitor.commons.SecurityAnalysisConfig;
 import org.gridsuite.monitor.server.dto.ProcessExecution;
 import org.gridsuite.monitor.server.dto.ReportPage;
-import org.gridsuite.monitor.server.services.MonitorService;
+import org.gridsuite.monitor.server.services.internal.MonitorService;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.*;
 

diff --git a/...vices/SecurityAnalysisResultProvider.java → ...apter/SecurityAnalysisResultProvider.java b/...vices/SecurityAnalysisResultProvider.java → ...apter/SecurityAnalysisResultProvider.java
@@ -4,9 +4,11 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
-package org.gridsuite.monitor.server.services;
+package org.gridsuite.monitor.server.services.external.adapter;
 
 import org.gridsuite.monitor.commons.ResultType;
+import org.gridsuite.monitor.server.services.internal.ResultProvider;
+import org.gridsuite.monitor.server.services.external.client.SecurityAnalysisRestClient;
 import org.springframework.stereotype.Service;
 
 import java.util.UUID;
@@ -16,10 +18,10 @@
  */
 @Service
 public class SecurityAnalysisResultProvider implements ResultProvider {
-    private final SecurityAnalysisService securityAnalysisService;
+    private final SecurityAnalysisRestClient securityAnalysisRestClient;
 
-    public SecurityAnalysisResultProvider(SecurityAnalysisService securityAnalysisService) {
-        this.securityAnalysisService = securityAnalysisService;
+    public SecurityAnalysisResultProvider(SecurityAnalysisRestClient securityAnalysisRestClient) {
+        this.securityAnalysisRestClient = securityAnalysisRestClient;
     }
 
     @Override
@@ -29,11 +31,11 @@ public ResultType getType() {
 
     @Override
     public String getResult(UUID resultId) {
-        return securityAnalysisService.getResult(resultId);
+        return securityAnalysisRestClient.getResult(resultId);
     }
 
     @Override
     public void deleteResult(UUID resultId) {
-        securityAnalysisService.deleteResult(resultId);
+        securityAnalysisRestClient.deleteResult(resultId);
     }
 }
diff --git a/...onitor/server/services/ReportService.java → ...ces/external/client/ReportRestClient.java b/...onitor/server/services/ReportService.java → ...ces/external/client/ReportRestClient.java
@@ -4,7 +4,7 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
-package org.gridsuite.monitor.server.services;
+package org.gridsuite.monitor.server.services.external.client;
 
 import org.gridsuite.monitor.server.dto.ReportPage;
 import org.springframework.beans.factory.annotation.Value;
@@ -24,7 +24,7 @@
  * @author Franck Lecuyer <franck.lecuyer at rte-france.com>
  */
 @Service
-public class ReportService {
+public class ReportRestClient {
     private static final String REPORT_API_VERSION = "v1";
 
     private static final String DELIMITER = "/";
@@ -33,8 +33,8 @@ public class ReportService {
 
     private final RestTemplate restTemplate;
 
-    public ReportService(@Value("${gridsuite.services.report-server.base-uri:http://report-server/}") String reportServerBaseUri,
-                         RestTemplateBuilder restTemplateBuilder) {
+    public ReportRestClient(@Value("${gridsuite.services.report-server.base-uri:http://report-server/}") String reportServerBaseUri,
+                            RestTemplateBuilder restTemplateBuilder) {
         this.reportServerBaseUri = reportServerBaseUri;
         this.restTemplate = restTemplateBuilder.build();
     }

diff --git a/...ver/services/SecurityAnalysisService.java → ...al/client/SecurityAnalysisRestClient.java b/...ver/services/SecurityAnalysisService.java → ...al/client/SecurityAnalysisRestClient.java
@@ -4,7 +4,7 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
-package org.gridsuite.monitor.server.services;
+package org.gridsuite.monitor.server.services.external.client;
 
 import lombok.Setter;
 import org.slf4j.Logger;
@@ -23,8 +23,8 @@
  * @author Kevin Le Saulnier <kevin.le-saulnier at rte-france.com>
  */
 @Service
-public class SecurityAnalysisService {
-    private static final Logger LOGGER = LoggerFactory.getLogger(SecurityAnalysisService.class);
+public class SecurityAnalysisRestClient {
+    private static final Logger LOGGER = LoggerFactory.getLogger(SecurityAnalysisRestClient.class);
     static final String SA_API_VERSION = "v1";
     private static final String DELIMITER = "/";
 
@@ -37,7 +37,7 @@ private String getSecurityAnalysisServerBaseUri() {
         return this.securityAnalysisServerBaseUri + DELIMITER + SA_API_VERSION + DELIMITER;
     }
 
-    public SecurityAnalysisService(
+    public SecurityAnalysisRestClient(
         RestTemplateBuilder restTemplateBuilder,
         @Value("${gridsuite.services.security-analysis-server.base-uri:http://security-analysis-server/}") String securityAnalysisServerBaseUri) {
         this.securityAnalysisServerBaseUri = securityAnalysisServerBaseUri;

diff --git a/...nitor/server/services/MonitorService.java → ...ver/services/internal/MonitorService.java b/...nitor/server/services/MonitorService.java → ...ver/services/internal/MonitorService.java
@@ -4,7 +4,7 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
-package org.gridsuite.monitor.server.services;
+package org.gridsuite.monitor.server.services.internal;
 
 import org.gridsuite.monitor.commons.ProcessConfig;
 import org.gridsuite.monitor.commons.ProcessExecutionStep;
@@ -18,6 +18,8 @@
 import org.gridsuite.monitor.server.mapper.ProcessExecutionMapper;
 import org.gridsuite.monitor.server.mapper.ProcessExecutionStepMapper;
 import org.gridsuite.monitor.server.repositories.ProcessExecutionRepository;
+import org.gridsuite.monitor.server.services.messaging.NotificationService;
+import org.gridsuite.monitor.server.services.external.client.ReportRestClient;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
@@ -35,16 +37,16 @@ public class MonitorService {
 
     private final ProcessExecutionRepository executionRepository;
     private final NotificationService notificationService;
-    private final ReportService reportService;
+    private final ReportRestClient reportRestClient;
     private final ResultService resultService;
 
     public MonitorService(ProcessExecutionRepository executionRepository,
                           NotificationService notificationService,
-                          ReportService reportService,
+                          ReportRestClient reportRestClient,
                           ResultService resultService) {
         this.executionRepository = executionRepository;
         this.notificationService = notificationService;
-        this.reportService = reportService;
+        this.reportRestClient = reportRestClient;
         this.resultService = resultService;
     }
 
@@ -142,7 +144,7 @@ private ProcessExecutionStepEntity toStepEntity(ProcessExecutionStep processExec
     public List<ReportPage> getReports(UUID executionId) {
         List<UUID> reportIds = getReportIds(executionId);
         return reportIds.stream()
-                .map(reportService::getReport)
+                .map(reportRestClient::getReport)
                 .toList();
     }
 
@@ -206,7 +208,7 @@ public boolean deleteExecution(UUID executionId) {
                 }
             });
             resultIds.forEach(resultService::deleteResult);
-            reportIds.forEach(reportService::deleteReport);
+            reportIds.forEach(reportRestClient::deleteReport);
 
             executionRepository.deleteById(executionId);
 

diff --git a/...nitor/server/services/ResultProvider.java → ...ver/services/internal/ResultProvider.java b/...nitor/server/services/ResultProvider.java → ...ver/services/internal/ResultProvider.java
@@ -4,7 +4,7 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
-package org.gridsuite.monitor.server.services;
+package org.gridsuite.monitor.server.services.internal;
 
 import org.gridsuite.monitor.commons.ResultType;
 

diff --git a/...onitor/server/services/ResultService.java → ...rver/services/internal/ResultService.java b/...onitor/server/services/ResultService.java → ...rver/services/internal/ResultService.java
@@ -4,7 +4,7 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
-package org.gridsuite.monitor.server.services;
+package org.gridsuite.monitor.server.services.internal;
 
 import org.gridsuite.monitor.commons.ResultInfos;
 import org.gridsuite.monitor.commons.ResultType;

diff --git a/...itor/server/services/ConsumerService.java → ...r/services/messaging/ConsumerService.java b/...itor/server/services/ConsumerService.java → ...r/services/messaging/ConsumerService.java
@@ -4,14 +4,15 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
-package org.gridsuite.monitor.server.services;
+package org.gridsuite.monitor.server.services.messaging;
 
 import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.core.type.TypeReference;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import org.gridsuite.monitor.commons.MessageType;
 import org.gridsuite.monitor.commons.ProcessExecutionStatusUpdate;
 import org.gridsuite.monitor.commons.ProcessExecutionStep;
+import org.gridsuite.monitor.server.services.internal.MonitorService;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Autowired;

diff --git a/.../server/services/NotificationService.java → ...rvices/messaging/NotificationService.java b/.../server/services/NotificationService.java → ...rvices/messaging/NotificationService.java
@@ -4,7 +4,7 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
-package org.gridsuite.monitor.server.services;
+package org.gridsuite.monitor.server.services.messaging;
 
 import lombok.RequiredArgsConstructor;
 import org.gridsuite.monitor.commons.ProcessConfig;

diff --git a/monitor-server/src/test/java/org/gridsuite/monitor/server/MonitorIntegrationTest.java b/monitor-server/src/test/java/org/gridsuite/monitor/server/MonitorIntegrationTest.java
@@ -14,11 +14,11 @@
 import org.gridsuite.monitor.server.entities.ProcessExecutionEntity;
 import org.gridsuite.monitor.server.repositories.ProcessConfigRepository;
 import org.gridsuite.monitor.server.repositories.ProcessExecutionRepository;
-import org.gridsuite.monitor.server.services.ConsumerService;
+import org.gridsuite.monitor.server.services.messaging.ConsumerService;
 import org.gridsuite.monitor.server.services.ProcessConfigService;
-import org.gridsuite.monitor.server.services.ReportService;
-import org.gridsuite.monitor.server.services.MonitorService;
-import org.gridsuite.monitor.server.services.ResultService;
+import org.gridsuite.monitor.server.services.external.client.ReportRestClient;
+import org.gridsuite.monitor.server.services.internal.MonitorService;
+import org.gridsuite.monitor.server.services.internal.ResultService;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -78,7 +78,7 @@ class MonitorIntegrationTest {
     private MockMvc mockMvc;
 
     @MockitoBean
-    private ReportService reportService;
+    private ReportRestClient reportRestClient;
 
     @MockitoBean
     private ResultService resultService;
@@ -183,8 +183,8 @@ void securityAnalysisProcessIT() throws Exception {
             new ReportLog("message2", Severity.WARN, 2, UUID.randomUUID())), 100, 10);
         ReportPage reportPage1 = new ReportPage(2, List.of(new ReportLog("message3", Severity.ERROR, 3, UUID.randomUUID())), 200, 20);
 
-        when(reportService.getReport(reportId0)).thenReturn(reportPage0);
-        when(reportService.getReport(reportId1)).thenReturn(reportPage1);
+        when(reportRestClient.getReport(reportId0)).thenReturn(reportPage0);
+        when(reportRestClient.getReport(reportId1)).thenReturn(reportPage1);
 
         // Test the reports endpoint fetches correctly from database
         mockMvc.perform(get("/v1/executions/{executionId}/reports", executionId))

diff --git a/...-server/src/test/java/org/gridsuite/monitor/server/controllers/MonitorControllerTest.java b/...-server/src/test/java/org/gridsuite/monitor/server/controllers/MonitorControllerTest.java
@@ -16,7 +16,7 @@
 import org.gridsuite.monitor.server.dto.ReportLog;
 import org.gridsuite.monitor.server.dto.ReportPage;
 import org.gridsuite.monitor.server.dto.Severity;
-import org.gridsuite.monitor.server.services.MonitorService;
+import org.gridsuite.monitor.server.services.internal.MonitorService;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.web.servlet.WebMvcTest;

diff --git a/...s/SecurityAnalysisResultProviderTest.java → ...r/SecurityAnalysisResultProviderTest.java b/...s/SecurityAnalysisResultProviderTest.java → ...r/SecurityAnalysisResultProviderTest.java
@@ -4,9 +4,10 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
-package org.gridsuite.monitor.server.services;
+package org.gridsuite.monitor.server.services.external.adapter;
 
 import org.gridsuite.monitor.commons.ResultType;
+import org.gridsuite.monitor.server.services.external.client.SecurityAnalysisRestClient;
 import org.junit.jupiter.api.Test;
 import org.mockito.Mockito;
 
@@ -19,11 +20,11 @@
  * @author Kevin Le Saulnier <kevin.le-saulnier at rte-france.com>
  */
 class SecurityAnalysisResultProviderTest {
-    private final SecurityAnalysisService securityAnalysisService =
-        Mockito.mock(SecurityAnalysisService.class);
+    private final SecurityAnalysisRestClient securityAnalysisRestClient =
+        Mockito.mock(SecurityAnalysisRestClient.class);
 
     private final SecurityAnalysisResultProvider provider =
-        new SecurityAnalysisResultProvider(securityAnalysisService);
+        new SecurityAnalysisResultProvider(securityAnalysisRestClient);
 
     @Test
     void getTypeShouldReturnSecurityAnalysis() {
@@ -36,24 +37,24 @@ void getResultShouldDelegateToSecurityAnalysisService() {
         UUID id = UUID.randomUUID();
         String expected = "result";
 
-        when(securityAnalysisService.getResult(id)).thenReturn(expected);
+        when(securityAnalysisRestClient.getResult(id)).thenReturn(expected);
 
         String result = provider.getResult(id);
 
         assertThat(result).isEqualTo(expected);
-        verify(securityAnalysisService).getResult(id);
-        verifyNoMoreInteractions(securityAnalysisService);
+        verify(securityAnalysisRestClient).getResult(id);
+        verifyNoMoreInteractions(securityAnalysisRestClient);
     }
 
     @Test
     void deleteResultShouldDelegateToSecurityAnalysisService() {
         UUID id = UUID.randomUUID();
 
-        doNothing().when(securityAnalysisService).deleteResult(id);
+        doNothing().when(securityAnalysisRestClient).deleteResult(id);
 
         provider.deleteResult(id);
 
-        verify(securityAnalysisService).deleteResult(id);
-        verifyNoMoreInteractions(securityAnalysisService);
+        verify(securityAnalysisRestClient).deleteResult(id);
+        verifyNoMoreInteractions(securityAnalysisRestClient);
     }
 }
diff --git a/...or/server/services/ReportServiceTest.java → ...external/client/ReportRestClientTest.java b/...or/server/services/ReportServiceTest.java → ...external/client/ReportRestClientTest.java
@@ -4,7 +4,7 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
-package org.gridsuite.monitor.server.services;
+package org.gridsuite.monitor.server.services.external.client;
 
 import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.databind.ObjectMapper;
@@ -32,11 +32,11 @@
 /**
  * @author Franck Lecuyer <franck.lecuyer at rte-france.com>
  */
-@RestClientTest(ReportService.class)
-@ContextConfiguration(classes = {ReportService.class})
-class ReportServiceTest {
+@RestClientTest(ReportRestClient.class)
+@ContextConfiguration(classes = {ReportRestClient.class})
+class ReportRestClientTest {
     @Autowired
-    private ReportService reportService;
+    private ReportRestClient reportRestClient;
 
     @Autowired
     private MockRestServiceServer server;
@@ -58,7 +58,7 @@ void getReport() throws JsonProcessingException {
                 .contentType(MediaType.APPLICATION_JSON)
                 .body(objectMapper.writeValueAsString(reportPage)));
 
-        ReportPage reportResult = reportService.getReport(reportId);
+        ReportPage reportResult = reportRestClient.getReport(reportId);
         assertThat(reportResult).usingRecursiveComparison().isEqualTo(reportPage);
     }
 
@@ -70,7 +70,7 @@ void getReportFailed() {
             .andExpect(MockRestRequestMatchers.requestTo("http://report-server/v1/reports/" + reportId + "/logs"))
             .andRespond(MockRestResponseCreators.withServerError());
 
-        assertThatThrownBy(() -> reportService.getReport(reportId)).isInstanceOf(RestClientException.class);
+        assertThatThrownBy(() -> reportRestClient.getReport(reportId)).isInstanceOf(RestClientException.class);
     }
 
     @Test
@@ -81,7 +81,7 @@ void deleteReport() {
             .andExpect(MockRestRequestMatchers.requestTo("http://report-server/v1/reports/" + reportId))
             .andRespond(MockRestResponseCreators.withSuccess());
 
-        assertThatNoException().isThrownBy(() -> reportService.deleteReport(reportId));
+        assertThatNoException().isThrownBy(() -> reportRestClient.deleteReport(reportId));
     }
 
     @Test
@@ -92,6 +92,6 @@ void deleteReportFailed() {
             .andExpect(MockRestRequestMatchers.requestTo("http://report-server/v1/reports/" + reportId))
             .andRespond(MockRestResponseCreators.withServerError());
 
-        assertThatThrownBy(() -> reportService.deleteReport(reportId)).isInstanceOf(RestClientException.class);
+        assertThatThrownBy(() -> reportRestClient.deleteReport(reportId)).isInstanceOf(RestClientException.class);
     }
 }