Bug 27421: (QA follow-up) BatchCommitImportRecords needs param for skipping commits
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Mon, 5 Sep 2022 11:57:52 +0000 (11:57 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Mon, 19 Sep 2022 18:14:59 +0000 (15:14 -0300)
commitbdb107bcec6d87649d6c38ac8bee960a321c9ab0
tree091c65406a74781d43f52a677846e6d070854cb6
parent3f79fcd5aa5efed7883045e3ce467ead79727131
Bug 27421: (QA follow-up) BatchCommitImportRecords needs param for skipping commits

When you submit a background jobs, and it fails, you do not expect
partial results in the database.
Note that when the Background feature would support a partially
completed status, things might change again.

Note that the >0 test was superfluous if you check for ^\d+$.

Test plan:
Run t/db_dependent/Koha/BackgroundJobs/StageMARCForImport.t

Note: This serves to verify that it still runs as expected.
The test plan of the following patch covers the new param.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
C4/ImportBatch.pm
Koha/BackgroundJob/MARCImportCommitBatch.pm