Skip to content

feat: add tests folder with phpstan level 8 solved all errors #417

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

Open
shakaran wants to merge 11 commits into
base: master
Choose a base branch
from
Open

feat: add tests folder with phpstan level 8 solved all errors #417

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
73826fc
feat: solve phpstan error with null type declared
shakaran Dec 8, 2025
22f988d
feat: resolve phpstan errors in tests
shakaran Dec 9, 2025
65678ec
feat: update phpstan type for tests and errors
shakaran Dec 9, 2025
6e591d5
feat: resolve all errors in phpstan tests
shakaran Dec 9, 2025
f1db5be
fix: avoid assert not null changing MenuTestCase and ItemInterface
shakaran Dec 9, 2025
3f6fef6
fix: use IteratorIterator as suggested by @stof
shakaran Dec 9, 2025
2d5281f
fix: refactor tests for back to use ArrayAccess
shakaran Dec 9, 2025
b70d44a
fix: remove intermediate variables not needed anymore
shakaran Dec 9, 2025
584ec1e
fix: restore cases of ArrayAccess instead getChildren
shakaran Dec 9, 2025
2ccbffb
fix: remove intermediate variables and avoid getChildren()
shakaran Dec 9, 2025
cf50474
feat: add return types only nullables
shakaran Dec 9, 2025
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
Prev Previous commit
Next Next commit
fix: restore cases of ArrayAccess instead getChildren
  • Loading branch information
@shakaran
shakaran committed Dec 9, 2025
commit 584ec1e135bf7b0fc3acd15d54ed7e8fb0376a29
17 changes: 9 additions & 8 deletions tests/Knp/Menu/Tests/MenuItemTreeTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -182,9 +182,8 @@ public function testCountable(): void

public function testGetChildren(): void
{
$children = $this->ch4->getChildren();
$this->assertCount(1, $children);
$this->assertEquals($this->gc1->getName(), $children['Grandchild 1']->getName());
$this->assertCount(1, $this->ch4->getChildren());
$this->assertSame($this->gc1, $this->ch4['Grandchild 1']);
}

public function testGetFirstChild(): void
Expand Down Expand Up @@ -233,8 +232,8 @@ public function testAddChildFailsIfInAnotherMenu(): void

public function testGetChild(): void
{
$this->assertSame($this->gc1, $this->ch4->getChild('Grandchild 1'));
$this->assertNull($this->ch4->getChild('nonexistentchild'));
$this->assertSame($this->gc1, $this->ch4['Grandchild 1']);
$this->assertNull($this->ch4['nonexistentchild']);
}

public function testRemoveChild(): void
Expand Down Expand Up @@ -274,9 +273,9 @@ public function testUpdateChildAfterRename(): void
$pt1 = $this->pt1;
$menu = $this->menu;
$pt1->setName('Temp name');
$this->assertSame($pt1, $menu->getChild('Temp name'));
$this->assertSame($pt1, $menu['Temp name']);
$this->assertEquals(['Temp name', 'Parent 2'], \array_keys($menu->getChildren()));
$this->assertNull($menu->getChild('Parent 1'));
$this->assertNull($menu['Parent 1']);
}

public function testRenameToExistingSiblingNameThrowAnException(): void
Expand All @@ -290,7 +289,9 @@ public function testGetUri(): void
{
$this->addChildWithExternalUrl();
$this->assertNull($this->pt1->getUri());
$this->assertEquals('http://www.symfony-reloaded.org', $this->menu->getChildren()['child']->getUri());
$child = $this->menu['child'];
/** @var ItemInterface $child */
$this->assertEquals('http://www.symfony-reloaded.org', $child->getUri());
}

protected function addChildWithExternalUrl(): void
Expand Down