Do not produce reflection error for unknown constants in class constants

Author: ondrejmirtes

src/Reflection/ClassConstantReflection.php

@@ -2,6 +2,7 @@
 
 namespace PHPStan\Reflection;
 
+use PHPStan\BetterReflection\NodeCompiler\Exception\UnableToCompileNode;
 use PHPStan\Php\PhpVersion;
 use PHPStan\TrinaryLogic;
 use PHPStan\Type\ConstantTypeHelper;
@@ -65,7 +66,11 @@ public function getFileName(): ?string
 	 */
 	public function getValue()
 	{
-		return $this->reflection->getValue();
+		try {
+			return $this->reflection->getValue();
+		} catch (UnableToCompileNode $e) {
+			return NAN;
+		}
 	}
 
 	public function hasPhpDocType(): bool

src/Type/ConstantTypeHelper.php

@@ -20,6 +20,9 @@ public static function getTypeFromValue($value): Type
 		if (is_int($value)) {
 			return new ConstantIntegerType($value);
 		} elseif (is_float($value)) {
+			if (is_nan($value)) {
+				return new MixedType();
+			}
 			return new ConstantFloatType($value);
 		} elseif (is_bool($value)) {
 			return new ConstantBooleanType($value);

tests/PHPStan/Analyser/AnalyserIntegrationTest.php

@@ -278,9 +278,8 @@ public function testBug3379(): void
 			$this->markTestSkipped('Test requires static reflection');
 		}
 		$errors = $this->runAnalyse(__DIR__ . '/data/bug-3379.php');
-		$this->assertCount(2, $errors);
+		$this->assertCount(1, $errors);
 		$this->assertSame('Constant SOME_UNKNOWN_CONST not found.', $errors[0]->getMessage());
-		$this->assertSame('Reflection error: Could not locate constant "SOME_UNKNOWN_CONST" while evaluating expression in Bug3379\Foo at line 8', $errors[1]->getMessage());
 	}
 
 	public function testBug3798(): void

tests/PHPStan/Analyser/NodeScopeResolverTest.php

@@ -463,6 +463,11 @@ public function dataFileAsserts(): iterable
 		}
 
 		yield from $this->gatherAssertTypes(__DIR__ . '/data/class-constant-types.php');
+
+		if (self::$useStaticReflectionProvider) {
+			yield from $this->gatherAssertTypes(__DIR__ . '/data/bug-3379.php');
+		}
+
 		yield from $this->gatherAssertTypes(__DIR__ . '/data/modulo-operator.php');
 	}
 

tests/PHPStan/Analyser/data/bug-3379.php

@@ -2,6 +2,8 @@
 
 namespace Bug3379;
 
+use function PHPStan\Testing\assertType;
+
 class Foo
 {
 
@@ -11,4 +13,5 @@ class Foo
 
 function () {
 	echo Foo::URL;
+	assertType('mixed', Foo::URL);
 };