npm run test - Fehler in Travis und lokal

guenterh commented 5 years ago

@liowalter @edelm fyi

npm run test "failed" bei drei Tests https://travis-ci.org/swissbib/vufind/jobs/452410082#L1811
dies passiert auch lokal
der Fehler tritt bei https://github.com/swissbib/vufind/releases/tag/4.1.0-core4.1.0 nicht auf (habe ich lokal getestet)
gemacht wurde
- Austausch des package.json mit demjenigen von 4.1.0-core4.1.0 (Fehler tritt weiterhin auf)
- Update des ts-jest uit-test frameworks auf die letzte stabile version 23.10 (aktuell 21...) Es treten weiterhin Fehler auf, allerdings nicht die gleich wie in unseren branches. Iregndwie scheinen die generic types von Promise-Typen nicht kompatibel zu sein. Einen solchen Fehler zeigt mir auch die IDE in der statischen Codeanalyse an. Allerdings überblicke ich im Moment nicht, wo das Problem liegt.

Mein aktuelles Zeitbudget für diesen task ist bereits überzogen. Ich stoppe deswegen an dieser Stelle

Was man aus meiner Perspektive machen sollte:

[ ] entferne "temporär" https://github.com/swissbib/vufind/blob/development/package.json#L28 aus package.json. Damit sollte der travis Fehler nicht mehr auftreten und wir haben erstmal wieder fehlerfreie deployments
[ ] wie kann man solche Java - Script Tests debuggen (IDE)? Das würde helfen zu erkennen, was die Ursache für die failures sind. Ich kann mir eigentlich nur vorstellen, dass die Anpassungen, aufgrund der Änderungen von VF5 bei uns im code, dazu führen, dass die Tests nicht mehr durchlaufen. Allerdings habe ich ohne weitere Einarbeitung keinen Überblick mehr darüber, was gemacht wurde
[ ] wir sollten / müssen uns ein besseres Verständnis der neuen Java-Script Umgebungen erarbeiten. Aktuell ist "mein" Schwerpunkt im Bereich Java (Scala) und Python. Ich werde das Thema jedoch nicht aus den Augen verlieren. Vieles (nicht nur UI auch auch im linked-Bereich) läuft auf diese Implementierungsform hin

liowalter commented 5 years ago

Ok, thanks for the hints ! For now, I disabled the npm run test on travis : https://github.com/swissbib/vufind/commit/0ab91de937ec808d67f02f6a47f4bc2f434de586 Let's keep that for the next sprint

liowalter commented 5 years ago

swissbib / vufind