Skip to content

Simplify some assertions #1695

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Mar 2, 2025
Merged

Simplify some assertions #1695

merged 1 commit into from
Mar 2, 2025

Conversation

bkoelman
Copy link
Member

@bkoelman bkoelman commented Mar 2, 2025

Affects tests only.

QUALITY CHECKLIST

Copy link

codecov bot commented Mar 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.64%. Comparing base (32b2624) to head (19625a3).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1695   +/-   ##
=======================================
  Coverage   90.64%   90.64%           
=======================================
  Files         355      355           
  Lines       11566    11566           
  Branches     1908     1908           
=======================================
  Hits        10484    10484           
  Misses        706      706           
  Partials      376      376           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bkoelman bkoelman force-pushed the simplify-assertions branch from f97efbc to 19625a3 Compare March 2, 2025 13:07
@bkoelman bkoelman marked this pull request as ready for review March 2, 2025 13:28
@bkoelman bkoelman merged commit 73e9e33 into master Mar 2, 2025
16 checks passed
@bkoelman bkoelman deleted the simplify-assertions branch March 2, 2025 13:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant