Add endpoint to compare 2 process configs

[FranckLecuyer]

PR Summary

Waiting for the loadflowParametersUuid to be added in the process config

[khouadrired]

can you provide an example of the output structure?

[antoinebhs]

LGTM I would suggest #64 to simplify/avoid sonar issue

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In
`@monitor-server/src/main/java/org/gridsuite/monitor/server/services/ProcessConfigService.java`:
- Around line 120-150: The helper compareSecurityAnalysisConfigs currently
compares only modificationUuids, parametersUuid, and contingencies; add a fourth
comparison for loadflowParametersUuid by creating a ProcessConfigFieldComparison
entry named "loadflowParameters" that checks
Objects.equals(config1.loadflowParametersUuid(),
config2.loadflowParametersUuid()) and includes both values; also add a
unit/service test for ProcessConfigService.compareSecurityAnalysisConfigs that
constructs SecurityAnalysisConfig instances differing each field in turn
(including loadflowParametersUuid) to assert each difference is detected so all
SecurityAnalysisConfig fields are enumerated by the comparison.

In
`@monitor-server/src/test/java/org/gridsuite/monitor/server/controllers/ProcessConfigControllerTest.java`:
- Around line 309-321: The controller currently lets IllegalArgumentException
from ProcessConfigService.compareProcessConfigs propagate as a ServletException
(500); add an exception mapping to convert IllegalArgumentException into a 400
Bad Request (either an `@ExceptionHandler` in ProcessConfigController or a small
`@ControllerAdvice` class that returns ResponseEntity.badRequest()) and update the
test in ProcessConfigControllerTest (the
compareProcessConfigsShouldThrowExceptionWhenDifferentTypes case) to expect
status().isBadRequest() from the
mockMvc.perform(get("/v1/process-configs/compare")...) call instead of
assertThrows(ServletException), ensuring the test still verifies the service was
invoked with compareProcessConfigs(uuid1, uuid2).

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f99c26f1-fdc7-4e0d-87d4-268253d53945

📥 Commits

Reviewing files that changed from the base of the PR and between 1160609 and 769c542.

📒 Files selected for processing (4)

• monitor-server/src/main/java/org/gridsuite/monitor/server/controllers/ProcessConfigController.java
• monitor-server/src/main/java/org/gridsuite/monitor/server/services/ProcessConfigService.java
• monitor-server/src/test/java/org/gridsuite/monitor/server/controllers/ProcessConfigControllerTest.java
• monitor-server/src/test/java/org/gridsuite/monitor/server/services/ProcessConfigServiceTest.java

🚧 Files skipped from review as they are similar to previous changes (2)

• monitor-server/src/main/java/org/gridsuite/monitor/server/controllers/ProcessConfigController.java
• monitor-server/src/test/java/org/gridsuite/monitor/server/services/ProcessConfigServiceTest.java

[sonarqubecloud]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
90.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud