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

download: provide description for an assertion error on attempts_allowed_or_not #1550

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

yarikoptic
Copy link
Member

Somehow, with some prior version of the code, but we did get this assertion triggered as shown in
#1549 (comment) although it remains unclear how that could have happened if type checking is correct since we must not end up there with None or even 0.

Unfortunately with a regular assert we just get an announcement of the fact and no details. After this change we hope to be able to see more details which might hint on the original issue.

Somehow, with some prior version of the code, but we did
get this assertion triggered as shown in
#1549 (comment)
although it remains unclear how that could have happened if type checking
is correct since we must not end up there with None or even 0.

Unfortunately with a regular assert we just get an announcement of the fact
and no details.  After this change we hope to be able to see more details
which might hint on the original issue.
@yarikoptic yarikoptic added the internal Changes only affect the internal API label Dec 11, 2024
@yarikoptic yarikoptic requested a review from jwodder December 11, 2024 20:27
@yarikoptic yarikoptic added the DX Developer eXperience label Dec 11, 2024
Copy link

codecov bot commented Dec 11, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 88.56%. Comparing base (38409e8) to head (db4c0e1).
Report is 11 commits behind head on master.

Files with missing lines Patch % Lines
dandi/download.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1550      +/-   ##
==========================================
+ Coverage   88.44%   88.56%   +0.12%     
==========================================
  Files          78       78              
  Lines       10742    10742              
==========================================
+ Hits         9501     9514      +13     
+ Misses       1241     1228      -13     
Flag Coverage Δ
unittests 88.56% <0.00%> (+0.12%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@yarikoptic yarikoptic merged commit 3ea8c87 into master Dec 12, 2024
25 of 26 checks passed
@yarikoptic yarikoptic deleted the enh-assert branch December 12, 2024 03:26
Copy link

🚀 PR was released in 0.66.2 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DX Developer eXperience internal Changes only affect the internal API released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants