diff options
| author | Greg Roach <greg@subaqua.co.uk> | 2023-10-01 20:39:56 +0100 |
|---|---|---|
| committer | Greg Roach <greg@subaqua.co.uk> | 2023-10-04 00:04:15 +0100 |
| commit | a26ec5ede3c2ecdb45b783755ede967935aaecba (patch) | |
| tree | 817d6af978a82882a9917be41e089d65a7912c43 /tests/app/Http/RequestHandlers | |
| parent | 6f3a4c3c03fa8481d67a2fd8f1e8be3e64e9691a (diff) | |
| download | webtrees-a26ec5ede3c2ecdb45b783755ede967935aaecba.tar.gz webtrees-a26ec5ede3c2ecdb45b783755ede967935aaecba.tar.bz2 webtrees-a26ec5ede3c2ecdb45b783755ede967935aaecba.zip | |
Fix broken tests
Diffstat (limited to 'tests/app/Http/RequestHandlers')
71 files changed, 72 insertions, 512 deletions
diff --git a/tests/app/Http/RequestHandlers/AccountDeleteTest.php b/tests/app/Http/RequestHandlers/AccountDeleteTest.php index c5e65c74d9..2d2e6b06bd 100644 --- a/tests/app/Http/RequestHandlers/AccountDeleteTest.php +++ b/tests/app/Http/RequestHandlers/AccountDeleteTest.php @@ -30,9 +30,6 @@ use Fisharebest\Webtrees\TestCase; */ class AccountDeleteTest extends TestCase { - /** - * @return void - */ public function testHandler(): void { $user_service = $this->createStub(UserService::class); diff --git a/tests/app/Http/RequestHandlers/AccountEditTest.php b/tests/app/Http/RequestHandlers/AccountEditTest.php index 27af86019a..5f3388e540 100644 --- a/tests/app/Http/RequestHandlers/AccountEditTest.php +++ b/tests/app/Http/RequestHandlers/AccountEditTest.php @@ -35,9 +35,6 @@ class AccountEditTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testHandler(): void { $user = $this->createStub(User::class); diff --git a/tests/app/Http/RequestHandlers/AccountUpdateTest.php b/tests/app/Http/RequestHandlers/AccountUpdateTest.php index ef14784301..6c95ddb88d 100644 --- a/tests/app/Http/RequestHandlers/AccountUpdateTest.php +++ b/tests/app/Http/RequestHandlers/AccountUpdateTest.php @@ -33,9 +33,6 @@ use Fisharebest\Webtrees\User; */ class AccountUpdateTest extends TestCase { - /** - * @return void - */ public function testHandler(): void { $user_service = $this->createMock(UserService::class); diff --git a/tests/app/Http/RequestHandlers/BroadcastPageTest.php b/tests/app/Http/RequestHandlers/BroadcastPageTest.php index 68730facac..6799bea23e 100644 --- a/tests/app/Http/RequestHandlers/BroadcastPageTest.php +++ b/tests/app/Http/RequestHandlers/BroadcastPageTest.php @@ -32,9 +32,6 @@ class BroadcastPageTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testMissingParameterTo(): void { $message_service = $this->createStub(MessageService::class); @@ -49,9 +46,6 @@ class BroadcastPageTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testHandler(): void { $message_service = $this->createStub(MessageService::class); diff --git a/tests/app/Http/RequestHandlers/ControlPanelControllerTest.php b/tests/app/Http/RequestHandlers/ControlPanelControllerTest.php index d8ec46c59f..2b6607c639 100644 --- a/tests/app/Http/RequestHandlers/ControlPanelControllerTest.php +++ b/tests/app/Http/RequestHandlers/ControlPanelControllerTest.php @@ -42,9 +42,6 @@ class ControlPanelControllerTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testControlPanel(): void { $admin_service = new AdminService(); diff --git a/tests/app/Http/RequestHandlers/DeleteUserTest.php b/tests/app/Http/RequestHandlers/DeleteUserTest.php index ee17fd8be6..eb38c2b748 100644 --- a/tests/app/Http/RequestHandlers/DeleteUserTest.php +++ b/tests/app/Http/RequestHandlers/DeleteUserTest.php @@ -34,9 +34,6 @@ class DeleteUserTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testDeleteUser(): void { $user = $this->createMock(User::class); @@ -53,9 +50,6 @@ class DeleteUserTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_NO_CONTENT, $response->getStatusCode()); } - /** - * @return void - */ public function testDeleteNonExistingUser(): void { $this->expectException(HttpNotFoundException::class); @@ -70,9 +64,6 @@ class DeleteUserTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testCannotDeleteAdministrator(): void { $this->expectException(HttpAccessDeniedException::class); diff --git a/tests/app/Http/RequestHandlers/FixLevel0MediaActionTest.php b/tests/app/Http/RequestHandlers/FixLevel0MediaActionTest.php index ca6d933e14..adbf964931 100644 --- a/tests/app/Http/RequestHandlers/FixLevel0MediaActionTest.php +++ b/tests/app/Http/RequestHandlers/FixLevel0MediaActionTest.php @@ -34,9 +34,6 @@ class FixLevel0MediaActionTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testFixLevel0MediaAction(): void { $gedcom_import_service = new GedcomImportService(); diff --git a/tests/app/Http/RequestHandlers/FixLevel0MediaDataTest.php b/tests/app/Http/RequestHandlers/FixLevel0MediaDataTest.php index 5a229006d9..4bfafb65ad 100644 --- a/tests/app/Http/RequestHandlers/FixLevel0MediaDataTest.php +++ b/tests/app/Http/RequestHandlers/FixLevel0MediaDataTest.php @@ -35,9 +35,6 @@ class FixLevel0MediaDataTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testFixLevel0MediaData(): void { $datatables_service = new DatatablesService(); diff --git a/tests/app/Http/RequestHandlers/FixLevel0MediaPageTest.php b/tests/app/Http/RequestHandlers/FixLevel0MediaPageTest.php index fcdb68cf5c..a374c10309 100644 --- a/tests/app/Http/RequestHandlers/FixLevel0MediaPageTest.php +++ b/tests/app/Http/RequestHandlers/FixLevel0MediaPageTest.php @@ -29,9 +29,8 @@ use Fisharebest\Webtrees\TestCase; */ class FixLevel0MediaPageTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testFixLevel0Media(): void { $handler = new FixLevel0MediaPage(); diff --git a/tests/app/Http/RequestHandlers/ImportThumbnailsActionTest.php b/tests/app/Http/RequestHandlers/ImportThumbnailsActionTest.php index 7ae95f0a45..ccad9a3001 100644 --- a/tests/app/Http/RequestHandlers/ImportThumbnailsActionTest.php +++ b/tests/app/Http/RequestHandlers/ImportThumbnailsActionTest.php @@ -34,9 +34,6 @@ class ImportThumbnailsActionTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testWebtrees1ThumbnailsAction(): void { $gedcom_import_service = new GedcomImportService(); diff --git a/tests/app/Http/RequestHandlers/ImportThumbnailsDataTest.php b/tests/app/Http/RequestHandlers/ImportThumbnailsDataTest.php index e6953c3088..c7ffc272e8 100644 --- a/tests/app/Http/RequestHandlers/ImportThumbnailsDataTest.php +++ b/tests/app/Http/RequestHandlers/ImportThumbnailsDataTest.php @@ -34,9 +34,6 @@ class ImportThumbnailsDataTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testWebtrees1ThumbnailsData(): void { $gedcom_import_service = new GedcomImportService(); diff --git a/tests/app/Http/RequestHandlers/ImportThumbnailsPageTest.php b/tests/app/Http/RequestHandlers/ImportThumbnailsPageTest.php index 6758ed0fb7..f642391f3b 100644 --- a/tests/app/Http/RequestHandlers/ImportThumbnailsPageTest.php +++ b/tests/app/Http/RequestHandlers/ImportThumbnailsPageTest.php @@ -31,9 +31,6 @@ class ImportThumbnailsPageTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testWebtrees1Thumbnails(): void { $handler = new ImportThumbnailsPage(); diff --git a/tests/app/Http/RequestHandlers/LoginPageTest.php b/tests/app/Http/RequestHandlers/LoginPageTest.php index d709860250..cb6180af0f 100644 --- a/tests/app/Http/RequestHandlers/LoginPageTest.php +++ b/tests/app/Http/RequestHandlers/LoginPageTest.php @@ -30,9 +30,8 @@ use Fisharebest\Webtrees\User; */ class LoginPageTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testLoginPage(): void { $gedcom_import_service = new GedcomImportService(); @@ -44,9 +43,6 @@ class LoginPageTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_OK, $response->getStatusCode()); } - /** - * @return void - */ public function testLoginPageAlreadyLoggedIn(): void { $gedcom_import_service = new GedcomImportService(); diff --git a/tests/app/Http/RequestHandlers/ManageMediaDataTest.php b/tests/app/Http/RequestHandlers/ManageMediaDataTest.php index 6f950cbc73..8b857ad488 100644 --- a/tests/app/Http/RequestHandlers/ManageMediaDataTest.php +++ b/tests/app/Http/RequestHandlers/ManageMediaDataTest.php @@ -37,9 +37,6 @@ class ManageMediaDataTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testDataLocal(): void { $datatables_service = new DatatablesService(); @@ -61,9 +58,6 @@ class ManageMediaDataTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_OK, $response->getStatusCode()); } - /** - * @return void - */ public function testDataExternal(): void { $datatables_service = new DatatablesService(); @@ -85,9 +79,6 @@ class ManageMediaDataTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_OK, $response->getStatusCode()); } - /** - * @return void - */ public function testDataUnused(): void { $datatables_service = new DatatablesService(); diff --git a/tests/app/Http/RequestHandlers/ManageMediaPageTest.php b/tests/app/Http/RequestHandlers/ManageMediaPageTest.php index 4b487b8af3..77479fe4d1 100644 --- a/tests/app/Http/RequestHandlers/ManageMediaPageTest.php +++ b/tests/app/Http/RequestHandlers/ManageMediaPageTest.php @@ -32,9 +32,6 @@ class ManageMediaPageTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testIndex(): void { $media_file_service = new MediaFileService(); diff --git a/tests/app/Http/RequestHandlers/MapDataExportCSVTest.php b/tests/app/Http/RequestHandlers/MapDataExportCSVTest.php index f418e9bbbb..d3b67b5f61 100644 --- a/tests/app/Http/RequestHandlers/MapDataExportCSVTest.php +++ b/tests/app/Http/RequestHandlers/MapDataExportCSVTest.php @@ -32,9 +32,6 @@ class MapDataExportCSVTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testExportCSV(): void { $map_data_service = new MapDataService(); diff --git a/tests/app/Http/RequestHandlers/MapDataExportGeoJsonTest.php b/tests/app/Http/RequestHandlers/MapDataExportGeoJsonTest.php index ba42f8a3c7..494efeb90f 100644 --- a/tests/app/Http/RequestHandlers/MapDataExportGeoJsonTest.php +++ b/tests/app/Http/RequestHandlers/MapDataExportGeoJsonTest.php @@ -32,9 +32,6 @@ class MapDataExportGeoJsonTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testExportGeoJson(): void { $map_data_service = new MapDataService(); diff --git a/tests/app/Http/RequestHandlers/MapDataImportActionTest.php b/tests/app/Http/RequestHandlers/MapDataImportActionTest.php index 98cb4dbafb..307588c783 100644 --- a/tests/app/Http/RequestHandlers/MapDataImportActionTest.php +++ b/tests/app/Http/RequestHandlers/MapDataImportActionTest.php @@ -35,9 +35,6 @@ class MapDataImportActionTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testImportAction(): void { $csv = $this->createUploadedFile(dirname(__DIR__, 3) . '/data/places.csv', 'text/csv'); diff --git a/tests/app/Http/RequestHandlers/MapDataImportPageTest.php b/tests/app/Http/RequestHandlers/MapDataImportPageTest.php index cb737f6230..68534f70fb 100644 --- a/tests/app/Http/RequestHandlers/MapDataImportPageTest.php +++ b/tests/app/Http/RequestHandlers/MapDataImportPageTest.php @@ -31,9 +31,6 @@ class MapDataImportPageTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testImportPage(): void { $handler = new MapDataImportPage(); diff --git a/tests/app/Http/RequestHandlers/MasqueradeTest.php b/tests/app/Http/RequestHandlers/MasqueradeTest.php index 2610815dee..940ed0cd06 100644 --- a/tests/app/Http/RequestHandlers/MasqueradeTest.php +++ b/tests/app/Http/RequestHandlers/MasqueradeTest.php @@ -32,9 +32,8 @@ use Fisharebest\Webtrees\User; */ class MasqueradeTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testMasqueradeAsUser(): void { $user1 = $this->createMock(User::class); @@ -58,9 +57,6 @@ class MasqueradeTest extends TestCase self::assertSame('1', Session::get('masquerade')); } - /** - * @return void - */ public function testCannotMasqueradeAsSelf(): void { $user = $this->createMock(User::class); @@ -80,9 +76,6 @@ class MasqueradeTest extends TestCase self::assertNull(Session::get('masquerade')); } - /** - * @return void - */ public function testMasqueradeAsNonExistingUser(): void { $this->expectException(HttpNotFoundException::class); diff --git a/tests/app/Http/RequestHandlers/ModuleActionTest.php b/tests/app/Http/RequestHandlers/ModuleActionTest.php index d89018a059..0474d30c59 100644 --- a/tests/app/Http/RequestHandlers/ModuleActionTest.php +++ b/tests/app/Http/RequestHandlers/ModuleActionTest.php @@ -36,9 +36,6 @@ use function response; */ class ModuleActionTest extends TestCase { - /** - * @return void - */ public function testModuleAction(): void { $module_service = $this->createMock(ModuleService::class); @@ -60,9 +57,6 @@ class ModuleActionTest extends TestCase self::assertSame('It works!', (string) $response->getBody()); } - /** - * @return void - */ public function testNonExistingAction(): void { $this->expectException(HttpNotFoundException::class); @@ -84,9 +78,6 @@ class ModuleActionTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNonExistingModule(): void { $this->expectException(HttpNotFoundException::class); @@ -108,9 +99,6 @@ class ModuleActionTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testAdminAction(): void { $this->expectException(HttpAccessDeniedException::class); diff --git a/tests/app/Http/RequestHandlers/PasswordRequestPageTest.php b/tests/app/Http/RequestHandlers/PasswordRequestPageTest.php index afa33571da..6eeb6d6961 100644 --- a/tests/app/Http/RequestHandlers/PasswordRequestPageTest.php +++ b/tests/app/Http/RequestHandlers/PasswordRequestPageTest.php @@ -28,9 +28,8 @@ use Fisharebest\Webtrees\User; */ class PasswordRequestPageTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testPasswordRequestPage(): void { $request = self::createRequest(); @@ -40,9 +39,6 @@ class PasswordRequestPageTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_OK, $response->getStatusCode()); } - /** - * @return void - */ public function testPasswordRequestPageAlreadyLoggedIn(): void { $user = $this->createMock(User::class); diff --git a/tests/app/Http/RequestHandlers/PasswordResetPageTest.php b/tests/app/Http/RequestHandlers/PasswordResetPageTest.php index d0c0f84126..a0e988c2be 100644 --- a/tests/app/Http/RequestHandlers/PasswordResetPageTest.php +++ b/tests/app/Http/RequestHandlers/PasswordResetPageTest.php @@ -30,9 +30,8 @@ use Fisharebest\Webtrees\User; */ class PasswordResetPageTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testPasswordResetPageWithValidToken(): void { $user = $this->createMock(User::class); @@ -52,9 +51,6 @@ class PasswordResetPageTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_OK, $response->getStatusCode()); } - /** - * @return void - */ public function testPasswordResetPageWithoutValidToken(): void { $user_service = $this->createMock(UserService::class); diff --git a/tests/app/Http/RequestHandlers/PingTest.php b/tests/app/Http/RequestHandlers/PingTest.php index 7da18a470e..080678d2f8 100644 --- a/tests/app/Http/RequestHandlers/PingTest.php +++ b/tests/app/Http/RequestHandlers/PingTest.php @@ -29,9 +29,6 @@ use Illuminate\Support\Collection; */ class PingTest extends TestCase { - /** - * @return void - */ public function testPingOK(): void { $server_check_service = $this->createMock(ServerCheckService::class); @@ -46,9 +43,6 @@ class PingTest extends TestCase self::assertSame('OK', (string) $response->getBody()); } - /** - * @return void - */ public function testPingWarnings(): void { $server_check_service = $this->createMock(ServerCheckService::class); @@ -63,9 +57,6 @@ class PingTest extends TestCase self::assertSame('WARNING', (string) $response->getBody()); } - /** - * @return void - */ public function testPingErrors(): void { $server_check_service = $this->createMock(ServerCheckService::class); diff --git a/tests/app/Http/RequestHandlers/RedirectAncestryPhpTest.php b/tests/app/Http/RequestHandlers/RedirectAncestryPhpTest.php index 97b4a972ed..d83e3600c7 100644 --- a/tests/app/Http/RequestHandlers/RedirectAncestryPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectAncestryPhpTest.php @@ -37,9 +37,8 @@ use Illuminate\Support\Collection; */ class RedirectAncestryPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -90,9 +89,6 @@ class RedirectAncestryPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -121,9 +117,6 @@ class RedirectAncestryPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(AncestorsChartModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectBranchesPhpTest.php b/tests/app/Http/RequestHandlers/RedirectBranchesPhpTest.php index 2cec7ca583..d22ea59bb8 100644 --- a/tests/app/Http/RequestHandlers/RedirectBranchesPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectBranchesPhpTest.php @@ -34,9 +34,8 @@ use Illuminate\Support\Collection; */ class RedirectBranchesPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -78,9 +77,6 @@ class RedirectBranchesPhpTest extends TestCase ); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -109,9 +105,6 @@ class RedirectBranchesPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(BranchesListModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectCalendarPhpTest.php b/tests/app/Http/RequestHandlers/RedirectCalendarPhpTest.php index 4a0e08e0aa..3c43256098 100644 --- a/tests/app/Http/RequestHandlers/RedirectCalendarPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectCalendarPhpTest.php @@ -32,9 +32,8 @@ use Illuminate\Support\Collection; */ class RedirectCalendarPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -67,9 +66,6 @@ class RedirectCalendarPhpTest extends TestCase ); } - /** - * @return void - */ public function testNoSuchTree(): void { $tree_service = $this->createStub(TreeService::class); diff --git a/tests/app/Http/RequestHandlers/RedirectCompactPhpTest.php b/tests/app/Http/RequestHandlers/RedirectCompactPhpTest.php index 65dd211eb4..705c3a5cb0 100644 --- a/tests/app/Http/RequestHandlers/RedirectCompactPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectCompactPhpTest.php @@ -37,9 +37,8 @@ use Illuminate\Support\Collection; */ class RedirectCompactPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -90,9 +89,6 @@ class RedirectCompactPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -121,9 +117,6 @@ class RedirectCompactPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(CompactTreeChartModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectDescendancyPhpTest.php b/tests/app/Http/RequestHandlers/RedirectDescendancyPhpTest.php index 1d2f3d048d..1d1df605c0 100644 --- a/tests/app/Http/RequestHandlers/RedirectDescendancyPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectDescendancyPhpTest.php @@ -37,9 +37,8 @@ use Illuminate\Support\Collection; */ class RedirectDescendancyPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -86,9 +85,6 @@ class RedirectDescendancyPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -117,9 +113,6 @@ class RedirectDescendancyPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(DescendancyChartModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectDescendencyPhpTest.php b/tests/app/Http/RequestHandlers/RedirectDescendencyPhpTest.php deleted file mode 100644 index 6a689d5742..0000000000 --- a/tests/app/Http/RequestHandlers/RedirectDescendencyPhpTest.php +++ /dev/null @@ -1,35 +0,0 @@ -<?php - -/** - * webtrees: online genealogy - * Copyright (C) 2023 webtrees development team - * This program is free software: you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation, either version 3 of the License, or - * (at your option) any later version. - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * You should have received a copy of the GNU General Public License - * along with this program. If not, see <https://www.gnu.org/licenses/>. - */ - -declare(strict_types=1); - -namespace Fisharebest\Webtrees\Http\RequestHandlers; - -use Fisharebest\Webtrees\TestCase; - -/** - * Test harness for the class RedirectDescendencyPhp - * - * @covers Fisharebest\Webtrees\Http\RequestHandlers\RedirectDescendencyPhp - */ -class RedirectDescendencyPhpTest extends TestCase -{ - public function testClass(): void - { - $this->assertTrue(class_exists(\Fisharebest\Webtrees\Http\RequestHandlers\RedirectDescendencyPhp::class)); - } -} diff --git a/tests/app/Http/RequestHandlers/RedirectFamListPhpTest.php b/tests/app/Http/RequestHandlers/RedirectFamListPhpTest.php index d8b00618ca..5583b0d557 100644 --- a/tests/app/Http/RequestHandlers/RedirectFamListPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectFamListPhpTest.php @@ -34,9 +34,8 @@ use Illuminate\Support\Collection; */ class RedirectFamListPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -73,9 +72,6 @@ class RedirectFamListPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -101,9 +97,6 @@ class RedirectFamListPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(FamilyListModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectFamilyBookPhpTest.php b/tests/app/Http/RequestHandlers/RedirectFamilyBookPhpTest.php index ff2b26101c..3e44db2c1e 100644 --- a/tests/app/Http/RequestHandlers/RedirectFamilyBookPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectFamilyBookPhpTest.php @@ -37,9 +37,8 @@ use Illuminate\Support\Collection; */ class RedirectFamilyBookPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -86,9 +85,6 @@ class RedirectFamilyBookPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -116,9 +112,6 @@ class RedirectFamilyBookPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(FamilyBookChartModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectFamilyPhpTest.php b/tests/app/Http/RequestHandlers/RedirectFamilyPhpTest.php index 53d3c191b5..a4094d1542 100644 --- a/tests/app/Http/RequestHandlers/RedirectFamilyPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectFamilyPhpTest.php @@ -36,9 +36,8 @@ use Illuminate\Support\Collection; */ class RedirectFamilyPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -79,9 +78,6 @@ class RedirectFamilyPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testNoSuchRecord(): void { $tree = $this->createStub(Tree::class); @@ -104,9 +100,6 @@ class RedirectFamilyPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testMissingXrefParameter(): void { $tree_service = $this->createStub(TreeService::class); diff --git a/tests/app/Http/RequestHandlers/RedirectFanChartPhpTest.php b/tests/app/Http/RequestHandlers/RedirectFanChartPhpTest.php index 7dda5db817..2b599ac2d4 100644 --- a/tests/app/Http/RequestHandlers/RedirectFanChartPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectFanChartPhpTest.php @@ -37,9 +37,8 @@ use Illuminate\Support\Collection; */ class RedirectFanChartPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -90,9 +89,6 @@ class RedirectFanChartPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -121,9 +117,6 @@ class RedirectFanChartPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(FanChartModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectGedRecordPhpTest.php b/tests/app/Http/RequestHandlers/RedirectGedRecordPhpTest.php index c920eb0ab3..7d8371e390 100644 --- a/tests/app/Http/RequestHandlers/RedirectGedRecordPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectGedRecordPhpTest.php @@ -36,9 +36,8 @@ use Illuminate\Support\Collection; */ class RedirectGedRecordPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -79,9 +78,6 @@ class RedirectGedRecordPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testNoSuchRecord(): void { $tree = $this->createStub(Tree::class); @@ -104,9 +100,6 @@ class RedirectGedRecordPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testMissingXrefParameter(): void { $tree_service = $this->createStub(TreeService::class); diff --git a/tests/app/Http/RequestHandlers/RedirectHourGlassPhpTest.php b/tests/app/Http/RequestHandlers/RedirectHourGlassPhpTest.php index d19c0b701d..908c009efa 100644 --- a/tests/app/Http/RequestHandlers/RedirectHourGlassPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectHourGlassPhpTest.php @@ -37,9 +37,8 @@ use Illuminate\Support\Collection; */ class RedirectHourGlassPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -90,9 +89,6 @@ class RedirectHourGlassPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -121,9 +117,6 @@ class RedirectHourGlassPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(HourglassChartModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectIndiListPhpTest.php b/tests/app/Http/RequestHandlers/RedirectIndiListPhpTest.php index f723831b64..1408a6b49d 100644 --- a/tests/app/Http/RequestHandlers/RedirectIndiListPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectIndiListPhpTest.php @@ -34,9 +34,8 @@ use Illuminate\Support\Collection; */ class RedirectIndiListPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -73,9 +72,6 @@ class RedirectIndiListPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -101,9 +97,6 @@ class RedirectIndiListPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(IndividualListModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectIndividualPhpTest.php b/tests/app/Http/RequestHandlers/RedirectIndividualPhpTest.php index cd7dcce302..f681cfe917 100644 --- a/tests/app/Http/RequestHandlers/RedirectIndividualPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectIndividualPhpTest.php @@ -36,9 +36,8 @@ use Illuminate\Support\Collection; */ class RedirectIndividualPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -79,9 +78,6 @@ class RedirectIndividualPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testNoSuchRecord(): void { $tree = $this->createStub(Tree::class); @@ -104,9 +100,6 @@ class RedirectIndividualPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $tree_service = $this->createStub(TreeService::class); @@ -127,9 +120,6 @@ class RedirectIndividualPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testMissingXrefParameter(): void { $tree_service = $this->createStub(TreeService::class); diff --git a/tests/app/Http/RequestHandlers/RedirectLifeSpanPhpTest.php b/tests/app/Http/RequestHandlers/RedirectLifeSpanPhpTest.php index 0073e05f25..4a03f3fd66 100644 --- a/tests/app/Http/RequestHandlers/RedirectLifeSpanPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectLifeSpanPhpTest.php @@ -34,9 +34,8 @@ use Illuminate\Support\Collection; */ class RedirectLifeSpanPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -76,9 +75,6 @@ class RedirectLifeSpanPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -107,9 +103,6 @@ class RedirectLifeSpanPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(LifespansChartModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectMediaListPhpTest.php b/tests/app/Http/RequestHandlers/RedirectMediaListPhpTest.php index 072b255b6d..02d6fb7bef 100644 --- a/tests/app/Http/RequestHandlers/RedirectMediaListPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectMediaListPhpTest.php @@ -34,9 +34,8 @@ use Illuminate\Support\Collection; */ class RedirectMediaListPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -73,9 +72,6 @@ class RedirectMediaListPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -101,9 +97,6 @@ class RedirectMediaListPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(MediaListModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectMediaViewerPhpTest.php b/tests/app/Http/RequestHandlers/RedirectMediaViewerPhpTest.php index e20357484b..d0df853cb1 100644 --- a/tests/app/Http/RequestHandlers/RedirectMediaViewerPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectMediaViewerPhpTest.php @@ -36,9 +36,8 @@ use Illuminate\Support\Collection; */ class RedirectMediaViewerPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -79,9 +78,6 @@ class RedirectMediaViewerPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testNoSuchRecord(): void { $tree = $this->createStub(Tree::class); @@ -104,9 +100,6 @@ class RedirectMediaViewerPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $tree_service = $this->createStub(TreeService::class); @@ -127,9 +120,6 @@ class RedirectMediaViewerPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testMissingTreeParameter(): void { $tree_service = $this->createStub(TreeService::class); @@ -143,9 +133,6 @@ class RedirectMediaViewerPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testMissingXrefParameter(): void { $tree_service = $this->createStub(TreeService::class); diff --git a/tests/app/Http/RequestHandlers/RedirectModulePhpTest.php b/tests/app/Http/RequestHandlers/RedirectModulePhpTest.php index db99bb0f9c..74b9447031 100644 --- a/tests/app/Http/RequestHandlers/RedirectModulePhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectModulePhpTest.php @@ -38,9 +38,8 @@ use Illuminate\Support\Collection; */ class RedirectModulePhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirectPedigreeMap(): void { $tree = $this->createStub(Tree::class); @@ -91,9 +90,6 @@ class RedirectModulePhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testRedirectInteractiveTree(): void { $tree = $this->createStub(Tree::class); @@ -145,9 +141,6 @@ class RedirectModulePhpTest extends TestCase } - /** - * @return void - */ public function testNoSuchTree(): void { $module_service = $this->createStub(ModuleService::class); @@ -169,9 +162,6 @@ class RedirectModulePhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchIndividual(): void { $tree = $this->createStub(Tree::class); @@ -206,9 +196,6 @@ class RedirectModulePhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testPedigreeMapModuleDisabled(): void { $tree = $this->createStub(Tree::class); @@ -252,9 +239,6 @@ class RedirectModulePhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testInteractiveTreeModuleDisabled(): void { $tree = $this->createStub(Tree::class); diff --git a/tests/app/Http/RequestHandlers/RedirectNoteListPhpTest.php b/tests/app/Http/RequestHandlers/RedirectNoteListPhpTest.php index cd9c3f031d..456c069e6f 100644 --- a/tests/app/Http/RequestHandlers/RedirectNoteListPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectNoteListPhpTest.php @@ -34,9 +34,8 @@ use Illuminate\Support\Collection; */ class RedirectNoteListPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -73,9 +72,6 @@ class RedirectNoteListPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -101,9 +97,6 @@ class RedirectNoteListPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(NoteListModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectNotePhpTest.php b/tests/app/Http/RequestHandlers/RedirectNotePhpTest.php index a4d75b9b90..d08fe27a4a 100644 --- a/tests/app/Http/RequestHandlers/RedirectNotePhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectNotePhpTest.php @@ -36,9 +36,8 @@ use Illuminate\Support\Collection; */ class RedirectNotePhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -79,9 +78,6 @@ class RedirectNotePhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testNoSuchRecord(): void { $tree = $this->createStub(Tree::class); @@ -104,9 +100,6 @@ class RedirectNotePhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testMissingXrefParameter(): void { $tree_service = $this->createStub(TreeService::class); diff --git a/tests/app/Http/RequestHandlers/RedirectPedigreePhpTest.php b/tests/app/Http/RequestHandlers/RedirectPedigreePhpTest.php index c73891a160..78b2627083 100644 --- a/tests/app/Http/RequestHandlers/RedirectPedigreePhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectPedigreePhpTest.php @@ -37,9 +37,8 @@ use Illuminate\Support\Collection; */ class RedirectPedigreePhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -90,9 +89,6 @@ class RedirectPedigreePhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -121,9 +117,6 @@ class RedirectPedigreePhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(PedigreeChartModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectPlaceListPhpTest.php b/tests/app/Http/RequestHandlers/RedirectPlaceListPhpTest.php index d8214a1359..5e928868e6 100644 --- a/tests/app/Http/RequestHandlers/RedirectPlaceListPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectPlaceListPhpTest.php @@ -34,9 +34,8 @@ use Illuminate\Support\Collection; */ class RedirectPlaceListPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -73,9 +72,6 @@ class RedirectPlaceListPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -101,9 +97,6 @@ class RedirectPlaceListPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(PlaceHierarchyListModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectRelationshipPhpTest.php b/tests/app/Http/RequestHandlers/RedirectRelationshipPhpTest.php index 1eab91ac61..ab153de190 100644 --- a/tests/app/Http/RequestHandlers/RedirectRelationshipPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectRelationshipPhpTest.php @@ -37,9 +37,8 @@ use Illuminate\Support\Collection; */ class RedirectRelationshipPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -90,9 +89,6 @@ class RedirectRelationshipPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -121,9 +117,6 @@ class RedirectRelationshipPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(RelationshipsChartModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectRepoListPhpTest.php b/tests/app/Http/RequestHandlers/RedirectRepoListPhpTest.php index 4659f09e87..62a819d450 100644 --- a/tests/app/Http/RequestHandlers/RedirectRepoListPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectRepoListPhpTest.php @@ -34,9 +34,8 @@ use Illuminate\Support\Collection; */ class RedirectRepoListPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -73,9 +72,6 @@ class RedirectRepoListPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -101,9 +97,6 @@ class RedirectRepoListPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(RepositoryListModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectReportEnginePhpTest.php b/tests/app/Http/RequestHandlers/RedirectReportEnginePhpTest.php index fd28cc971a..67a6c9de9b 100644 --- a/tests/app/Http/RequestHandlers/RedirectReportEnginePhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectReportEnginePhpTest.php @@ -32,9 +32,8 @@ use Illuminate\Support\Collection; */ class RedirectReportEnginePhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -67,9 +66,6 @@ class RedirectReportEnginePhpTest extends TestCase ); } - /** - * @return void - */ public function testNoSuchTree(): void { $tree_service = $this->createStub(TreeService::class); diff --git a/tests/app/Http/RequestHandlers/RedirectRepositoryPhpTest.php b/tests/app/Http/RequestHandlers/RedirectRepositoryPhpTest.php index 8bb3ff6a4b..d6ae8ddd1e 100644 --- a/tests/app/Http/RequestHandlers/RedirectRepositoryPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectRepositoryPhpTest.php @@ -36,9 +36,8 @@ use Illuminate\Support\Collection; */ class RedirectRepositoryPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -79,9 +78,6 @@ class RedirectRepositoryPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testNoSuchRecord(): void { $tree = $this->createStub(Tree::class); @@ -104,9 +100,6 @@ class RedirectRepositoryPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testMissingXrefParameter(): void { $tree_service = $this->createStub(TreeService::class); diff --git a/tests/app/Http/RequestHandlers/RedirectSourceListPhpTest.php b/tests/app/Http/RequestHandlers/RedirectSourceListPhpTest.php index 14ba060191..0eb23b0d1a 100644 --- a/tests/app/Http/RequestHandlers/RedirectSourceListPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectSourceListPhpTest.php @@ -34,9 +34,8 @@ use Illuminate\Support\Collection; */ class RedirectSourceListPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -73,9 +72,6 @@ class RedirectSourceListPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -101,9 +97,6 @@ class RedirectSourceListPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(SourceListModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectSourcePhpTest.php b/tests/app/Http/RequestHandlers/RedirectSourcePhpTest.php index fb775bd90f..3a2bc211ba 100644 --- a/tests/app/Http/RequestHandlers/RedirectSourcePhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectSourcePhpTest.php @@ -36,9 +36,8 @@ use Illuminate\Support\Collection; */ class RedirectSourcePhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -79,9 +78,6 @@ class RedirectSourcePhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testNoSuchRecord(): void { $tree = $this->createStub(Tree::class); @@ -104,9 +100,6 @@ class RedirectSourcePhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testMissingXrefParameter(): void { $tree_service = $this->createStub(TreeService::class); diff --git a/tests/app/Http/RequestHandlers/RedirectStatisticsPhpTest.php b/tests/app/Http/RequestHandlers/RedirectStatisticsPhpTest.php index 36a071daee..33be8d9c99 100644 --- a/tests/app/Http/RequestHandlers/RedirectStatisticsPhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectStatisticsPhpTest.php @@ -34,9 +34,8 @@ use Illuminate\Support\Collection; */ class RedirectStatisticsPhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -73,9 +72,6 @@ class RedirectStatisticsPhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -101,9 +97,6 @@ class RedirectStatisticsPhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(StatisticsChartModule::class); diff --git a/tests/app/Http/RequestHandlers/RedirectTimeLinePhpTest.php b/tests/app/Http/RequestHandlers/RedirectTimeLinePhpTest.php index 68e4dea6fb..8abc4b661a 100644 --- a/tests/app/Http/RequestHandlers/RedirectTimeLinePhpTest.php +++ b/tests/app/Http/RequestHandlers/RedirectTimeLinePhpTest.php @@ -37,9 +37,8 @@ use Illuminate\Support\Collection; */ class RedirectTimeLinePhpTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testRedirect(): void { $tree = $this->createStub(Tree::class); @@ -90,9 +89,6 @@ class RedirectTimeLinePhpTest extends TestCase self::assertSame('https://www.example.com', $response->getHeaderLine('Location')); } - /** - * @return void - */ public function testModuleDisabled(): void { $module_service = $this->createStub(ModuleService::class); @@ -121,9 +117,6 @@ class RedirectTimeLinePhpTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testNoSuchTree(): void { $module = $this->createStub(TimelineChartModule::class); diff --git a/tests/app/Http/RequestHandlers/SelectLanguageTest.php b/tests/app/Http/RequestHandlers/SelectLanguageTest.php index ec3acf4930..f6c4108069 100644 --- a/tests/app/Http/RequestHandlers/SelectLanguageTest.php +++ b/tests/app/Http/RequestHandlers/SelectLanguageTest.php @@ -29,9 +29,8 @@ use Fisharebest\Webtrees\TestCase; */ class SelectLanguageTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testSelectLanguageForGuest(): void { $user = new GuestUser(); @@ -44,9 +43,6 @@ class SelectLanguageTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_NO_CONTENT, $response->getStatusCode()); } - /** - * @return void - */ public function testSelectLanguageForUser(): void { $user_service = new UserService(); diff --git a/tests/app/Http/RequestHandlers/SelectThemeTest.php b/tests/app/Http/RequestHandlers/SelectThemeTest.php index e6b031b6ed..5e44500ec6 100644 --- a/tests/app/Http/RequestHandlers/SelectThemeTest.php +++ b/tests/app/Http/RequestHandlers/SelectThemeTest.php @@ -29,9 +29,6 @@ use Fisharebest\Webtrees\User; */ class SelectThemeTest extends TestCase { - /** - * @return void - */ public function testSelectThemeForGuest(): void { $user = $this->createMock(GuestUser::class); @@ -47,9 +44,6 @@ class SelectThemeTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_NO_CONTENT, $response->getStatusCode()); } - /** - * @return void - */ public function testSelectThemeForUser(): void { $user = $this->createMock(User::class); diff --git a/tests/app/Http/RequestHandlers/SiteLogsDataTest.php b/tests/app/Http/RequestHandlers/SiteLogsDataTest.php index f6fa31a000..ff9c801254 100644 --- a/tests/app/Http/RequestHandlers/SiteLogsDataTest.php +++ b/tests/app/Http/RequestHandlers/SiteLogsDataTest.php @@ -32,9 +32,6 @@ use Psr\Http\Message\ResponseInterface; */ class SiteLogsDataTest extends TestCase { - /** - * @return void - */ public function testResponse(): void { $request = self::createRequest( diff --git a/tests/app/Http/RequestHandlers/SiteLogsDeleteTest.php b/tests/app/Http/RequestHandlers/SiteLogsDeleteTest.php index e5159cb402..502dfd68d9 100644 --- a/tests/app/Http/RequestHandlers/SiteLogsDeleteTest.php +++ b/tests/app/Http/RequestHandlers/SiteLogsDeleteTest.php @@ -29,9 +29,6 @@ use Illuminate\Database\Query\Builder; */ class SiteLogsDeleteTest extends TestCase { - /** - * @return void - */ public function testResponse(): void { $request = self::createRequest(); diff --git a/tests/app/Http/RequestHandlers/SiteLogsDownloadTest.php b/tests/app/Http/RequestHandlers/SiteLogsDownloadTest.php index 039a638d91..9f45f10dc6 100644 --- a/tests/app/Http/RequestHandlers/SiteLogsDownloadTest.php +++ b/tests/app/Http/RequestHandlers/SiteLogsDownloadTest.php @@ -30,9 +30,6 @@ use Illuminate\Support\Collection; */ class SiteLogsDownloadTest extends TestCase { - /** - * @return void - */ public function testResponse(): void { $request = self::createRequest(); diff --git a/tests/app/Http/RequestHandlers/SiteLogsPageTest.php b/tests/app/Http/RequestHandlers/SiteLogsPageTest.php index f5f268dd05..bbc331801d 100644 --- a/tests/app/Http/RequestHandlers/SiteLogsPageTest.php +++ b/tests/app/Http/RequestHandlers/SiteLogsPageTest.php @@ -30,9 +30,8 @@ use Illuminate\Support\Collection; */ class SiteLogsPageTest extends TestCase { - /** - * @return void - */ + protected static bool $uses_database = true; + public function testResponse(): void { $request = self::createRequest(); diff --git a/tests/app/Http/RequestHandlers/UpgradeWizardConfirmTest.php b/tests/app/Http/RequestHandlers/UpgradeWizardConfirmTest.php index fda3d37a3f..231d32efcc 100644 --- a/tests/app/Http/RequestHandlers/UpgradeWizardConfirmTest.php +++ b/tests/app/Http/RequestHandlers/UpgradeWizardConfirmTest.php @@ -29,9 +29,6 @@ use Fisharebest\Webtrees\TestCase; */ class UpgradeWizardConfirmTest extends TestCase { - /** - * @return void - */ public function testWizardContinue(): void { $handler = new UpgradeWizardConfirm(); diff --git a/tests/app/Http/RequestHandlers/UpgradeWizardPageTest.php b/tests/app/Http/RequestHandlers/UpgradeWizardPageTest.php index e4b8b3a258..ea6090d37a 100644 --- a/tests/app/Http/RequestHandlers/UpgradeWizardPageTest.php +++ b/tests/app/Http/RequestHandlers/UpgradeWizardPageTest.php @@ -35,9 +35,6 @@ class UpgradeWizardPageTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testWizard(): void { $timeout_service = new TimeoutService(); diff --git a/tests/app/Http/RequestHandlers/UpgradeWizardStepTest.php b/tests/app/Http/RequestHandlers/UpgradeWizardStepTest.php index 6e73e31385..d940e2c7dd 100644 --- a/tests/app/Http/RequestHandlers/UpgradeWizardStepTest.php +++ b/tests/app/Http/RequestHandlers/UpgradeWizardStepTest.php @@ -43,9 +43,6 @@ class UpgradeWizardStepTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testIgnoreStepInvalid(): void { $handler = new UpgradeWizardStep( @@ -61,9 +58,6 @@ class UpgradeWizardStepTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_NO_CONTENT, $response->getStatusCode()); } - /** - * @return void - */ public function testStepCheckOK(): void { $mock_upgrade_service = $this->createMock(UpgradeService::class); @@ -80,9 +74,6 @@ class UpgradeWizardStepTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_OK, $response->getStatusCode()); } - /** - * @return void - */ public function testStepCheckUnavailable(): void { $this->expectException(HttpServerErrorException::class); @@ -99,9 +90,6 @@ class UpgradeWizardStepTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testStepCheckFail(): void { $this->expectException(HttpServerErrorException::class); @@ -118,9 +106,6 @@ class UpgradeWizardStepTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testStepPrepare(): void { $handler = new UpgradeWizardStep( @@ -135,9 +120,6 @@ class UpgradeWizardStepTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_OK, $response->getStatusCode()); } - /** - * @return void - */ public function testStepPending(): void { $handler = new UpgradeWizardStep( @@ -152,9 +134,6 @@ class UpgradeWizardStepTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_OK, $response->getStatusCode()); } - /** - * @return void - */ public function testStepPendingExist(): void { $tree_service = new TreeService(new GedcomImportService()); @@ -176,9 +155,6 @@ class UpgradeWizardStepTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_INTERNAL_SERVER_ERROR, $response->getStatusCode()); } - /** - * @return void - */ public function testStepExport(): void { $tree = $this->importTree('demo.ged'); @@ -203,9 +179,6 @@ class UpgradeWizardStepTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_OK, $response->getStatusCode()); } - /** - * @return void - */ public function testStepDownloadFails(): void { $this->expectException(HttpServerErrorException::class); @@ -222,9 +195,6 @@ class UpgradeWizardStepTest extends TestCase $handler->handle($request); } - /** - * @return void - */ public function testStepDownload(): void { $mock_upgrade_service = $this->createMock(UpgradeService::class); @@ -241,9 +211,6 @@ class UpgradeWizardStepTest extends TestCase self::assertSame(StatusCodeInterface::STATUS_OK, $response->getStatusCode()); } - /** - * @return void - */ public function testStepUnzip(): void { $mock_upgrade_service = $this->createMock(UpgradeService::class); diff --git a/tests/app/Http/RequestHandlers/UploadMediaActionTest.php b/tests/app/Http/RequestHandlers/UploadMediaActionTest.php index 71af2b80d1..5847cea28e 100644 --- a/tests/app/Http/RequestHandlers/UploadMediaActionTest.php +++ b/tests/app/Http/RequestHandlers/UploadMediaActionTest.php @@ -32,9 +32,6 @@ class UploadMediaActionTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testResponseIsOK(): void { $media_file_service = new MediaFileService(); diff --git a/tests/app/Http/RequestHandlers/UploadMediaPageTest.php b/tests/app/Http/RequestHandlers/UploadMediaPageTest.php index 3647da6377..837f0338fb 100644 --- a/tests/app/Http/RequestHandlers/UploadMediaPageTest.php +++ b/tests/app/Http/RequestHandlers/UploadMediaPageTest.php @@ -32,9 +32,6 @@ class UploadMediaPageTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testResponseIsOK(): void { $media_file_service = new MediaFileService(); diff --git a/tests/app/Http/RequestHandlers/UserAddActionTest.php b/tests/app/Http/RequestHandlers/UserAddActionTest.php index 9ddb2a0daa..e02d93fde5 100644 --- a/tests/app/Http/RequestHandlers/UserAddActionTest.php +++ b/tests/app/Http/RequestHandlers/UserAddActionTest.php @@ -33,9 +33,6 @@ class UserAddActionTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testHandler(): void { $user_service = new UserService(); diff --git a/tests/app/Http/RequestHandlers/UserAddPageTest.php b/tests/app/Http/RequestHandlers/UserAddPageTest.php index 21a82a6242..b5d09bbe11 100644 --- a/tests/app/Http/RequestHandlers/UserAddPageTest.php +++ b/tests/app/Http/RequestHandlers/UserAddPageTest.php @@ -31,9 +31,6 @@ class UserAddPageTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testHandler(): void { $handler = new UserAddPage(); diff --git a/tests/app/Http/RequestHandlers/UserEditActionTest.php b/tests/app/Http/RequestHandlers/UserEditActionTest.php index fe92cf5c4f..a1798e3421 100644 --- a/tests/app/Http/RequestHandlers/UserEditActionTest.php +++ b/tests/app/Http/RequestHandlers/UserEditActionTest.php @@ -36,9 +36,6 @@ class UserEditActionTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testHandler(): void { $mail_service = new EmailService(); diff --git a/tests/app/Http/RequestHandlers/UserEditPageTest.php b/tests/app/Http/RequestHandlers/UserEditPageTest.php index 9694287e9b..513c8cf951 100644 --- a/tests/app/Http/RequestHandlers/UserEditPageTest.php +++ b/tests/app/Http/RequestHandlers/UserEditPageTest.php @@ -38,9 +38,6 @@ class UserEditPageTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testHandler(): void { $mail_service = new EmailService(); diff --git a/tests/app/Http/RequestHandlers/UserListDataTest.php b/tests/app/Http/RequestHandlers/UserListDataTest.php index c90122d4ca..63d1944ca6 100644 --- a/tests/app/Http/RequestHandlers/UserListDataTest.php +++ b/tests/app/Http/RequestHandlers/UserListDataTest.php @@ -34,9 +34,6 @@ class UserListDataTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testHandler(): void { $datatables_service = new DatatablesService(); diff --git a/tests/app/Http/RequestHandlers/UserListTest.php b/tests/app/Http/RequestHandlers/UserListTest.php index 2cc72cddbf..4aa8bd8b9e 100644 --- a/tests/app/Http/RequestHandlers/UserListTest.php +++ b/tests/app/Http/RequestHandlers/UserListTest.php @@ -32,9 +32,6 @@ class UserListTest extends TestCase { protected static bool $uses_database = true; - /** - * @return void - */ public function testHandler(): void { $handler = new UserListPage(); |