Skip to content

Conversation

@RobertoRoos
Copy link
Contributor

@RobertoRoos RobertoRoos commented Jan 14, 2026

@coveralls
Copy link

coveralls commented Jan 14, 2026

Pull Request Test Coverage Report for Build 21254060182

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall first build on bugfix/del-notification at 93.389%

Totals Coverage Status
Change from base Build 20993886363: 93.4%
Covered Lines: 10708
Relevant Lines: 11466

💛 - Coveralls

@stlehmann
Copy link
Owner

Not sure about it but generally I like the idea of getting an exception if something unintended (e.g. KeyError) happens. It might be just to log a Warning. Wouldn't want to let it slip unnoticed.

@RobertoRoos RobertoRoos changed the title Removed possible exception when deleting from callback_store Removed pytest warning from deleting from callback_store Jan 22, 2026
@stlehmann stlehmann merged commit a13d02d into stlehmann:master Jan 22, 2026
19 of 22 checks passed
@RobertoRoos RobertoRoos deleted the bugfix/del-notification branch January 22, 2026 20:00
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.

pytest.PytestUnraisableExceptionWarning for a KeyError in tests

3 participants