Skip to content
This repository has been archived by the owner on Mar 7, 2023. It is now read-only.

Add unit tests for all DB functions in bridge.go #68

Draft
wants to merge 15 commits into
base: main
Choose a base branch
from
Draft

Conversation

pinpox
Copy link
Member

@pinpox pinpox commented Feb 14, 2021

No description provided.

@ghost
Copy link

ghost commented Feb 14, 2021

Congratulations 🎉. DeepCode analyzed your code in 0.812 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

@codecov-io
Copy link

codecov-io commented Feb 14, 2021

Codecov Report

Merging #68 (da22dc6) into main (b02bb7a) will increase coverage by 14.23%.
The diff coverage is 53.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##            main      #68       +/-   ##
==========================================
+ Coverage   9.00%   23.24%   +14.23%     
==========================================
  Files         12       12               
  Lines        722      796       +74     
==========================================
+ Hits          65      185      +120     
+ Misses       646      581       -65     
- Partials      11       30       +19     
Impacted Files Coverage Δ
call.go 0.00% <0.00%> (ø)
handler_person.go 0.00% <0.00%> (ø)
handler_upload.go 0.00% <0.00%> (ø)
helper.go 44.44% <0.00%> (+11.11%) ⬆️
person.go 0.00% <0.00%> (ø)
sender.go 60.00% <25.00%> (+50.19%) ⬆️
main.go 22.00% <60.00%> (-6.85%) ⬇️
bridge.go 44.29% <68.00%> (+28.57%) ⬆️

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 b02bb7a...da22dc6. Read the comment docs.

@pinpox pinpox changed the title add test for full check Add unit test for DB functions in bridge.go Feb 14, 2021
@pinpox pinpox changed the title Add unit test for DB functions in bridge.go Add unit tests for all DB functions in bridge.go Feb 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants