-
-
Notifications
You must be signed in to change notification settings - Fork 31.3k
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
Fixed #34370 -- Added integer fields validation as 64-bit on SQLite. #16601
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nabil-rady Thanks 👍
|
||
with self.assertRaises(ValidationError): | ||
instance_overflow.full_clean() | ||
instance_underflow.full_clean() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is unreachable.
85e0718
to
2083ff4
Compare
instance = self.model(value=min_value) | ||
instance.full_clean() | ||
instance.save() | ||
qs = self.model.objects.filter(value__lte=min_value) | ||
self.assertEqual(qs.count(), 1) | ||
self.assertEqual(qs[0].value, min_value) | ||
|
||
if max_value is not None: | ||
if max_value <= max_documented_value: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes in test_backend_range_save
looks unnecessary, please revert.
instance_underflow.full_clean() | ||
instance_overflow = self.model(value=max_value + 1) | ||
with self.assertRaises(ValidationError): | ||
instance_overflow.full_clean() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test_overflow_and_underflow
seems unnecessary, it's already tested in test_backend_range_validation
.
5a5cb3d
to
d3ffdc7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nabil-rady Thanks 👍 I added a release note.
Fixes 34370