Skip to content

test: init tests in internal/admin#56

Closed
g1eny0ung wants to merge 1 commit intohstreamdb:mainfrom
g1eny0ung:test/internal-admin
Closed

test: init tests in internal/admin#56
g1eny0ung wants to merge 1 commit intohstreamdb:mainfrom
g1eny0ung:test/internal-admin

Conversation

@g1eny0ung
Copy link
Contributor

Starting to supplement tests in the internal/admin package. Still in progress.

@codecov
Copy link

codecov bot commented Oct 13, 2023

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (e885e2a) 84.17% compared to head (c3fb52a) 84.27%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #56      +/-   ##
==========================================
+ Coverage   84.17%   84.27%   +0.09%     
==========================================
  Files          31       32       +1     
  Lines        2054     2060       +6     
==========================================
+ Hits         1729     1736       +7     
  Misses        231      231              
+ Partials       94       93       -1     
Files Coverage Δ
internal/admin/admin_client.go 33.33% <ø> (ø)
internal/admin/remote_cmd.go 71.95% <ø> (ø)
internal/admin/client.go 62.50% <50.00%> (ø)
internal/admin/remote_cmd_mock.go 33.33% <33.33%> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@g1eny0ung
Copy link
Contributor Author

Close due to #79 having been refactored internal/admin, and some of the tests have been added in packages.

@g1eny0ung g1eny0ung closed this Dec 12, 2023
@g1eny0ung g1eny0ung deleted the test/internal-admin branch December 12, 2023 09:11
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

Successfully merging this pull request may close these issues.

1 participant