InjectExtension: added type checking

src/DI/Extensions/InjectExtension.php

@@ -66,6 +66,9 @@ private function updateDefinition(Definitions\ServiceDefinition $def): void
 				}
 			}
 
+			if ($builder) {
+				self::checkType($class, $property, $type, $builder, $def);
+			}
 			array_unshift($setups, $inject);
 		}
 
@@ -158,7 +161,26 @@ public static function callInjects(DI\Container $container, $service): void
 		}
 
 		foreach (self::getInjectProperties(get_class($service)) as $property => $type) {
+			self::checkType($service, $property, $type, $container, null);
 			$service->$property = $container->getByType($type);
 		}
 	}
+
+
+	private static function checkType(
+		$class,
+		string $name,
+		?string $type,
+		$container,
+		?Definitions\Definition $def
+	): void
+	{
+		if (!$container->getByType($type, false)) {
+			throw new Nette\DI\MissingServiceException(sprintf(
+				'Service of type %s required by %s not found. Did you add it to configuration file?',
+				$type,
+				Reflection::toString(new \ReflectionProperty($class, $name))
+			));
+		}
+	}
 }

tests/DI/InjectExtension.errors.phpt

@@ -58,7 +58,7 @@ services:
 		factory: ServiceA
 		inject: yes
 ');
-}, InvalidStateException::class, "Service 'service' (type of ServiceA): Service of type DateTimeImmutable not found. Did you add it to configuration file?");
+}, InvalidStateException::class, 'Service of type DateTimeImmutable required by ServiceA::$a not found. Did you add it to configuration file?');
 
 
 Assert::exception(function () {