Fix: doppelte Mitarbeits-Sitzung durch Unterrichtsmodus-Instanz

EnsureTodaySession() prüfte nur den lokalen Instanz-Zustand, nicht das Repository. Da der Unterrichtsmodus eine eigene, per DI frisch aufgelöste SeatingPlanTabViewModel-Instanz nutzt (getrennt vom Sitzplan-Tab der Kursübersicht), legte eine spätere Bewertung im normalen Tab eine zweite, unverknüpfte Sitzung für denselben Tag an, statt die bereits über den Unterrichtsmodus verknüpfte zu erkennen. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
2026-08-24 11:43:19 +02:00
parent 07788dcfcb
commit a30aab0fa5
2 changed files with 68 additions and 1 deletions
@@ -543,6 +543,50 @@ public sealed class SeatingPlanViewModelTests
        Assert.Equal(session.Id, vm.SelectedSession?.Id);
    }

+    [Fact]
+    public async Task AssessStudent_SitzungBereitsUeberAndereInstanzVerknuepft_LegtKeineZweiteAn()
+    {
+        // Regressionstest: Der Unterrichtsmodus läuft auf einer eigenen, per DI frisch aufgelösten
+        // SeatingPlanTabViewModel-Instanz (TimetableView.ShowTeachingMode), getrennt von der
+        // Instanz im Sitzplan-Tab der Kursübersicht — beide teilen sich nur das Repository. Wurde
+        // (wie hier simuliert) die Sitzung bereits über den Unterrichtsmodus mit der Stunde
+        // verknüpft, bevor im echten Sitzplan-Tab bewertet wird, darf letzterer keine zweite,
+        // unverknüpfte "Sitzplan"-Sitzung für denselben Tag anlegen.
+        var groupId = Guid.NewGuid();
+        var student = new Student { FirstName = "Anna", LastName = "A" };
+        var plan = new SeatingPlan
+        {
+            GroupId = groupId, Name = "Standard", Rows = 1, Columns = 1,
+            Assignments = [new SeatAssignment { StudentId = student.Id }],
+        };
+        var sessions = new FakeSessions([]);
+        var memberships = new FakeMemberships([new GroupMembership { GroupId = groupId, StudentId = student.Id }]);
+
+        // Instanz A: Sitzplan-Tab der Kursübersicht, bereits initialisiert (kein SelectedSession).
+        var tabVm = new SeatingPlanTabViewModel(new FakeSeatingPlans([plan]), new FakeStudents([student]),
+            memberships, sessions, new FakeEntries(), new FakeAspects());
+        tabVm.Initialize(groupId, isReadOnly: false);
+        tabVm.OnAssessStudent = _ => Task.CompletedTask;
+
+        // Instanz B: eigene, frische Instanz wie im Unterrichtsmodus — verknüpft die Sitzung mit
+        // der Stunde, ohne dass Instanz A davon erfährt.
+        var lesson = new Lesson { GroupId = groupId, Date = DateOnly.FromDateTime(DateTime.Today), Topic = "Redox" };
+        var teachingModeVm = new SeatingPlanTabViewModel(new FakeSeatingPlans([plan]), new FakeStudents([student]),
+            memberships, sessions, new FakeEntries(), new FakeAspects());
+        teachingModeVm.Initialize(groupId, isReadOnly: false);
+        teachingModeVm.SelectOrCreateSessionForLesson(lesson);
+
+        var linked = Assert.Single(sessions.GetByGroup(groupId));
+        Assert.Equal(lesson.Id, linked.LessonId);
+
+        await tabVm.AssessStudent(tabVm.Seats.Single());
+
+        var session = Assert.Single(sessions.GetByGroup(groupId));
+        Assert.Equal(linked.Id, session.Id);
+        Assert.Equal(lesson.Id, session.LessonId);
+        Assert.Equal(session.Id, tabVm.SelectedSession?.Id);
+    }
+
    [Fact]
    public void ShuffleSeats_ErzeugtNeuenPlanMitVertauschtenPlaetzenUndBehaeltDenUrsprungsplan()
    {