add UserClient with updateUser-Method and test

fh.fd.ci.client/src/main/java/de/fd/fh/network/FeignClients.java

@@ -12,10 +12,12 @@ public class FeignClients
     private static final String BASE_URL = "http://localhost:4567";
 
     private AccessClient accessClient;
+    private UserClient userClient;
 
     public FeignClients()
     {
         accessClient = buildClient(AccessClient.class);
+        userClient = buildClient(UserClient.class);
     }
 
     <T> T buildClient(final Class<T> client)

fh.fd.ci.client/src/main/java/de/fd/fh/network/UserClient.java

@@ -0,0 +1,18 @@
+package de.fd.fh.network;
+
+import de.fd.fh.shared.Utils;
+import de.fd.fh.shared.network.messages.ChangeUserRequest;
+import de.fd.fh.shared.network.messages.PlayerResponse;
+import feign.Headers;
+import feign.Param;
+import feign.RequestLine;
+
+public interface UserClient
+{
+    @RequestLine("POST /users")
+    @Headers(Utils.AUTHENTICATION_HEADER + ":Bearer {authToken}")
+    PlayerResponse updatePlayer(
+            @Param("authToken") final String authenticationHeader,
+            final ChangeUserRequest request
+    );
+}

fh.fd.ci.client/src/test/java/de/fd/fh/network/FeignClientsTest.java

@@ -0,0 +1,17 @@
+package de.fd.fh.network;
+
+import org.junit.jupiter.api.Test;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+class FeignClientsTest
+{
+    @Test
+    void testInitFeignClients()
+    {
+        final FeignClients feignClients = new FeignClients();
+
+        assertNotNull(feignClients.getAccessClient());
+        assertNotNull(feignClients.getUserClient());
+    }
+}

fh.fd.ci.client/src/test/java/de/fd/fh/network/UserClientTest.java

@@ -0,0 +1,57 @@
+package de.fd.fh.network;
+
+import de.fd.fh.shared.network.messages.ChangeUserRequest;
+import de.fd.fh.shared.network.messages.PlayerResponse;
+import feign.Feign;
+import feign.gson.GsonDecoder;
+import feign.gson.GsonEncoder;
+import feign.mock.HttpMethod;
+import feign.mock.MockClient;
+import feign.mock.MockTarget;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+
+class UserClientTest
+{
+    private UserClient client;
+    private MockClient mockClient;
+
+    @BeforeEach
+    void before()
+    {
+        mockClient = new MockClient();
+
+        client = Feign.builder()
+                .decoder(new GsonDecoder())
+                .encoder(new GsonEncoder())
+                .client(mockClient)
+                .target(new MockTarget<>(UserClient.class));
+    }
+
+    @Test
+    void updatePlayer()
+    {
+        mockClient.ok(HttpMethod.POST, "/users",
+                "{\n" +
+                        "   \"id\": {\n" +
+                        "       \"identifier\":\"12345\"\n" +
+                        "   },\n" +
+                        "   \"name\":\"TestName\"\n" +
+                        "}");
+
+        final ChangeUserRequest request = new ChangeUserRequest(
+                "TestName",
+                "TestPassword");
+
+        final PlayerResponse response = client.updatePlayer("TestAuthHeader", request);
+
+        assertNotNull(response);
+        assertEquals("TestName", response.getName());
+        assertEquals("12345", response.getId().getIdentifier());
+
+        mockClient.verifyOne(HttpMethod.POST, "/users");
+    }
+}

fh.fd.ci.server/src/main/java/de/fd/fh/server/user/UserService.java

@@ -2,7 +2,7 @@ package de.fd.fh.server.user;
 
 import de.fd.fh.server.user.events.ChangePasswordEvent;
 import de.fd.fh.server.user.events.ChangeUserNameEvent;
-import de.fd.fh.server.user.web.ChangeUserRequest;
+import de.fd.fh.shared.network.messages.ChangeUserRequest;
 import de.fd.fh.server.user.web.UserRequest;
 import lombok.RequiredArgsConstructor;
 

fh.fd.ci.server/src/main/java/de/fd/fh/server/user/web/UserController.java

@@ -6,6 +6,7 @@ import de.fd.fh.server.access.AccessToken;
 import de.fd.fh.server.user.User;
 import de.fd.fh.server.user.UserId;
 import de.fd.fh.server.user.UserService;
+import de.fd.fh.shared.network.messages.ChangeUserRequest;
 import lombok.RequiredArgsConstructor;
 import spark.Request;
 import spark.Response;

fh.fd.ci.server/src/test/java/de/fd/fh/server/user/UserServiceTest.java

@@ -1,6 +1,6 @@
 package de.fd.fh.server.user;
 
-import de.fd.fh.server.user.web.ChangeUserRequest;
+import de.fd.fh.shared.network.messages.ChangeUserRequest;
 import de.fd.fh.server.user.web.UserRequest;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;

fh.fd.ci.server/src/main/java/de/fd/fh/server/user/web/ChangeUserRequest.java → fh.fd.ci.shared/src/main/java/de/fd/fh/shared/network/messages/ChangeUserRequest.java

@@ -1,4 +1,4 @@
-package de.fd.fh.server.user.web;
+package de.fd.fh.shared.network.messages;
 
 import lombok.AllArgsConstructor;
 import lombok.Getter;

fh.fd.ci.shared/src/main/java/de/fd/fh/shared/network/messages/Identifier.java

@@ -0,0 +1,12 @@
+package de.fd.fh.shared.network.messages;
+
+
+import lombok.AllArgsConstructor;
+import lombok.Getter;
+
+@AllArgsConstructor
+@Getter
+public class Identifier
+{
+    private String identifier;
+}

fh.fd.ci.shared/src/main/java/de/fd/fh/shared/network/messages/PlayerResponse.java

@@ -0,0 +1,13 @@
+package de.fd.fh.shared.network.messages;
+
+import lombok.AllArgsConstructor;
+import lombok.Getter;
+
+@AllArgsConstructor
+@Getter
+public class PlayerResponse
+{
+    private final Identifier id;
+
+    private String name;
+}