Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update tests for phpunit 10 and 11 #219

Merged
merged 1 commit into from
Dec 7, 2024
Merged

chore: update tests for phpunit 10 and 11 #219

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
52 changes: 30 additions & 22 deletions tests/PaginationTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@

namespace tests;

use PHPUnit\Framework\Attributes\DataProvider;
use PHPUnit\Framework\TestCase;
use Rancoud\Pagination\Item;
use Rancoud\Pagination\Pagination;
Expand Down Expand Up @@ -115,7 +116,7 @@ public function testConfiguration(): void
/**
* @return array
*/
public function dataCeil(): array
public static function dataCeil(): array
{
$configuration = [
'use_previous' => true,
Expand Down Expand Up @@ -192,7 +193,7 @@ public function dataCeil(): array
'count' => 1,
'per_page' => 1
],
'data_out' => [
'dataOut' => [
'previous' => $previous,
'links' => [],
'next' => null
Expand All @@ -205,7 +206,7 @@ public function dataCeil(): array
'count' => 1,
'per_page' => 1
],
'data_out' => [
'dataOut' => [
'previous' => null,
'links' => [$linkPage1Current],
'next' => null,
Expand All @@ -218,7 +219,7 @@ public function dataCeil(): array
'count' => 1,
'per_page' => -9
],
'data_out' => [
'dataOut' => [
'previous' => null,
'links' => [$linkPage1Current],
'next' => null,
Expand All @@ -231,7 +232,7 @@ public function dataCeil(): array
'count' => -9,
'per_page' => 1
],
'data_out' => [
'dataOut' => [
'previous' => null,
'links' => [],
'next' => null,
Expand All @@ -244,7 +245,7 @@ public function dataCeil(): array
'count' => 6,
'per_page' => 2
],
'data_out' => [
'dataOut' => [
'previous' => $previousPage3,
'links' => [],
'next' => null
Expand All @@ -257,7 +258,7 @@ public function dataCeil(): array
'count' => 6,
'per_page' => 2
],
'data_out' => [
'dataOut' => [
'previous' => null,
'links' => [
$linkPage1Current,
Expand All @@ -274,7 +275,7 @@ public function dataCeil(): array
'count' => 6,
'per_page' => -9
],
'data_out' => [
'dataOut' => [
'previous' => $previous,
'links' => [
$linkPage1,
Expand All @@ -292,7 +293,7 @@ public function dataCeil(): array
'count' => -9,
'per_page' => 2
],
'data_out' => [
'dataOut' => [
'previous' => null,
'links' => [],
'next' => null,
Expand All @@ -310,6 +311,7 @@ public function dataCeil(): array
*
* @throws PaginationException
*/
#[DataProvider('dataCeil')]
public function testIncorrectCeilCompute(array $configuration, array $params, array $dataOut): void
{
$p = new Pagination();
Expand All @@ -322,7 +324,7 @@ public function testIncorrectCeilCompute(array $configuration, array $params, ar
/**
* @return array
*/
public function dataShowAllLinks(): array
public static function dataShowAllLinks(): array
{
$configuration = [
'use_previous' => true,
Expand Down Expand Up @@ -395,7 +397,7 @@ public function dataShowAllLinks(): array
'count' => 50,
'per_page' => 1
],
'data_out' => [
'dataOut' => [
'previous' => null,
'links' => $linksCurrentPage1,
'next' => $nextPage2
Expand All @@ -408,7 +410,7 @@ public function dataShowAllLinks(): array
'count' => 50,
'per_page' => 1
],
'data_out' => [
'dataOut' => [
'previous' => $previousPage1,
'links' => $linksCurrentPage2,
'next' => $nextPage3
Expand All @@ -421,7 +423,7 @@ public function dataShowAllLinks(): array
'count' => 50,
'per_page' => 1
],
'data_out' => [
'dataOut' => [
'previous' => $previousPage49,
'links' => $linksCurrentPage50,
'next' => null
Expand All @@ -439,6 +441,7 @@ public function dataShowAllLinks(): array
*
* @throws PaginationException
*/
#[DataProvider('dataShowAllLinks')]
public function testShowAllLinks(array $configuration, array $params, array $dataOut): void
{
$p = new Pagination();
Expand All @@ -451,7 +454,7 @@ public function testShowAllLinks(array $configuration, array $params, array $dat
/**
* @return array
*/
public function dataAdjacentAndLimitConfiguration(): array
public static function dataAdjacentAndLimitConfiguration(): array
{
$linkCurrentPage20 = new Item();
$linkCurrentPage20->isDots = false;
Expand Down Expand Up @@ -527,7 +530,7 @@ public function dataAdjacentAndLimitConfiguration(): array
'count_pages_pair_adjacent' => 0,
],
'params' => $params,
'data_out' => [
'dataOut' => [
'links' => $currentPage,
]
],
Expand All @@ -537,7 +540,7 @@ public function dataAdjacentAndLimitConfiguration(): array
'count_pages_pair_adjacent' => 0,
],
'params' => $params,
'data_out' => [
'dataOut' => [
'links' => \array_merge($limitPagesLeft, $currentPage, $limitPagesRight),
]
],
Expand All @@ -547,7 +550,7 @@ public function dataAdjacentAndLimitConfiguration(): array
'count_pages_pair_adjacent' => 5,
],
'params' => $params,
'data_out' => [
'dataOut' => [
'links' => \array_merge($adjacentPagesLeft, $currentPage, $adjacentPagesRight),
]
],
Expand All @@ -557,7 +560,7 @@ public function dataAdjacentAndLimitConfiguration(): array
'count_pages_pair_adjacent' => 5,
],
'params' => $params,
'data_out' => [
'dataOut' => [
'links' => \array_merge($limitPagesLeft, $adjacentPagesLeft, $currentPage, $adjacentPagesRight, $limitPagesRight), // phpcs:ignore
]
]
Expand All @@ -573,6 +576,7 @@ public function dataAdjacentAndLimitConfiguration(): array
*
* @throws PaginationException
*/
#[DataProvider('dataAdjacentAndLimitConfiguration')]
public function testAdjacentAndLimitConfiguration(array $configuration, array $params, array $dataOut): void
{
$p = new Pagination();
Expand All @@ -585,7 +589,7 @@ public function testAdjacentAndLimitConfiguration(array $configuration, array $p
/**
* @return array
*/
public function dataRenderHtml(): array
public static function dataRenderHtml(): array
{
return [
'pretty html off' => [
Expand Down Expand Up @@ -913,6 +917,7 @@ public function dataRenderHtml(): array
*
* @throws PaginationException
*/
#[DataProvider('dataRenderHtml')]
public function testRenderHtml(array $configuration, array $params, string $expectedHtml): void
{
$p = new Pagination();
Expand All @@ -924,7 +929,7 @@ public function testRenderHtml(array $configuration, array $params, string $expe
/**
* @return array
*/
public function dataCountPages(): array
public static function dataCountPages(): array
{
return [
'1 item + 1 per page' => [1, 1, 1],
Expand All @@ -941,6 +946,7 @@ public function dataCountPages(): array
* @param int $countElementPerPage
* @param int $expected
*/
#[DataProvider('dataCountPages')]
public function testCountPages(int $countElements, int $countElementPerPage, int $expected): void
{
static::assertSame($expected, Pagination::countPages($countElements, $countElementPerPage));
Expand All @@ -949,7 +955,7 @@ public function testCountPages(int $countElements, int $countElementPerPage, int
/**
* @return array
*/
public function dataLocateItemInPage(): array
public static function dataLocateItemInPage(): array
{
return [
'1 per page + item index 1' => [1, 1, 1],
Expand All @@ -970,12 +976,13 @@ public function dataLocateItemInPage(): array
* @param int $itemIndex
* @param int $expected
*/
#[DataProvider('dataLocateItemInPage')]
public function testLocateItemInPage(int $countElementPerPage, int $itemIndex, int $expected): void
{
static::assertSame($expected, Pagination::locateItemInPage($countElementPerPage, $itemIndex));
}

public function dataPaginationException(): array
public static function dataPaginationException(): array
{
return [
'aria_label_nav' => [
Expand Down Expand Up @@ -1017,6 +1024,7 @@ public function dataPaginationException(): array
*
* @throws PaginationException
*/
#[DataProvider('dataPaginationException')]
public function testException(array $conf, string $message): void
{
$this->expectException(PaginationException::class);
Expand Down
Loading