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

Improve test coverage #4

Open
1 of 2 tasks
teekaay opened this issue Oct 16, 2017 · 0 comments
Open
1 of 2 tasks

Improve test coverage #4

teekaay opened this issue Oct 16, 2017 · 0 comments

Comments

@teekaay
Copy link
Collaborator

teekaay commented Oct 16, 2017

Scope

I'm submitting a ...

  • bug report
  • feature request

Expected Behavior

The code should be tested in more depth. Also, no frontend tests are available.

Actual Behavior

Parts of the code (e.g. endpoints, frontend etc) are not tested at all. Too much is mocked
away, which typically means tightly coupled code. Some parts (like the ones that first have to wait for
Promises to resolve on the toplevel (see this) cannot be tested easily/not at all.

Steps to Reproduce the Problem

  1. npm run test
  2. Use your favourite test coverage tool (e.g. istanbul)

Specifications

  • Version: All
  • Platform: All
  • Subsystem: All

Additional Information

  • We could use a coverage tool (istanbul?)
  • Refactor after writing a test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant