archive/kevinpapst_kimai2
0
0
Fork 0
mirror of https://github.com/kevinpapst/kimai2.git synced 2025-05-11 20:25:44 +00:00
Code Issues Projects Releases Wiki Activity

use static for assertCount

Browse source
This commit is contained in:
Kevin Papst 2024-12-21 18:49:30 +01:00
parent 9bd37fb695
commit 4e11c59b58
17 changed files with 25 additions and 25 deletions

4
tests/API/ConfigurationControllerTest.php
View file

@ -32,7 +32,7 @@ class ConfigurationControllerTest extends APIControllerBaseTest
$this->assertIsArray($result);
$this->assertNotEmpty($result);
$expectedKeys = ['activeEntriesHardLimit', 'defaultBeginTime', 'isAllowFutureTimes', 'isAllowOverlapping', 'trackingMode'];
$this->assertCount(\count($expectedKeys), $result);
self::assertCount(\count($expectedKeys), $result);
$actual = array_keys($result);
sort($actual);
sort($expectedKeys);
@ -84,7 +84,7 @@ class ConfigurationControllerTest extends APIControllerBaseTest
'Rose' => '#ffe4e1',
'Lavender' => '#E6E6FA',
];
$this->assertCount(\count($expected), $actual);
self::assertCount(\count($expected), $actual);
$this->assertEquals($expected, $actual, 'Color structure does not match');
}

2
tests/API/TimesheetControllerTest.php
View file

@ -440,7 +440,7 @@ class TimesheetControllerTest extends APIControllerBaseTest
$client = $this->getClientForAuthenticatedUser(User::ROLE_USER);
$this->importFixtureForUser(User::ROLE_USER);
$timesheets = $this->importFixtureForUser(User::ROLE_ADMIN);
$this->assertCount(10, $timesheets);
self::assertCount(10, $timesheets);
$this->assertApiAccessDenied($client, '/api/timesheets/' . $timesheets[0]->getId(), 'Access denied.');
}

2
tests/Controller/InvoiceControllerTest.php
View file

@ -214,7 +214,7 @@ class InvoiceControllerTest extends ControllerBaseTest
$em = $this->getEntityManager();
$em->clear();
$timesheets = $em->getRepository(Timesheet::class)->findAll();
$this->assertCount(20, $timesheets);
self::assertCount(20, $timesheets);
/** @var Timesheet $timesheet */
foreach ($timesheets as $timesheet) {
$this->assertTrue($timesheet->isExported());

2
tests/Controller/PermissionControllerTest.php
View file

@ -212,7 +212,7 @@ class PermissionControllerTest extends ControllerBaseTest
self::assertArrayHasKey('token', $result);
$rolePermissions = $em->getRepository(RolePermission::class)->findAll();
$this->assertCount(1, $rolePermissions);
self::assertCount(1, $rolePermissions);
$permission = $rolePermissions[0];
self::assertInstanceOf(RolePermission::class, $permission);
self::assertEquals('view_user', $permission->getPermission());

2
tests/Controller/ProfileControllerTest.php
View file

@ -381,7 +381,7 @@ class ProfileControllerTest extends ControllerBaseTest
$user = $this->getUserByRole(User::ROLE_USER);
$this->assertCount(1, $user->getTeams());
self::assertCount(1, $user->getTeams());
}
public function getPreferencesTestData(): array

6
tests/Controller/ReportingControllerTest.php
View file

@ -26,7 +26,7 @@ class ReportingControllerTest extends ControllerBaseTest
$client = $this->getClientForAuthenticatedUser(User::ROLE_ADMIN);
$this->request($client, '/reporting/');
$nodes = $client->getCrawler()->filter('section.content div.row-cards a.card-link');
$this->assertCount(11, $nodes);
self::assertCount(11, $nodes);
}
public function testAllReports(): void
@ -34,7 +34,7 @@ class ReportingControllerTest extends ControllerBaseTest
$client = $this->getClientForAuthenticatedUser(User::ROLE_SUPER_ADMIN);
$this->request($client, '/reporting/');
$nodes = $client->getCrawler()->filter('section.content div.row-cards a.card-link');
$this->assertCount(11, $nodes);
self::assertCount(11, $nodes);
foreach ($nodes as $node) {
self::assertNotNull($node->attributes);
$link = $node->attributes->getNamedItem('href');
@ -52,6 +52,6 @@ class ReportingControllerTest extends ControllerBaseTest
$client = $this->getClientForAuthenticatedUser(User::ROLE_USER);
$this->request($client, '/reporting/');
$nodes = $client->getCrawler()->filter('section.content div.row-cards a.card-link');
$this->assertCount(3, $nodes);
self::assertCount(3, $nodes);
}
}

2
tests/Controller/TimesheetTeamControllerTest.php
View file

@ -231,7 +231,7 @@ class TimesheetTeamControllerTest extends ControllerBaseTest
$em = $this->getEntityManager();
/** @var Timesheet[] $timesheets */
$timesheets = $em->getRepository(Timesheet::class)->findAll();
$this->assertCount(2, $timesheets);
self::assertCount(2, $timesheets);
foreach ($timesheets as $timesheet) {
$this->assertInstanceOf(\DateTime::class, $timesheet->getBegin());
$this->assertNull($timesheet->getEnd());

2
tests/Event/ThemeJavascriptTranslationsEventTest.php
View file

@ -23,7 +23,7 @@ class ThemeJavascriptTranslationsEventTest extends TestCase
{
$sut = new ThemeJavascriptTranslationsEvent();
$this->assertCount(self::COUNTER, $sut->getTranslations());
self::assertCount(self::COUNTER, $sut->getTranslations());
}
public function testGetterAndSetter(): void

2
tests/Pdf/MPdfConverterTest.php
View file

@ -29,6 +29,6 @@ class MPdfConverterTest extends KernelTestCase
// Yeah, thats not a real test, I know ;-)
$this->assertNotEmpty($result);
preg_match('/\/Creator \((.*)\)/', $result, $matches);
$this->assertCount(2, $matches);
self::assertCount(2, $matches);
}
}

2
tests/Repository/InvoiceDocumentRepositoryTest.php
View file

@ -87,7 +87,7 @@ class InvoiceDocumentRepositoryTest extends TestCase
{
$sut = new InvoiceDocumentRepository(self::$defaultDirectories);
$all = $sut->findAll();
$this->assertCount(\count(self::$defaultDocuments), $all);
self::assertCount(\count(self::$defaultDocuments), $all);
foreach ($all as $document) {
$this->assertTrue(\in_array($document->getName(), self::$defaultDocuments), 'Missing template: ' . $document->getName());

2
tests/Repository/Query/TeamQueryTest.php
View file

@ -47,6 +47,6 @@ class TeamQueryTest extends BaseQueryTest
$sut->addUser($user);
$sut->removeUser($user);
$this->assertCount(2, $sut->getUsers());
self::assertCount(2, $sut->getUsers());
}
}

2
tests/Repository/Query/UserQueryTest.php
View file

@ -39,7 +39,7 @@ class UserQueryTest extends BaseQueryTest
$this->assertIsArray($sut->getSearchTeams());
$this->assertEmpty($sut->getSearchTeams());
$sut->setSearchTeams([$team, new Team('foo')]);
$this->assertCount(2, $sut->getSearchTeams());
self::assertCount(2, $sut->getSearchTeams());
$this->assertSame($team, $sut->getSearchTeams()[0]);
}

2
tests/Twig/DatatableExtensionsTest.php
View file

@ -32,7 +32,7 @@ class DatatableExtensionsTest extends TestCase
$functions = ['initialize_datatable', 'datatable_column_class'];
$sut = $this->getSut('de');
$twigFunctions = $sut->getFunctions();
$this->assertCount(\count($functions), $twigFunctions);
self::assertCount(\count($functions), $twigFunctions);
$i = 0;
foreach ($twigFunctions as $function) {
$this->assertInstanceOf(TwigFunction::class, $function);

6
tests/Twig/ExtensionsTest.php
View file

@ -46,7 +46,7 @@ class ExtensionsTest extends TestCase
$filters = ['report_date', 'docu_link', 'multiline_indent', 'color', 'font_contrast', 'default_color', 'nl2str'];
$sut = $this->getSut();
$twigFilters = $sut->getFilters();
$this->assertCount(\count($filters), $twigFilters);
self::assertCount(\count($filters), $twigFilters);
$i = 0;
foreach ($twigFilters as $filter) {
@ -67,7 +67,7 @@ class ExtensionsTest extends TestCase
$functions = ['report_date', 'class_name', 'iso_day_by_name', 'random_color'];
$sut = $this->getSut();
$twigFunctions = $sut->getFunctions();
$this->assertCount(\count($functions), $twigFunctions);
self::assertCount(\count($functions), $twigFunctions);
$i = 0;
/** @var TwigFunction $filter */
foreach ($twigFunctions as $filter) {
@ -83,7 +83,7 @@ class ExtensionsTest extends TestCase
$sut = $this->getSut();
$twigTests = $sut->getTests();
$this->assertCount(\count($tests), $twigTests);
self::assertCount(\count($tests), $twigTests);
/** @var TwigTest $test */
foreach ($twigTests as $test) {

6
tests/Twig/LocaleFormatExtensionsTest.php
View file

@ -84,7 +84,7 @@ class LocaleFormatExtensionsTest extends TestCase
$sut = $this->getSut('de', []);
$twigFilters = $sut->getFilters();
//$this->assertCount(\count($filters), $twigFilters);
//self::assertCount(\count($filters), $twigFilters);
foreach ($twigFilters as $filter) {
$this->assertInstanceOf(TwigFilter::class, $filter);
@ -99,7 +99,7 @@ class LocaleFormatExtensionsTest extends TestCase
$sut = $this->getSut('de', []);
$twigFunctions = $sut->getFunctions();
$this->assertCount(\count($functions), $twigFunctions);
self::assertCount(\count($functions), $twigFunctions);
/** @var TwigFunction $filter */
foreach ($twigFunctions as $filter) {
@ -115,7 +115,7 @@ class LocaleFormatExtensionsTest extends TestCase
$sut = $this->getSut('de', []);
$twigTests = $sut->getTests();
$this->assertCount(\count($tests), $twigTests);
self::assertCount(\count($tests), $twigTests);
/** @var TwigTest $test */
foreach ($twigTests as $test) {

4
tests/Twig/RuntimeExtensionsTest.php
View file

@ -27,7 +27,7 @@ class RuntimeExtensionsTest extends TestCase
$sut = new RuntimeExtensions();
$twigFilters = $sut->getFilters();
$this->assertCount(\count($expected), $twigFilters);
self::assertCount(\count($expected), $twigFilters);
foreach ($twigFilters as $filter) {
$this->assertInstanceOf(TwigFilter::class, $filter);
@ -57,7 +57,7 @@ class RuntimeExtensionsTest extends TestCase
$sut = new RuntimeExtensions();
$twigFunctions = $sut->getFunctions();
$this->assertCount(\count($expected), $twigFunctions);
self::assertCount(\count($expected), $twigFunctions);
/** @var TwigFunction $filter */
foreach ($twigFunctions as $filter) {

2
tests/WorkingTime/Mode/WorkingTimeModeDayTest.php
View file

@ -29,6 +29,6 @@ class WorkingTimeModeDayTest extends TestCase
$this->assertEquals('hours_per_day', $sut->getName());
$this->assertInstanceOf(WorkingTimeCalculatorDay::class, $sut->getCalculator($user));
$fields = $sut->getFormFields();
$this->assertCount(7, $fields);
self::assertCount(7, $fields);
}
}