diff --git a/src/documents/bulk_edit.py b/src/documents/bulk_edit.py index 13df775c09..c313d52e23 100644 --- a/src/documents/bulk_edit.py +++ b/src/documents/bulk_edit.py @@ -427,6 +427,7 @@ def delete(doc_ids: list[int]) -> Literal["OK"]: "Detected a possible incompatible database column. See https://docs.paperless-ngx.com/troubleshooting/#convert-uuid-field", ) logger.error(f"Error deleting documents: {e!s}") + raise return "OK" diff --git a/src/documents/tests/test_bulk_edit.py b/src/documents/tests/test_bulk_edit.py index b341f57444..8e7d2e1546 100644 --- a/src/documents/tests/test_bulk_edit.py +++ b/src/documents/tests/test_bulk_edit.py @@ -698,10 +698,10 @@ class TestBulkEdit(DirectoriesMixin, TestCase): def test_delete_documents_old_uuid_field(self, m) -> None: m.side_effect = Exception("Data too long for column 'transaction_id' at row 1") doc_ids = [self.doc1.id, self.doc2.id, self.doc3.id] - bulk_edit.delete(doc_ids) with self.assertLogs(level="WARNING") as cm: - bulk_edit.delete(doc_ids) - self.assertIn("possible incompatible database column", cm.output[0]) + with self.assertRaisesRegex(Exception, "Data too long for column"): + bulk_edit.delete(doc_ids) + self.assertIn("possible incompatible database column", cm.output[0]) class TestPDFActions(DirectoriesMixin, TestCase):