Aufgefallen bei der Prüfung von #72.
Befund
Die Schreibhoheitstabelle in 13-15_Parallelbetrieb.md beansprucht Vollständigkeit ausdrücklich nur für Assets/_Project/Scripts/. Für tools/Nova.SimRunner.Tests/ gibt es keine Zeile — und beide Stränge schreiben dort faktisch:
Damit entsteht ein Graubereich bei bestehenden Testdateien: „eigene neue Testdatei" ist geregelt, „vom anderen Strang angelegte, jetzt geänderte Datei" nicht.
Warum das jetzt auffällt
#72 ändert zwei bestehende Testdateien, die der Beitragende selbst angelegt hatte (AiProfileTests.cs aus #36, SkirmishAiTests.cs aus #38), die zwischenzeitlich aber von Maintainer-Seite angefasst wurden. Keine Überschreitung — aber die Regel sagt dazu nichts.
Dazu ein zweiter Fall aus demselben PR: der neue Test MatchRunnerPassesTheSameFourNumbersTheShippedProfileCarries liest Gameplay/Match/MatchRunner.cs per File.ReadAllText und prüft vier Literale per Regex. Lesen ist kein Schreiben — aber der Netzstrang bekommt damit einen roten Test aus fremdem Strang, wenn er seine eigene Datei ändert. Auch dafür hat die Tabelle keine Kategorie.
Vorschlag
Zwei Zeilen in die Tabelle:
tools/Nova.SimRunner.Tests/ — geteilt, jeder Strang besitzt die Dateien zu seinem Gegenstand; Änderungen an fremden Testdateien werden angesagt- eine dritte Vertragsflächen-Zeile für Tests, die fremden Quelltext lesen — erlaubt, aber ansagepflichtig, weil sie den anderen Strang rot machen können
Dieselbe Art Lücke, die Fassung 1.2.0 schon für tools/Nova.AiLab/ und Presentation/UI/DebugHud.cs schließen musste — beide fehlten dort aus demselben Grund: die Tabelle war nur für einen Teilbaum vollständig.
Aufgefallen bei der Prüfung von #72.
Befund
Die Schreibhoheitstabelle in
13-15_Parallelbetrieb.mdbeansprucht Vollständigkeit ausdrücklich nur fürAssets/_Project/Scripts/. Fürtools/Nova.SimRunner.Tests/gibt es keine Zeile — und beide Stränge schreiben dort faktisch:tools/Nova.SimRunner.Tests/" mit.AiProfileTests.cs, ursprünglich vom Einheitenstrang angelegt).Damit entsteht ein Graubereich bei bestehenden Testdateien: „eigene neue Testdatei" ist geregelt, „vom anderen Strang angelegte, jetzt geänderte Datei" nicht.
Warum das jetzt auffällt
#72 ändert zwei bestehende Testdateien, die der Beitragende selbst angelegt hatte (
AiProfileTests.csaus #36,SkirmishAiTests.csaus #38), die zwischenzeitlich aber von Maintainer-Seite angefasst wurden. Keine Überschreitung — aber die Regel sagt dazu nichts.Dazu ein zweiter Fall aus demselben PR: der neue Test
MatchRunnerPassesTheSameFourNumbersTheShippedProfileCarriesliestGameplay/Match/MatchRunner.csperFile.ReadAllTextund prüft vier Literale per Regex. Lesen ist kein Schreiben — aber der Netzstrang bekommt damit einen roten Test aus fremdem Strang, wenn er seine eigene Datei ändert. Auch dafür hat die Tabelle keine Kategorie.Vorschlag
Zwei Zeilen in die Tabelle:
tools/Nova.SimRunner.Tests/— geteilt, jeder Strang besitzt die Dateien zu seinem Gegenstand; Änderungen an fremden Testdateien werden angesagtDieselbe Art Lücke, die Fassung 1.2.0 schon für
tools/Nova.AiLab/undPresentation/UI/DebugHud.csschließen musste — beide fehlten dort aus demselben Grund: die Tabelle war nur für einen Teilbaum vollständig.