More precise array_chunk signature

fix Scheduler fix cs type __construct() Update Scheduler.php adjust test
phpstan · Jan 11, 2022 · c416825 · c416825
1 parent fb12769
commit c416825
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/resources/functionMap.php b/resources/functionMap.php
@@ -259,7 +259,7 @@
 'AppendIterator::rewind' => ['void'],
 'AppendIterator::valid' => ['bool'],
 'array_change_key_case' => ['array', 'input'=>'array', 'case='=>'int'],
-'array_chunk' => ['array[]', 'input'=>'array', 'size'=>'int', 'preserve_keys='=>'bool'],
+'array_chunk' => ['array[]', 'input'=>'array', 'size'=>'positive-int', 'preserve_keys='=>'bool'],
 'array_column' => ['array', 'array'=>'array', 'column_key'=>'mixed', 'index_key='=>'mixed'],
 'array_combine' => ['array|false', 'keys'=>'array', 'values'=>'array'],
 'array_count_values' => ['array<positive-int>', 'input'=>'array'],

diff --git a/src/Parallel/Scheduler.php b/src/Parallel/Scheduler.php
@@ -11,6 +11,16 @@
 class Scheduler
 {
 
+	/** @var positive-int */
+	private int $jobSize;
+
+	private int $maximumNumberOfProcesses;
+
+	private int $minimumNumberOfJobsPerProcess;
+
+	/**
+	 * @param positive-int $jobSize
+	 */
 	public function __construct(
 		private int $jobSize,
 		private int $maximumNumberOfProcesses,

diff --git a/tests/PHPStan/Parallel/SchedulerTest.php b/tests/PHPStan/Parallel/SchedulerTest.php
@@ -72,6 +72,7 @@ public function dataSchedule(): array
 
 	/**
 	 * @dataProvider dataSchedule
+	 * @param positive-int $jobSize
 	 * @param 0|positive-int $numberOfFiles
 	 * @param array<int> $expectedJobSizes
 	 */