Fix creds validation endpoint (#740) [ci skip]

Signed-off-by: Paolo Di Tommaso <[email protected]>

Author: pditommaso

src/main/groovy/io/seqera/wave/controller/ValidateController.groovy

@@ -18,15 +18,13 @@
 
 package io.seqera.wave.controller
 
-import io.micronaut.http.annotation.Body
 import io.micronaut.http.annotation.Controller
 import io.micronaut.http.annotation.Post
 import io.micronaut.scheduling.TaskExecutors
 import io.micronaut.scheduling.annotation.ExecuteOn
 import io.seqera.wave.auth.RegistryAuthService
 import jakarta.inject.Inject
 import jakarta.validation.Valid
-import reactor.core.publisher.Mono
 
 @ExecuteOn(TaskExecutors.IO)
 @Controller("/validate-creds")
@@ -35,10 +33,8 @@ class ValidateController {
     @Inject RegistryAuthService loginService
 
     @Post
-    Mono<Boolean> validateCreds(@Valid @Body ValidateRegistryCredsRequest request){
-        Mono.just(
-            loginService.validateUser(request.registry, request.userName, request.password)
-        )
+    Boolean validateCreds(@Valid ValidateRegistryCredsRequest request){
+        loginService.validateUser(request.registry, request.userName, request.password)
     }
 
 }

src/test/groovy/io/seqera/wave/controller/ValidateCredsControllerTest.groovy

@@ -67,30 +67,30 @@ class ValidateCredsControllerTest extends Specification implements SecureDockerR
 
     void 'should validate username required'() {
         when:
-        HttpRequest request = HttpRequest.POST("/validate-creds", [
+        HttpRequest request = HttpRequest.POST("/validate-creds", [request: [
                 password: 'test',
-        ])
+        ]])
         client.toBlocking().exchange(request, Boolean)
         then:
         def e = thrown(HttpClientResponseException)
     }
 
     void 'should validate pwd required'() {
         when:
-        HttpRequest request = HttpRequest.POST("/validate-creds", [
+        HttpRequest request = HttpRequest.POST("/validate-creds", [request:[
                 userName: 'test',
-        ])
+        ]])
         client.toBlocking().exchange(request, Boolean)
         then:
         def e = thrown(HttpClientResponseException)
     }
 
     void 'should validate the test user'() {
         given:
-        def req = [
+        def req = [request: [
                 userName:'test',
                 password:'test',
-                registry: getTestRegistryUrl('test') ]
+                registry: getTestRegistryUrl('test') ]]
         and:
         HttpRequest request = HttpRequest.POST("/validate-creds", req)
         when:
@@ -103,11 +103,11 @@ class ValidateCredsControllerTest extends Specification implements SecureDockerR
 
     void 'test validateController valid login'() {
         given:
-        def req = [
+        def req = [request: [
                 userName: USER,
                 password: PWD,
                 registry: getTestRegistryUrl(REGISTRY_URL)
-        ]
+        ]]
         HttpRequest request = HttpRequest.POST("/validate-creds", req)
         when:
         HttpResponse<Boolean> response = client.toBlocking().exchange(request, Boolean)