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

Add sqlite details to unit testing guide #7603

Merged
merged 1 commit into from Sep 4, 2020

Conversation

jasonprado
Copy link
Contributor

Description of what you did:

Added sqlite3 to the list of modules to install for testing, as the guide assumes it is there. In the configuration I also dropped the pool size to 1; I was consistently hitting sqlite3 database lock errors until I did this.

@codecov
Copy link

codecov bot commented Aug 26, 2020

Codecov Report

Merging #7603 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #7603   +/-   ##
=======================================
  Coverage   26.38%   26.38%           
=======================================
  Files        1131     1131           
  Lines       15442    15442           
  Branches     2439     2439           
=======================================
  Hits         4074     4074           
  Misses       9559     9559           
  Partials     1809     1809           
Flag Coverage Δ
#front 18.25% <ø> (ø)
#unit 53.85% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


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 e6589f6...091d60a. Read the comment docs.

@alexandrebodin
Copy link
Member

Thanks for the update !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue: enhancement Issue suggesting an enhancement to an existing feature source: docs Documentation changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants