[alex1702]

No description provided.

[xaverW]

läuft bei mir (http://5.45.104.194/akt/filme-akt.xz), könnte nach develop
Da hat @derreisende77 eine super Arbeit abgeliefert!

[alex1702]

@pidoubleyou Bitte nochmal die Test Logik hinter AddToFilmlist überprüfen/überdenken. Es sollte bei einem normalen Build nicht die Laufzeit beeinträchtigen.

[derreisende77]

Der ZDF crawler wird nicht beendet...AddToFilmList reagiert auf das terminieren. Also ist eher MediathekZdf das Problem.

[pidoubleyou]

@alex1702 die Tests für AddFilmToList sind aktuell auskommentiert worden, da sie noch nicht an die Änderungen in diesem Branch angepasst wurden. Werde ich mir mal anschauen, was ich umbauen muss, damit diese wieder funktionieren.

Im Develop benötigen alle Tests bei mir um die 15 Sekunden. Wenn diese Laufzeiten jetzt schon zu lange sind, dann müssen wir wohl darüber nachdenken, die Ausführung der Tests aus dem normalen Builds des Entwicklers herauszunehmen. Der Entwickler muss die Tests dann explizit ausführen und der Travis führt sie natürlich automatisch aus.

[derreisende77]

Also ein separater Test ist mir deutlich am liebsten.

[alex1702]

Ich bin auch dafür, dass die Tests eher seperat ausgeführt werden. Und wie gesagt soll Code für Tests nicht den Produktivcode beeinflussen.

[pidoubleyou]

@alex1702 grundsätzlich stimme ich dir zu, dass Testcode den Produktivcode nicht beeinflussen soll. Allerdings sollte Produktivcode so strukturiert sein, dass er gut testbar ist. Da dies bei MServer aktuell nur bedingt der Fall ist, wird wohl die ein oder andere Anpassung am Produktivcode nicht ausbleiben, wenn wir Test haben wollen (siehe Diskussion in #92)