[Code Qualität, Backend] Findings Testabdeckung erhöhen / Package mapper

[clean-coder]

No description provided.

[clean-coder]

Bei der Umsetzung von #914 sind ein paar Unschönheiten (Findings) aufgetaucht, die man kurz anschauen und ev. anpassen sollte. Alles Findings sind in den Tests als TODOs markiert.

writeable
Viele Objekte, die via Mapper von obj -> dto und umgekehrt gemappt werden können, haben ein Property writable.
Die Transformation obj -> dto passt. Bei der Transformation von dto -> obj wird writable immer false. Ist das so gewollt?

CheckInMapper (bei Metric und Ordinal)
CheckIn checkIn = checkInMetricMapper.toCheckInMetric(checkInMetricDto);
CreatedOn und ModifiedOn werden nicht abgefüllt (sind null). Die Werte sind gesetzt im Dto aber nicht im Obj.

API Unschönheit CheckInMapper
In CheckInOrdinal haben wir ein Property Zone zone (mit Getters und Setters)
In CheckInOrdinalDto haben wir ein Property Zone value
Dies ist unschön. Vorschlag: in CheckInOrdinalDto Zone von value auf zone umbenennen

API Frage
KeyResultMapper und CheckInMapper sind ähnlich designed. Beim CheckInMapper kann man auf den Typ zugreifen, beim KeyResultMapper nicht. Wollte das in den Test verwenden :-(

KeyResultMapper (bei Metric + Ordinal)
KeyResult keyResult = keyResultMetricMapper.toKeyResultMetric(keyResultMetricDto);
Modified wird nicht abgefüllt (vom dto -> obj)

API Unschönheit KeyResultMapper
Die gleiche API Unschönheit wie wie bei CheckInMapper. Property heisst im Object Zone zone und im Dto Zone value

API Unschönheit KeyResultMapper
In allen anderen Mappern ist obj -> dto immer eine Methode mit Namen: toDto(....) Beim KeyResultMapper heisst es aber: toKeyResultMetricDto() bzw. toKeyResultOrdinalDto(). Vorschlag: umbennenen in toDto()