paradizelost/paperless-ngx
1
0
Fork 0
mirror of https://github.com/paperless-ngx/paperless-ngx.git synced 2025-08-18 00:46:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix: Allows pre-consume scripts to modify the working path again (#5260)

Browse Source 
* Allows pre-consume scripts to modify the working path again and generally cleans up some confusion about working copy vs original
This commit is contained in:
Trenton H
2024-01-05 21:01:57 -08:00
committed by GitHub
parent a96bf3c165
commit eb2caa5118
5 changed files with 37 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/documents/tests/test_consumer.py
View File

@@ -928,7 +928,7 @@ class PreConsumeTestCase(TestCase):
@override_settings(PRE_CONSUME_SCRIPT=None)
def test_no_pre_consume_script(self, m):
c = Consumer()
c.path = "path-to-file"
c.working_copy = "path-to-file"
c.run_pre_consume_script()
m.assert_not_called()
@@ -938,7 +938,7 @@ class PreConsumeTestCase(TestCase):
def test_pre_consume_script_not_found(self, m, m2):
c = Consumer()
c.filename = "somefile.pdf"
c.path = "path-to-file"
c.working_copy = "path-to-file"
self.assertRaises(ConsumerError, c.run_pre_consume_script)
@mock.patch("documents.consumer.run")
@@ -947,7 +947,7 @@ class PreConsumeTestCase(TestCase):
with override_settings(PRE_CONSUME_SCRIPT=script.name):
c = Consumer()
c.original_path = "path-to-file"
c.path = "/tmp/somewhere/path-to-file"
c.working_copy = "/tmp/somewhere/path-to-file"
c.task_id = str(uuid.uuid4())
c.run_pre_consume_script()
@@ -963,7 +963,7 @@ class PreConsumeTestCase(TestCase):
subset = {
"DOCUMENT_SOURCE_PATH": c.original_path,
"DOCUMENT_WORKING_PATH": c.path,
"DOCUMENT_WORKING_PATH": c.working_copy,
"TASK_ID": c.task_id,
}
self.assertDictEqual(environment, {**environment, **subset})
@@ -991,7 +991,7 @@ class PreConsumeTestCase(TestCase):
with override_settings(PRE_CONSUME_SCRIPT=script.name):
with self.assertLogs("paperless.consumer", level="INFO") as cm:
c = Consumer()
c.path = "path-to-file"
c.working_copy = "path-to-file"
c.run_pre_consume_script()
self.assertIn(
@@ -1024,7 +1024,7 @@ class PreConsumeTestCase(TestCase):
with override_settings(PRE_CONSUME_SCRIPT=script.name):
c = Consumer()
c.path = "path-to-file"
c.working_copy = "path-to-file"
self.assertRaises(
ConsumerError,
c.run_pre_consume_script,