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

Increase test coverage to >90% #3

Closed
danielberkompas opened this issue Jan 2, 2018 · 2 comments
Closed

Increase test coverage to >90% #3

danielberkompas opened this issue Jan 2, 2018 · 2 comments

Comments

@danielberkompas
Copy link
Owner

No description provided.

@DanielZendejas
Copy link

Hi, first of all thanks for the great library!

I'd be happy to help with this. The command mix test --cover yields the following results:

COV    FILE                                        LINES RELEVANT   MISSED
 73.1% lib/elasticsearch.ex                          354       26        7
  0.0% lib/elasticsearch/api/api.ex                   20        0        0
 91.7% lib/elasticsearch/api/http.ex                  52       12        1
100.0% lib/elasticsearch/application.ex               20        3        0
 91.7% lib/elasticsearch/config.ex                   132       12        1
 87.5% lib/elasticsearch/exception.ex                 65        8        1
 50.0% lib/elasticsearch/executable.ex                43        8        4
 75.0% lib/elasticsearch/indexing/bulk.ex            132       24        6
 97.3% lib/elasticsearch/indexing/index.ex           285       37        1
100.0% lib/elasticsearch/storage/data_stream.ex       75        5        0
  0.0% lib/elasticsearch/storage/store.ex              8        0        0
 69.0% lib/mix/elasticsearch.build.ex                148       29        9
  0.0% lib/mix/elasticsearch.install.ex               71       20       20
100.0% test/support/post.ex                           17        4        0
100.0% test/support/store.ex                          21        3        0
[TOTAL]  73.8%

Has anybody already started with something? I'd like to avoid double efforts.

@danielberkompas
Copy link
Owner Author

Nobody is working on this right now, so if you would like to try your hand at it, feel free!

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

2 participants