@@ -15,8 +15,7 @@ |
||
15 | 15 | use Kitodo\Dlf\Domain\Repository\TokenRepository; |
16 | 16 | use Kitodo\Dlf\Tests\Functional\FunctionalTestCase; |
17 | 17 | |
18 | -class TokenRepositoryTest extends FunctionalTestCase |
|
19 | -{ |
|
18 | +class TokenRepositoryTest extends FunctionalTestCase { |
|
20 | 19 | /** |
21 | 20 | * @var TokenRepository |
22 | 21 | */ |
@@ -16,8 +16,7 @@ |
||
16 | 16 | use Kitodo\Dlf\Domain\Repository\CollectionRepository; |
17 | 17 | use Kitodo\Dlf\Tests\Functional\FunctionalTestCase; |
18 | 18 | |
19 | -class CollectionRepositoryTest extends FunctionalTestCase |
|
20 | -{ |
|
19 | +class CollectionRepositoryTest extends FunctionalTestCase { |
|
21 | 20 | /** |
22 | 21 | * @var CollectionRepository |
23 | 22 | */ |
@@ -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'); |