groove/Part-DB-server
1
0
Fork 0
mirror of https://github.com/Part-DB/Part-DB-server.git synced 2026-03-01 04:49:36 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

Fix test failures: add locale prefix to URLs, correct log directory path

Browse source
This commit is contained in:
Sebastian Almberg 2026-02-21 00:04:08 +01:00
parent e6ac77ffde
commit 597f0e6280
2 changed files with 12 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

22
tests/Controller/UpdateManagerControllerTest.php
View file

@ -47,7 +47,7 @@ final class UpdateManagerControllerTest extends WebTestCase
{ {
$client = static::createClient(); $client = static::createClient();
$client->request('GET', '/system/update-manager'); $client->request('GET', '/en/system/update-manager');
// Should redirect to login // Should redirect to login
$this->assertResponseRedirects(); $this->assertResponseRedirects();
@ -58,7 +58,7 @@ final class UpdateManagerControllerTest extends WebTestCase
$client = static::createClient(); $client = static::createClient();
$this->loginAsAdmin($client); $this->loginAsAdmin($client);
$client->request('GET', '/system/update-manager'); $client->request('GET', '/en/system/update-manager');
$this->assertResponseIsSuccessful(); $this->assertResponseIsSuccessful();
} }
@ -68,12 +68,12 @@ final class UpdateManagerControllerTest extends WebTestCase
$client = static::createClient(); $client = static::createClient();
$this->loginAsAdmin($client); $this->loginAsAdmin($client);
$client->request('POST', '/system/update-manager/backup', [ $client->request('POST', '/en/system/update-manager/backup', [
'_token' => 'invalid', '_token' => 'invalid',
]); ]);
// Should redirect with error flash // Should redirect with error flash
$this->assertResponseRedirects('/system/update-manager'); $this->assertResponseRedirects();
} }
public function testDeleteBackupRequiresCsrf(): void public function testDeleteBackupRequiresCsrf(): void
@ -81,12 +81,12 @@ final class UpdateManagerControllerTest extends WebTestCase
$client = static::createClient(); $client = static::createClient();
$this->loginAsAdmin($client); $this->loginAsAdmin($client);
$client->request('POST', '/system/update-manager/backup/delete', [ $client->request('POST', '/en/system/update-manager/backup/delete', [
'_token' => 'invalid', '_token' => 'invalid',
'filename' => 'test.zip', 'filename' => 'test.zip',
]); ]);
$this->assertResponseRedirects('/system/update-manager'); $this->assertResponseRedirects();
} }
public function testDeleteLogRequiresCsrf(): void public function testDeleteLogRequiresCsrf(): void
@ -94,12 +94,12 @@ final class UpdateManagerControllerTest extends WebTestCase
$client = static::createClient(); $client = static::createClient();
$this->loginAsAdmin($client); $this->loginAsAdmin($client);
$client->request('POST', '/system/update-manager/log/delete', [ $client->request('POST', '/en/system/update-manager/log/delete', [
'_token' => 'invalid', '_token' => 'invalid',
'filename' => 'test.log', 'filename' => 'test.log',
]); ]);
$this->assertResponseRedirects('/system/update-manager'); $this->assertResponseRedirects();
} }
public function testDownloadBackupReturns404ForNonExistent(): void public function testDownloadBackupReturns404ForNonExistent(): void
@ -107,7 +107,7 @@ final class UpdateManagerControllerTest extends WebTestCase
$client = static::createClient(); $client = static::createClient();
$this->loginAsAdmin($client); $this->loginAsAdmin($client);
$client->request('GET', '/system/update-manager/backup/download/nonexistent.zip'); $client->request('GET', '/en/system/update-manager/backup/download/nonexistent.zip');
$this->assertResponseStatusCodeSame(404); $this->assertResponseStatusCodeSame(404);
} }
@ -117,7 +117,7 @@ final class UpdateManagerControllerTest extends WebTestCase
$client = static::createClient(); $client = static::createClient();
$this->loginAsAdmin($client); $this->loginAsAdmin($client);
$client->request('GET', '/system/update-manager/backup/nonexistent.zip'); $client->request('GET', '/en/system/update-manager/backup/nonexistent.zip');
$this->assertResponseStatusCodeSame(404); $this->assertResponseStatusCodeSame(404);
} }
@ -128,7 +128,7 @@ final class UpdateManagerControllerTest extends WebTestCase
$this->loginAsAdmin($client); $this->loginAsAdmin($client);
// DISABLE_BACKUP_RESTORE=1 is the default in .env, so this should return 403 // DISABLE_BACKUP_RESTORE=1 is the default in .env, so this should return 403
$client->request('POST', '/system/update-manager/restore', [ $client->request('POST', '/en/system/update-manager/restore', [
'_token' => 'invalid', '_token' => 'invalid',
'filename' => 'test.zip', 'filename' => 'test.zip',
]); ]);

2
tests/Services/System/UpdateExecutorTest.php
View file

@ -158,7 +158,7 @@ final class UpdateExecutorTest extends KernelTestCase
{ {
// Create a temporary log file in the update logs directory // Create a temporary log file in the update logs directory
$projectDir = self::getContainer()->getParameter('kernel.project_dir'); $projectDir = self::getContainer()->getParameter('kernel.project_dir');
$logDir = $projectDir . '/var/update_logs'; $logDir = $projectDir . '/var/log/updates';
if (!is_dir($logDir)) { if (!is_dir($logDir)) {
mkdir($logDir, 0755, true); mkdir($logDir, 0755, true);