Merge pull request #223 from phpcr/phpstan

dbu · web-flow · commit 778bc5a48165 · 2023-12-02T11:15:07.000+01:00
cleanup phpstan
diff --git a/phpstan.tests.neon.dist b/phpstan.tests.neon.dist
@@ -8,17 +8,6 @@ parameters:
             - tests/*/Fixtures/*
 
     ignoreErrors:
-        # not sure what is going on here
-        -
-            message: "#^Interface Iterator specifies template type TKey of interface Traversable as string but it's already specified as mixed\\.$#"
-            count: 1
-            path: tests/PHPCR/Tests/Stubs/MockNodeTypeManager.php
-
-        -
-            message: "#^Interface Iterator specifies template type TValue of interface Traversable as PHPCR\\\\NodeType\\\\NodeTypeInterface but it's already specified as mixed\\.$#"
-            count: 1
-            path: tests/PHPCR/Tests/Stubs/MockNodeTypeManager.php
-
         # too pedantic for tests
         -
             message: "#^Parameter \\#1 \\.\\.\\.\\$arrays of function array_merge expects array, array\\<int, string\\>\\|false given\\.$#"
diff --git a/tests/PHPCR/Tests/Stubs/MockNodeTypeManager.php b/tests/PHPCR/Tests/Stubs/MockNodeTypeManager.php
@@ -8,7 +8,7 @@
 use PHPCR\NodeType\NodeTypeManagerInterface;
 
 /**
- * @implements \Iterator<string, NodeTypeInterface>
+ * @implements \Iterator<mixed, NodeTypeInterface>
  */
 abstract class MockNodeTypeManager implements \Iterator, NodeTypeManagerInterface
 {

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`use PHPCR\NodeType\NodeTypeManagerInterface;`
`9`	`9`
`10`	`10`	`/**`
`11`		`- * @implements \Iterator<string, NodeTypeInterface>`
	`11`	`+ * @implements \Iterator<mixed, NodeTypeInterface>`
`12`	`12`	`*/`
`13`	`13`	`abstract class MockNodeTypeManager implements \Iterator, NodeTypeManagerInterface`
`14`	`14`	`{`