jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-07-17 10:17:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

Raise on database error in recorder.migration._add_constraint (#123646)

Browse Source 
* Raise on database error in recorder.migration._add_constraint

* Fix test
This commit is contained in:
Erik Montnemery 2024-08-14 13:59:06 +02:00 committed by GitHub
parent b698dd8f32
commit 80f5683cd6
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
homeassistant/components/recorder/migration.py
View File

@ -727,6 +727,7 @@ def _add_constraint(
connection.execute(add_constraint)
except (InternalError, OperationalError):
_LOGGER.exception("Could not update foreign options in %s table", table)
raise
def _delete_foreign_key_violations(

7
tests/components/recorder/test_migrate.py
View File

@ -962,9 +962,10 @@ def test_restore_foreign_key_constraints_with_error(
engine = Mock()
session_maker = Mock(return_value=session)
migration._restore_foreign_key_constraints(
session_maker, engine, constraints_to_restore
)
with pytest.raises(InternalError):
migration._restore_foreign_key_constraints(
session_maker, engine, constraints_to_restore
)
assert "Could not update foreign options in events table" in caplog.text