feat(discovery): support containers with autowiring (tempestphp#2084)

Author: innocenzi

packages/discovery/src/BootDiscovery.php

@@ -4,11 +4,13 @@
 
 namespace Tempest\Discovery;
 
+use ArgumentCountError;
 use AssertionError;
 use Closure;
 use Pest\Exceptions\InvalidPestCommand;
 use Psr\Container\ContainerInterface;
-use Tempest\Container\GenericContainer;
+use Psr\Container\NotFoundExceptionInterface;
+use Tempest\Discovery\Exceptions\DiscoveryClassCouldNotBeResolved;
 use Tempest\Reflection\ClassReflector;
 use Tempest\Support\Filesystem;
 use Throwable;
@@ -260,18 +262,28 @@ private function discoverPath(string $input, DiscoveryLocation $location, array
     /**
      * Create a discovery instance from a class name.
      * Optionally set the cached discovery items whenever caching is enabled.
+     *
      * @template T of Discovery
      * @param class-string<T> $discoveryClass
      * @return T
      */
     private function resolveDiscovery(string $discoveryClass): Discovery
     {
-        if ($this->container instanceof GenericContainer || $this->container->has($discoveryClass)) {
+        $discovery = null;
+
+        try {
             /** @var Discovery $discovery */
             $discovery = $this->container->get($discoveryClass);
-        } else {
-            /** @var Discovery $discovery */
-            $discovery = new $discoveryClass();
+        } catch (NotFoundExceptionInterface) {
+            // @mago-expect lint:no-empty-catch-clause
+        }
+
+        if ($discovery === null) {
+            try {
+                $discovery = new $discoveryClass();
+            } catch (ArgumentCountError) { // @phpstan-ignore catch.neverThrown
+                throw DiscoveryClassCouldNotBeResolved::forDiscoveryClass($discoveryClass);
+            }
         }
 
         $discovery->setItems(new DiscoveryItems());

packages/discovery/src/Exceptions/DiscoveryClassCouldNotBeResolved.php

@@ -0,0 +1,13 @@
+<?php
+
+namespace Tempest\Discovery\Exceptions;
+
+use Exception;
+
+final class DiscoveryClassCouldNotBeResolved extends Exception implements DiscoveryException
+{
+    public static function forDiscoveryClass(string $discoveryClass): self
+    {
+        return new self("Failed to resolve discovery class [{$discoveryClass}]: it is not bound in the container and cannot be instantiated without constructor arguments.");
+    }
+}

packages/discovery/src/Exceptions/DiscoveryException.php

@@ -0,0 +1,5 @@
+<?php
+
+namespace Tempest\Discovery\Exceptions;
+
+interface DiscoveryException {}