Improved error message for failing background job

diff --git a/lib/private/backgroundjob/job.php b/lib/private/backgroundjob/job.php
index 677b37e9297b66db2244f60b52913d09b9dab6fd..4c4da94d4fdde465322b6ec2ce9ffb51d7ccef53 100644 (file)
--- a/lib/private/backgroundjob/job.php
+++ b/lib/private/backgroundjob/job.php
@@ -52,7 +52,7 @@ abstract class Job implements IJob {
                        $this->run($this->argument);
                } catch (\Exception $e) {
                        if ($logger) {
-                               $logger->error('Error while running background job: ' . $e->getMessage());
+                               $logger->error('Error while running background job (class: ' . get_class($this) . ', arguments: ' . print_r($this->argument, true) . '): ' . $e->getMessage());
                        }
                }
        }

diff --git a/tests/lib/backgroundjob/job.php b/tests/lib/backgroundjob/job.php
index 912e0e13b57b6a0719d21cae36312ca70c4b52b9..f2a5ff894fe92e8f7e95f755827f822f9312dc2e 100644 (file)
--- a/tests/lib/backgroundjob/job.php
+++ b/tests/lib/backgroundjob/job.php
@@ -28,7 +28,7 @@ class Job extends \Test\TestCase {
                        ->getMock();
                $logger->expects($this->once())
                        ->method('error')
-                       ->with('Error while running background job: ');
+                       ->with('Error while running background job (class: Test\BackgroundJob\TestJob, arguments: ): ');
 
                $this->assertCount(1, $jobList->getAll());
                $job->execute($jobList, $logger);