From cc7a324ca1030b7d9eab95e269d491e0c91c16f4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Thomas=20M=C3=BCller?= Date: Mon, 18 Dec 2023 11:37:03 +0100 Subject: [PATCH] update DocBlock --- phpstan.neon | 3 +- src/Config/NavigationConfig.php | 6 +- src/Config/NavigationConfigInterface.php | 4 +- src/ContainerInterface.php | 5 +- src/ContainerTrait.php | 2 +- src/Exception/ExceptionInterface.php | 4 +- src/Navigation.php | 4 ++ src/Page/PageInterface.php | 45 ++++++------ src/Page/PageTrait.php | 71 +++++++++---------- src/Page/Route.php | 2 +- src/Page/Uri.php | 2 +- .../NavigationAbstractServiceFactory.php | 36 +--------- src/Service/NavigationFactoryTrait.php | 2 +- tests/NavigationTest.php | 3 + tests/Page/RouteTest.php | 5 +- tests/Page/UriTest.php | 5 +- .../NavigationAbstractServiceFactoryTest.php | 3 + 17 files changed, 85 insertions(+), 117 deletions(-) diff --git a/phpstan.neon b/phpstan.neon index 66f3c66..ba01646 100644 --- a/phpstan.neon +++ b/phpstan.neon @@ -102,4 +102,5 @@ parameters: - '~Parameter #1 \$pages of method Mimmi20\\Mezzio\\Navigation\\(Navigation|Page\\(Route|Uri))::addPages\(\) expects iterable, array given~' - '~Access to private property Mimmi20\\Mezzio\\Navigation\\Page\\(Route|Uri)::\$target~' - '~Access to an undefined property Mimmi20\\Mezzio\\Navigation\\Page\\(Route|Uri)::~' - - '~Offset .uri. does not exist on array~' + - '~::preparePages\(\) expects array\|string>>, array given~' + - '~will always evaluate to false~' diff --git a/src/Config/NavigationConfig.php b/src/Config/NavigationConfig.php index ab253b7..7ed766b 100644 --- a/src/Config/NavigationConfig.php +++ b/src/Config/NavigationConfig.php @@ -29,7 +29,7 @@ final class NavigationConfig implements NavigationConfigInterface private ServerRequestInterface | null $request = null; private AuthorizationInterface | null $authorization = null; - /** @var array>|null */ + /** @var array>|null */ private array | null $pages = null; /** @throws void */ @@ -93,7 +93,7 @@ public function setAuthorization(AuthorizationInterface | null $authorization = } /** - * @return array|string>>>|null + * @return array>|null * * @throws void */ @@ -103,7 +103,7 @@ public function getPages(): array | null } /** - * @param array> $pages + * @param array> $pages * * @throws void */ diff --git a/src/Config/NavigationConfigInterface.php b/src/Config/NavigationConfigInterface.php index 90ba098..f4abf52 100644 --- a/src/Config/NavigationConfigInterface.php +++ b/src/Config/NavigationConfigInterface.php @@ -54,14 +54,14 @@ public function getAuthorization(): AuthorizationInterface | null; public function setAuthorization(AuthorizationInterface | null $authorization = null): void; /** - * @return array|string>>>|null + * @return array>|null * * @throws void */ public function getPages(): array | null; /** - * @param array> $pages + * @param array> $pages * * @throws void */ diff --git a/src/ContainerInterface.php b/src/ContainerInterface.php index 9022147..7d080b0 100644 --- a/src/ContainerInterface.php +++ b/src/ContainerInterface.php @@ -20,7 +20,8 @@ /** * ContainerInterface class for Mimmi20\Mezzio\Navigation\Navigation classes. * - * @extends RecursiveIterator + * @template TPage of PageInterface + * @extends RecursiveIterator */ interface ContainerInterface extends Countable, RecursiveIterator { @@ -139,7 +140,7 @@ public function findAllBy(string $property, mixed $value): array; /** * Returns an array representation of all pages in container * - * @return array, array> + * @return array * * @throws void */ diff --git a/src/ContainerTrait.php b/src/ContainerTrait.php index 21d5df0..ef5d30d 100644 --- a/src/ContainerTrait.php +++ b/src/ContainerTrait.php @@ -371,7 +371,7 @@ final public function findAllBy(string $property, mixed $value): array /** * Returns an array representation of all pages in container * - * @return array, array> + * @return array * * @throws void */ diff --git a/src/Exception/ExceptionInterface.php b/src/Exception/ExceptionInterface.php index 6fb5851..bdc98d6 100644 --- a/src/Exception/ExceptionInterface.php +++ b/src/Exception/ExceptionInterface.php @@ -12,9 +12,11 @@ namespace Mimmi20\Mezzio\Navigation\Exception; +use Throwable; + /** * Navigation exception */ -interface ExceptionInterface +interface ExceptionInterface extends Throwable { } diff --git a/src/Navigation.php b/src/Navigation.php index 6229df5..3b4a1fe 100644 --- a/src/Navigation.php +++ b/src/Navigation.php @@ -12,8 +12,12 @@ namespace Mimmi20\Mezzio\Navigation; +use Mimmi20\Mezzio\Navigation\Page\PageInterface; + /** * A simple container class for {@link \Mimmi20\Mezzio\Navigation\Navigation} pages + * + * @implements ContainerInterface */ final class Navigation implements ContainerInterface { diff --git a/src/Page/PageInterface.php b/src/Page/PageInterface.php index 864fa79..d9a5e48 100644 --- a/src/Page/PageInterface.php +++ b/src/Page/PageInterface.php @@ -14,10 +14,11 @@ use Mimmi20\Mezzio\Navigation\ContainerInterface; use Mimmi20\Mezzio\Navigation\Exception; -use Traversable; /** * Interface for Mimmi20\Mezzio\Navigation\Page pages + * + * @extends ContainerInterface */ interface PageInterface extends ContainerInterface { @@ -29,7 +30,7 @@ interface PageInterface extends ContainerInterface * corresponds to setTarget(), and the option 'reset_params' corresponds to * the method setResetParams(). * - * @param iterable $options associative array of options to set + * @param iterable|bool|string> $options associative array of options to set * * @throws Exception\InvalidArgumentException if invalid options are given */ @@ -174,7 +175,7 @@ public function getTarget(): string | null; * prev, next, help, etc), and the value is a mixed value that could somehow * be considered a page. * - * @param iterable|PageInterface|string>|null $relations [optional] an associative array of + * @param iterable|iterable|PageInterface|string>|null $relations [optional] an associative array of * forward links to other pages * * @throws void @@ -192,7 +193,7 @@ public function setRel(iterable | null $relations = null): void; * @param string|null $relation [optional] name of relation to return. If not * given, all relations will be returned. * - * @return ContainerInterface|iterable|PageInterface|string>|PageInterface|string|null an array of relations. If $relation is not + * @return ContainerInterface|iterable|iterable|PageInterface|string>|PageInterface|string|null an array of relations. If $relation is not * specified, all relations will be returned in * an associative array. * @@ -208,7 +209,7 @@ public function getRel(string | null $relation = null): iterable | ContainerInte * prev, next, help, etc), and the value is a mixed value that could somehow * be considered a page. * - * @param iterable|PageInterface|string>|null $relations [optional] an associative array of + * @param iterable|iterable|PageInterface|string>|null $relations [optional] an associative array of * reverse links to other pages * * @throws void @@ -226,7 +227,7 @@ public function setRev(iterable | null $relations = null): void; * @param string|null $relation [optional] name of relation to return. If not * given, all relations will be returned. * - * @return ContainerInterface|iterable|PageInterface|string>|PageInterface|string|null an array of relations. If $relation is not + * @return ContainerInterface|iterable|iterable|PageInterface|string>|PageInterface|string|null an array of relations. If $relation is not * specified, all relations will be returned in * an associative array. * @@ -241,7 +242,6 @@ public function getRev(string | null $relation = null): iterable | ContainerInte * Default is null, which sets no * specific order. * - * @throws Exception\InvalidArgumentException if order is not integer or null * @throws void */ public function setOrder(int | float | string | null $order = null): void; @@ -263,7 +263,6 @@ public function getOrder(): int | null; * sets no resource. * * @throws Exception\InvalidArgumentException if $resource is invalid - * @throws void */ public function setResource(string $resource): void; @@ -412,7 +411,7 @@ public function getVisible(bool $recursive = false): bool; /** * Sets parent container * - * @param ContainerInterface|null $parent [optional] new parent to set. + * @param ContainerInterface|null $parent [optional] new parent to set. * Default is null which will set no parent. * * @throws Exception\InvalidArgumentException @@ -422,7 +421,7 @@ public function setParent(ContainerInterface | null $parent = null): void; /** * Returns parent container * - * @return ContainerInterface|null parent container or null + * @return ContainerInterface|null parent container or null * * @throws void */ @@ -434,8 +433,8 @@ public function getParent(): ContainerInterface | null; * If the given property is native (id, class, title, etc), the matching * set method will be used. Otherwise, it will be set as a custom property. * - * @param string $property property name - * @param bool|float|int|iterable|string|null $value value to set + * @param string $property property name + * @param bool|ContainerInterface|float|int|iterable|string)>|PageInterface|string|null $value value to set * * @throws Exception\InvalidArgumentException if property name is invalid */ @@ -450,7 +449,7 @@ public function set(string $property, bool | float | int | iterable | string | n * * @param string $property property name * - * @return bool|float|int|iterable|string|null the property's value or null + * @return bool|ContainerInterface|float|int|iterable|string)>|PageInterface|string|null the property's value or null * * @throws Exception\InvalidArgumentException if property name is invalid */ @@ -459,24 +458,22 @@ public function get(string $property): bool | float | int | iterable | string | /** * Adds a forward relation to the page * - * @param string $relation relation name (e.g. alternate, glossary, - * canonical, etc) - * @param array|ContainerInterface|PageInterface|string|Traversable $value value to set for relation + * @param string $relation relation name (e.g. alternate, glossary, canonical, etc) + * @param ContainerInterface|iterable|PageInterface|string $value value to set for relation * * @throws void */ - public function addRel(string $relation, array | ContainerInterface | self | string | Traversable $value): void; + public function addRel(string $relation, iterable | ContainerInterface | self | string $value): void; /** * Adds a reverse relation to the page * - * @param string $relation relation name (e.g. alternate, glossary, - * canonical, etc) - * @param array|ContainerInterface|PageInterface|string|Traversable $value value to set for relation + * @param string $relation relation name (e.g. alternate, glossary, canonical, etc) + * @param ContainerInterface|iterable|PageInterface|string $value value to set for relation * * @throws void */ - public function addRev(string $relation, array | ContainerInterface | self | string | Traversable $value): void; + public function addRev(string $relation, iterable | ContainerInterface | self | string $value): void; /** * Removes a forward relation from the page @@ -499,7 +496,7 @@ public function removeRev(string $relation): void; /** * Returns an array containing the defined forward relations * - * @return array defined forward relations + * @return array defined forward relations * * @throws void */ @@ -508,7 +505,7 @@ public function getDefinedRel(): array; /** * Returns an array containing the defined reverse relations * - * @return array defined reverse relations + * @return array defined reverse relations * * @throws void */ @@ -517,7 +514,7 @@ public function getDefinedRev(): array; /** * Returns custom properties as an array * - * @return array|bool|float|int|iterable|string|null> an array containing custom properties + * @return array|float|int|iterable|string)>|PageInterface|string|null> an array containing custom properties * * @throws void */ diff --git a/src/Page/PageTrait.php b/src/Page/PageTrait.php index 85e4011..47c5565 100644 --- a/src/Page/PageTrait.php +++ b/src/Page/PageTrait.php @@ -15,7 +15,6 @@ use Mimmi20\Mezzio\Navigation\ContainerInterface; use Mimmi20\Mezzio\Navigation\ContainerTrait; use Mimmi20\Mezzio\Navigation\Exception; -use Traversable; use function array_keys; use function array_merge; @@ -84,7 +83,7 @@ trait PageTrait * * @see http://www.w3.org/TR/html4/struct/links.html#h-12.3.1 * - * @var array|ContainerInterface|PageInterface|string|Traversable> + * @var array|iterable|PageInterface|string> */ private array $rel = []; @@ -93,7 +92,7 @@ trait PageTrait * * @see http://www.w3.org/TR/html4/struct/links.html#h-12.3.1 * - * @var array|ContainerInterface|PageInterface|string|Traversable> + * @var array|iterable|PageInterface|string> */ private array $rev = []; @@ -134,18 +133,20 @@ trait PageTrait /** * Parent container + * + * @var ContainerInterface|null */ private ContainerInterface | null $parent = null; /** * Custom page properties, used by __set(), __get() and __isset() * - * @var array|bool|float|int|iterable|string|null> + * @var array|float|int|iterable|string)>|PageInterface|string|null> */ private array $properties = []; /** - * @param array|iterable|null $options [optional] page options. Default is null, which should set defaults. + * @param iterable|bool|string|null>|null $options [optional] page options. Default is null, which should set defaults. * * @throws Exception\InvalidArgumentException if invalid options are given */ @@ -165,8 +166,8 @@ public function __construct(iterable | null $options = null) * * Magic overload for enabling $page->propname = $value. * - * @param string $name property name - * @param bool|ContainerInterface|float|int|iterable|string)>|PageInterface|string|null $value value to set + * @param string $name property name + * @param bool|ContainerInterface|float|int|iterable|string)>|PageInterface|string|null $value value to set * * @throws Exception\InvalidArgumentException if property name is invalid */ @@ -184,7 +185,7 @@ public function __set( * * @param string $name property name * - * @return bool|ContainerInterface|float|int|iterable|string)>|PageInterface|string|null property value or null + * @return bool|ContainerInterface|float|int|iterable|string)>|PageInterface|string|null property value or null * * @throws Exception\InvalidArgumentException if property name is invalid */ @@ -271,7 +272,7 @@ public function __toString(): string * corresponds to setTarget(), and the option 'reset_params' corresponds to * the method setResetParams(). * - * @param array|iterable $options associative array of options to set + * @param iterable|bool|string|null> $options associative array of options to set * * @throws Exception\InvalidArgumentException if invalid options are given */ @@ -462,7 +463,7 @@ public function getTarget(): string | null * prev, next, help, etc), and the value is a mixed value that could somehow * be considered a page. * - * @param iterable|PageInterface|string>|null $relations [optional] an associative array of forward links to other pages + * @param iterable|iterable|PageInterface|string>|null $relations [optional] an associative array of forward links to other pages * * @throws void */ @@ -494,7 +495,7 @@ public function setRel(iterable | null $relations = null): void * @param string|null $relation [optional] name of relation to return. If not * given, all relations will be returned. * - * @return ContainerInterface|iterable|ContainerInterface|PageInterface|string|Traversable>|PageInterface|string|null an array of relations. If $relation is not + * @return ContainerInterface|iterable|iterable|PageInterface|string>|PageInterface|string|null an array of relations. If $relation is not * specified, all relations will be returned in * an associative array. * @@ -518,7 +519,7 @@ public function getRel( * prev, next, help, etc), and the value is a mixed value that could somehow * be considered a page. * - * @param iterable|PageInterface|string>|null $relations [optional] an associative array of reverse links to other pages + * @param iterable|iterable|PageInterface|string>|null $relations [optional] an associative array of reverse links to other pages * * @throws void */ @@ -550,7 +551,7 @@ public function setRev(iterable | null $relations = null): void * @param string|null $relation [optional] name of relation to return. If not * given, all relations will be returned. * - * @return ContainerInterface|iterable|PageInterface|string>|PageInterface|string|null an array of relations. If $relation is not + * @return ContainerInterface|iterable|iterable|PageInterface|string>|PageInterface|string|null an array of relations. If $relation is not * specified, all relations will be returned in * an associative array. * @@ -569,7 +570,7 @@ public function getRev( /** * Sets parent container * - * @param ContainerInterface|null $parent [optional] new parent to set. + * @param ContainerInterface|null $parent [optional] new parent to set. * Default is null which will set no parent. * * @throws Exception\InvalidArgumentException @@ -604,7 +605,7 @@ public function setParent(ContainerInterface | null $parent = null): void /** * Returns parent container * - * @return ContainerInterface|null parent container or null + * @return ContainerInterface|null parent container or null * * @throws void */ @@ -620,7 +621,7 @@ public function getParent(): ContainerInterface | null * Default is null, which sets no * specific order. * - * @throws Exception\InvalidArgumentException if order is not integer or null + * @throws void */ public function setOrder(int | float | string | null $order = null): void { @@ -874,8 +875,8 @@ public function getVisible(bool $recursive = false): bool * If the given property is native (id, class, title, etc), the matching * set method will be used. Otherwise, it will be set as a custom property. * - * @param string $property property name - * @param bool|ContainerInterface|float|int|iterable|string)>|PageInterface|string|null $value value to set + * @param string $property property name + * @param bool|ContainerInterface|float|int|iterable|string)>|PageInterface|string|null $value value to set * * @throws Exception\InvalidArgumentException if property name is invalid */ @@ -907,7 +908,7 @@ public function set( * * @param string $property property name * - * @return bool|ContainerInterface|float|int|iterable|string)>|PageInterface|string|null the property's value or null + * @return bool|ContainerInterface|float|int|iterable|string)>|PageInterface|string|null the property's value or null * * @throws Exception\InvalidArgumentException if property name is invalid */ @@ -934,32 +935,28 @@ public function get( /** * Adds a forward relation to the page * - * @param string $relation relation name (e.g. alternate, glossary, - * canonical, etc) - * @param array|ContainerInterface|PageInterface|string|Traversable $value value to set for relation + * @param string $relation relation name (e.g. alternate, glossary, + * canonical, etc) + * @param ContainerInterface|iterable|PageInterface|string $value value to set for relation * * @throws void */ - public function addRel( - string $relation, - array | ContainerInterface | PageInterface | string | Traversable $value, - ): void { + public function addRel(string $relation, iterable | ContainerInterface | PageInterface | string $value): void + { $this->rel[$relation] = $value; } /** * Adds a reverse relation to the page * - * @param string $relation relation name (e.g. alternate, glossary, - * canonical, etc) - * @param array|ContainerInterface|PageInterface|string|Traversable $value value to set for relation + * @param string $relation relation name (e.g. alternate, glossary, + * canonical, etc) + * @param ContainerInterface|iterable|PageInterface|string $value value to set for relation * * @throws void */ - public function addRev( - string $relation, - array | ContainerInterface | PageInterface | string | Traversable $value, - ): void { + public function addRev(string $relation, iterable | ContainerInterface | PageInterface | string $value): void + { $this->rev[$relation] = $value; } @@ -998,7 +995,7 @@ public function removeRev(string $relation): void /** * Returns an array containing the defined forward relations * - * @return array defined forward relations + * @return array defined forward relations * * @throws void */ @@ -1010,7 +1007,7 @@ public function getDefinedRel(): array /** * Returns an array containing the defined reverse relations * - * @return array defined reverse relations + * @return array defined reverse relations * * @throws void */ @@ -1022,7 +1019,7 @@ public function getDefinedRev(): array /** * Returns custom properties as an array * - * @return array|bool|float|int|iterable|string|null> an array containing custom properties + * @return array|float|int|iterable|string)>|PageInterface|string|null> an array containing custom properties * * @throws void */ @@ -1046,7 +1043,7 @@ public function hashCode(): string /** * Returns an array representation of the page * - * @return array|bool|float|int|iterable|string|null> associative array containing all page properties + * @return array associative array containing all page properties * * @throws void */ diff --git a/src/Page/Route.php b/src/Page/Route.php index 2d0d572..1bd891e 100644 --- a/src/Page/Route.php +++ b/src/Page/Route.php @@ -373,7 +373,7 @@ public function setRouter(RouterInterface | null $router): void /** * Returns an array representation of the page * - * @return array|bool|float|int|iterable|RouteResult|RouterInterface|string|null> associative array containing all page properties + * @return array associative array containing all page properties * * @throws void */ diff --git a/src/Page/Uri.php b/src/Page/Uri.php index bd55eb7..c509752 100644 --- a/src/Page/Uri.php +++ b/src/Page/Uri.php @@ -126,7 +126,7 @@ public function isActive(bool $recursive = false): bool /** * Returns an array representation of the page * - * @return array|bool|float|int|iterable|string|null> + * @return array * * @throws void */ diff --git a/src/Service/NavigationAbstractServiceFactory.php b/src/Service/NavigationAbstractServiceFactory.php index 0d838a6..5333999 100644 --- a/src/Service/NavigationAbstractServiceFactory.php +++ b/src/Service/NavigationAbstractServiceFactory.php @@ -13,7 +13,6 @@ namespace Mimmi20\Mezzio\Navigation\Service; use Laminas\ServiceManager\Factory\AbstractFactoryInterface; -use Laminas\ServiceManager\ServiceLocatorInterface; use Mimmi20\Mezzio\Navigation\Config\NavigationConfigInterface; use Mimmi20\Mezzio\Navigation\Exception\InvalidArgumentException; use Mimmi20\Mezzio\Navigation\Navigation; @@ -47,6 +46,7 @@ final class NavigationAbstractServiceFactory implements AbstractFactoryInterface * * @throws ContainerExceptionInterface * @throws InvalidArgumentException + * @throws \Laminas\Stdlib\Exception\InvalidArgumentException * * @phpcsSuppress SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingNativeTypeHint * @phpcsSuppress SlevomatCodingStandard.Functions.UnusedParameter.UnusedParameter @@ -83,40 +83,6 @@ public function canCreate(ContainerInterface $container, $requestedName): bool return $this->hasNamedConfig($requestedName, $config); } - /** - * Determine if we can create a service with name - * - * @param string $name - * @param string $requestedName - * - * @throws ContainerExceptionInterface - * - * @codeCoverageIgnore - * @phpcsSuppress SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingNativeTypeHint - * @phpcsSuppress SlevomatCodingStandard.Functions.UnusedParameter.UnusedParameter - */ - public function canCreateServiceWithName(ServiceLocatorInterface $serviceLocator, $name, $requestedName): bool - { - return $this->canCreate($serviceLocator, $requestedName); - } - - /** - * Create service with name - * - * @param string $name - * @param string $requestedName - * - * @throws void - * - * @codeCoverageIgnore - * @phpcsSuppress SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingNativeTypeHint - * @phpcsSuppress SlevomatCodingStandard.Functions.UnusedParameter.UnusedParameter - */ - public function createServiceWithName(ServiceLocatorInterface $serviceLocator, $name, $requestedName): Navigation - { - return $this($serviceLocator, $requestedName); - } - /** * Extract config name from service name * diff --git a/src/Service/NavigationFactoryTrait.php b/src/Service/NavigationFactoryTrait.php index 6b2938a..065c5d0 100644 --- a/src/Service/NavigationFactoryTrait.php +++ b/src/Service/NavigationFactoryTrait.php @@ -96,7 +96,7 @@ private function getPages(ContainerInterface $container, NavigationConfigInterfa } /** - * @param array|string>> $pages + * @param array|string>> $pages * * @return array * diff --git a/tests/NavigationTest.php b/tests/NavigationTest.php index b3aa26b..36c34e8 100644 --- a/tests/NavigationTest.php +++ b/tests/NavigationTest.php @@ -74,8 +74,11 @@ public function testNavigationArraySortsCorrectly(): void self::assertCount(3, $pages); self::assertCount(3, $this->navigation); + self::assertIsArray($pages[0]); self::assertSame('page3', $pages[0]['uri'], var_export($pages, true)); + self::assertIsArray($pages[1]); self::assertSame('page1', $pages[1]['uri']); + self::assertIsArray($pages[2]); self::assertSame('page2', $pages[2]['uri']); } diff --git a/tests/Page/RouteTest.php b/tests/Page/RouteTest.php index e24b81e..3de6060 100644 --- a/tests/Page/RouteTest.php +++ b/tests/Page/RouteTest.php @@ -480,10 +480,7 @@ public function testSetTwoParents(): void self::assertSame($parent2, $this->page->getParent()); } - /** - * @throws Exception - * @throws InvalidArgumentException - */ + /** @throws Exception */ public function testSetOrder(): void { $order = 42; diff --git a/tests/Page/UriTest.php b/tests/Page/UriTest.php index 46b8b39..105c358 100644 --- a/tests/Page/UriTest.php +++ b/tests/Page/UriTest.php @@ -443,10 +443,7 @@ public function testSetTwoParents(): void self::assertSame($parent2, $this->page->getParent()); } - /** - * @throws Exception - * @throws InvalidArgumentException - */ + /** @throws Exception */ public function testSetOrder(): void { $order = 42; diff --git a/tests/Service/NavigationAbstractServiceFactoryTest.php b/tests/Service/NavigationAbstractServiceFactoryTest.php index 5f04fed..1b0c493 100644 --- a/tests/Service/NavigationAbstractServiceFactoryTest.php +++ b/tests/Service/NavigationAbstractServiceFactoryTest.php @@ -141,6 +141,7 @@ public function testCanCreateLowercased(): void * @throws Exception * @throws InvalidArgumentException * @throws ContainerExceptionInterface + * @throws \Laminas\Stdlib\Exception\InvalidArgumentException */ public function testInvoke(): void { @@ -197,6 +198,7 @@ static function (string $id) use ($matcher, $navigationConfig, $pageFactory): mi * @throws Exception * @throws InvalidArgumentException * @throws ContainerExceptionInterface + * @throws \Laminas\Stdlib\Exception\InvalidArgumentException */ public function testInvokeLowercased(): void { @@ -253,6 +255,7 @@ static function (string $id) use ($matcher, $navigationConfig, $pageFactory): mi * @throws Exception * @throws InvalidArgumentException * @throws ContainerExceptionInterface + * @throws \Laminas\Stdlib\Exception\InvalidArgumentException */ public function testCanNotInvoleWithoutConfig(): void {