Skip to content

Fix bug where crash tests weren't being run.#511

Merged
sangjinhan merged 1 commit into
masterfrom
actually_run_crash_tests
Jun 12, 2017
Merged

Fix bug where crash tests weren't being run.#511
sangjinhan merged 1 commit into
masterfrom
actually_run_crash_tests

Conversation

@justinemarie

Copy link
Copy Markdown
Contributor

Okay this is the actual commit.

@justinemarie
justinemarie requested a review from melvinw June 9, 2017 20:15
@codecov

codecov Bot commented Jun 9, 2017

Copy link
Copy Markdown

Codecov Report

Merging #511 into master will decrease coverage by <1%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #511    +/-   ##
=======================================
- Coverage      65%     64%   -<1%     
=======================================
  Files         198     198            
  Lines       12062   12062            
=======================================
- Hits         7804    7760    -44     
- Misses       4258    4302    +44
Impacted Files Coverage Δ
core/modules/port_inc.h 0% <0%> (-100%) ⬇️
core/modules/port_inc.cc 5% <0%> (-68%) ⬇️
core/traffic_class.cc 91% <0%> (ø) ⬆️
core/drivers/unix_socket.cc 86% <0%> (+1%) ⬆️

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 39d1229...a78a0e9. Read the comment docs.

@sangjinhan
sangjinhan merged commit 47ee7c9 into master Jun 12, 2017
@sangjinhan

Copy link
Copy Markdown
Member

x3rmp1hjoomt4zbblyoxa0xuo1_500

@sangjinhan
sangjinhan deleted the actually_run_crash_tests branch June 14, 2017 22:58
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.

2 participants