diff options
| author |  Alexandre Alapetite <alexandre@alapetite.fr>
| 2024-12-28 23:58:00 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2024-12-28 23:58:00 +0100 |
| commit | c29cbb7b8be95fee249ed1a21dce98a4772d92e2 (patch) | |
| tree | b116f6033ea081c6cc5f82ce83156f2a7021166c /tests/app/Models/UserQueryTest.php | |
| parent | 33cdfbb309c61167cf1c81273eb242f94ca8f996 (diff) | |
Fix regressions on some array structures (#7155)
regressions from https://github.com/FreshRSS/FreshRSS/pull/7131
fix https://github.com/FreshRSS/FreshRSS/issues/7154
Diffstat (limited to 'tests/app/Models/UserQueryTest.php')
| -rw-r--r-- | tests/app/Models/UserQueryTest.php | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/tests/app/Models/UserQueryTest.php b/tests/app/Models/UserQueryTest.php index 04fb74fd0..3017f7897 100644 --- a/tests/app/Models/UserQueryTest.php +++ b/tests/app/Models/UserQueryTest.php @@ -23,12 +23,13 @@ class UserQueryTest extends TestCase { $category_name = 'some category name'; /** @var FreshRSS_Category&PHPUnit\Framework\MockObject\MockObject */ $cat = $this->createMock(FreshRSS_Category::class); + $cat->method('id')->withAnyParameters()->willReturn(1); $cat->expects(self::atLeastOnce()) ->method('name') ->withAnyParameters() ->willReturn($category_name); $query = ['get' => 'c_1']; - $user_query = new FreshRSS_UserQuery($query, [1 => $cat], []); + $user_query = new FreshRSS_UserQuery($query, [$cat], []); self::assertSame($category_name, $user_query->getGetName()); self::assertSame('category', $user_query->getGetType()); } @@ -47,12 +48,13 @@ class UserQueryTest extends TestCase { ->willReturn($feed_name); /** @var FreshRSS_Category&PHPUnit\Framework\MockObject\MockObject */ $cat = $this->createMock(FreshRSS_Category::class); + $cat->method('id')->withAnyParameters()->willReturn(1); $cat->expects(self::atLeastOnce()) ->method('feeds') ->withAnyParameters() ->willReturn([1 => $feed]); $query = ['get' => 'f_1']; - $user_query = new FreshRSS_UserQuery($query, [1 => $cat], []); + $user_query = new FreshRSS_UserQuery($query, [$cat], []); self::assertSame($feed_name, $user_query->getGetName()); self::assertSame('feed', $user_query->getGetType()); } @@ -143,12 +145,13 @@ class UserQueryTest extends TestCase { public function testIsDeprecated_whenCategoryExists_returnFalse(): void { /** @var FreshRSS_Category&PHPUnit\Framework\MockObject\MockObject */ $cat = $this->createMock(FreshRSS_Category::class); + $cat->method('id')->withAnyParameters()->willReturn(1); $cat->expects(self::atLeastOnce()) ->method('name') ->withAnyParameters() ->willReturn('cat 1'); $query = ['get' => 'c_1']; - $user_query = new FreshRSS_UserQuery($query, [1 => $cat], []); + $user_query = new FreshRSS_UserQuery($query, [$cat], []); self::assertFalse($user_query->isDeprecated()); } @@ -171,24 +174,26 @@ class UserQueryTest extends TestCase { ->willReturn('feed 1'); /** @var FreshRSS_Category&PHPUnit\Framework\MockObject\MockObject */ $cat = $this->createMock(FreshRSS_Category::class); + $cat->method('id')->withAnyParameters()->willReturn(1); $cat->expects(self::atLeastOnce()) ->method('feeds') ->withAnyParameters() ->willReturn([1 => $feed]); $query = ['get' => 'f_1']; - $user_query = new FreshRSS_UserQuery($query, [1 => $cat], []); + $user_query = new FreshRSS_UserQuery($query, [$cat], []); self::assertFalse($user_query->isDeprecated()); } public function testIsDeprecated_whenFeedDoesNotExist_returnTrue(): void { /** @var FreshRSS_Category&PHPUnit\Framework\MockObject\MockObject */ $cat = $this->createMock(FreshRSS_Category::class); + $cat->method('id')->withAnyParameters()->willReturn(1); $cat->expects(self::atLeastOnce()) ->method('feeds') ->withAnyParameters() ->willReturn([]); $query = ['get' => 'f_1']; - $user_query = new FreshRSS_UserQuery($query, [1 => $cat], []); + $user_query = new FreshRSS_UserQuery($query, [$cat], []); self::assertTrue($user_query->isDeprecated()); } |