From 9546e537855348f5a970434bfe3b7e331ef68997 Mon Sep 17 00:00:00 2001
From: gianmarcoplutino <119858159+gianmarcoplutino@users.noreply.github.com>
Date: Thu, 19 Sep 2024 12:49:14 +0200
Subject: [PATCH] [SELC-5596] fix: fixed checkManager (#491)

---
 apps/onboarding-ms/src/main/docs/openapi.json                 | 4 ++--
 apps/onboarding-ms/src/main/docs/openapi.yaml                 | 4 ++--
 .../selfcare/onboarding/controller/OnboardingController.java  | 3 ++-
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/apps/onboarding-ms/src/main/docs/openapi.json b/apps/onboarding-ms/src/main/docs/openapi.json
index 8de5f6358..8c696f973 100644
--- a/apps/onboarding-ms/src/main/docs/openapi.json
+++ b/apps/onboarding-ms/src/main/docs/openapi.json
@@ -355,7 +355,7 @@
     "/v1/onboarding/check-manager" : {
       "post" : {
         "tags" : [ "Onboarding Controller" ],
-        "summary" : "In the addition administrator flow, it checks if the new manager is equal from old one, returning true ",
+        "summary" : "In the addition administrator flow, it checks if the new manager is equal to the old one, returning true or false",
         "requestBody" : {
           "content" : {
             "application/json" : {
@@ -369,7 +369,7 @@
           "200" : {
             "description" : "OK",
             "content" : {
-              "text/plain" : {
+              "application/json" : {
                 "schema" : {
                   "type" : "boolean"
                 }
diff --git a/apps/onboarding-ms/src/main/docs/openapi.yaml b/apps/onboarding-ms/src/main/docs/openapi.yaml
index acdf00811..a01ec7418 100644
--- a/apps/onboarding-ms/src/main/docs/openapi.yaml
+++ b/apps/onboarding-ms/src/main/docs/openapi.yaml
@@ -249,7 +249,7 @@ paths:
       tags:
       - Onboarding Controller
       summary: "In the addition administrator flow, it checks if the new manager is\
-        \ equal from old one, returning true "
+        \ equal to the old one, returning true or false"
       requestBody:
         content:
           application/json:
@@ -259,7 +259,7 @@ paths:
         "200":
           description: OK
           content:
-            text/plain:
+            application/json:
               schema:
                 type: boolean
         "401":
diff --git a/apps/onboarding-ms/src/main/java/it/pagopa/selfcare/onboarding/controller/OnboardingController.java b/apps/onboarding-ms/src/main/java/it/pagopa/selfcare/onboarding/controller/OnboardingController.java
index 2f9022b28..c669e106d 100644
--- a/apps/onboarding-ms/src/main/java/it/pagopa/selfcare/onboarding/controller/OnboardingController.java
+++ b/apps/onboarding-ms/src/main/java/it/pagopa/selfcare/onboarding/controller/OnboardingController.java
@@ -336,9 +336,10 @@ public Uni<Response> update(@PathParam(value = "onboardingId") String onboarding
     }
 
     @Operation(summary = "In the addition administrator flow, it checks " +
-            "if the new manager is equal from old one, returning true ")
+            "if the new manager is equal to the old one, returning true or false")
     @POST
     @Consumes(MediaType.APPLICATION_JSON)
+    @Produces(MediaType.APPLICATION_JSON)
     @Path("/check-manager")
     public Uni<Boolean> checkManager(OnboardingUserRequest onboardingUserRequest) {
         return onboardingService.checkManager(onboardingUserRequest);