Skip to content

Commit 094eba0

Browse files
d-s-codappixd-s-codappix
committed
Increase Test coverage
In order to ensure that every single line works with TYPO3 version.
1 parent 8d84d74 commit 094eba0

6 files changed

Lines changed: 297 additions & 3 deletions

File tree

‎Classes/Domain/Model/Marker.php‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ public function __construct(
2121
public int $uid,
2222
public string $key,
2323
public string $replacement,
24-
// public ?int $smallestValueFromTimeFields,
2524
) {
2625
}
2726

‎Tests/Functional/Caching/FlushViaDataHandlerChangesTest.php‎

Lines changed: 92 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,14 +38,25 @@
3838
#[CoversClass(DataHandler::class)]
3939
class FlushViaDataHandlerChangesTest extends FunctionalTestCase
4040
{
41+
protected array $testExtensionsToLoad = [
42+
'typo3conf/ext/variables',
43+
];
44+
45+
protected function setUp(): void
46+
{
47+
parent::setUp();
48+
49+
$this->importCSVDataSet(__DIR__ . '/../Fixtures/Frontend/Content.csv');
50+
}
51+
4152
/**
4253
* @param array<mixed> $params
4354
*/
4455
#[DataProvider('possibleNoneTriggeringParams')]
4556
#[Test]
4657
public function doesNotInteractWithCacheManagerOnUnkownData(array $params): void
4758
{
48-
$connectionPool = self::createMock(ConnectionPool::class);
59+
$connectionPool = $this->get(ConnectionPool::class);
4960
$cacheManager = self::createMock(CacheManager::class);
5061
$cacheManager
5162
->expects(self::never())
@@ -81,7 +92,7 @@ public static function possibleNoneTriggeringParams(): \Generator
8192
#[Test]
8293
public function flushCachesByGroupForMarker(): void
8394
{
84-
$connectionPool = self::createMock(ConnectionPool::class);
95+
$connectionPool = $this->get(ConnectionPool::class);
8596
$cacheManager = self::createMock(CacheManager::class);
8697
$cacheManager
8798
->expects(self::once())
@@ -105,4 +116,83 @@ public function flushCachesByGroupForMarker(): void
105116
'marker' => 'TEST'
106117
], $dataHandler);
107118
}
119+
120+
#[Test]
121+
public function flushCachesByGroupForHiddenMarker(): void
122+
{
123+
$connectionPool = $this->get(ConnectionPool::class);
124+
$connectionPool->getConnectionForTable('tx_variables_marker')
125+
->insert('tx_variables_marker',[
126+
'uid' => '100',
127+
'hidden' => '1',
128+
'marker' => 'hidden_marker',
129+
'replacement' => 'replacement',
130+
]);
131+
132+
$cacheManager = self::createMock(CacheManager::class);
133+
$cacheManager
134+
->expects(self::once())
135+
->method('flushCachesInGroupByTag')
136+
->with('pages', $this->stringStartsWith('tx_variables_key_hash_'))
137+
;
138+
139+
$dataHandler = self::createStub(Typo3DataHandler::class);
140+
141+
$subject = new DataHandler($connectionPool, $cacheManager);
142+
$subject->clearCachePostProc([
143+
'table' => 'tx_variables_marker',
144+
'uid' => '100',
145+
'marker' => 'TEST'
146+
], $dataHandler);
147+
}
148+
149+
#[Test]
150+
public function flushCachesByGroupForDeletedMarker(): void
151+
{
152+
$connectionPool = $this->get(ConnectionPool::class);
153+
$connectionPool->getConnectionForTable('tx_variables_marker')
154+
->insert('tx_variables_marker',[
155+
'uid' => '100',
156+
'deleted' => '1',
157+
'marker' => 'hidden_marker',
158+
'replacement' => 'replacement',
159+
]);
160+
161+
$cacheManager = self::createMock(CacheManager::class);
162+
$cacheManager
163+
->expects(self::once())
164+
->method('flushCachesInGroupByTag')
165+
->with('pages', $this->stringStartsWith('tx_variables_key_hash_'))
166+
;
167+
168+
$dataHandler = self::createStub(Typo3DataHandler::class);
169+
170+
$subject = new DataHandler($connectionPool, $cacheManager);
171+
$subject->clearCachePostProc([
172+
'table' => 'tx_variables_marker',
173+
'uid' => '100',
174+
'marker' => 'TEST'
175+
], $dataHandler);
176+
}
177+
178+
#[Test]
179+
public function doesNotFlushCachesByGroupForUnkownMarker(): void
180+
{
181+
$connectionPool = $this->get(ConnectionPool::class);
182+
183+
$cacheManager = self::createMock(CacheManager::class);
184+
$cacheManager
185+
->expects(self::never())
186+
->method('flushCachesInGroupByTag')
187+
;
188+
189+
$dataHandler = self::createStub(Typo3DataHandler::class);
190+
191+
$subject = new DataHandler($connectionPool, $cacheManager);
192+
$subject->clearCachePostProc([
193+
'table' => 'tx_variables_marker',
194+
'uid' => '100',
195+
'marker' => 'TEST'
196+
], $dataHandler);
197+
}
108198
}

‎Tests/Unit/Domain/Model/MarkerCollectionTest.php‎

Lines changed: 81 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,81 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
/*
6+
* Copyright (C) 2025 Daniel Siepmann <daniel.siepmann@codappix.com>
7+
*
8+
* This program is free software; you can redistribute it and/or
9+
* modify it under the terms of the GNU General Public License
10+
* as published by the Free Software Foundation; either version 2
11+
* of the License, or (at your option) any later version.
12+
*
13+
* This program is distributed in the hope that it will be useful,
14+
* but WITHOUT ANY WARRANTY; without even the implied warranty of
15+
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16+
* GNU General Public License for more details.
17+
*
18+
* You should have received a copy of the GNU General Public License
19+
* along with this program; if not, write to the Free Software
20+
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
21+
* 02110-1301, USA.
22+
*/
23+
24+
namespace Sinso\Variables\Tests\Unit\Domain\Model;
25+
26+
use RuntimeException;
27+
use Sinso\Variables\Domain\Model\Marker;
28+
use Sinso\Variables\Domain\Model\MarkerCollection;
29+
use PHPUnit\Framework\Attributes\CoversClass;
30+
use PHPUnit\Framework\Attributes\Test;
31+
use PHPUnit\Framework\TestCase;
32+
33+
#[CoversClass(MarkerCollection::class)]
34+
final class MarkerCollectionTest extends TestCase
35+
{
36+
#[Test]
37+
public function returnsType(): void
38+
{
39+
$subject = new MarkerCollection();
40+
41+
self::assertSame('Sinso\Variables\Domain\Model\Marker', $subject->getType());
42+
}
43+
44+
#[Test]
45+
public function throwsExceptionWhenRequestingUnavailableMarker(): void
46+
{
47+
$subject = new MarkerCollection();
48+
49+
$this->expectException(RuntimeException::class);
50+
$this->expectExceptionMessage('Marker not found');
51+
$this->expectExceptionCode(6494338102);
52+
53+
$subject->get('{{unavailable}}');
54+
}
55+
56+
#[Test]
57+
public function returnsAddedMarker(): void
58+
{
59+
$marker = new Marker(0, 'key', 'replacement');
60+
61+
$subject = new MarkerCollection();
62+
63+
$subject->add($marker);
64+
65+
self::assertSame($marker, $subject->get('{{key}}'));
66+
}
67+
68+
#[Test]
69+
public function returnsMarkerKeys(): void
70+
{
71+
$subject = new MarkerCollection();
72+
73+
$subject->add(new Marker(0, 'key_1', 'replacement_1'));
74+
$subject->add(new Marker(0, 'key_2', 'replacement_2'));
75+
76+
self::assertSame([
77+
'{{key_1}}',
78+
'{{key_2}}',
79+
], $subject->getMarkerKeys());
80+
}
81+
}

‎Tests/Unit/Domain/Model/MarkerTest.php‎

Lines changed: 65 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,65 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
/*
6+
* Copyright (C) 2025 Daniel Siepmann <daniel.siepmann@codappix.com>
7+
*
8+
* This program is free software; you can redistribute it and/or
9+
* modify it under the terms of the GNU General Public License
10+
* as published by the Free Software Foundation; either version 2
11+
* of the License, or (at your option) any later version.
12+
*
13+
* This program is distributed in the hope that it will be useful,
14+
* but WITHOUT ANY WARRANTY; without even the implied warranty of
15+
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16+
* GNU General Public License for more details.
17+
*
18+
* You should have received a copy of the GNU General Public License
19+
* along with this program; if not, write to the Free Software
20+
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
21+
* 02110-1301, USA.
22+
*/
23+
24+
namespace Sinso\Variables\Tests\Unit\Domain\Model;
25+
26+
use Sinso\Variables\Domain\Model\Marker;
27+
use PHPUnit\Framework\Attributes\CoversClass;
28+
use PHPUnit\Framework\Attributes\Test;
29+
use PHPUnit\Framework\TestCase;
30+
31+
#[CoversClass(Marker::class)]
32+
final class MarkerTest extends TestCase
33+
{
34+
#[Test]
35+
public function exposesUid(): void
36+
{
37+
$subject = new Marker(10, '', '');
38+
39+
self::assertSame(10, $subject->uid);
40+
}
41+
42+
#[Test]
43+
public function exposesKey(): void
44+
{
45+
$subject = new Marker(0, 'key', '');
46+
47+
self::assertSame('key', $subject->key);
48+
}
49+
50+
#[Test]
51+
public function exposesReplacement(): void
52+
{
53+
$subject = new Marker(0, '', 'replacement');
54+
55+
self::assertSame('replacement', $subject->replacement);
56+
}
57+
58+
#[Test]
59+
public function returnsMarkerWithBrackets(): void
60+
{
61+
$subject = new Marker(0, 'key', '');
62+
63+
self::assertSame('{{key}}', $subject->getMarkerWithBrackets());
64+
}
65+
}

‎Tests/Unit/Utility/CacheKeyUtilityTest.php‎

Lines changed: 56 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,56 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
/*
6+
* Copyright (C) 2025 Daniel Siepmann <daniel.siepmann@codappix.com>
7+
*
8+
* This program is free software; you can redistribute it and/or
9+
* modify it under the terms of the GNU General Public License
10+
* as published by the Free Software Foundation; either version 2
11+
* of the License, or (at your option) any later version.
12+
*
13+
* This program is distributed in the hope that it will be useful,
14+
* but WITHOUT ANY WARRANTY; without even the implied warranty of
15+
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16+
* GNU General Public License for more details.
17+
*
18+
* You should have received a copy of the GNU General Public License
19+
* along with this program; if not, write to the Free Software
20+
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
21+
* 02110-1301, USA.
22+
*/
23+
24+
namespace Sinso\Variables\Tests\Unit\Utility;
25+
26+
use PHPUnit\Framework\Attributes\DataProvider;
27+
use Sinso\Variables\Utility\CacheKeyUtility;
28+
use PHPUnit\Framework\Attributes\CoversClass;
29+
use PHPUnit\Framework\Attributes\Test;
30+
use PHPUnit\Framework\TestCase;
31+
32+
#[CoversClass(CacheKeyUtility::class)]
33+
final class CacheKeyUtilityTest extends TestCase
34+
{
35+
#[DataProvider('possibleCacheKeys')]
36+
#[Test]
37+
public function returnsCacheKey(string $cacheKey): void
38+
{
39+
self::assertSame('tx_variables_key_hash_3d70412c7e9ea2d96fa23d4f1f1f0a1c', CacheKeyUtility::getCacheKey($cacheKey));
40+
}
41+
42+
public static function possibleCacheKeys(): iterable
43+
{
44+
yield 'Simple Cache Key' => [
45+
'cacheKey' => 'some_key',
46+
];
47+
48+
yield 'Cache Key with leading white space' => [
49+
'cacheKey' => ' some_key',
50+
];
51+
52+
yield 'Cache Key with trailing white space' => [
53+
'cacheKey' => 'some_key ',
54+
];
55+
}
56+
}

‎phpunit.xml.dist‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,9 @@
1313
>
1414

1515
<testsuites>
16+
<testsuite name="unit">
17+
<directory>Tests/Unit/</directory>
18+
</testsuite>
1619
<testsuite name="functional">
1720
<directory>Tests/Functional/</directory>
1821
</testsuite>

0 commit comments

Comments
 (0)