Change upsert video to update video

muttalipkucuk · Dec 15, 2019 · d5cfd6c · d5cfd6c
1 parent 3ccfd7f
commit d5cfd6c
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 10 deletions.
diff --git a/video-api/src/main/java/nl/muttalipkucuk/video/api/VideoRequest.java b/video-api/src/main/java/nl/muttalipkucuk/video/api/VideoRequest.java
@@ -29,7 +29,7 @@ class Create {
     @Value
     @Builder
     @AllArgsConstructor
-    class Upsert {
+    class Update {
         @NonNull
         @Builder.Default
         Option<String> name = Option.none();

diff --git a/video-api/src/main/java/nl/muttalipkucuk/video/api/VideoService.java b/video-api/src/main/java/nl/muttalipkucuk/video/api/VideoService.java
@@ -21,7 +21,7 @@ public interface VideoService extends Service {
 
     ServiceCall<NotUsed, Video> get(UUID id);
 
-    ServiceCall<VideoRequest.Upsert, Video> upsert(UUID id);
+    ServiceCall<VideoRequest.Update, Video> update(UUID id);
 
     ServiceCall<NotUsed, Done> remove(UUID id);
 
@@ -34,7 +34,7 @@ default Descriptor descriptor() {
                 restCall(Method.GET, "/health", this::health),
                 restCall(Method.POST, "api/video", this::create),
                 restCall(Method.GET, "api/video?id", this::get),
-                restCall(Method.PUT, "api/video?id", this::upsert),
+                restCall(Method.PUT, "api/video?id", this::update),
                 restCall(Method.DELETE, "api/video?id", this::remove)
             )
             .withTopics(

diff --git a/video-impl/src/main/java/nl/muttalipkucuk/video/impl/VideoCommand.java b/video-impl/src/main/java/nl/muttalipkucuk/video/impl/VideoCommand.java
@@ -48,7 +48,7 @@ final class Get implements VideoCommand, CompressedJsonable, PersistentEntity.Re
     @Wither
     @Immutable
     @Getter
-    final class Upsert implements VideoCommand, CompressedJsonable, PersistentEntity.ReplyType<VideoState> {
+    final class Update implements VideoCommand, CompressedJsonable, PersistentEntity.ReplyType<VideoState> {
         @NonNull
         @Builder.Default
         Option<String> name = Option.none();

diff --git a/video-impl/src/main/java/nl/muttalipkucuk/video/impl/VideoServiceImpl.java b/video-impl/src/main/java/nl/muttalipkucuk/video/impl/VideoServiceImpl.java
@@ -61,9 +61,9 @@ public ServiceCall<NotUsed, Video> get(UUID id) {
     }
 
     @Override
-    public ServiceCall<VideoRequest.Upsert, Video> upsert(UUID id) {
+    public ServiceCall<VideoRequest.Update, Video> update(UUID id) {
         return req -> RequestValidator.validate(req)
-            .map(validatedRequest -> upsertVideo(id, CommandBuilder.build(validatedRequest)))
+            .map(validatedRequest -> updateVideo(id, CommandBuilder.build(validatedRequest)))
             .fold(
                 err -> {
                     throw ErrorHandler.handle(err);
@@ -95,7 +95,7 @@ private CompletionStage<VideoState> getVideo(UUID id) {
             .ask(CommandBuilder.buildGet());
     }
 
-    private CompletionStage<VideoState> upsertVideo(UUID id, VideoCommand.Upsert cmd) {
+    private CompletionStage<VideoState> updateVideo(UUID id, VideoCommand.Update cmd) {
         return persistentEntityRegistry
             .refFor(VideoEntity.class, id.toString())
             .ask(cmd);

diff --git a/video-impl/src/main/java/nl/muttalipkucuk/video/impl/builder/CommandBuilder.java b/video-impl/src/main/java/nl/muttalipkucuk/video/impl/builder/CommandBuilder.java
@@ -19,8 +19,8 @@ public static VideoCommand.Get buildGet() {
         return VideoCommand.Get.builder().build();
     }
 
-    public static VideoCommand.Upsert build(VideoRequest.Upsert req) {
-        return VideoCommand.Upsert.builder()
+    public static VideoCommand.Update build(VideoRequest.Update req) {
+        return VideoCommand.Update.builder()
             .name(req.getName())
             .description(req.getDescription())
             .likes(req.getLikes())

diff --git a/video-impl/src/main/java/nl/muttalipkucuk/video/impl/validator/RequestValidator.java b/video-impl/src/main/java/nl/muttalipkucuk/video/impl/validator/RequestValidator.java
@@ -20,7 +20,7 @@ public static Validation<Seq<DomainError>, VideoRequest.Create> validate(VideoRe
         ).ap((notUsed, notUsed2) -> request);
     }
 
-    public static Validation<Seq<DomainError>, VideoRequest.Upsert> validate(VideoRequest.Upsert request) {
+    public static Validation<Seq<DomainError>, VideoRequest.Update> validate(VideoRequest.Update request) {
         return Validation.valid(request);
     }