BE-#97: Update test for map widget

SE-TINF22B2 · Apr 25, 2024 · 71a7fc9 · 71a7fc9
1 parent b3aa648
commit 71a7fc9
Show file tree

Hide file tree

Showing 3 changed files with 50 additions and 6 deletions.
diff --git a/.../src/main/java/com/dhbw/get2gether/backend/widget/application/mapper/MapWidgetMapper.java b/.../src/main/java/com/dhbw/get2gether/backend/widget/application/mapper/MapWidgetMapper.java
@@ -12,6 +12,7 @@ public interface MapWidgetMapper {
 
     @Mapping(target = "id", ignore = true)
     @Mapping(target = "creationDate", ignore = true)
+    @Mapping(target = "locations", ignore = true)
     MapWidget mapToMapWidget(MapWidgetCreateCommand createCommand);
 
     @Mapping(target = "id", ignore = true)

diff --git a/backend/src/main/java/com/dhbw/get2gether/backend/widget/model/map/MapWidget.java b/backend/src/main/java/com/dhbw/get2gether/backend/widget/model/map/MapWidget.java
@@ -7,6 +7,7 @@
 import lombok.Getter;
 
 import java.time.LocalDateTime;
+import java.util.ArrayList;
 import java.util.List;
 
 @Builder(toBuilder = true)
@@ -15,7 +16,8 @@
 public class MapWidget extends Widget {
     private final String id;
     private final LocalDateTime creationDate;
-    private List<Location> locations;
+    @Builder.Default
+    private List<Location> locations = new ArrayList<>();
 
     @Override
     public WidgetType getWidgetType() {

diff --git a/...nd/src/test/java/com/dhbw/get2gether/backend/widget/application/MapWidgetServiceTest.java b/...nd/src/test/java/com/dhbw/get2gether/backend/widget/application/MapWidgetServiceTest.java
@@ -7,6 +7,7 @@
 import com.dhbw.get2gether.backend.exceptions.OperationNotAllowedException;
 import com.dhbw.get2gether.backend.utils.WithMockGuestUser;
 import com.dhbw.get2gether.backend.utils.WithMockOAuth2User;
+import com.dhbw.get2gether.backend.widget.model.map.Location;
 import com.dhbw.get2gether.backend.widget.model.map.LocationAddCommand;
 import com.dhbw.get2gether.backend.widget.model.map.MapWidget;
 import com.dhbw.get2gether.backend.widget.model.map.MapWidgetCreateCommand;
@@ -62,7 +63,7 @@ void shouldCreateWidget() {
         // then
         assertThat(returnedEvent).isNotNull();
         assertThat(returnedEvent.getWidgets()).hasSize(1);
-        assertThat(returnedEvent.getWidgets().get(0)).isInstanceOf(MapWidget.class);
+        assertThat(returnedEvent.getWidgets().get(0)).isInstanceOf(MapWidget.class).hasNoNullFieldsOrProperties();
     }
 
     @Test
@@ -111,11 +112,11 @@ void shouldAddLocation() {
         AuthenticatedPrincipal principal = (AuthenticatedPrincipal)
                 SecurityContextHolder.getContext().getAuthentication().getPrincipal();
         MapWidget widget = MapWidget.builder()
-                .id("123")
+                .id("wi-123")
                 .locations(new ArrayList<>())
                 .build();
         Event event = Event.builder()
-                .id("123")
+                .id("ev-123")
                 .widgets(new ArrayList<>(List.of(widget)))
                 .build();
         LocationAddCommand addCommand = LocationAddCommand.builder()
@@ -154,11 +155,11 @@ void shouldNotAddLocationIfUserIsGuest() {
         AuthenticatedPrincipal principal = (AuthenticatedPrincipal)
                 SecurityContextHolder.getContext().getAuthentication().getPrincipal();
         MapWidget widget = MapWidget.builder()
-                .id("123")
+                .id("wi-123")
                 .locations(new ArrayList<>())
                 .build();
         Event event = Event.builder()
-                .id("123")
+                .id("ev-123")
                 .widgets(new ArrayList<>(List.of(widget)))
                 .build();
         LocationAddCommand addCommand = LocationAddCommand.builder()
@@ -176,4 +177,44 @@ void shouldNotAddLocationIfUserIsGuest() {
         assertThatThrownBy(() -> mapWidgetService.addLocation(principal, event.getId(), widget.getId(), addCommand))
                 .isInstanceOf(AccessDeniedException.class);
     }
+
+    @Test
+    @WithMockOAuth2User
+    void shouldRemoveLocation() {
+        // given
+        AuthenticatedPrincipal principal = (AuthenticatedPrincipal)
+                SecurityContextHolder.getContext().getAuthentication().getPrincipal();
+        Location location = Location.builder()
+                .id("lo-123")
+                .name("Test")
+                .placeId("place-123")
+                .address("address-123")
+                .latitude(12.34)
+                .longitude(56.78)
+                .build();
+        MapWidget widget = MapWidget.builder()
+                .id("wi-123")
+                .locations(new ArrayList<>(List.of(location)))
+                .build();
+        Event event = Event.builder()
+                .id("ev-123")
+                .widgets(new ArrayList<>(List.of(widget)))
+                .build();
+
+        when(eventService.getSingleEvent(any(), eq(event.getId()))).thenReturn(event);
+        when(eventService.updateEventWidgets(any(), eq(event.getId()), any()))
+                .thenAnswer(i -> event.toBuilder()
+                        .widgets(
+                                i.getArgument(2, EventWidgetUpdateCommand.class).getWidgets()
+                        )
+                        .build()
+                );
+
+        // when
+        MapWidget returnedWidget = mapWidgetService.removeLocation(principal, event.getId(), widget.getId(), location.getId());
+
+        // then
+        assertThat(returnedWidget).isNotNull();
+        assertThat(returnedWidget.getLocations()).isEmpty();
+    }
 }