@@ -16,8 +16,7 @@ |
||
| 16 | 16 | use Kitodo\Dlf\Domain\Repository\MailRepository; |
| 17 | 17 | use Kitodo\Dlf\Tests\Functional\FunctionalTestCase; |
| 18 | 18 | |
| 19 | -class MailRepositoryTest extends FunctionalTestCase |
|
| 20 | -{ |
|
| 19 | +class MailRepositoryTest extends FunctionalTestCase { |
|
| 21 | 20 | /** |
| 22 | 21 | * @var MailRepository |
| 23 | 22 | */ |
@@ -16,8 +16,7 @@ discard block |
||
| 16 | 16 | use Kitodo\Dlf\Domain\Repository\MetadataRepository; |
| 17 | 17 | use Kitodo\Dlf\Tests\Functional\FunctionalTestCase; |
| 18 | 18 | |
| 19 | -class MetadataRepositoryTest extends FunctionalTestCase |
|
| 20 | -{ |
|
| 19 | +class MetadataRepositoryTest extends FunctionalTestCase { |
|
| 21 | 20 | /** |
| 22 | 21 | * @var MetadataRepository |
| 23 | 22 | */ |
@@ -40,8 +39,7 @@ discard block |
||
| 40 | 39 | * @param $settings |
| 41 | 40 | * @return array |
| 42 | 41 | */ |
| 43 | - protected function findBySettings($settings) |
|
| 44 | - { |
|
| 42 | + protected function findBySettings($settings) { |
|
| 45 | 43 | $metadata = $this->metadataRepository->findBySettings($settings); |
| 46 | 44 | self::assertNotNull($metadata); |
| 47 | 45 | self::assertInstanceOf(QueryResult::class, $metadata); |
@@ -21,8 +21,7 @@ discard block |
||
| 21 | 21 | use Phpoaipmh\Exception\OaipmhException; |
| 22 | 22 | use SimpleXMLElement; |
| 23 | 23 | |
| 24 | -class OaiPmhTest extends FunctionalTestCase |
|
| 25 | -{ |
|
| 24 | +class OaiPmhTest extends FunctionalTestCase { |
|
| 26 | 25 | protected $disableJsonWrappedResponse = true; |
| 27 | 26 | |
| 28 | 27 | protected $coreExtensionsToLoad = [ |
@@ -66,8 +65,7 @@ discard block |
||
| 66 | 65 | $this->setUpOaiSolr(); |
| 67 | 66 | } |
| 68 | 67 | |
| 69 | - protected function setUpOaiSolr() |
|
| 70 | - { |
|
| 68 | + protected function setUpOaiSolr() { |
|
| 71 | 69 | // Setup Solr only once for all tests in this suite |
| 72 | 70 | static $solr = null; |
| 73 | 71 | |
@@ -87,8 +85,7 @@ discard block |
||
| 87 | 85 | /** |
| 88 | 86 | * @test |
| 89 | 87 | */ |
| 90 | - public function correctlyRespondsOnBadVerb() |
|
| 91 | - { |
|
| 88 | + public function correctlyRespondsOnBadVerb() { |
|
| 92 | 89 | $client = new HttpClient(); |
| 93 | 90 | $response = $client->get($this->baseUrl, [ |
| 94 | 91 | 'query' => [ |
@@ -113,8 +110,7 @@ discard block |
||
| 113 | 110 | /** |
| 114 | 111 | * @test |
| 115 | 112 | */ |
| 116 | - public function canIdentify() |
|
| 117 | - { |
|
| 113 | + public function canIdentify() { |
|
| 118 | 114 | $oai = Endpoint::build($this->oaiUrl); |
| 119 | 115 | $identity = $oai->identify(); |
| 120 | 116 | |
@@ -127,8 +123,7 @@ discard block |
||
| 127 | 123 | /** |
| 128 | 124 | * @test |
| 129 | 125 | */ |
| 130 | - public function identifyGivesFallbackDatestampWhenNoDocuments() |
|
| 131 | - { |
|
| 126 | + public function identifyGivesFallbackDatestampWhenNoDocuments() { |
|
| 132 | 127 | $oai = Endpoint::build($this->oaiUrlNoStoragePid); |
| 133 | 128 | $identity = $oai->identify(); |
| 134 | 129 | |
@@ -138,8 +133,7 @@ discard block |
||
| 138 | 133 | /** |
| 139 | 134 | * @test |
| 140 | 135 | */ |
| 141 | - public function canListMetadataFormats() |
|
| 142 | - { |
|
| 136 | + public function canListMetadataFormats() { |
|
| 143 | 137 | $oai = Endpoint::build($this->oaiUrl); |
| 144 | 138 | $formats = $oai->listMetadataFormats(); |
| 145 | 139 | |
@@ -154,8 +148,7 @@ discard block |
||
| 154 | 148 | /** |
| 155 | 149 | * @test |
| 156 | 150 | */ |
| 157 | - public function canListRecords() |
|
| 158 | - { |
|
| 151 | + public function canListRecords() { |
|
| 159 | 152 | $oai = Endpoint::build($this->oaiUrl); |
| 160 | 153 | $result = $oai->listRecords('mets'); |
| 161 | 154 | |
@@ -168,8 +161,7 @@ discard block |
||
| 168 | 161 | /** |
| 169 | 162 | * @test |
| 170 | 163 | */ |
| 171 | - public function noRecordsUntil1900() |
|
| 172 | - { |
|
| 164 | + public function noRecordsUntil1900() { |
|
| 173 | 165 | $this->expectException(OaipmhException::class); |
| 174 | 166 | $this->expectExceptionMessage('empty list'); |
| 175 | 167 | |
@@ -182,8 +174,7 @@ discard block |
||
| 182 | 174 | /** |
| 183 | 175 | * @test |
| 184 | 176 | */ |
| 185 | - public function canUseResumptionToken() |
|
| 186 | - { |
|
| 177 | + public function canUseResumptionToken() { |
|
| 187 | 178 | // NOTE: cursor and expirationDate are optional by the specification, |
| 188 | 179 | // but we include them in our implementation |
| 189 | 180 | |
@@ -241,8 +232,7 @@ discard block |
||
| 241 | 232 | /** |
| 242 | 233 | * @test |
| 243 | 234 | */ |
| 244 | - public function noResumptionTokenForCompleteList() |
|
| 245 | - { |
|
| 235 | + public function noResumptionTokenForCompleteList() { |
|
| 246 | 236 | $client = new HttpClient(); |
| 247 | 237 | |
| 248 | 238 | foreach (['ListIdentifiers', 'ListRecords'] as $verb) { |
@@ -264,8 +254,7 @@ discard block |
||
| 264 | 254 | /** |
| 265 | 255 | * @test |
| 266 | 256 | */ |
| 267 | - public function canListAndResumeIdentifiers() |
|
| 268 | - { |
|
| 257 | + public function canListAndResumeIdentifiers() { |
|
| 269 | 258 | $oai = Endpoint::build($this->oaiUrl); |
| 270 | 259 | $result = $oai->listIdentifiers('mets'); |
| 271 | 260 | |
@@ -278,18 +267,15 @@ discard block |
||
| 278 | 267 | self::assertEquals('oai:de:slub-dresden:db:id-476248086', $record->identifier); |
| 279 | 268 | } |
| 280 | 269 | |
| 281 | - protected function parseUtc(string $dateTime) |
|
| 282 | - { |
|
| 270 | + protected function parseUtc(string $dateTime) { |
|
| 283 | 271 | return DateTime::createFromFormat('Y-m-d\TH:i:s\Z', $dateTime); |
| 284 | 272 | } |
| 285 | 273 | |
| 286 | - protected function assertUtcDateString(string $dateTime) |
|
| 287 | - { |
|
| 274 | + protected function assertUtcDateString(string $dateTime) { |
|
| 288 | 275 | self::assertInstanceOf(DateTime::class, $this->parseUtc($dateTime)); |
| 289 | 276 | } |
| 290 | 277 | |
| 291 | - protected function assertInFuture(string $dateTime) |
|
| 292 | - { |
|
| 278 | + protected function assertInFuture(string $dateTime) { |
|
| 293 | 279 | self::assertGreaterThan(new DateTime(), $this->parseUtc($dateTime)); |
| 294 | 280 | } |
| 295 | 281 | } |
@@ -20,8 +20,7 @@ |
||
| 20 | 20 | /** |
| 21 | 21 | * @covers JsFooterViewHelper |
| 22 | 22 | */ |
| 23 | -class JsFooterViewHelperTest extends FunctionalTestCase |
|
| 24 | -{ |
|
| 23 | +class JsFooterViewHelperTest extends FunctionalTestCase { |
|
| 25 | 24 | /** |
| 26 | 25 | * @var bool Speed up this test case, it needs no database |
| 27 | 26 | */ |
@@ -18,8 +18,7 @@ |
||
| 18 | 18 | /** |
| 19 | 19 | * @covers MetadataWrapVariableViewHelper |
| 20 | 20 | */ |
| 21 | -class MetadataWrapVariableViewHelperTest extends FunctionalTestCase |
|
| 22 | -{ |
|
| 21 | +class MetadataWrapVariableViewHelperTest extends FunctionalTestCase { |
|
| 23 | 22 | /** |
| 24 | 23 | * @var bool Speed up this test case, it needs no database |
| 25 | 24 | */ |
@@ -18,8 +18,7 @@ |
||
| 18 | 18 | /** |
| 19 | 19 | * @covers StdWrapViewHelper |
| 20 | 20 | */ |
| 21 | -class StdWrapViewHelperTest extends FunctionalTestCase |
|
| 22 | -{ |
|
| 21 | +class StdWrapViewHelperTest extends FunctionalTestCase { |
|
| 23 | 22 | /** |
| 24 | 23 | * @var bool Speed up this test case, it needs no database |
| 25 | 24 | */ |
@@ -15,8 +15,7 @@ discard block |
||
| 15 | 15 | use Kitodo\Dlf\Common\Helper; |
| 16 | 16 | use Kitodo\Dlf\Tests\Functional\FunctionalTestCase; |
| 17 | 17 | |
| 18 | -class HelperTest extends FunctionalTestCase |
|
| 19 | -{ |
|
| 18 | +class HelperTest extends FunctionalTestCase { |
|
| 20 | 19 | public function setUp(): void |
| 21 | 20 | { |
| 22 | 21 | parent::setUp(); |
@@ -28,8 +27,7 @@ discard block |
||
| 28 | 27 | /** |
| 29 | 28 | * @test |
| 30 | 29 | */ |
| 31 | - public function canGetIndexNameFromUid() |
|
| 32 | - { |
|
| 30 | + public function canGetIndexNameFromUid() { |
|
| 33 | 31 | // Repeat to make sure caching isn't broken |
| 34 | 32 | for ($n = 0; $n < 2; $n++) { |
| 35 | 33 | // Good UID, no PID |
@@ -78,8 +76,7 @@ discard block |
||
| 78 | 76 | * @test |
| 79 | 77 | * @group getLanguageName |
| 80 | 78 | */ |
| 81 | - public function canTranslateLanguageNameToEnglish() |
|
| 82 | - { |
|
| 79 | + public function canTranslateLanguageNameToEnglish() { |
|
| 83 | 80 | // NOTE: This only tests in BE mode |
| 84 | 81 | |
| 85 | 82 | $this->initLanguageService('default'); |
@@ -93,8 +90,7 @@ discard block |
||
| 93 | 90 | * @test |
| 94 | 91 | * @group getLanguageName |
| 95 | 92 | */ |
| 96 | - public function canTranslateLanguageNameToGerman() |
|
| 97 | - { |
|
| 93 | + public function canTranslateLanguageNameToGerman() { |
|
| 98 | 94 | // NOTE: This only tests in BE mode |
| 99 | 95 | |
| 100 | 96 | $this->initLanguageService('de'); |
@@ -15,14 +15,12 @@ |
||
| 15 | 15 | use Kitodo\Dlf\Common\IiifUrlReader; |
| 16 | 16 | use Kitodo\Dlf\Tests\Functional\FunctionalTestCase; |
| 17 | 17 | |
| 18 | -class IiifUrlReaderTest extends FunctionalTestCase |
|
| 19 | -{ |
|
| 18 | +class IiifUrlReaderTest extends FunctionalTestCase { |
|
| 20 | 19 | /** |
| 21 | 20 | * @test |
| 22 | 21 | * @group getContent |
| 23 | 22 | */ |
| 24 | - public function getContentCheck() |
|
| 25 | - { |
|
| 23 | + public function getContentCheck() { |
|
| 26 | 24 | $iiifUrlReader = new IiifUrlReader(); |
| 27 | 25 | |
| 28 | 26 | $correctUrl = 'http://web:8001/Tests/Fixtures/Common/correct.txt'; |
@@ -18,8 +18,7 @@ discard block |
||
| 18 | 18 | use Kitodo\Dlf\Domain\Repository\SolrCoreRepository; |
| 19 | 19 | use Kitodo\Dlf\Tests\Functional\FunctionalTestCase; |
| 20 | 20 | |
| 21 | -class SolrSearchTest extends FunctionalTestCase |
|
| 22 | -{ |
|
| 21 | +class SolrSearchTest extends FunctionalTestCase { |
|
| 23 | 22 | private static array $databaseFixtures = [ |
| 24 | 23 | __DIR__ . '/../../Fixtures/Common/solrcores.csv' |
| 25 | 24 | ]; |
@@ -41,8 +40,7 @@ discard block |
||
| 41 | 40 | /** |
| 42 | 41 | * @test |
| 43 | 42 | */ |
| 44 | - public function canPrepareAndSubmit() |
|
| 45 | - { |
|
| 43 | + public function canPrepareAndSubmit() { |
|
| 46 | 44 | $documentRepository = $this->initializeRepository(DocumentRepository::class, 0); |
| 47 | 45 | $solrCoreName = $this->solrCoreRepository->findByUid(5)->getIndexName(); |
| 48 | 46 | $settings = ['solrcore' => $solrCoreName, 'storagePid' => 0]; |
@@ -80,8 +78,7 @@ discard block |
||
| 80 | 78 | $this->initializeRepository(DocumentRepository::class, 0); |
| 81 | 79 | } |
| 82 | 80 | |
| 83 | - protected function setUpSolr($uid, $storagePid, $solrFixtures) |
|
| 84 | - { |
|
| 81 | + protected function setUpSolr($uid, $storagePid, $solrFixtures) { |
|
| 85 | 82 | $this->solrCoreRepository = $this->initializeRepository(SolrCoreRepository::class, $storagePid); |
| 86 | 83 | |
| 87 | 84 | // Setup Solr only once for all tests in this suite |