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

Bring code coverage to 100% #12

Open
1 of 4 tasks
toh82 opened this issue Jan 23, 2016 · 6 comments
Open
1 of 4 tasks

Bring code coverage to 100% #12

toh82 opened this issue Jan 23, 2016 · 6 comments

Comments

@toh82
Copy link
Member

toh82 commented Jan 23, 2016

current status: Coverage Status

npm run test:coverage

...
=============================== Coverage summary ===============================
Statements   : 73.33% ( 33/45 )
Branches     : 47.62% ( 10/21 )
Functions    : 85.71% ( 6/7 )
Lines        : 74.42% ( 32/43 )
================================================================================

  • claim this issue (@ddmck)
  • start a pull request
  • write tests until 100% coverage is reached
  • review & merge

Ping us in the Hoodie Chat or on Twitter if you have ​​_any_​​ questions

@hobbyexplorer
Copy link

Hi @toh82,
I would like to work on this.

@gr2m
Copy link
Member

gr2m commented Feb 21, 2016

All yours 👍

@gr2m gr2m added in progress and removed ready labels Mar 18, 2016
@ddmck
Copy link

ddmck commented May 28, 2016

Hi @gr2m, I'd like to claim this.

@ddmck ddmck self-assigned this May 28, 2016
@gr2m
Copy link
Member

gr2m commented May 28, 2016

all yours :) Note the code is a bit out of scope so might not be super up to date, but we will need 100% coverage anyway earlier or later, so go! :)

@inator
Copy link
Contributor

inator commented Jan 16, 2017

@gr2m - Still applicable post refactor?

@gr2m
Copy link
Member

gr2m commented Jan 16, 2017

yes but let’s put this on hold until we have a first version of hoodie-task with the latest client & server running.

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

No branches or pull requests

5 participants