Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests #4

Open
tobinski opened this issue Mar 4, 2018 · 3 comments · Fixed by #5
Open

Tests #4

tobinski opened this issue Mar 4, 2018 · 3 comments · Fixed by #5
Labels
question Further information is requested

Comments

@tobinski
Copy link
Contributor

tobinski commented Mar 4, 2018

Aktuell schlagen die Tests bei mir fehlt. Dies sollten wir bereits jetzt anpassen, damit wir kontinuierlich testen können.

ERROR in src/app/image-viewer/osd-viewport.directive.spec.ts(5,23): error TS2554: Expected 1 arguments, but got 0.

@tobinski tobinski mentioned this issue Mar 4, 2018
Merged
@sschuepbach
Copy link
Member

@tobinski, vielen Dank für das Anpassen der specs. Bin im Übrigen ganz deiner Meinung, dass wir eine solide Testsuite haben sollten. Ich schlage vor, dass wir uns primär auf Unit Tests konzentrieren, also Jasmine + Karma verwenden, und E2E Tests wenn überhaupt eher sparsam einsetzen. Was denkst du?

@sschuepbach
Copy link
Member

Öffne das Issue noch einmal, da wir uns hier vielleicht noch auf Details des Testing verständigen müssen.

@tobinski
Copy link
Contributor Author

tobinski commented Mar 4, 2018

Ich teste Angular Apps relativ sparsam, da es sehr viel Arbeit bedeuten und dann bei jedem kleinsten Update failen. Ich denke Jasmine + Karma sollten reichen und wir sollten zumindest den Core unserer App absichern.

@sschuepbach sschuepbach added the question Further information is requested label Mar 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants