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

Improve coverage via optimized seeds #4504

Merged
merged 1 commit into from
Jun 6, 2019
Merged

Conversation

npinto
Copy link
Contributor

@npinto npinto commented Jun 6, 2019

Improving coverage by adding seeds that have been optimized for it.


For Admin Use:

  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

Improving coverage by adding four seeds that have been optimized for it.
@npinto npinto changed the title WIP: Improve coverage via four optimized seeds WIP: Improve coverage via optimized seeds Jun 6, 2019
@codecov
Copy link

codecov bot commented Jun 6, 2019

Codecov Report

Merging #4504 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4504      +/-   ##
==========================================
- Coverage   53.18%   53.16%   -0.02%     
==========================================
  Files         259      259              
  Lines       16190    16190              
==========================================
- Hits         8610     8608       -2     
- Misses       6934     6936       +2     
  Partials      646      646

1 similar comment
@codecov
Copy link

codecov bot commented Jun 6, 2019

Codecov Report

Merging #4504 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4504      +/-   ##
==========================================
- Coverage   53.18%   53.16%   -0.02%     
==========================================
  Files         259      259              
  Lines       16190    16190              
==========================================
- Hits         8610     8608       -2     
- Misses       6934     6936       +2     
  Partials      646      646

@npinto
Copy link
Contributor Author

npinto commented Jun 6, 2019

Is the coverage report produced by codecov including coverage from the sim (e.g. test_sim_app_multi_seed) ?

@alexanderbez
Copy link
Contributor

@npinto no, I do not believe so. Is this ready for review?

@npinto
Copy link
Contributor Author

npinto commented Jun 6, 2019 via email

@alexanderbez
Copy link
Contributor

@npinto yes please share your results. With regards to code coverage, since it's ran via go test ..., maybe we can pass the --coverage flag?

@alexanderbez alexanderbez changed the title WIP: Improve coverage via optimized seeds Improve coverage via optimized seeds Jun 6, 2019
Copy link
Contributor

@alexanderbez alexanderbez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@alessio alessio merged commit 3862b74 into cosmos:master Jun 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants