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: refactor tests for back to use ArrayAccess
  • Loading branch information
@shakaran
shakaran committed Dec 9, 2025
commit 2d5281f9d73c3ccd2c095bbe024dbdcb94bcdd52
13 changes: 9 additions & 4 deletions tests/Knp/Menu/Tests/MenuItemTreeTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
namespace Knp\Menu\Tests;

use Knp\Menu\FactoryInterface;
use Knp\Menu\ItemInterface;
use Knp\Menu\MenuItem;

final class TestMenuItem extends MenuItem
Expand Down Expand Up @@ -165,13 +166,17 @@ public function testArrayAccess(): void
{
$menu = $this->menu;
$menu->addChild('Child Menu');
$this->assertEquals('Child Menu', $menu->getChildren()['Child Menu']->getName());
$childMenu = $menu['Child Menu'];
/** @var ItemInterface $childMenu */
$this->assertEquals('Child Menu', $childMenu->getName());
$this->assertNull($menu->getChild('Fake'));

$menu->addChild('New Child', ['label' => 'New Label']);
$this->assertEquals(MenuItem::class, \get_class($menu->getChildren()['New Child']));
$this->assertEquals('New Child', $menu->getChildren()['New Child']->getName());
$this->assertEquals('New Label', $menu->getChildren()['New Child']->getLabel());
$newChild = $menu['New Child'];
/** @var ItemInterface $newChild */
$this->assertEquals(MenuItem::class, \get_class($newChild));
$this->assertEquals('New Child', $newChild->getName());
$this->assertEquals('New Label', $newChild->getLabel());

unset($menu['New Child']);
$this->assertNull($menu['New Child']);
Expand Down