Import schon vorhandener Testgruppen lässt sich nicht abbrechen

[schumaki]

Beschreibung

Wenn ich im Reiter Testergebnisse einen Test Center Import machen will, werden mir sowohl schon vorhandene als auch noch nicht vorhandene Testgruppen angezeigt.

Klicke ich auf "Alle auswählen" und "Importieren" wird mir eine Warnmeldung angezeigt und gefragt, ob ich die schon vorhandenen Testgruppen überschreiben will. Soweit so gut.

Das Problem ist: Klicke ich auf "Abbrechen" werden trotzdem alle Testgruppen importiert (und ggf. die Daten überschrieben?)

Erwartetes Verhalten

• neben "Abbrechen" und "Überschreiben" sollte eine dritte Option angeboten werden, in der nur neue Fälle importiert werden
• "Abbrechen" sollte im Auswahldialog bleiben und nicht mit dem Import beginnen
• neben "Alle auswählen" sollte es eine Option geben, alle nicht vollständig importieren Testgruppen neu zu importieren.
• Testgruppen sollten nach Status gefiltert/sortiert werden können.

Reproduktionsschritte

s. Beschreibung

Beispiele

• V3-Ma_Pilo 26: KKS Testbereich auf dem Produktivserver

---

Versionen

• Kodierboxversion 1.16.2

Weitere Informationen

Abgesehen von den oben beschriebenen Problemen ist mir nicht ganz klar, was passiert, wenn innerhalb einer Testgruppe neue Fälle (z. B. durch eine Nachtestung einen Tag später o. Ä.) hinzugekommen sind. Wie würden diese angezeigt werden? Wie ließen sich die Daten für diese Fälle importieren (ohne den Rest der Testgruppe zu überschreiben)?

[jurei733]

Danke, das sollten wir m. E. in mehrere Punkte trennen:

1. Abbrechen darf keinen Import starten. Aktuell wirkt der Pfad eher wie „ohne Überschreiben fortfahren“. Das sollte ein eigener Button sein. Abbrechen sollte im Auswahldialog bleiben und gar keinen Import auslösen.

2. „Überschreiben“ sollte präziser benannt werden. Beim Testcenter-Import betrifft die Nachfrage aktuell vor allem vorhandene Logs. Responses werden im Testcenter-Import derzeit nicht über denselben Importmodus-Dialog gesteuert wie beim Datei-Upload, sondern backendseitig mit Skip-Semantik verarbeitet.

Zur Frage mit Nachtestungen/neuen Fällen innerhalb einer bestehenden Testgruppe:

Aktuell würde diese Testgruppe weiterhin nur als „Vorhanden“ angezeigt, weil die Anzeige auf Gruppenebene prüft, ob der groupName bereits in der Datenbank existiert. Einen Diff auf Fall-/Personenebene gibt es in der Auswahl derzeit nicht. Neue Fälle innerhalb derselben Testgruppe wären also nicht explizit als „neu“ sichtbar.

Technisch lassen sich solche Fälle grundsätzlich importieren, indem man die bestehende Testgruppe erneut auswählt: Neue Personen/Booklets/Units können ergänzt werden, ohne vorhandene Responses zu ersetzen. Für Logs gilt aktuell: Wenn man nicht überschreibt, werden vorhandene Booklet-/Unit-Logs übersprungen; Logs für neue Fälle funktionieren sinnvollerweise erst, wenn die zugehörigen Responses/Booklets vorhanden sind.

Ich denke deshalb, es wäre sinnvoll, die Importmodi, die es beim Datei-Upload von Testergebnissen bereits gibt (skip, merge, replace), zumindest für Responses auch beim Testcenter-Import verfügbar zu machen. Das würde den Nachtestungsfall transparenter machen:

• skip: vorhandene Units/Responses nicht verändern
• merge: neue Antwortwerte ergänzen, vorhandene nicht überschreiben
• replace: vorhandene Antwortwerte gezielt ersetzen

Für Logs gibt es beim manuellen Upload zwar ebenfalls den skip/merge/replace-Dialog, praktisch scheint das aktuell aber auf „vorhandene Logs überspringen“ vs. „vorhandene Logs überschreiben“ abgebildet zu werden. Damit verhalten sich merge und replace für Logs vermutlich gleich. Das wäre m. E. ein eigener Bug bzw. zumindest eine irreführende UI: Entweder sollte die UI für Logs nur die tatsächlich unterstützten Optionen anzeigen, oder die Log-Importlogik müsste eine echte merge-Semantik bekommen.

Zusätzlich wäre für die UI ein weiterer Status hilfreich, z. B. „Vorhanden, im Testcenter erweitert“ oder „unvollständig importiert“, basierend auf einem Vergleich zwischen Testcenter-Zahlen und lokalen Zahlen pro Gruppe. Darauf könnte dann auch die gewünschte Aktion „alle nicht vollständig importierten Testgruppen auswählen“ aufbauen.

[schumaki]

Ich denke deshalb, es wäre sinnvoll, die Importmodi, die es beim Datei-Upload von Testergebnissen bereits gibt (skip, merge, replace), zumindest für Responses auch beim Testcenter-Import verfügbar zu machen.

Das klingt sinnvoll. Bitte so umsetzen.

[jurei733]

Umgesetzt und gemerged. Kurzüberblick:

• Abbrechen im Testcenter-Import startet keinen Import mehr, sondern bricht den Start sauber ab.
• Für Antworten aus dem Testcenter gibt es jetzt die Importmodi skip, merge und replace. Damit können neue Antwortwerte/Fälle ergänzt werden, ohne bestehende Antworten zu überschreiben.
• Der Log-Konflikt-Dialog trennt jetzt zwischen Überschreiben, Ohne Überschreiben fortfahren und Abbrechen.
• Die weitergehenden UI-Punkte zu unvollständig importierten Gruppen, Auswahl aller unvollständigen Gruppen sowie Statusfilter/-sortierung sind als Folge-Issue Testcenter-Import: unvollständig importierte Testgruppen erkennen und auswählbar machen #806 ausgelagert.