Passed
Pull Request — master (#123)
by
unknown
04:23
created
Tests/Functional/Repository/TokenRepositoryTest.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -15,8 +15,7 @@
 block discarded – undo
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
      */
Please login to merge, or discard this patch.
Tests/Functional/Repository/CollectionRepositoryTest.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -16,8 +16,7 @@
 block discarded – undo
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
      */
Please login to merge, or discard this patch.
Tests/Functional/Repository/MailRepositoryTest.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -16,8 +16,7 @@
 block discarded – undo
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
      */
Please login to merge, or discard this patch.
Tests/Functional/Repository/MetatdataRepositoryTest.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -16,8 +16,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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);
Please login to merge, or discard this patch.
Tests/Functional/Api/OaiPmhTest.php 1 patch
Braces   +14 added lines, -28 removed lines patch added patch discarded remove patch
@@ -21,8 +21,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 }
Please login to merge, or discard this patch.
Tests/Functional/ViewHelpers/JsFooterViewHelperTest.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -20,8 +20,7 @@
 block discarded – undo
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
      */
Please login to merge, or discard this patch.
Tests/Functional/ViewHelpers/MetadataWrapVariableViewHelperTest.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -18,8 +18,7 @@
 block discarded – undo
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
      */
Please login to merge, or discard this patch.
Tests/Functional/ViewHelpers/StdWrapViewHelperTest.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -18,8 +18,7 @@
 block discarded – undo
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
      */
Please login to merge, or discard this patch.
Tests/Functional/Common/HelperTest.php 1 patch
Braces   +4 added lines, -8 removed lines patch added patch discarded remove patch
@@ -15,8 +15,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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');
Please login to merge, or discard this patch.