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

Add initial testcase #4

Merged
merged 2 commits into from
Sep 21, 2022
Merged

Add initial testcase #4

merged 2 commits into from
Sep 21, 2022

Conversation

zchee
Copy link
Owner

@zchee zchee commented Sep 21, 2022

No description provided.

Signed-off-by: Koichi Shiraishi <zchee.io@gmail.com>
@zchee zchee changed the title pkg/detector: add initial testcase Add initial testcase Sep 21, 2022
@codecov
Copy link

codecov bot commented Sep 21, 2022

Codecov Report

Merging #4 (2ec68b9) into main (7bff2d4) will increase coverage by 25.5%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##            main      #4      +/-   ##
========================================
+ Coverage   22.5%   48.1%   +25.5%     
========================================
  Files         14      14              
  Lines        478     476       -2     
========================================
+ Hits         108     229     +121     
+ Misses       370     247     -123     
Impacted Files Coverage Δ
pkg/detector/detector.go 76.1% <ø> (+76.1%) ⬆️
pkg/monitoredresource/monitoredresource.go 75.7% <0.0%> (+75.7%) ⬆️
pkg/detector/appengine.go 100.0% <0.0%> (+100.0%) ⬆️
pkg/detector/container.go 100.0% <0.0%> (+100.0%) ⬆️
pkg/detector/serverless.go 100.0% <0.0%> (+100.0%) ⬆️

Signed-off-by: Koichi Shiraishi <zchee.io@gmail.com>
@zchee zchee marked this pull request as ready for review September 21, 2022 22:35
@zchee zchee merged commit 1793db4 into main Sep 21, 2022
@zchee zchee deleted the testcase branch September 21, 2022 22: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.

1 participant