fixup! feat(user-stats): wire UserStatsJob to send stats to PSS via API client

Pass the Throwable object to the logger instead of its message string.
Nextcloud's logger treats 'exception' => $throwable specially and
serializes the full stack trace; passing a plain string silently drops
the trace, making transient but unexpected errors hard to diagnose.
Update the corresponding test assertion accordingly.

Signed-off-by: Misha M.-Kupriyanov <kupriyanov@strato.de>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: printminion-co

lib/BackgroundJob/UserStatsJob.php

@@ -74,7 +74,7 @@ protected function run(mixed $argument): void {
 			$api->updateStats($brand, $extRef, $request);
 		} catch (\Throwable $e) {
 			$this->logger->error('UserStatsJob: failed to push stats to PSS', [
-				'exception' => $e->getMessage(),
+				'exception' => $e,
 			]);
 		}
 	}

tests/unit/BackgroundJob/UserStatsJobTest.php

@@ -90,7 +90,8 @@ public function testRunLogsErrorWhenApiThrows(): void {
 		$this->logger->expects($this->once())
 			->method('error')
 			->with('UserStatsJob: failed to push stats to PSS', $this->callback(function (array $ctx): bool {
-				return str_contains($ctx['exception'], 'connection refused');
+				return $ctx['exception'] instanceof \Exception
+					&& str_contains($ctx['exception']->getMessage(), 'connection refused');
 			}));
 
 		$this->invokePrivate($this->job, 'run', [null]);