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

fix: filter ran items using selected items, close #451 #549

Merged
merged 1 commit into from
Aug 29, 2021
Merged

Conversation

noahnu
Copy link
Collaborator

@noahnu noahnu commented Aug 29, 2021

Description

Only set the ran items if there is a corresponding selected item.

Related Issues

Checklist

  • This PR has sufficient documentation.
  • This PR has sufficient test coverage.
  • This PR title satisfies semantic convention.

Additional Comments

Need to give this more thought on how to test it to prevent a regression. Open to ideas.

@codecov
Copy link

codecov bot commented Aug 29, 2021

Codecov Report

Merging #549 (4d2b62c) into master (0b4e544) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #549   +/-   ##
=======================================
  Coverage   99.91%   99.91%           
=======================================
  Files          19       19           
  Lines        1129     1130    +1     
=======================================
+ Hits         1128     1129    +1     
  Misses          1        1           

@noahnu noahnu merged commit 7374862 into master Aug 29, 2021
@noahnu noahnu deleted the bugfix/451 branch August 29, 2021 15:33
tophat-opensource-bot pushed a commit that referenced this pull request Aug 29, 2021
## [1.4.5](v1.4.4...v1.4.5) (2021-08-29)

### Bug Fixes

* filter ran items using selected items, close [#451](#451) ([#549](#549)) ([7374862](7374862))
@tophat-opensource-bot
Copy link
Collaborator

🎉 This PR is included in version 1.4.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

KeyError during report generation in syrupy when pytest is also doing doctests
2 participants