Skip to content
New issue

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

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

Already on GitHub? # to your account

[elasticsearch] Add tests #261

Merged
merged 1 commit into from
Dec 27, 2021
Merged

[elasticsearch] Add tests #261

merged 1 commit into from
Dec 27, 2021

Conversation

ricoberger
Copy link
Member

Add tests for Elasticsearch plugin to improve code coverage.

Add tests for Elasticsearch plugin to improve code coverage.
@codecov-commenter
Copy link

Codecov Report

Merging #261 (c306ed5) into main (4c8d592) will increase coverage by 1.84%.
The diff coverage is 93.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #261      +/-   ##
==========================================
+ Coverage   14.36%   16.20%   +1.84%     
==========================================
  Files          89       89              
  Lines        7534     7533       -1     
==========================================
+ Hits         1082     1221     +139     
+ Misses       6435     6295     -140     
  Partials       17       17              
Impacted Files Coverage Δ
cmd/kobs/plugins/plugins.go 0.00% <0.00%> (ø)
plugins/elasticsearch/elasticsearch.go 100.00% <100.00%> (+100.00%) ⬆️
plugins/elasticsearch/pkg/instance/instance.go 100.00% <100.00%> (+100.00%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4c8d592...c306ed5. Read the comment docs.

@ricoberger ricoberger merged commit 53665a6 into main Dec 27, 2021
@ricoberger ricoberger deleted the elasticsearch-add-tests branch December 27, 2021 20:35
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants