Skip to content
This repository has been archived by the owner on Jan 25, 2021. It is now read-only.

npm run test - Fehler in Travis und lokal #662

Open
3 tasks
guenterh opened this issue Nov 12, 2018 · 2 comments
Open
3 tasks

npm run test - Fehler in Travis und lokal #662

guenterh opened this issue Nov 12, 2018 · 2 comments

Comments

@guenterh
Copy link
Member

@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
Copy link
Member

Ok, thanks for the hints ! For now, I disabled the npm run test on travis :
0ab91de
Let's keep that for the next sprint

@liowalter
Copy link
Member

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants