Skip to content

feature/user-model#4

Open
leosocy wants to merge 14 commits intomasterfrom
feature-app-models-user
Open

feature/user-model#4
leosocy wants to merge 14 commits intomasterfrom
feature-app-models-user

Conversation

@leosocy
Copy link
Copy Markdown
Collaborator

@leosocy leosocy commented Jul 26, 2018

No description provided.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 26, 2018

Codecov Report

Merging #4 into master will increase coverage by 5.32%.
The diff coverage is 98.59%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   94.04%   99.36%   +5.32%     
==========================================
  Files          12       12              
  Lines         218      313      +95     
==========================================
+ Hits          205      311     +106     
+ Misses         13        2      -11
Impacted Files Coverage Δ
app/db/smart_session.h 100% <ø> (ø) ⬆️
app/models/user.h 100% <100%> (+50%) ⬆️
tests/test_models/user_unittest.cpp 100% <100%> (ø) ⬆️
app/db/smart_session.cpp 100% <100%> (ø) ⬆️
app/db/session_pool.h 100% <100%> (ø) ⬆️
app/models/user.cpp 100% <100%> (+52.94%) ⬆️
app/db/session_pool.cpp 98.36% <93.75%> (+1.75%) ⬆️
tests/test_db/session_pool_unittest.cpp 98.55% <97.06%> (-1.45%) ⬇️

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