adding tests

phpstan · Oct 2, 2022 · 3e8df38 · 3e8df38
1 parent e515b25
commit 3e8df38
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/src/Reflection/ParametersAcceptorSelector.php b/src/Reflection/ParametersAcceptorSelector.php
@@ -728,10 +728,10 @@ private static function getCurlOptValueType(int $curlOpt): ?Type
 		];
 		foreach ($arrayOrStringConstants as $constName) {
 			if (defined($constName) && constant($constName) === $curlOpt) {
-				return TypeCombinator::union(
+				return new UnionType([
 					new StringType(),
 					new ArrayType(new MixedType(), new MixedType()),
-				);
+				]);
 			}
 		}
 

diff --git a/tests/PHPStan/Rules/Functions/CallToFunctionParametersRuleTest.php b/tests/PHPStan/Rules/Functions/CallToFunctionParametersRuleTest.php
@@ -1213,6 +1213,10 @@ public function testCurlSetOpt(): void
 				'Parameter #3 $value of function curl_setopt expects resource, string given.',
 				24,
 			],
+			[
+				'Parameter #3 $value of function curl_setopt expects array|string, int given.',
+				26,
+			],
 		]);
 	}
 

diff --git a/tests/PHPStan/Rules/Functions/data/curl_setopt.php b/tests/PHPStan/Rules/Functions/data/curl_setopt.php
@@ -22,6 +22,8 @@ public function errors(int $i, string $s) {
 		curl_setopt($curl, CURLOPT_CONNECT_TO, $s);
 		// expecting resource
 		curl_setopt($curl, CURLOPT_FILE, $s);
+		// expecting string or array
+		curl_setopt($curl, CURLOPT_POSTFIELDS, $i);
 	}
 
 	/**
@@ -45,5 +47,8 @@ public function allGood(string $url, array $header) {
 		curl_setopt($curl, CURLOPT_FILE, $fp);
 		curl_setopt($curl, CURLOPT_HEADER, false);
 		curl_setopt($curl, CURLOPT_HTTPHEADER, array('Content-type: text/plain', 'Content-length: 100'));
+		curl_setopt($curl, CURLOPT_POSTFIELDS, array('foo' => 'bar'));
+		curl_setopt($curl, CURLOPT_POSTFIELDS, '');
+		curl_setopt($curl, CURLOPT_POSTFIELDS, 'para1=val1&para2=val2');
 	}
 }