From 709964f2babb070ba4d166c959112bbb5c58bb76 Mon Sep 17 00:00:00 2001 From: DanielOber <145556033+DanielOber@users.noreply.github.com> Date: Tue, 2 Apr 2024 10:02:54 +0200 Subject: [PATCH 1/2] bug/zeitraum-wird-nicht-ausgewertet into sprint (#64) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * fix einfügen des zeitraums * sortierung für list aus localDates hinzugefügt * unused import entfernt --- .../muenchen/dave/services/messstelle/MesswerteService.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/main/java/de/muenchen/dave/services/messstelle/MesswerteService.java b/src/main/java/de/muenchen/dave/services/messstelle/MesswerteService.java index 5982c910..59ee709e 100644 --- a/src/main/java/de/muenchen/dave/services/messstelle/MesswerteService.java +++ b/src/main/java/de/muenchen/dave/services/messstelle/MesswerteService.java @@ -11,6 +11,7 @@ import de.muenchen.dave.geodateneai.gen.model.MeasurementValuesResponse; import de.muenchen.dave.geodateneai.gen.model.TotalSumPerMessquerschnitt; import de.muenchen.dave.util.OptionsUtil; +import java.util.Collections; import java.util.List; import lombok.AllArgsConstructor; import lombok.extern.slf4j.Slf4j; @@ -72,10 +73,12 @@ protected MeasurementValuesResponse ladeMesswerteIntervall(final MessstelleOptio if (StringUtils.isNotEmpty(options.getTagesTyp())) { request.setTagesTyp(GetMeasurementValuesRequest.TagesTypEnum.valueOf(options.getTagesTyp())); } - request.setZeitpunktStart(options.getZeitraum().get(0)); if (options.getZeitraum().size() == 2) { + Collections.sort(options.getZeitraum()); + request.setZeitpunktStart(options.getZeitraum().get(0)); request.setZeitpunktEnde(options.getZeitraum().get(1)); } else { + request.setZeitpunktStart(options.getZeitraum().get(0)); request.setZeitpunktEnde(options.getZeitraum().get(0)); } request.setUhrzeitStart(options.getZeitblock().getStart().toLocalTime()); From 914a3edeb0f8be85dee47fdefbb29677e4e55e6b Mon Sep 17 00:00:00 2001 From: Der-Alex-K <18263366+Der-Alex-K@users.noreply.github.com> Date: Thu, 4 Apr 2024 10:37:02 +0200 Subject: [PATCH 2/2] Koordinaten der Messstelle sind nun bearbeitbar (#66) --- .../mapper/detektor/MessstelleMapper.java | 3 + .../detektor/MessstelleReceiverMapper.java | 5 +- .../gen/model/MessquerschnittDto.java | 58 +++++++++---------- .../geodateneai/gen/model/MessstelleDto.java | 58 +++++++++---------- .../geodateneai/gen/model/StadtbezirkDto.java | 58 +++++++++---------- .../domain/mapper/MessstelleMapperTests.java | 3 +- .../mapper/MessstelleReceiverMapperTests.java | 2 +- .../messstelle/MessstelleReceiverTest.java | 28 +++++---- 8 files changed, 109 insertions(+), 106 deletions(-) diff --git a/src/main/java/de/muenchen/dave/domain/mapper/detektor/MessstelleMapper.java b/src/main/java/de/muenchen/dave/domain/mapper/detektor/MessstelleMapper.java index 13ec3d8e..2c3761aa 100644 --- a/src/main/java/de/muenchen/dave/domain/mapper/detektor/MessstelleMapper.java +++ b/src/main/java/de/muenchen/dave/domain/mapper/detektor/MessstelleMapper.java @@ -20,6 +20,7 @@ import org.mapstruct.Mapper; import org.mapstruct.Mapping; import org.mapstruct.MappingTarget; +import org.springframework.data.elasticsearch.core.geo.GeoPoint; @Mapper(componentModel = "spring") public interface MessstelleMapper { @@ -85,6 +86,8 @@ default void updateMessstelleAfterMapping(@MappingTarget Messstelle actual, Edit updateMessquerschnitt(messquerschnitt, dto1); } })); + + actual.setPunkt(new GeoPoint(dto.getLatitude(), dto.getLongitude())); } ReadMessquerschnittDTO bean2readDto(Messquerschnitt bean); diff --git a/src/main/java/de/muenchen/dave/domain/mapper/detektor/MessstelleReceiverMapper.java b/src/main/java/de/muenchen/dave/domain/mapper/detektor/MessstelleReceiverMapper.java index ab353901..6e4243b9 100644 --- a/src/main/java/de/muenchen/dave/domain/mapper/detektor/MessstelleReceiverMapper.java +++ b/src/main/java/de/muenchen/dave/domain/mapper/detektor/MessstelleReceiverMapper.java @@ -10,6 +10,7 @@ import java.util.Set; import java.util.UUID; import org.apache.commons.collections4.CollectionUtils; +import org.apache.commons.lang3.ObjectUtils; import org.apache.commons.lang3.StringUtils; import org.mapstruct.AfterMapping; import org.mapstruct.Mapper; @@ -32,7 +33,7 @@ default void createMessstelleAfterMapping(@MappingTarget Messstelle bean, Messst bean.setId(UUID.randomUUID().toString()); } - if (dto.getXcoordinate() != null && dto.getYcoordinate() != null) { + if (ObjectUtils.isEmpty(bean.getPunkt()) && dto.getXcoordinate() != null && dto.getYcoordinate() != null) { bean.setPunkt(new GeoPoint(dto.getXcoordinate(), dto.getYcoordinate())); } @@ -58,7 +59,7 @@ default void createMessquerschnittAfterMapping(@MappingTarget Messquerschnitt be if (StringUtils.isEmpty(bean.getId())) { bean.setId(UUID.randomUUID().toString()); } - if (dto.getXcoordinate() != null && dto.getYcoordinate() != null) { + if (ObjectUtils.isEmpty(bean.getPunkt()) && dto.getXcoordinate() != null && dto.getYcoordinate() != null) { bean.setPunkt(new GeoPoint(dto.getXcoordinate(), dto.getYcoordinate())); } } diff --git a/src/main/java/de/muenchen/dave/geodateneai/gen/model/MessquerschnittDto.java b/src/main/java/de/muenchen/dave/geodateneai/gen/model/MessquerschnittDto.java index bc7f4eba..00fa92f7 100644 --- a/src/main/java/de/muenchen/dave/geodateneai/gen/model/MessquerschnittDto.java +++ b/src/main/java/de/muenchen/dave/geodateneai/gen/model/MessquerschnittDto.java @@ -34,8 +34,8 @@ MessquerschnittDto.JSON_PROPERTY_FAHRTRICHTUNG, MessquerschnittDto.JSON_PROPERTY_ANZAHL_FAHRSPUREN, MessquerschnittDto.JSON_PROPERTY_ANZAHL_DETEKTOREN, - MessquerschnittDto.JSON_PROPERTY_YCOORDINATE, - MessquerschnittDto.JSON_PROPERTY_XCOORDINATE + MessquerschnittDto.JSON_PROPERTY_XCOORDINATE, + MessquerschnittDto.JSON_PROPERTY_YCOORDINATE }) @javax.annotation.Generated(value = "org.openapitools.codegen.languages.JavaClientCodegen") public class MessquerschnittDto { @@ -60,12 +60,12 @@ public class MessquerschnittDto { public static final String JSON_PROPERTY_ANZAHL_DETEKTOREN = "anzahlDetektoren"; private Integer anzahlDetektoren; - public static final String JSON_PROPERTY_YCOORDINATE = "ycoordinate"; - private Double ycoordinate; - public static final String JSON_PROPERTY_XCOORDINATE = "xcoordinate"; private Double xcoordinate; + public static final String JSON_PROPERTY_YCOORDINATE = "ycoordinate"; + private Double ycoordinate; + public MessquerschnittDto() { } @@ -251,55 +251,55 @@ public void setAnzahlDetektoren(Integer anzahlDetektoren) { } - public MessquerschnittDto ycoordinate(Double ycoordinate) { + public MessquerschnittDto xcoordinate(Double xcoordinate) { - this.ycoordinate = ycoordinate; + this.xcoordinate = xcoordinate; return this; } /** - * Get ycoordinate - * @return ycoordinate + * Get xcoordinate + * @return xcoordinate **/ @javax.annotation.Nullable - @JsonProperty(JSON_PROPERTY_YCOORDINATE) + @JsonProperty(JSON_PROPERTY_XCOORDINATE) @JsonInclude(value = JsonInclude.Include.USE_DEFAULTS) - public Double getYcoordinate() { - return ycoordinate; + public Double getXcoordinate() { + return xcoordinate; } - @JsonProperty(JSON_PROPERTY_YCOORDINATE) + @JsonProperty(JSON_PROPERTY_XCOORDINATE) @JsonInclude(value = JsonInclude.Include.USE_DEFAULTS) - public void setYcoordinate(Double ycoordinate) { - this.ycoordinate = ycoordinate; + public void setXcoordinate(Double xcoordinate) { + this.xcoordinate = xcoordinate; } - public MessquerschnittDto xcoordinate(Double xcoordinate) { + public MessquerschnittDto ycoordinate(Double ycoordinate) { - this.xcoordinate = xcoordinate; + this.ycoordinate = ycoordinate; return this; } /** - * Get xcoordinate - * @return xcoordinate + * Get ycoordinate + * @return ycoordinate **/ @javax.annotation.Nullable - @JsonProperty(JSON_PROPERTY_XCOORDINATE) + @JsonProperty(JSON_PROPERTY_YCOORDINATE) @JsonInclude(value = JsonInclude.Include.USE_DEFAULTS) - public Double getXcoordinate() { - return xcoordinate; + public Double getYcoordinate() { + return ycoordinate; } - @JsonProperty(JSON_PROPERTY_XCOORDINATE) + @JsonProperty(JSON_PROPERTY_YCOORDINATE) @JsonInclude(value = JsonInclude.Include.USE_DEFAULTS) - public void setXcoordinate(Double xcoordinate) { - this.xcoordinate = xcoordinate; + public void setYcoordinate(Double ycoordinate) { + this.ycoordinate = ycoordinate; } @Override @@ -318,13 +318,13 @@ public boolean equals(Object o) { Objects.equals(this.fahrtrichtung, messquerschnittDto.fahrtrichtung) && Objects.equals(this.anzahlFahrspuren, messquerschnittDto.anzahlFahrspuren) && Objects.equals(this.anzahlDetektoren, messquerschnittDto.anzahlDetektoren) && - Objects.equals(this.ycoordinate, messquerschnittDto.ycoordinate) && - Objects.equals(this.xcoordinate, messquerschnittDto.xcoordinate); + Objects.equals(this.xcoordinate, messquerschnittDto.xcoordinate) && + Objects.equals(this.ycoordinate, messquerschnittDto.ycoordinate); } @Override public int hashCode() { - return Objects.hash(mqId, mstId, strassenname, lageMessquerschnitt, fahrtrichtung, anzahlFahrspuren, anzahlDetektoren, ycoordinate, xcoordinate); + return Objects.hash(mqId, mstId, strassenname, lageMessquerschnitt, fahrtrichtung, anzahlFahrspuren, anzahlDetektoren, xcoordinate, ycoordinate); } @Override @@ -338,8 +338,8 @@ public String toString() { sb.append(" fahrtrichtung: ").append(toIndentedString(fahrtrichtung)).append("\n"); sb.append(" anzahlFahrspuren: ").append(toIndentedString(anzahlFahrspuren)).append("\n"); sb.append(" anzahlDetektoren: ").append(toIndentedString(anzahlDetektoren)).append("\n"); - sb.append(" ycoordinate: ").append(toIndentedString(ycoordinate)).append("\n"); sb.append(" xcoordinate: ").append(toIndentedString(xcoordinate)).append("\n"); + sb.append(" ycoordinate: ").append(toIndentedString(ycoordinate)).append("\n"); sb.append("}"); return sb.toString(); } diff --git a/src/main/java/de/muenchen/dave/geodateneai/gen/model/MessstelleDto.java b/src/main/java/de/muenchen/dave/geodateneai/gen/model/MessstelleDto.java index 6d3d69b0..fcf7b6be 100644 --- a/src/main/java/de/muenchen/dave/geodateneai/gen/model/MessstelleDto.java +++ b/src/main/java/de/muenchen/dave/geodateneai/gen/model/MessstelleDto.java @@ -44,8 +44,8 @@ MessstelleDto.JSON_PROPERTY_HERSTELLER, MessstelleDto.JSON_PROPERTY_DATUM_LETZTE_PLAUSIBLE_MESSUNG, MessstelleDto.JSON_PROPERTY_MESSQUERSCHNITTE, - MessstelleDto.JSON_PROPERTY_YCOORDINATE, - MessstelleDto.JSON_PROPERTY_XCOORDINATE + MessstelleDto.JSON_PROPERTY_XCOORDINATE, + MessstelleDto.JSON_PROPERTY_YCOORDINATE }) @javax.annotation.Generated(value = "org.openapitools.codegen.languages.JavaClientCodegen") public class MessstelleDto { @@ -126,12 +126,12 @@ public static StatusEnum fromValue(String value) { public static final String JSON_PROPERTY_MESSQUERSCHNITTE = "messquerschnitte"; private List messquerschnitte; - public static final String JSON_PROPERTY_YCOORDINATE = "ycoordinate"; - private Double ycoordinate; - public static final String JSON_PROPERTY_XCOORDINATE = "xcoordinate"; private Double xcoordinate; + public static final String JSON_PROPERTY_YCOORDINATE = "ycoordinate"; + private Double ycoordinate; + public MessstelleDto() { } @@ -455,55 +455,55 @@ public void setMessquerschnitte(List messquerschnitte) { } - public MessstelleDto ycoordinate(Double ycoordinate) { + public MessstelleDto xcoordinate(Double xcoordinate) { - this.ycoordinate = ycoordinate; + this.xcoordinate = xcoordinate; return this; } /** - * Get ycoordinate - * @return ycoordinate + * Get xcoordinate + * @return xcoordinate **/ @javax.annotation.Nullable - @JsonProperty(JSON_PROPERTY_YCOORDINATE) + @JsonProperty(JSON_PROPERTY_XCOORDINATE) @JsonInclude(value = JsonInclude.Include.USE_DEFAULTS) - public Double getYcoordinate() { - return ycoordinate; + public Double getXcoordinate() { + return xcoordinate; } - @JsonProperty(JSON_PROPERTY_YCOORDINATE) + @JsonProperty(JSON_PROPERTY_XCOORDINATE) @JsonInclude(value = JsonInclude.Include.USE_DEFAULTS) - public void setYcoordinate(Double ycoordinate) { - this.ycoordinate = ycoordinate; + public void setXcoordinate(Double xcoordinate) { + this.xcoordinate = xcoordinate; } - public MessstelleDto xcoordinate(Double xcoordinate) { + public MessstelleDto ycoordinate(Double ycoordinate) { - this.xcoordinate = xcoordinate; + this.ycoordinate = ycoordinate; return this; } /** - * Get xcoordinate - * @return xcoordinate + * Get ycoordinate + * @return ycoordinate **/ @javax.annotation.Nullable - @JsonProperty(JSON_PROPERTY_XCOORDINATE) + @JsonProperty(JSON_PROPERTY_YCOORDINATE) @JsonInclude(value = JsonInclude.Include.USE_DEFAULTS) - public Double getXcoordinate() { - return xcoordinate; + public Double getYcoordinate() { + return ycoordinate; } - @JsonProperty(JSON_PROPERTY_XCOORDINATE) + @JsonProperty(JSON_PROPERTY_YCOORDINATE) @JsonInclude(value = JsonInclude.Include.USE_DEFAULTS) - public void setXcoordinate(Double xcoordinate) { - this.xcoordinate = xcoordinate; + public void setYcoordinate(Double ycoordinate) { + this.ycoordinate = ycoordinate; } @Override @@ -527,13 +527,13 @@ public boolean equals(Object o) { Objects.equals(this.hersteller, messstelleDto.hersteller) && Objects.equals(this.datumLetztePlausibleMessung, messstelleDto.datumLetztePlausibleMessung) && Objects.equals(this.messquerschnitte, messstelleDto.messquerschnitte) && - Objects.equals(this.ycoordinate, messstelleDto.ycoordinate) && - Objects.equals(this.xcoordinate, messstelleDto.xcoordinate); + Objects.equals(this.xcoordinate, messstelleDto.xcoordinate) && + Objects.equals(this.ycoordinate, messstelleDto.ycoordinate); } @Override public int hashCode() { - return Objects.hash(mstId, name, status, realisierungsdatum, abbaudatum, stadtbezirkNummer, bemerkung, fahrzeugKlassen, detektierteVerkehrsarten, hersteller, datumLetztePlausibleMessung, messquerschnitte, ycoordinate, xcoordinate); + return Objects.hash(mstId, name, status, realisierungsdatum, abbaudatum, stadtbezirkNummer, bemerkung, fahrzeugKlassen, detektierteVerkehrsarten, hersteller, datumLetztePlausibleMessung, messquerschnitte, xcoordinate, ycoordinate); } @Override @@ -552,8 +552,8 @@ public String toString() { sb.append(" hersteller: ").append(toIndentedString(hersteller)).append("\n"); sb.append(" datumLetztePlausibleMessung: ").append(toIndentedString(datumLetztePlausibleMessung)).append("\n"); sb.append(" messquerschnitte: ").append(toIndentedString(messquerschnitte)).append("\n"); - sb.append(" ycoordinate: ").append(toIndentedString(ycoordinate)).append("\n"); sb.append(" xcoordinate: ").append(toIndentedString(xcoordinate)).append("\n"); + sb.append(" ycoordinate: ").append(toIndentedString(ycoordinate)).append("\n"); sb.append("}"); return sb.toString(); } diff --git a/src/main/java/de/muenchen/dave/geodateneai/gen/model/StadtbezirkDto.java b/src/main/java/de/muenchen/dave/geodateneai/gen/model/StadtbezirkDto.java index b638ca68..8bba50b8 100644 --- a/src/main/java/de/muenchen/dave/geodateneai/gen/model/StadtbezirkDto.java +++ b/src/main/java/de/muenchen/dave/geodateneai/gen/model/StadtbezirkDto.java @@ -32,8 +32,8 @@ StadtbezirkDto.JSON_PROPERTY_STADTBEZIRK_NUMMER, StadtbezirkDto.JSON_PROPERTY_NAME, StadtbezirkDto.JSON_PROPERTY_FLAECHE_QM, - StadtbezirkDto.JSON_PROPERTY_YCOORDINATE, - StadtbezirkDto.JSON_PROPERTY_XCOORDINATE + StadtbezirkDto.JSON_PROPERTY_XCOORDINATE, + StadtbezirkDto.JSON_PROPERTY_YCOORDINATE }) @javax.annotation.Generated(value = "org.openapitools.codegen.languages.JavaClientCodegen") public class StadtbezirkDto { @@ -49,12 +49,12 @@ public class StadtbezirkDto { public static final String JSON_PROPERTY_FLAECHE_QM = "flaecheQm"; private BigDecimal flaecheQm; - public static final String JSON_PROPERTY_YCOORDINATE = "ycoordinate"; - private BigDecimal ycoordinate; - public static final String JSON_PROPERTY_XCOORDINATE = "xcoordinate"; private BigDecimal xcoordinate; + public static final String JSON_PROPERTY_YCOORDINATE = "ycoordinate"; + private BigDecimal ycoordinate; + public StadtbezirkDto() { } @@ -162,55 +162,55 @@ public void setFlaecheQm(BigDecimal flaecheQm) { } - public StadtbezirkDto ycoordinate(BigDecimal ycoordinate) { + public StadtbezirkDto xcoordinate(BigDecimal xcoordinate) { - this.ycoordinate = ycoordinate; + this.xcoordinate = xcoordinate; return this; } /** - * Get ycoordinate - * @return ycoordinate + * Get xcoordinate + * @return xcoordinate **/ @javax.annotation.Nullable - @JsonProperty(JSON_PROPERTY_YCOORDINATE) + @JsonProperty(JSON_PROPERTY_XCOORDINATE) @JsonInclude(value = JsonInclude.Include.USE_DEFAULTS) - public BigDecimal getYcoordinate() { - return ycoordinate; + public BigDecimal getXcoordinate() { + return xcoordinate; } - @JsonProperty(JSON_PROPERTY_YCOORDINATE) + @JsonProperty(JSON_PROPERTY_XCOORDINATE) @JsonInclude(value = JsonInclude.Include.USE_DEFAULTS) - public void setYcoordinate(BigDecimal ycoordinate) { - this.ycoordinate = ycoordinate; + public void setXcoordinate(BigDecimal xcoordinate) { + this.xcoordinate = xcoordinate; } - public StadtbezirkDto xcoordinate(BigDecimal xcoordinate) { + public StadtbezirkDto ycoordinate(BigDecimal ycoordinate) { - this.xcoordinate = xcoordinate; + this.ycoordinate = ycoordinate; return this; } /** - * Get xcoordinate - * @return xcoordinate + * Get ycoordinate + * @return ycoordinate **/ @javax.annotation.Nullable - @JsonProperty(JSON_PROPERTY_XCOORDINATE) + @JsonProperty(JSON_PROPERTY_YCOORDINATE) @JsonInclude(value = JsonInclude.Include.USE_DEFAULTS) - public BigDecimal getXcoordinate() { - return xcoordinate; + public BigDecimal getYcoordinate() { + return ycoordinate; } - @JsonProperty(JSON_PROPERTY_XCOORDINATE) + @JsonProperty(JSON_PROPERTY_YCOORDINATE) @JsonInclude(value = JsonInclude.Include.USE_DEFAULTS) - public void setXcoordinate(BigDecimal xcoordinate) { - this.xcoordinate = xcoordinate; + public void setYcoordinate(BigDecimal ycoordinate) { + this.ycoordinate = ycoordinate; } @Override @@ -226,13 +226,13 @@ public boolean equals(Object o) { Objects.equals(this.stadtbezirkNummer, stadtbezirkDto.stadtbezirkNummer) && Objects.equals(this.name, stadtbezirkDto.name) && Objects.equals(this.flaecheQm, stadtbezirkDto.flaecheQm) && - Objects.equals(this.ycoordinate, stadtbezirkDto.ycoordinate) && - Objects.equals(this.xcoordinate, stadtbezirkDto.xcoordinate); + Objects.equals(this.xcoordinate, stadtbezirkDto.xcoordinate) && + Objects.equals(this.ycoordinate, stadtbezirkDto.ycoordinate); } @Override public int hashCode() { - return Objects.hash(objectId, stadtbezirkNummer, name, flaecheQm, ycoordinate, xcoordinate); + return Objects.hash(objectId, stadtbezirkNummer, name, flaecheQm, xcoordinate, ycoordinate); } @Override @@ -243,8 +243,8 @@ public String toString() { sb.append(" stadtbezirkNummer: ").append(toIndentedString(stadtbezirkNummer)).append("\n"); sb.append(" name: ").append(toIndentedString(name)).append("\n"); sb.append(" flaecheQm: ").append(toIndentedString(flaecheQm)).append("\n"); - sb.append(" ycoordinate: ").append(toIndentedString(ycoordinate)).append("\n"); sb.append(" xcoordinate: ").append(toIndentedString(xcoordinate)).append("\n"); + sb.append(" ycoordinate: ").append(toIndentedString(ycoordinate)).append("\n"); sb.append("}"); return sb.toString(); } diff --git a/src/test/java/de/muenchen/dave/domain/mapper/MessstelleMapperTests.java b/src/test/java/de/muenchen/dave/domain/mapper/MessstelleMapperTests.java index ffdb5edf..41cec747 100644 --- a/src/test/java/de/muenchen/dave/domain/mapper/MessstelleMapperTests.java +++ b/src/test/java/de/muenchen/dave/domain/mapper/MessstelleMapperTests.java @@ -19,6 +19,7 @@ import lombok.extern.slf4j.Slf4j; import org.assertj.core.api.Assertions; import org.junit.jupiter.api.Test; +import org.springframework.data.elasticsearch.core.geo.GeoPoint; @Slf4j class MessstelleMapperTests { @@ -138,7 +139,7 @@ void testUpdateMessstelle() { expected.setAbbaudatum(bean.getAbbaudatum()); expected.setDatumLetztePlausibleMessung(bean.getDatumLetztePlausibleMessung()); expected.setMessquerschnitte(bean.getMessquerschnitte()); - expected.setPunkt(bean.getPunkt()); + expected.setPunkt(new GeoPoint(updatedData.getLatitude(), updatedData.getLongitude())); expected.setHersteller(bean.getHersteller()); expected.setFahrzeugKlassen(bean.getFahrzeugKlassen()); expected.setDetektierteVerkehrsarten(bean.getDetektierteVerkehrsarten()); diff --git a/src/test/java/de/muenchen/dave/domain/mapper/MessstelleReceiverMapperTests.java b/src/test/java/de/muenchen/dave/domain/mapper/MessstelleReceiverMapperTests.java index d5aad99a..a71b4e8a 100644 --- a/src/test/java/de/muenchen/dave/domain/mapper/MessstelleReceiverMapperTests.java +++ b/src/test/java/de/muenchen/dave/domain/mapper/MessstelleReceiverMapperTests.java @@ -113,7 +113,6 @@ void testUpdateMessstelle() { expected.setHersteller(updatedData.getHersteller()); expected.setFahrzeugKlassen(updatedData.getFahrzeugKlassen()); expected.setDetektierteVerkehrsarten(updatedData.getDetektierteVerkehrsarten()); - expected.setPunkt(new GeoPoint(updatedData.getXcoordinate(), updatedData.getYcoordinate())); expected.setSuchwoerter(new ArrayList<>()); expected.getSuchwoerter().addAll(bean.getCustomSuchwoerter()); expected.getSuchwoerter().add(updatedData.getMstId()); @@ -132,6 +131,7 @@ void testUpdateMessstelle() { expected.setKommentar(bean.getKommentar()); expected.setStandort(bean.getStandort()); expected.setCustomSuchwoerter(bean.getCustomSuchwoerter()); + expected.setPunkt(bean.getPunkt()); expected.setMessquerschnitte(bean.getMessquerschnitte()); final Messstelle actual = this.mapper.updateMessstelle(bean, updatedData); diff --git a/src/test/java/de/muenchen/dave/services/messstelle/MessstelleReceiverTest.java b/src/test/java/de/muenchen/dave/services/messstelle/MessstelleReceiverTest.java index 4dfcd311..7313d0a0 100644 --- a/src/test/java/de/muenchen/dave/services/messstelle/MessstelleReceiverTest.java +++ b/src/test/java/de/muenchen/dave/services/messstelle/MessstelleReceiverTest.java @@ -13,7 +13,7 @@ @Slf4j public class MessstelleReceiverTest { - private MessstelleReceiver messstelleReceiver = new MessstelleReceiver( + private final MessstelleReceiver messstelleReceiver = new MessstelleReceiver( null, null, null, @@ -31,13 +31,12 @@ void updateMessquerschnitteOfMessstelleWithoutCreate() { }); final List expected = new MessstelleReceiverMapperImpl().createMessquerschnitte(messquerschnitteDto); - expected.forEach(expectedMessquerschnitt -> { - messquerschnitte.forEach(messquerschnitt1 -> { - if (expectedMessquerschnitt.getMqId().equalsIgnoreCase(messquerschnitt1.getMqId())) { - expectedMessquerschnitt.setStandort(messquerschnitt1.getStandort()); - } - }); - }); + expected.forEach(expectedMessquerschnitt -> messquerschnitte.forEach(messquerschnitt1 -> { + if (expectedMessquerschnitt.getMqId().equalsIgnoreCase(messquerschnitt1.getMqId())) { + expectedMessquerschnitt.setStandort(messquerschnitt1.getStandort()); + expectedMessquerschnitt.setPunkt(messquerschnitt1.getPunkt()); + } + })); final List result = messstelleReceiver.updateMessquerschnitteOfMessstelle(messquerschnitte, messquerschnitteDto); Assertions.assertThat(result) @@ -64,13 +63,12 @@ void updateMessquerschnitteOfMessstelleWithCreate() { messquerschnitteDto.add(addNewMessquerschnitt); final List expected = new MessstelleReceiverMapperImpl().createMessquerschnitte(messquerschnitteDto); - expected.forEach(expectedMessquerschnitt -> { - messquerschnitte.forEach(messquerschnitt1 -> { - if (expectedMessquerschnitt.getMqId().equalsIgnoreCase(messquerschnitt1.getMqId())) { - expectedMessquerschnitt.setStandort(messquerschnitt1.getStandort()); - } - }); - }); + expected.forEach(expectedMessquerschnitt -> messquerschnitte.forEach(messquerschnitt1 -> { + if (expectedMessquerschnitt.getMqId().equalsIgnoreCase(messquerschnitt1.getMqId())) { + expectedMessquerschnitt.setStandort(messquerschnitt1.getStandort()); + expectedMessquerschnitt.setPunkt(messquerschnitt1.getPunkt()); + } + })); final List result = messstelleReceiver.updateMessquerschnitteOfMessstelle(messquerschnitte, messquerschnitteDto); Assertions.assertThat(result)