From aa44a5f6edd044e3acab21a60d8126868ee93f83 Mon Sep 17 00:00:00 2001 From: Ralf Haferkamp Date: Wed, 26 Nov 2025 16:14:30 +0100 Subject: [PATCH] tests: Wait for post-processing to finish before editing the file Otherwise the test is racy. The file might still be stuck in post-processing so the admin would update an empty file. --- tests/acceptance/bootstrap/CliContext.php | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/acceptance/bootstrap/CliContext.php b/tests/acceptance/bootstrap/CliContext.php index 35889cc88..082d3894a 100644 --- a/tests/acceptance/bootstrap/CliContext.php +++ b/tests/acceptance/bootstrap/CliContext.php @@ -675,6 +675,9 @@ class CliContext implements Context { * @return void */ public function theAdministratorChangesFileContent(string $content, string $file, string $user): void { + // this downloads the file using WebDAV and by that checks if it's still in + // postprocessing. So its effectively a check for finished postprocessing + $this->featureContext->userDownloadsFileUsingTheAPI($user, $file); $userUuid = $this->featureContext->getAttributeOfCreatedUser($user, 'id'); $storagePath = $this->getUsersStoragePath(); $safeContent = escapeshellarg($content);