[BCB] Constructor parameter $templateTypes in CallableTypeNode made required

Author: ondrejmirtes

UPGRADING.md

@@ -124,3 +124,4 @@ The line with `some text in the middle` in phpdoc-parser 2.0 is now part of the
 * Constructor parameter `$templateTypes` in `MethodTagValueNode` made required
 * Constructor parameter `$isReference` in `ParamTagValueNode` made required
 * Constructor parameter `$isReference` in `TypelessParamTagValueNode` made required
+* Constructor parameter `$templateTypes` in `CallableTypeNode` made required

src/Ast/Type/CallableTypeNode.php

@@ -25,7 +25,7 @@ class CallableTypeNode implements TypeNode
 	 * @param CallableTypeParameterNode[] $parameters
 	 * @param TemplateTagValueNode[]  $templateTypes
 	 */
-	public function __construct(IdentifierTypeNode $identifier, array $parameters, TypeNode $returnType, array $templateTypes = [])
+	public function __construct(IdentifierTypeNode $identifier, array $parameters, TypeNode $returnType, array $templateTypes)
 	{
 		$this->identifier = $identifier;
 		$this->parameters = $parameters;

tests/PHPStan/Ast/ToString/TypeToStringTest.php

@@ -92,34 +92,34 @@ public static function provideCallableCases(): Generator
 		yield from [
 			[
 				'\\Closure(): string',
-				new CallableTypeNode(new IdentifierTypeNode('\Closure'), [], new IdentifierTypeNode('string')),
+				new CallableTypeNode(new IdentifierTypeNode('\Closure'), [], new IdentifierTypeNode('string'), []),
 			],
 			[
 				'callable(int, int $foo): void',
 				new CallableTypeNode(new IdentifierTypeNode('callable'), [
 					new CallableTypeParameterNode(new IdentifierTypeNode('int'), false, false, '', false),
 					new CallableTypeParameterNode(new IdentifierTypeNode('int'), false, false, '$foo', false),
-				], new IdentifierTypeNode('void')),
+				], new IdentifierTypeNode('void'), []),
 			],
 			[
 				'callable(int=, int $foo=): void',
 				new CallableTypeNode(new IdentifierTypeNode('callable'), [
 					new CallableTypeParameterNode(new IdentifierTypeNode('int'), false, false, '', true),
 					new CallableTypeParameterNode(new IdentifierTypeNode('int'), false, false, '$foo', true),
-				], new IdentifierTypeNode('void')),
+				], new IdentifierTypeNode('void'), []),
 			],
 			[
 				'callable(int &, int &$foo): void',
 				new CallableTypeNode(new IdentifierTypeNode('callable'), [
 					new CallableTypeParameterNode(new IdentifierTypeNode('int'), true, false, '', false),
 					new CallableTypeParameterNode(new IdentifierTypeNode('int'), true, false, '$foo', false),
-				], new IdentifierTypeNode('void')),
+				], new IdentifierTypeNode('void'), []),
 			],
 			[
 				'callable(string ...$foo): void',
 				new CallableTypeNode(new IdentifierTypeNode('callable'), [
 					new CallableTypeParameterNode(new IdentifierTypeNode('string'), false, true, '$foo', false),
-				], new IdentifierTypeNode('void')),
+				], new IdentifierTypeNode('void'), []),
 			],
 		];
 	}

tests/PHPStan/Parser/PhpDocParserTest.php

@@ -1754,6 +1754,7 @@ public function provideReturnTagsData(): Iterator
 								),
 							],
 							new IdentifierTypeNode('string'),
+							[],
 						),
 						'',
 					),
@@ -2909,6 +2910,7 @@ public function provideSingleLinePhpDocData(): Iterator
 								new CallableTypeParameterNode(new IdentifierTypeNode('int'), false, false, '', false),
 							],
 							new IdentifierTypeNode('void'),
+							[],
 						),
 						'',
 						'',
@@ -5345,6 +5347,7 @@ public function provideRealWorldExampleData(): Iterator
 								new CallableTypeParameterNode(new IdentifierTypeNode('B'), false, false, '', false),
 							],
 							new IdentifierTypeNode('void'),
+							[],
 						),
 						false,
 						'$foo',
@@ -5374,6 +5377,7 @@ public function provideRealWorldExampleData(): Iterator
 								new CallableTypeParameterNode(new IdentifierTypeNode('B'), false, false, '', false),
 							],
 							new IdentifierTypeNode('void'),
+							[],
 						),
 						false,
 						'$foo',
@@ -5403,6 +5407,7 @@ public function provideRealWorldExampleData(): Iterator
 								new CallableTypeParameterNode(new IdentifierTypeNode('B'), false, false, '', false),
 							],
 							new IdentifierTypeNode('void'),
+							[],
 						),
 						false,
 						'$foo',