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 test for email_case_deletion(). Refs #878 #969

Merged
merged 1 commit into from
Jun 18, 2019
Merged

Add test for email_case_deletion(). Refs #878 #969

merged 1 commit into from
Jun 18, 2019

Conversation

atodorov
Copy link
Member

Cherry-pick only the tests from #878 since the rest has already been introduced into master.

@codecov-io
Copy link

codecov-io commented Jun 18, 2019

Codecov Report

Merging #969 into master will increase coverage by 0.18%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #969      +/-   ##
==========================================
+ Coverage   73.15%   73.34%   +0.18%     
==========================================
  Files         112      112              
  Lines        4764     4764              
  Branches      609      609              
==========================================
+ Hits         3485     3494       +9     
+ Misses       1057     1044      -13     
- Partials      222      226       +4
Impacted Files Coverage Δ
tcms/core/history.py 90.69% <0%> (ø) ⬆️
tcms/signals.py 76.19% <0%> (+6.34%) ⬆️
tcms/testcases/helpers/email.py 82.85% <0%> (+14.28%) ⬆️

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 f810bc5...a3b63d1. Read the comment docs.

@atodorov atodorov merged commit b4273f7 into master Jun 18, 2019
@atodorov atodorov deleted the pr878 branch June 18, 2019 13:10
@coveralls
Copy link

Pull Request Test Coverage Report for Build 3316

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.2%) to 74.038%

Totals Coverage Status
Change from base Build 3313: 0.2%
Covered Lines: 3719
Relevant Lines: 4764

💛 - Coveralls

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.

4 participants