diff options
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/app/Models/CategoryTest.php | 33 | ||||
| -rw-r--r-- | tests/app/Models/LogDAOTest.php | 16 | ||||
| -rw-r--r-- | tests/app/Models/SearchTest.php | 66 | ||||
| -rw-r--r-- | tests/app/Models/UserQueryTest.php | 76 | ||||
| -rw-r--r-- | tests/app/Utils/passwordUtilTest.php | 6 | ||||
| -rw-r--r-- | tests/cli/i18n/I18nCompletionValidatorTest.php | 38 | ||||
| -rw-r--r-- | tests/cli/i18n/I18nDataTest.php | 132 | ||||
| -rw-r--r-- | tests/cli/i18n/I18nFileTest.php | 4 | ||||
| -rw-r--r-- | tests/cli/i18n/I18nUsageValidatorTest.php | 32 | ||||
| -rw-r--r-- | tests/cli/i18n/I18nValueTest.php | 60 | ||||
| -rw-r--r-- | tests/lib/CssXPath/CssXPathTest.php | 2 | ||||
| -rw-r--r-- | tests/lib/Minz/MigratorTest.php | 84 | ||||
| -rw-r--r-- | tests/lib/PHPMailer/PHPMailerTest.php | 2 |
13 files changed, 275 insertions, 276 deletions
diff --git a/tests/app/Models/CategoryTest.php b/tests/app/Models/CategoryTest.php index a438afe9e..b332cb524 100644 --- a/tests/app/Models/CategoryTest.php +++ b/tests/app/Models/CategoryTest.php @@ -4,8 +4,8 @@ class CategoryTest extends PHPUnit\Framework\TestCase { public function test__construct_whenNoParameters_createsObjectWithDefaultValues(): void { $category = new FreshRSS_Category(); - $this->assertEquals(0, $category->id()); - $this->assertEquals('', $category->name()); + self::assertEquals(0, $category->id()); + self::assertEquals('', $category->name()); } /** @@ -13,7 +13,7 @@ class CategoryTest extends PHPUnit\Framework\TestCase { */ public function test_name_whenValidValue_storesModifiedValue(string $input, string $expected): void { $category = new FreshRSS_Category($input); - $this->assertEquals($expected, $category->name()); + self::assertEquals($expected, $category->name()); } /** @return array<array{string,string}> */ @@ -33,21 +33,21 @@ class CategoryTest extends PHPUnit\Framework\TestCase { $feed_1 = $this->getMockBuilder(FreshRSS_Feed::class) ->disableOriginalConstructor() ->getMock(); - $feed_1->expects($this->any()) + $feed_1->expects(self::any()) ->method('name') ->willReturn('AAA'); $feed_2 = $this->getMockBuilder(FreshRSS_Feed::class) ->disableOriginalConstructor() ->getMock(); - $feed_2->expects($this->any()) + $feed_2->expects(self::any()) ->method('name') ->willReturn('ZZZ'); $feed_3 = $this->getMockBuilder(FreshRSS_Feed::class) ->disableOriginalConstructor() ->getMock(); - $feed_3->expects($this->any()) + $feed_3->expects(self::any()) ->method('name') ->willReturn('lll'); @@ -58,25 +58,26 @@ class CategoryTest extends PHPUnit\Framework\TestCase { ]); $feeds = $category->feeds(); - $this->assertCount(3, $feeds); - $this->assertEquals('AAA', $feeds[0]->name()); - $this->assertEquals('lll', $feeds[1]->name()); - $this->assertEquals('ZZZ', $feeds[2]->name()); + self::assertCount(3, $feeds); + self::assertEquals('AAA', $feeds[0]->name()); + self::assertEquals('lll', $feeds[1]->name()); + self::assertEquals('ZZZ', $feeds[2]->name()); + /** @var FreshRSS_Feed&PHPUnit\Framework\MockObject\MockObject */ $feed_4 = $this->getMockBuilder(FreshRSS_Feed::class) ->disableOriginalConstructor() ->getMock(); - $feed_4->expects($this->any()) + $feed_4->expects(self::any()) ->method('name') ->willReturn('BBB'); $category->addFeed($feed_4); $feeds = $category->feeds(); - $this->assertCount(4, $feeds); - $this->assertEquals('AAA', $feeds[0]->name()); - $this->assertEquals('BBB', $feeds[1]->name()); - $this->assertEquals('lll', $feeds[2]->name()); - $this->assertEquals('ZZZ', $feeds[3]->name()); + self::assertCount(4, $feeds); + self::assertEquals('AAA', $feeds[0]->name()); + self::assertEquals('BBB', $feeds[1]->name()); + self::assertEquals('lll', $feeds[2]->name()); + self::assertEquals('ZZZ', $feeds[3]->name()); } } diff --git a/tests/app/Models/LogDAOTest.php b/tests/app/Models/LogDAOTest.php index abe6c6c1f..dc9fbd316 100644 --- a/tests/app/Models/LogDAOTest.php +++ b/tests/app/Models/LogDAOTest.php @@ -23,20 +23,20 @@ class LogDAOTest extends TestCase { } public function test_lines_is_array_and_truncate_function_work(): void { - $this->assertEquals(USERS_PATH . '/' . Minz_User::INTERNAL_USER . '/' . self::LOG_FILE_TEST, $this->logPath); + self::assertEquals(USERS_PATH . '/' . Minz_User::INTERNAL_USER . '/' . self::LOG_FILE_TEST, $this->logPath); $line = $this->logDAO::lines(self::LOG_FILE_TEST); - $this->assertIsArray($line); - $this->assertCount(1, $line); - $this->assertInstanceOf(FreshRSS_Log::class, $line[0]); - $this->assertEquals('Wed, 08 Feb 2023 15:35:05 +0000', $line[0]->date()); - $this->assertEquals('notice', $line[0]->level()); - $this->assertEquals("Migration 2019_12_22_FooBar: OK", $line[0]->info()); + self::assertIsArray($line); + self::assertCount(1, $line); + self::assertInstanceOf(FreshRSS_Log::class, $line[0]); + self::assertEquals('Wed, 08 Feb 2023 15:35:05 +0000', $line[0]->date()); + self::assertEquals('notice', $line[0]->level()); + self::assertEquals("Migration 2019_12_22_FooBar: OK", $line[0]->info()); $this->logDAO::truncate(self::LOG_FILE_TEST); - $this->assertStringContainsString('', file_get_contents($this->logPath) ?: ''); + self::assertStringContainsString('', file_get_contents($this->logPath) ?: ''); } protected function tearDown(): void { diff --git a/tests/app/Models/SearchTest.php b/tests/app/Models/SearchTest.php index 2b2501589..fa41542be 100644 --- a/tests/app/Models/SearchTest.php +++ b/tests/app/Models/SearchTest.php @@ -9,15 +9,15 @@ class SearchTest extends PHPUnit\Framework\TestCase { */ public function test__construct_whenInputIsEmpty_getsOnlyNullValues(?string $input): void { $search = new FreshRSS_Search($input); - $this->assertEquals('', $search->getRawInput()); - $this->assertNull($search->getIntitle()); - $this->assertNull($search->getMinDate()); - $this->assertNull($search->getMaxDate()); - $this->assertNull($search->getMinPubdate()); - $this->assertNull($search->getMaxPubdate()); - $this->assertNull($search->getAuthor()); - $this->assertNull($search->getTags()); - $this->assertNull($search->getSearch()); + self::assertEquals('', $search->getRawInput()); + self::assertNull($search->getIntitle()); + self::assertNull($search->getMinDate()); + self::assertNull($search->getMaxDate()); + self::assertNull($search->getMinPubdate()); + self::assertNull($search->getMaxPubdate()); + self::assertNull($search->getAuthor()); + self::assertNull($search->getTags()); + self::assertNull($search->getSearch()); } /** @@ -39,8 +39,8 @@ class SearchTest extends PHPUnit\Framework\TestCase { */ public function test__construct_whenInputContainsIntitle_setsIntitleProperty(string $input, ?array $intitle_value, ?array $search_value): void { $search = new FreshRSS_Search($input); - $this->assertEquals($intitle_value, $search->getIntitle()); - $this->assertEquals($search_value, $search->getSearch()); + self::assertEquals($intitle_value, $search->getIntitle()); + self::assertEquals($search_value, $search->getSearch()); } /** @@ -76,8 +76,8 @@ class SearchTest extends PHPUnit\Framework\TestCase { */ public function test__construct_whenInputContainsAuthor_setsAuthorValue(string $input, ?array $author_value, ?array $search_value): void { $search = new FreshRSS_Search($input); - $this->assertEquals($author_value, $search->getAuthor()); - $this->assertEquals($search_value, $search->getSearch()); + self::assertEquals($author_value, $search->getAuthor()); + self::assertEquals($search_value, $search->getSearch()); } /** @@ -113,8 +113,8 @@ class SearchTest extends PHPUnit\Framework\TestCase { */ public function test__construct_whenInputContainsInurl_setsInurlValue(string $input, ?array $inurl_value, ?array $search_value): void { $search = new FreshRSS_Search($input); - $this->assertEquals($inurl_value, $search->getInurl()); - $this->assertEquals($search_value, $search->getSearch()); + self::assertEquals($inurl_value, $search->getInurl()); + self::assertEquals($search_value, $search->getSearch()); } /** @@ -138,8 +138,8 @@ class SearchTest extends PHPUnit\Framework\TestCase { */ public function test__construct_whenInputContainsDate_setsDateValues(string $input, ?int $min_date_value, ?int $max_date_value): void { $search = new FreshRSS_Search($input); - $this->assertEquals($min_date_value, $search->getMinDate()); - $this->assertEquals($max_date_value, $search->getMaxDate()); + self::assertEquals($min_date_value, $search->getMinDate()); + self::assertEquals($max_date_value, $search->getMaxDate()); } /** @@ -161,8 +161,8 @@ class SearchTest extends PHPUnit\Framework\TestCase { */ public function test__construct_whenInputContainsPubdate_setsPubdateValues(string $input, ?int $min_pubdate_value, ?int $max_pubdate_value): void { $search = new FreshRSS_Search($input); - $this->assertEquals($min_pubdate_value, $search->getMinPubdate()); - $this->assertEquals($max_pubdate_value, $search->getMaxPubdate()); + self::assertEquals($min_pubdate_value, $search->getMinPubdate()); + self::assertEquals($max_pubdate_value, $search->getMaxPubdate()); } /** @@ -186,8 +186,8 @@ class SearchTest extends PHPUnit\Framework\TestCase { */ public function test__construct_whenInputContainsTags_setsTagsValue(string $input, ?array $tags_value, ?array $search_value): void { $search = new FreshRSS_Search($input); - $this->assertEquals($tags_value, $search->getTags()); - $this->assertEquals($search_value, $search->getSearch()); + self::assertEquals($tags_value, $search->getTags()); + self::assertEquals($search_value, $search->getSearch()); } /** @@ -218,16 +218,16 @@ class SearchTest extends PHPUnit\Framework\TestCase { ?int $max_date_value, ?array $intitle_value, ?array $inurl_value, ?int $min_pubdate_value, ?int $max_pubdate_value, ?array $tags_value, ?array $search_value): void { $search = new FreshRSS_Search($input); - $this->assertEquals($author_value, $search->getAuthor()); - $this->assertEquals($min_date_value, $search->getMinDate()); - $this->assertEquals($max_date_value, $search->getMaxDate()); - $this->assertEquals($intitle_value, $search->getIntitle()); - $this->assertEquals($inurl_value, $search->getInurl()); - $this->assertEquals($min_pubdate_value, $search->getMinPubdate()); - $this->assertEquals($max_pubdate_value, $search->getMaxPubdate()); - $this->assertEquals($tags_value, $search->getTags()); - $this->assertEquals($search_value, $search->getSearch()); - $this->assertEquals($input, $search->getRawInput()); + self::assertEquals($author_value, $search->getAuthor()); + self::assertEquals($min_date_value, $search->getMinDate()); + self::assertEquals($max_date_value, $search->getMaxDate()); + self::assertEquals($intitle_value, $search->getIntitle()); + self::assertEquals($inurl_value, $search->getInurl()); + self::assertEquals($min_pubdate_value, $search->getMinPubdate()); + self::assertEquals($max_pubdate_value, $search->getMaxPubdate()); + self::assertEquals($tags_value, $search->getTags()); + self::assertEquals($search_value, $search->getSearch()); + self::assertEquals($input, $search->getRawInput()); } /** @return array<array<mixed>> */ @@ -290,8 +290,8 @@ class SearchTest extends PHPUnit\Framework\TestCase { */ public function test__construct_parentheses(string $input, string $sql, array $values): void { list($filterValues, $filterSearch) = FreshRSS_EntryDAOPGSQL::sqlBooleanSearch('e.', new FreshRSS_BooleanSearch($input)); - $this->assertEquals($sql, $filterSearch); - $this->assertEquals($values, $filterValues); + self::assertEquals($sql, $filterSearch); + self::assertEquals($values, $filterValues); } /** @return array<array<mixed>> */ diff --git a/tests/app/Models/UserQueryTest.php b/tests/app/Models/UserQueryTest.php index c227024a9..aa66b1447 100644 --- a/tests/app/Models/UserQueryTest.php +++ b/tests/app/Models/UserQueryTest.php @@ -8,15 +8,15 @@ class UserQueryTest extends PHPUnit\Framework\TestCase { public function test__construct_whenAllQuery_storesAllParameters(): void { $query = array('get' => 'a'); $user_query = new FreshRSS_UserQuery($query); - $this->assertEquals('all', $user_query->getGetName()); - $this->assertEquals('all', $user_query->getGetType()); + self::assertEquals('all', $user_query->getGetName()); + self::assertEquals('all', $user_query->getGetType()); } public function test__construct_whenFavoriteQuery_storesFavoriteParameters(): void { $query = array('get' => 's'); $user_query = new FreshRSS_UserQuery($query); - $this->assertEquals('favorite', $user_query->getGetName()); - $this->assertEquals('favorite', $user_query->getGetType()); + self::assertEquals('favorite', $user_query->getGetName()); + self::assertEquals('favorite', $user_query->getGetType()); } public function test__construct_whenCategoryQueryAndNoDao_throwsException(): void { @@ -31,20 +31,20 @@ class UserQueryTest extends PHPUnit\Framework\TestCase { $category_name = 'some category name'; /** @var FreshRSS_Category&PHPUnit\Framework\MockObject\MockObject */ $cat = $this->createMock(FreshRSS_Category::class); - $cat->expects($this->atLeastOnce()) + $cat->expects(self::atLeastOnce()) ->method('name') ->withAnyParameters() ->willReturn($category_name); /** @var FreshRSS_CategoryDAO&PHPUnit\Framework\MockObject\MockObject */ $cat_dao = $this->createMock(FreshRSS_CategoryDAO::class); - $cat_dao->expects($this->atLeastOnce()) + $cat_dao->expects(self::atLeastOnce()) ->method('searchById') ->withAnyParameters() ->willReturn($cat); $query = array('get' => 'c_1'); $user_query = new FreshRSS_UserQuery($query, null, $cat_dao); - $this->assertEquals($category_name, $user_query->getGetName()); - $this->assertEquals('category', $user_query->getGetType()); + self::assertEquals($category_name, $user_query->getGetName()); + self::assertEquals('category', $user_query->getGetType()); } public function test__construct_whenFeedQueryAndNoDao_throwsException(): void { @@ -59,61 +59,60 @@ class UserQueryTest extends PHPUnit\Framework\TestCase { $feed_name = 'some feed name'; /** @var FreshRSS_Feed&PHPUnit\Framework\MockObject\MockObject */ $feed = $this->createMock(FreshRSS_Feed::class); - $feed->expects($this->atLeastOnce()) + $feed->expects(self::atLeastOnce()) ->method('name') ->withAnyParameters() ->willReturn($feed_name); /** @var FreshRSS_FeedDAO&PHPUnit\Framework\MockObject\MockObject */ $feed_dao = $this->createMock(FreshRSS_FeedDAO::class); - $feed_dao->expects($this->atLeastOnce()) + $feed_dao->expects(self::atLeastOnce()) ->method('searchById') ->withAnyParameters() ->willReturn($feed); $query = array('get' => 'f_1'); $user_query = new FreshRSS_UserQuery($query, $feed_dao, null); - $this->assertEquals($feed_name, $user_query->getGetName()); - $this->assertEquals('feed', $user_query->getGetType()); + self::assertEquals($feed_name, $user_query->getGetName()); + self::assertEquals('feed', $user_query->getGetType()); } public function test__construct_whenUnknownQuery_doesStoreParameters(): void { $query = array('get' => 'q'); $user_query = new FreshRSS_UserQuery($query); - $this->assertEmpty($user_query->getGetName()); - $this->assertEmpty($user_query->getGetType()); + self::assertEmpty($user_query->getGetName()); + self::assertEmpty($user_query->getGetType()); } public function test__construct_whenName_storesName(): void { $name = 'some name'; $query = array('name' => $name); $user_query = new FreshRSS_UserQuery($query); - $this->assertEquals($name, $user_query->getName()); + self::assertEquals($name, $user_query->getName()); } public function test__construct_whenOrder_storesOrder(): void { $order = 'some order'; $query = array('order' => $order); $user_query = new FreshRSS_UserQuery($query); - $this->assertEquals($order, $user_query->getOrder()); + self::assertEquals($order, $user_query->getOrder()); } public function test__construct_whenState_storesState(): void { $state = FreshRSS_Entry::STATE_ALL; $query = array('state' => $state); $user_query = new FreshRSS_UserQuery($query); - $this->assertEquals($state, $user_query->getState()); + self::assertEquals($state, $user_query->getState()); } public function test__construct_whenUrl_storesUrl(): void { $url = 'some url'; $query = array('url' => $url); $user_query = new FreshRSS_UserQuery($query); - $this->assertEquals($url, $user_query->getUrl()); + self::assertEquals($url, $user_query->getUrl()); } public function testToArray_whenNoData_returnsEmptyArray(): void { $user_query = new FreshRSS_UserQuery(array()); - $this->assertIsIterable($user_query->toArray()); - $this->assertCount(0, $user_query->toArray()); + self::assertCount(0, $user_query->toArray()); } public function testToArray_whenData_returnsArray(): void { @@ -126,9 +125,8 @@ class UserQueryTest extends PHPUnit\Framework\TestCase { 'url' => 'some url', ); $user_query = new FreshRSS_UserQuery($query); - $this->assertIsIterable($user_query->toArray()); - $this->assertCount(6, $user_query->toArray()); - $this->assertEquals($query, $user_query->toArray()); + self::assertCount(6, $user_query->toArray()); + self::assertEquals($query, $user_query->toArray()); } public function testHasSearch_whenSearch_returnsTrue(): void { @@ -136,30 +134,30 @@ class UserQueryTest extends PHPUnit\Framework\TestCase { 'search' => 'some search', ); $user_query = new FreshRSS_UserQuery($query); - $this->assertTrue($user_query->hasSearch()); + self::assertTrue($user_query->hasSearch()); } public function testHasSearch_whenNoSearch_returnsFalse(): void { $user_query = new FreshRSS_UserQuery(array()); - $this->assertFalse($user_query->hasSearch()); + self::assertFalse($user_query->hasSearch()); } public function testHasParameters_whenAllQuery_returnsFalse(): void { $query = array('get' => 'a'); $user_query = new FreshRSS_UserQuery($query); - $this->assertFalse($user_query->hasParameters()); + self::assertFalse($user_query->hasParameters()); } public function testHasParameters_whenNoParameter_returnsFalse(): void { $query = array(); $user_query = new FreshRSS_UserQuery($query); - $this->assertFalse($user_query->hasParameters()); + self::assertFalse($user_query->hasParameters()); } public function testHasParameters_whenParameter_returnTrue(): void { $query = array('get' => 's'); $user_query = new FreshRSS_UserQuery($query); - $this->assertTrue($user_query->hasParameters()); + self::assertTrue($user_query->hasParameters()); } public function testIsDeprecated_whenCategoryExists_returnFalse(): void { @@ -167,25 +165,25 @@ class UserQueryTest extends PHPUnit\Framework\TestCase { $cat = $this->createMock(FreshRSS_Category::class); /** @var FreshRSS_CategoryDAO&PHPUnit\Framework\MockObject\MockObject */ $cat_dao = $this->createMock(FreshRSS_CategoryDAO::class); - $cat_dao->expects($this->atLeastOnce()) + $cat_dao->expects(self::atLeastOnce()) ->method('searchById') ->withAnyParameters() ->willReturn($cat); $query = array('get' => 'c_1'); $user_query = new FreshRSS_UserQuery($query, null, $cat_dao); - $this->assertFalse($user_query->isDeprecated()); + self::assertFalse($user_query->isDeprecated()); } public function testIsDeprecated_whenCategoryDoesNotExist_returnTrue(): void { /** @var FreshRSS_CategoryDAO&PHPUnit\Framework\MockObject\MockObject */ $cat_dao = $this->createMock(FreshRSS_CategoryDAO::class); - $cat_dao->expects($this->atLeastOnce()) + $cat_dao->expects(self::atLeastOnce()) ->method('searchById') ->withAnyParameters() ->willReturn(null); $query = array('get' => 'c_1'); $user_query = new FreshRSS_UserQuery($query, null, $cat_dao); - $this->assertTrue($user_query->isDeprecated()); + self::assertTrue($user_query->isDeprecated()); } public function testIsDeprecated_whenFeedExists_returnFalse(): void { @@ -193,43 +191,43 @@ class UserQueryTest extends PHPUnit\Framework\TestCase { $feed = $this->createMock(FreshRSS_Feed::class); /** @var FreshRSS_FeedDAO&PHPUnit\Framework\MockObject\MockObject */ $feed_dao = $this->createMock(FreshRSS_FeedDAO::class); - $feed_dao->expects($this->atLeastOnce()) + $feed_dao->expects(self::atLeastOnce()) ->method('searchById') ->withAnyParameters() ->willReturn($feed); $query = array('get' => 'f_1'); $user_query = new FreshRSS_UserQuery($query, $feed_dao, null); - $this->assertFalse($user_query->isDeprecated()); + self::assertFalse($user_query->isDeprecated()); } public function testIsDeprecated_whenFeedDoesNotExist_returnTrue(): void { /** @var FreshRSS_FeedDAO&PHPUnit\Framework\MockObject\MockObject */ $feed_dao = $this->createMock(FreshRSS_FeedDAO::class); - $feed_dao->expects($this->atLeastOnce()) + $feed_dao->expects(self::atLeastOnce()) ->method('searchById') ->withAnyParameters() ->willReturn(null); $query = array('get' => 'f_1'); $user_query = new FreshRSS_UserQuery($query, $feed_dao, null); - $this->assertTrue($user_query->isDeprecated()); + self::assertTrue($user_query->isDeprecated()); } public function testIsDeprecated_whenAllQuery_returnFalse(): void { $query = array('get' => 'a'); $user_query = new FreshRSS_UserQuery($query); - $this->assertFalse($user_query->isDeprecated()); + self::assertFalse($user_query->isDeprecated()); } public function testIsDeprecated_whenFavoriteQuery_returnFalse(): void { $query = array('get' => 's'); $user_query = new FreshRSS_UserQuery($query); - $this->assertFalse($user_query->isDeprecated()); + self::assertFalse($user_query->isDeprecated()); } public function testIsDeprecated_whenUnknownQuery_returnFalse(): void { $query = array('get' => 'q'); $user_query = new FreshRSS_UserQuery($query); - $this->assertFalse($user_query->isDeprecated()); + self::assertFalse($user_query->isDeprecated()); } } diff --git a/tests/app/Utils/passwordUtilTest.php b/tests/app/Utils/passwordUtilTest.php index 113476c63..e2fe78b97 100644 --- a/tests/app/Utils/passwordUtilTest.php +++ b/tests/app/Utils/passwordUtilTest.php @@ -6,7 +6,7 @@ class passwordUtilTest extends PHPUnit\Framework\TestCase { $ok = FreshRSS_password_Util::check($password); - $this->assertTrue($ok); + self::assertTrue($ok); } public function testCheckReturnsFalseIfEmpty(): void { @@ -14,7 +14,7 @@ class passwordUtilTest extends PHPUnit\Framework\TestCase { $ok = FreshRSS_password_Util::check($password); - $this->assertFalse($ok); + self::assertFalse($ok); } public function testCheckReturnsFalseIfLessThan7Characters(): void { @@ -22,6 +22,6 @@ class passwordUtilTest extends PHPUnit\Framework\TestCase { $ok = FreshRSS_password_Util::check($password); - $this->assertFalse($ok); + self::assertFalse($ok); } } diff --git a/tests/cli/i18n/I18nCompletionValidatorTest.php b/tests/cli/i18n/I18nCompletionValidatorTest.php index 1a361823d..3c6fc99eb 100644 --- a/tests/cli/i18n/I18nCompletionValidatorTest.php +++ b/tests/cli/i18n/I18nCompletionValidatorTest.php @@ -16,23 +16,23 @@ class I18nCompletionValidatorTest extends PHPUnit\Framework\TestCase { public function testDisplayReport(): void { $validator = new I18nCompletionValidator([], []); - $this->assertEquals("There is no data.\n", $validator->displayReport()); + self::assertEquals("There is no data.\n", $validator->displayReport()); $reflectionTotalEntries = new ReflectionProperty(I18nCompletionValidator::class, 'totalEntries'); $reflectionTotalEntries->setAccessible(true); $reflectionTotalEntries->setValue($validator, 100); - $this->assertEquals("Translation is 0.0% complete.\n", $validator->displayReport()); + self::assertEquals("Translation is 0.0% complete.\n", $validator->displayReport()); $reflectionPassEntries = new ReflectionProperty(I18nCompletionValidator::class, 'passEntries'); $reflectionPassEntries->setAccessible(true); $reflectionPassEntries->setValue($validator, 25); - $this->assertEquals("Translation is 25.0% complete.\n", $validator->displayReport()); + self::assertEquals("Translation is 25.0% complete.\n", $validator->displayReport()); $reflectionPassEntries->setValue($validator, 100); - $this->assertEquals("Translation is 100.0% complete.\n", $validator->displayReport()); + self::assertEquals("Translation is 100.0% complete.\n", $validator->displayReport()); $reflectionPassEntries->setValue($validator, 200); @@ -43,8 +43,8 @@ class I18nCompletionValidatorTest extends PHPUnit\Framework\TestCase { public function testValidateWhenNoData(): void { $validator = new I18nCompletionValidator([], []); - $this->assertTrue($validator->validate()); - $this->assertEquals('', $validator->displayResult()); + self::assertTrue($validator->validate()); + self::assertEquals('', $validator->displayResult()); } public function testValidateWhenKeyIsMissing(): void { @@ -57,12 +57,12 @@ class I18nCompletionValidatorTest extends PHPUnit\Framework\TestCase { ], ], []); - $this->assertFalse($validator->validate()); - $this->assertEquals("Missing key file1.l1.l2.k1\nMissing key file2.l1.l2.k1\n", $validator->displayResult()); + self::assertFalse($validator->validate()); + self::assertEquals("Missing key file1.l1.l2.k1\nMissing key file2.l1.l2.k1\n", $validator->displayResult()); } public function testValidateWhenKeyIsIgnored(): void { - $this->value->expects($this->exactly(2)) + $this->value->expects(self::exactly(2)) ->method('isIgnore') ->willReturn(true); @@ -82,15 +82,15 @@ class I18nCompletionValidatorTest extends PHPUnit\Framework\TestCase { ], ]); - $this->assertTrue($validator->validate()); - $this->assertEquals('', $validator->displayResult()); + self::assertTrue($validator->validate()); + self::assertEquals('', $validator->displayResult()); } public function testValidateWhenValueIsEqual(): void { - $this->value->expects($this->exactly(2)) + $this->value->expects(self::exactly(2)) ->method('isIgnore') ->willReturn(false); - $this->value->expects($this->exactly(2)) + $this->value->expects(self::exactly(2)) ->method('equal') ->willReturn(true); @@ -110,15 +110,15 @@ class I18nCompletionValidatorTest extends PHPUnit\Framework\TestCase { ], ]); - $this->assertFalse($validator->validate()); - $this->assertEquals("Untranslated key file1.l1.l2.k1 - \nUntranslated key file2.l1.l2.k1 - \n", $validator->displayResult()); + self::assertFalse($validator->validate()); + self::assertEquals("Untranslated key file1.l1.l2.k1 - \nUntranslated key file2.l1.l2.k1 - \n", $validator->displayResult()); } public function testValidateWhenValueIsDifferent(): void { - $this->value->expects($this->exactly(2)) + $this->value->expects(self::exactly(2)) ->method('isIgnore') ->willReturn(false); - $this->value->expects($this->exactly(2)) + $this->value->expects(self::exactly(2)) ->method('equal') ->willReturn(false); @@ -138,7 +138,7 @@ class I18nCompletionValidatorTest extends PHPUnit\Framework\TestCase { ], ]); - $this->assertTrue($validator->validate()); - $this->assertEquals('', $validator->displayResult()); + self::assertTrue($validator->validate()); + self::assertEquals('', $validator->displayResult()); } } diff --git a/tests/cli/i18n/I18nDataTest.php b/tests/cli/i18n/I18nDataTest.php index 37811c193..fda4a4734 100644 --- a/tests/cli/i18n/I18nDataTest.php +++ b/tests/cli/i18n/I18nDataTest.php @@ -35,7 +35,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { public function testConstructWhenReferenceOnly(): void { $data = new I18nData($this->referenceData); - $this->assertEquals($this->referenceData, $data->getData()); + self::assertEquals($this->referenceData, $data->getData()); } public function testConstructorWhenLanguageIsMissingFile(): void { @@ -47,7 +47,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { ], ]); $data = new I18nData($rawData); - $this->assertEquals([ + self::assertEquals([ 'en' => [ 'file1.php' => [ 'file1.l1.l2.k1' => $this->value, @@ -93,7 +93,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { ], ]); $data = new I18nData($rawData); - $this->assertEquals([ + self::assertEquals([ 'en' => [ 'file1.php' => [ 'file1.l1.l2.k1' => $this->value, @@ -147,7 +147,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { ], ]); $data = new I18nData($rawData); - $this->assertEquals([ + self::assertEquals([ 'en' => [ 'file1.php' => [ 'file1.l1.l2.k1' => $this->value, @@ -185,12 +185,12 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { $value = $this->getMockBuilder(I18nValue::class) ->disableOriginalConstructor() ->getMock(); - $value->expects($this->exactly(2)) + $value->expects(self::exactly(2)) ->method('isIgnore') ->willReturn(true); - $value->expects($this->never()) + $value->expects(self::never()) ->method('markAsTodo'); - $value->expects($this->exactly(3)) + $value->expects(self::exactly(3)) ->method('equal') ->with($value) ->willReturn(true); @@ -210,12 +210,12 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { $value = $this->getMockBuilder(I18nValue::class) ->disableOriginalConstructor() ->getMock(); - $value->expects($this->exactly(2)) + $value->expects(self::exactly(2)) ->method('isIgnore') ->willReturn(false); - $value->expects($this->exactly(2)) + $value->expects(self::exactly(2)) ->method('markAsTodo'); - $value->expects($this->exactly(2)) + $value->expects(self::exactly(2)) ->method('equal') ->with($value) ->willReturn(true); @@ -235,10 +235,10 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { $value = $this->getMockBuilder(I18nValue::class) ->disableOriginalConstructor() ->getMock(); - $value->expects($this->once()) + $value->expects(self::once()) ->method('isTodo') ->willReturn(true); - $value->expects($this->once()) + $value->expects(self::once()) ->method('markAsDirty'); $rawData = array_merge($this->referenceData, [ @@ -255,10 +255,10 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { $value = $this->getMockBuilder(I18nValue::class) ->disableOriginalConstructor() ->getMock(); - $value->expects($this->once()) + $value->expects(self::once()) ->method('isTodo') ->willReturn(false); - $value->expects($this->never()) + $value->expects(self::never()) ->method('markAsDirty'); $rawData = array_merge($this->referenceData, [ @@ -277,7 +277,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { 'nl' => [], ]); $data = new I18nData($rawData); - $this->assertEquals([ + self::assertEquals([ 'en', 'fr', 'nl', @@ -291,7 +291,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { 'de' => [], ]); $data = new I18nData($rawData); - $this->assertEquals([ + self::assertEquals([ 'de', 'en', 'fr', @@ -309,7 +309,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { public function testAddLanguageWhenNoReferenceProvided(): void { $data = new I18nData($this->referenceData); $data->addLanguage('fr'); - $this->assertEquals([ + self::assertEquals([ 'en' => [ 'file1.php' => [ 'file1.l1.l2.k1' => $this->value, @@ -346,7 +346,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { public function testAddLanguageWhenUnknownReferenceProvided(): void { $data = new I18nData($this->referenceData); $data->addLanguage('fr', 'unknown'); - $this->assertEquals([ + self::assertEquals([ 'en' => [ 'file1.php' => [ 'file1.l1.l2.k1' => $this->value, @@ -383,7 +383,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { public function testAddLanguageWhenKnownReferenceProvided(): void { $data = new I18nData($this->referenceData); $data->addLanguage('fr', 'en'); - $this->assertEquals([ + self::assertEquals([ 'en' => [ 'file1.php' => [ 'file1.l1.l2.k1' => $this->value, @@ -419,12 +419,12 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { public function testIsKnownWhenKeyExists(): void { $data = new I18nData($this->referenceData); - $this->assertTrue($data->isKnown('file2.l1.l2.k2')); + self::assertTrue($data->isKnown('file2.l1.l2.k2')); } public function testIsKnownWhenKeyDoesNotExist(): void { $data = new I18nData($this->referenceData); - $this->assertFalse($data->isKnown('file2.l1.l2.k3')); + self::assertFalse($data->isKnown('file2.l1.l2.k3')); } public function testAddKeyWhenKeyExists(): void { @@ -440,13 +440,13 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { ]); $data = new I18nData($rawData); - $this->assertTrue($data->isKnown('file2.l1.l2.k1')); - $this->assertFalse($data->isKnown('file2.l1.l2.k1._')); - $this->assertFalse($data->isKnown('file2.l1.l2.k1.sk1')); + self::assertTrue($data->isKnown('file2.l1.l2.k1')); + self::assertFalse($data->isKnown('file2.l1.l2.k1._')); + self::assertFalse($data->isKnown('file2.l1.l2.k1.sk1')); $data->addKey('file2.l1.l2.k1.sk1', 'value'); - $this->assertFalse($data->isKnown('file2.l1.l2.k1')); - $this->assertTrue($data->isKnown('file2.l1.l2.k1._')); - $this->assertTrue($data->isKnown('file2.l1.l2.k1.sk1')); + self::assertFalse($data->isKnown('file2.l1.l2.k1')); + self::assertTrue($data->isKnown('file2.l1.l2.k1._')); + self::assertTrue($data->isKnown('file2.l1.l2.k1.sk1')); } public function testAddKeyWhenKeyIsParent(): void { @@ -455,13 +455,13 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { ]); $data = new I18nData($rawData); - $this->assertFalse($data->isKnown('file1.l1.l2._')); - $this->assertTrue($data->isKnown('file1.l1.l2.k1')); - $this->assertTrue($data->isKnown('file1.l1.l2.k2')); + self::assertFalse($data->isKnown('file1.l1.l2._')); + self::assertTrue($data->isKnown('file1.l1.l2.k1')); + self::assertTrue($data->isKnown('file1.l1.l2.k2')); $data->addKey('file1.l1.l2', 'value'); - $this->assertTrue($data->isKnown('file1.l1.l2._')); - $this->assertTrue($data->isKnown('file1.l1.l2.k1')); - $this->assertTrue($data->isKnown('file1.l1.l2.k2')); + self::assertTrue($data->isKnown('file1.l1.l2._')); + self::assertTrue($data->isKnown('file1.l1.l2.k1')); + self::assertTrue($data->isKnown('file1.l1.l2.k2')); } public function testAddKey(): void { @@ -474,16 +474,16 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { ]); $data = new I18nData($rawData); - $this->assertFalse($data->isKnown('file2.l1.l2.k3')); + self::assertFalse($data->isKnown('file2.l1.l2.k3')); $data->addKey('file2.l1.l2.k3', 'value'); - $this->assertTrue($data->isKnown('file2.l1.l2.k3')); + self::assertTrue($data->isKnown('file2.l1.l2.k3')); $enValue = $getTargetedValue($data, 'en'); $frValue = $getTargetedValue($data, 'fr'); - $this->assertInstanceOf(I18nValue::class, $enValue); - $this->assertEquals('value', $enValue->getValue()); - $this->assertTrue($enValue->isTodo()); - $this->assertEquals($frValue, $enValue); + self::assertInstanceOf(I18nValue::class, $enValue); + self::assertEquals('value', $enValue->getValue()); + self::assertTrue($enValue->isTodo()); + self::assertEquals($frValue, $enValue); } public function testAddValueWhenLanguageDoesNotExist(): void { @@ -505,7 +505,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { return $data->getData()[$language]['file2.php']['file2.l1.l2.k2']; }; - $this->value->expects($this->atLeast(2)) + $this->value->expects(self::atLeast(2)) ->method('equal') ->with($this->value) ->willReturn(true); @@ -520,12 +520,12 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { $afterEnValue = $getTargetedValue($data, 'en'); $afterFrValue = $getTargetedValue($data, 'fr'); - $this->assertEquals($this->value, $beforeEnValue); - $this->assertEquals($this->value, $beforeFrValue); - $this->assertInstanceOf(I18nValue::class, $afterEnValue); - $this->assertEquals('new value', $afterEnValue->getValue()); - $this->assertInstanceOf(I18nValue::class, $afterFrValue); - $this->assertEquals('new value', $afterFrValue->getValue()); + self::assertEquals($this->value, $beforeEnValue); + self::assertEquals($this->value, $beforeFrValue); + self::assertInstanceOf(I18nValue::class, $afterEnValue); + self::assertEquals('new value', $afterEnValue->getValue()); + self::assertInstanceOf(I18nValue::class, $afterFrValue); + self::assertEquals('new value', $afterFrValue->getValue()); } public function testAddValueWhenLanguageIsReferenceAndValueInOtherLanguageHasChange(): void { @@ -533,7 +533,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { return $data->getData()[$language]['file2.php']['file2.l1.l2.k2']; }; - $this->value->expects($this->any()) + $this->value->expects(self::any()) ->method('equal') ->with($this->value) ->willReturn(true); @@ -556,11 +556,11 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { $afterEnValue = $getTargetedValue($data, 'en'); $afterFrValue = $getTargetedValue($data, 'fr'); - $this->assertEquals($this->value, $beforeEnValue); - $this->assertEquals($value, $beforeFrValue); - $this->assertInstanceOf(I18nValue::class, $afterEnValue); - $this->assertEquals('new value', $afterEnValue->getValue()); - $this->assertEquals($value, $afterFrValue); + self::assertEquals($this->value, $beforeEnValue); + self::assertEquals($value, $beforeFrValue); + self::assertInstanceOf(I18nValue::class, $afterEnValue); + self::assertEquals('new value', $afterEnValue->getValue()); + self::assertEquals($value, $afterFrValue); } public function testAddValueWhenLanguageIsNotReference(): void { @@ -578,11 +578,11 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { $afterEnValue = $getTargetedValue($data, 'en'); $afterFrValue = $getTargetedValue($data, 'fr'); - $this->assertEquals($this->value, $beforeEnValue); - $this->assertEquals($this->value, $beforeFrValue); - $this->assertEquals($this->value, $afterEnValue); - $this->assertInstanceOf(I18nValue::class, $afterFrValue); - $this->assertEquals('new value', $afterFrValue->getValue()); + self::assertEquals($this->value, $beforeEnValue); + self::assertEquals($this->value, $beforeFrValue); + self::assertEquals($this->value, $afterEnValue); + self::assertInstanceOf(I18nValue::class, $afterFrValue); + self::assertEquals('new value', $afterFrValue->getValue()); } public function testRemoveKeyWhenKeyDoesNotExist(): void { @@ -605,7 +605,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { ]); $data = new I18nData($rawData); $data->removeKey('file2.l1.l2'); - $this->assertEquals([ + self::assertEquals([ 'en' => [ 'file1.php' => [ 'file1.l1.l2.k1' => $this->value, @@ -643,7 +643,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { ]); $data = new I18nData($rawData); $data->removeKey('file3.l1.l2.k1'); - $this->assertEquals([ + self::assertEquals([ 'en' => [ 'file1.php' => [ 'file1.l1.l2.k1' => $this->value, @@ -679,9 +679,9 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { $value = $this->getMockBuilder(I18nValue::class) ->disableOriginalConstructor() ->getMock(); - $value->expects($this->exactly(2)) + $value->expects(self::exactly(2)) ->method('unmarkAsIgnore'); - $value->expects($this->once()) + $value->expects(self::once()) ->method('markAsIgnore'); $rawData = array_merge($this->referenceData, [ @@ -701,12 +701,12 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { $value = $this->getMockBuilder(I18nValue::class) ->disableOriginalConstructor() ->getMock(); - $value->expects($this->exactly(2)) + $value->expects(self::exactly(2)) ->method('unmarkAsIgnore'); - $value->expects($this->once()) + $value->expects(self::once()) ->method('markAsIgnore'); - $this->value->expects($this->atLeast(2)) + $this->value->expects(self::atLeast(2)) ->method('equal') ->with($value) ->willReturn(true); @@ -730,7 +730,7 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { 'nl' => [], ]); $data = new I18nData($rawData); - $this->assertEquals($this->referenceData['en'], $data->getLanguage('en')); + self::assertEquals($this->referenceData['en'], $data->getLanguage('en')); } public function testGetReferenceLanguage(): void { @@ -739,6 +739,6 @@ class I18nDataTest extends PHPUnit\Framework\TestCase { 'nl' => [], ]); $data = new I18nData($rawData); - $this->assertEquals($this->referenceData['en'], $data->getReferenceLanguage()); + self::assertEquals($this->referenceData['en'], $data->getReferenceLanguage()); } } diff --git a/tests/cli/i18n/I18nFileTest.php b/tests/cli/i18n/I18nFileTest.php index d584fdd22..47464c2e2 100644 --- a/tests/cli/i18n/I18nFileTest.php +++ b/tests/cli/i18n/I18nFileTest.php @@ -12,7 +12,7 @@ class I18nFileTest extends PHPUnit\Framework\TestCase { $after = $this->computeFilesHash(); - $this->assertEquals($before, $after); + self::assertEquals($before, $after); } /** @return array<string,string|false> */ @@ -30,7 +30,7 @@ class I18nFileTest extends PHPUnit\Framework\TestCase { continue; } - $hashes[$file->getPathName()] = sha1_file($file->getPathName()); + $hashes[$file->getPathname()] = sha1_file($file->getPathname()); } } diff --git a/tests/cli/i18n/I18nUsageValidatorTest.php b/tests/cli/i18n/I18nUsageValidatorTest.php index fdd75bf64..163c0ce82 100644 --- a/tests/cli/i18n/I18nUsageValidatorTest.php +++ b/tests/cli/i18n/I18nUsageValidatorTest.php @@ -16,23 +16,23 @@ class I18nUsageValidatorTest extends PHPUnit\Framework\TestCase { public function testDisplayReport(): void { $validator = new I18nUsageValidator([], []); - $this->assertEquals("There is no data.\n", $validator->displayReport()); + self::assertEquals("There is no data.\n", $validator->displayReport()); $reflectionTotalEntries = new ReflectionProperty(I18nUsageValidator::class, 'totalEntries'); $reflectionTotalEntries->setAccessible(true); $reflectionTotalEntries->setValue($validator, 100); - $this->assertEquals(" 0.0% of translation keys are unused.\n", $validator->displayReport()); + self::assertEquals(" 0.0% of translation keys are unused.\n", $validator->displayReport()); $reflectionFailedEntries = new ReflectionProperty(I18nUsageValidator::class, 'failedEntries'); $reflectionFailedEntries->setAccessible(true); $reflectionFailedEntries->setValue($validator, 25); - $this->assertEquals(" 25.0% of translation keys are unused.\n", $validator->displayReport()); + self::assertEquals(" 25.0% of translation keys are unused.\n", $validator->displayReport()); $reflectionFailedEntries->setValue($validator, 100); - $this->assertEquals("100.0% of translation keys are unused.\n", $validator->displayReport()); + self::assertEquals("100.0% of translation keys are unused.\n", $validator->displayReport()); $reflectionFailedEntries->setValue($validator, 200); @@ -43,8 +43,8 @@ class I18nUsageValidatorTest extends PHPUnit\Framework\TestCase { public function testValidateWhenNoData(): void { $validator = new I18nUsageValidator([], []); - $this->assertTrue($validator->validate()); - $this->assertEquals('', $validator->displayResult()); + self::assertTrue($validator->validate()); + self::assertEquals('', $validator->displayResult()); } public function testValidateWhenParentKeyExistsWithoutTransformation(): void { @@ -59,8 +59,8 @@ class I18nUsageValidatorTest extends PHPUnit\Framework\TestCase { 'file1.l1.l2._', 'file2.l1.l2._', ]); - $this->assertTrue($validator->validate()); - $this->assertEquals('', $validator->displayResult()); + self::assertTrue($validator->validate()); + self::assertEquals('', $validator->displayResult()); } public function testValidateWhenParentKeyExistsWithTransformation(): void { @@ -75,8 +75,8 @@ class I18nUsageValidatorTest extends PHPUnit\Framework\TestCase { 'file1.l1.l2', 'file2.l1.l2', ]); - $this->assertTrue($validator->validate()); - $this->assertEquals('', $validator->displayResult()); + self::assertTrue($validator->validate()); + self::assertEquals('', $validator->displayResult()); } public function testValidateWhenParentKeyDoesNotExist(): void { @@ -88,8 +88,8 @@ class I18nUsageValidatorTest extends PHPUnit\Framework\TestCase { 'file2.l1.l2._' => $this->value, ], ], []); - $this->assertFalse($validator->validate()); - $this->assertEquals("Unused key file1.l1.l2._ - \nUnused key file2.l1.l2._ - \n", $validator->displayResult()); + self::assertFalse($validator->validate()); + self::assertEquals("Unused key file1.l1.l2._ - \nUnused key file2.l1.l2._ - \n", $validator->displayResult()); } public function testValidateWhenChildKeyExists(): void { @@ -104,8 +104,8 @@ class I18nUsageValidatorTest extends PHPUnit\Framework\TestCase { 'file1.l1.l2.k1', 'file2.l1.l2.k1', ]); - $this->assertTrue($validator->validate()); - $this->assertEquals('', $validator->displayResult()); + self::assertTrue($validator->validate()); + self::assertEquals('', $validator->displayResult()); } public function testValidateWhenChildKeyDoesNotExist(): void { @@ -117,7 +117,7 @@ class I18nUsageValidatorTest extends PHPUnit\Framework\TestCase { 'file2.l1.l2.k1' => $this->value, ], ], []); - $this->assertFalse($validator->validate()); - $this->assertEquals("Unused key file1.l1.l2.k1 - \nUnused key file2.l1.l2.k1 - \n", $validator->displayResult()); + self::assertFalse($validator->validate()); + self::assertEquals("Unused key file1.l1.l2.k1 - \nUnused key file2.l1.l2.k1 - \n", $validator->displayResult()); } } diff --git a/tests/cli/i18n/I18nValueTest.php b/tests/cli/i18n/I18nValueTest.php index 7a1109a20..4454ea073 100644 --- a/tests/cli/i18n/I18nValueTest.php +++ b/tests/cli/i18n/I18nValueTest.php @@ -5,55 +5,55 @@ require_once __DIR__ . '/../../../cli/i18n/I18nValue.php'; class I18nValueTest extends PHPUnit\Framework\TestCase { public function testConstructorWithoutState(): void { $value = new I18nValue('some value'); - $this->assertEquals('some value', $value->getValue()); - $this->assertFalse($value->isIgnore()); - $this->assertFalse($value->isTodo()); + self::assertEquals('some value', $value->getValue()); + self::assertFalse($value->isIgnore()); + self::assertFalse($value->isTodo()); } public function testConstructorWithUnknownState(): void { $value = new I18nValue('some value -> unknown'); - $this->assertEquals('some value', $value->getValue()); - $this->assertFalse($value->isIgnore()); - $this->assertFalse($value->isTodo()); + self::assertEquals('some value', $value->getValue()); + self::assertFalse($value->isIgnore()); + self::assertFalse($value->isTodo()); } public function testConstructorWithTodoState(): void { $value = new I18nValue('some value -> todo'); - $this->assertEquals('some value', $value->getValue()); - $this->assertFalse($value->isIgnore()); - $this->assertTrue($value->isTodo()); + self::assertEquals('some value', $value->getValue()); + self::assertFalse($value->isIgnore()); + self::assertTrue($value->isTodo()); } public function testConstructorWithIgnoreState(): void { $value = new I18nValue('some value -> ignore'); - $this->assertEquals('some value', $value->getValue()); - $this->assertTrue($value->isIgnore()); - $this->assertFalse($value->isTodo()); + self::assertEquals('some value', $value->getValue()); + self::assertTrue($value->isIgnore()); + self::assertFalse($value->isTodo()); } public function testClone(): void { $value = new I18nValue('some value'); $clonedValue = clone $value; - $this->assertEquals('some value', $value->getValue()); - $this->assertEquals('some value', $clonedValue->getValue()); - $this->assertFalse($value->isIgnore()); - $this->assertFalse($clonedValue->isIgnore()); - $this->assertFalse($value->isTodo()); - $this->assertTrue($clonedValue->isTodo()); + self::assertEquals('some value', $value->getValue()); + self::assertEquals('some value', $clonedValue->getValue()); + self::assertFalse($value->isIgnore()); + self::assertFalse($clonedValue->isIgnore()); + self::assertFalse($value->isTodo()); + self::assertTrue($clonedValue->isTodo()); } public function testEqualWhenValueIsIdentical(): void { $value = new I18nValue('some value'); $clonedValue = clone $value; - $this->assertTrue($value->equal($clonedValue)); - $this->assertTrue($clonedValue->equal($value)); + self::assertTrue($value->equal($clonedValue)); + self::assertTrue($clonedValue->equal($value)); } public function testEqualWhenValueIsDifferent(): void { $value = new I18nValue('some value'); $otherValue = new I18nValue('some other value'); - $this->assertFalse($value->equal($otherValue)); - $this->assertFalse($otherValue->equal($value)); + self::assertFalse($value->equal($otherValue)); + self::assertFalse($otherValue->equal($value)); } public function testStates(): void { @@ -61,23 +61,23 @@ class I18nValueTest extends PHPUnit\Framework\TestCase { $reflectionProperty->setAccessible(true); $value = new I18nValue('some value'); - $this->assertNull($reflectionProperty->getValue($value)); + self::assertNull($reflectionProperty->getValue($value)); $value->markAsDirty(); - $this->assertEquals('dirty', $reflectionProperty->getValue($value)); + self::assertEquals('dirty', $reflectionProperty->getValue($value)); $value->unmarkAsIgnore(); - $this->assertEquals('dirty', $reflectionProperty->getValue($value)); + self::assertEquals('dirty', $reflectionProperty->getValue($value)); $value->markAsIgnore(); - $this->assertEquals('ignore', $reflectionProperty->getValue($value)); + self::assertEquals('ignore', $reflectionProperty->getValue($value)); $value->unmarkAsIgnore(); - $this->assertNull($reflectionProperty->getValue($value)); + self::assertNull($reflectionProperty->getValue($value)); $value->markAsTodo(); - $this->assertEquals('todo', $reflectionProperty->getValue($value)); + self::assertEquals('todo', $reflectionProperty->getValue($value)); } public function testToString(): void { $value = new I18nValue('some value'); - $this->assertEquals('some value', $value->__toString()); + self::assertEquals('some value', $value->__toString()); $value->markAsTodo(); - $this->assertEquals('some value -> todo', $value->__toString()); + self::assertEquals('some value -> todo', $value->__toString()); } } diff --git a/tests/lib/CssXPath/CssXPathTest.php b/tests/lib/CssXPath/CssXPathTest.php index 8d5a943ec..1cfbcf58f 100644 --- a/tests/lib/CssXPath/CssXPathTest.php +++ b/tests/lib/CssXPath/CssXPathTest.php @@ -3,6 +3,6 @@ class CssXPathTest extends PHPUnit\Framework\TestCase { public function testCssXPathTranslatorClassExists(): void { - $this->assertTrue(class_exists('Gt\\CssXPath\\Translator')); + self::assertTrue(class_exists('Gt\\CssXPath\\Translator')); } } diff --git a/tests/lib/Minz/MigratorTest.php b/tests/lib/Minz/MigratorTest.php index 7d8016139..4ff02e837 100644 --- a/tests/lib/Minz/MigratorTest.php +++ b/tests/lib/Minz/MigratorTest.php @@ -12,9 +12,9 @@ class MigratorTest extends TestCase }); $migrations = $migrator->migrations(); - $this->assertArrayHasKey('foo', $migrations); + self::assertArrayHasKey('foo', $migrations); $result = $migrations['foo'](); - $this->assertTrue($result); + self::assertTrue($result); } public function testAddMigrationFailsIfUncallableMigration(): void { @@ -40,7 +40,7 @@ class MigratorTest extends TestCase $migrations = $migrator->migrations(); - $this->assertSame($expected_versions, array_keys($migrations)); + self::assertSame($expected_versions, array_keys($migrations)); } public function testSetAppliedVersions(): void { @@ -51,7 +51,7 @@ class MigratorTest extends TestCase $migrator->setAppliedVersions(['foo']); - $this->assertSame(['foo'], $migrator->appliedVersions()); + self::assertSame(['foo'], $migrator->appliedVersions()); } public function testSetAppliedVersionsTrimArgument(): void { @@ -62,7 +62,7 @@ class MigratorTest extends TestCase $migrator->setAppliedVersions(["foo\n"]); - $this->assertSame(['foo'], $migrator->appliedVersions()); + self::assertSame(['foo'], $migrator->appliedVersions()); } public function testSetAppliedVersionsFailsIfMigrationDoesNotExist(): void { @@ -85,7 +85,7 @@ class MigratorTest extends TestCase $versions = $migrator->versions(); - $this->assertSame(['bar', 'foo'], $versions); + self::assertSame(['bar', 'foo'], $versions); } public function testMigrate(): void { @@ -95,13 +95,13 @@ class MigratorTest extends TestCase $spy = true; return true; }); - $this->assertEmpty($migrator->appliedVersions()); + self::assertEmpty($migrator->appliedVersions()); $result = $migrator->migrate(); - $this->assertTrue($spy); - $this->assertSame(['foo'], $migrator->appliedVersions()); - $this->assertSame([ + self::assertTrue($spy); + self::assertSame(['foo'], $migrator->appliedVersions()); + self::assertSame([ 'foo' => true, ], $result); } @@ -119,8 +119,8 @@ class MigratorTest extends TestCase $result = $migrator->migrate(); - $this->assertSame(['1_foo', '2_foo'], $migrator->appliedVersions()); - $this->assertSame([ + self::assertSame(['1_foo', '2_foo'], $migrator->appliedVersions()); + self::assertSame([ '1_foo' => true, '2_foo' => true, ], $result); @@ -137,8 +137,8 @@ class MigratorTest extends TestCase $result = $migrator->migrate(); - $this->assertFalse($spy); - $this->assertSame([], $result); + self::assertFalse($spy); + self::assertSame([], $result); } public function testMigrateCallNonAppliedBetweenTwoApplied(): void { @@ -156,8 +156,8 @@ class MigratorTest extends TestCase $result = $migrator->migrate(); - $this->assertSame(['1_foo', '2_foo', '3_foo'], $migrator->appliedVersions()); - $this->assertSame([ + self::assertSame(['1_foo', '2_foo', '3_foo'], $migrator->appliedVersions()); + self::assertSame([ '2_foo' => true, ], $result); } @@ -173,8 +173,8 @@ class MigratorTest extends TestCase $result = $migrator->migrate(); - $this->assertSame(['1_foo'], $migrator->appliedVersions()); - $this->assertSame([ + self::assertSame(['1_foo'], $migrator->appliedVersions()); + self::assertSame([ '1_foo' => true, '2_foo' => false, ], $result); @@ -193,9 +193,9 @@ class MigratorTest extends TestCase $result = $migrator->migrate(); - $this->assertEmpty($migrator->appliedVersions()); - $this->assertFalse($spy); - $this->assertSame([ + self::assertEmpty($migrator->appliedVersions()); + self::assertFalse($spy); + self::assertSame([ '1_foo' => false, ], $result); } @@ -208,8 +208,8 @@ class MigratorTest extends TestCase $result = $migrator->migrate(); - $this->assertEmpty($migrator->appliedVersions()); - $this->assertSame([ + self::assertEmpty($migrator->appliedVersions()); + self::assertSame([ 'foo' => 'Oops, it failed.', ], $result); } @@ -223,7 +223,7 @@ class MigratorTest extends TestCase $upToDate = $migrator->upToDate(); - $this->assertTrue($upToDate); + self::assertTrue($upToDate); } public function testUpToDateIfRemainingMigration(): void { @@ -238,7 +238,7 @@ class MigratorTest extends TestCase $upToDate = $migrator->upToDate(); - $this->assertFalse($upToDate); + self::assertFalse($upToDate); } public function testUpToDateIfNoMigrations(): void { @@ -246,7 +246,7 @@ class MigratorTest extends TestCase $upToDate = $migrator->upToDate(); - $this->assertTrue($upToDate); + self::assertTrue($upToDate); } public function testConstructorLoadsDirectory(): void { @@ -256,47 +256,47 @@ class MigratorTest extends TestCase $migrations = $migrator->migrations(); - $this->assertSame($expected_versions, array_keys($migrations)); + self::assertSame($expected_versions, array_keys($migrations)); } public function testExecute(): void { $migrations_path = TESTS_PATH . '/fixtures/migrations/'; $applied_migrations_path = tempnam('/tmp', 'applied_migrations.txt'); - $this->assertIsString($applied_migrations_path); + self::assertIsString($applied_migrations_path); $result = Minz_Migrator::execute($migrations_path, $applied_migrations_path); - $this->assertTrue($result); + self::assertTrue($result); $versions = file_get_contents($applied_migrations_path); - $this->assertSame("2019_12_22_FooBar\n2019_12_23_Baz", $versions); + self::assertSame("2019_12_22_FooBar\n2019_12_23_Baz", $versions); @unlink($applied_migrations_path); } public function testExecuteWithAlreadyAppliedMigration(): void { $migrations_path = TESTS_PATH . '/fixtures/migrations/'; $applied_migrations_path = tempnam('/tmp', 'applied_migrations.txt'); - $this->assertIsString($applied_migrations_path); + self::assertIsString($applied_migrations_path); file_put_contents($applied_migrations_path, '2019_12_22_FooBar'); $result = Minz_Migrator::execute($migrations_path, $applied_migrations_path); - $this->assertTrue($result); + self::assertTrue($result); $versions = file_get_contents($applied_migrations_path); - $this->assertSame("2019_12_22_FooBar\n2019_12_23_Baz", $versions); + self::assertSame("2019_12_22_FooBar\n2019_12_23_Baz", $versions); @unlink($applied_migrations_path); } public function testExecuteWithAppliedMigrationInDifferentOrder(): void { $migrations_path = TESTS_PATH . '/fixtures/migrations/'; $applied_migrations_path = tempnam('/tmp', 'applied_migrations.txt'); - $this->assertIsString($applied_migrations_path); + self::assertIsString($applied_migrations_path); file_put_contents($applied_migrations_path, "2019_12_23_Baz\n2019_12_22_FooBar"); $result = Minz_Migrator::execute($migrations_path, $applied_migrations_path); - $this->assertTrue($result); + self::assertTrue($result); $versions = file_get_contents($applied_migrations_path); // if the order changes, it probably means the first versions comparison test doesn’t work anymore - $this->assertSame("2019_12_23_Baz\n2019_12_22_FooBar", $versions); + self::assertSame("2019_12_23_Baz\n2019_12_22_FooBar", $versions); @unlink($applied_migrations_path); } @@ -304,11 +304,11 @@ class MigratorTest extends TestCase $migrations_path = TESTS_PATH . '/fixtures/migrations/'; $applied_migrations_path = tempnam('/tmp', 'applied_migrations.txt'); $expected_result = "Cannot open the {$applied_migrations_path} file"; - $this->assertIsString($applied_migrations_path); + self::assertIsString($applied_migrations_path); unlink($applied_migrations_path); $result = Minz_Migrator::execute($migrations_path, $applied_migrations_path); - $this->assertSame($expected_result, $result); + self::assertSame($expected_result, $result); @unlink($applied_migrations_path); } @@ -316,14 +316,14 @@ class MigratorTest extends TestCase $migrations_path = TESTS_PATH . '/fixtures/migrations_with_failing/'; $applied_migrations_path = tempnam('/tmp', 'applied_migrations.txt'); $expected_result = 'A migration failed to be applied, please see previous logs.'; - $this->assertIsString($applied_migrations_path); + self::assertIsString($applied_migrations_path); $result = Minz_Migrator::execute($migrations_path, $applied_migrations_path); - $this->assertIsString($result); + self::assertIsString($result); [$result,] = explode("\n", $result, 2); - $this->assertSame($expected_result, $result); + self::assertSame($expected_result, $result); $versions = file_get_contents($applied_migrations_path); - $this->assertSame('2020_01_11_FooBar', $versions); + self::assertSame('2020_01_11_FooBar', $versions); @unlink($applied_migrations_path); } } diff --git a/tests/lib/PHPMailer/PHPMailerTest.php b/tests/lib/PHPMailer/PHPMailerTest.php index b39cf2cb0..5873c58a9 100644 --- a/tests/lib/PHPMailer/PHPMailerTest.php +++ b/tests/lib/PHPMailer/PHPMailerTest.php @@ -3,6 +3,6 @@ class PHPMailerTest extends PHPUnit\Framework\TestCase { public function testPHPMailerClassExists(): void { - $this->assertTrue(class_exists('PHPMailer\\PHPMailer\\PHPMailer')); + self::assertTrue(class_exists('PHPMailer\\PHPMailer\\PHPMailer')); } } |
