Fixed #32224 -- Avoided suppressing connection errors in supports_json_field on SQLite.`

Regression in 6789ded0a6.

Thanks Juan Garcia Alvite for the report.
This commit is contained in:
Mariusz Felisiak 2020-11-25 12:19:45 +01:00 committed by GitHub
parent b996424199
commit f5e5aac59e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 28 additions and 5 deletions

View File

@ -70,8 +70,9 @@ class DatabaseFeatures(BaseDatabaseFeatures):
@cached_property @cached_property
def supports_json_field(self): def supports_json_field(self):
with self.connection.cursor() as cursor:
try: try:
with self.connection.cursor() as cursor, transaction.atomic(): with transaction.atomic(self.connection.alias):
cursor.execute('SELECT JSON(\'{"a": "b"}\')') cursor.execute('SELECT JSON(\'{"a": "b"}\')')
except OperationalError: except OperationalError:
return False return False

View File

@ -24,3 +24,7 @@ Bugfixes
* Fixed a regression in Django 3.1 that caused the incorrect grouping by a * Fixed a regression in Django 3.1 that caused the incorrect grouping by a
``Q`` object annotation (:ticket:`32200`). ``Q`` object annotation (:ticket:`32200`).
* Fixed a regression in Django 3.1 that caused suppressing connection errors
when :class:`~django.db.models.JSONField` is used on SQLite
(:ticket:`32224`).

View File

@ -0,0 +1,18 @@
from unittest import mock, skipUnless
from django.db import OperationalError, connection
from django.test import TestCase
@skipUnless(connection.vendor == 'sqlite', 'SQLite tests.')
class FeaturesTests(TestCase):
def test_supports_json_field_operational_error(self):
if hasattr(connection.features, 'supports_json_field'):
del connection.features.supports_json_field
msg = 'unable to open database file'
with mock.patch(
'django.db.backends.base.base.BaseDatabaseWrapper.cursor',
side_effect=OperationalError(msg),
):
with self.assertRaisesMessage(OperationalError, msg):
connection.features.supports_json_field