feat: deprecate foundry integration

Author: kbond

README.md

@@ -320,9 +320,6 @@ $browser
     // authenticate a user for subsequent actions
     ->actingAs($user) // \Symfony\Component\Security\Core\User\UserInterface
 
-    // If using zenstruck/foundry, you can pass a factory/proxy
-    ->actingAs(UserFactory::new())
-
     // fail if authenticated
     ->assertNotAuthenticated()
 
@@ -332,8 +329,7 @@ $browser
     // fails if NOT authenticated as "kbond"
     ->assertAuthenticated('kbond')
 
-    // \Symfony\Component\Security\Core\User\UserInterface or, if using
-    // zenstruck/foundry, you can pass a factory/proxy
+    // \Symfony\Component\Security\Core\User\UserInterface
     ->assertAuthenticated($user)
 ;
 ```

composer.json

@@ -30,8 +30,7 @@
         "symfony/mime": "^5.4|^6.0|^7.0",
         "symfony/panther": "^1.1|^2.0.1",
         "symfony/phpunit-bridge": "^6.0|^7.0",
-        "symfony/security-bundle": "^5.4|^6.0|^7.0",
-        "zenstruck/foundry": "^1.30"
+        "symfony/security-bundle": "^5.4|^6.0|^7.0"
     },
     "suggest": {
         "justinrainbow/json-schema": "Json schema validator. Needed to use Json::assertMatchesSchema().",

src/Browser/KernelBrowser.php

@@ -127,18 +127,20 @@ final public function withProfiling(): self
     }
 
     /**
-     * @param UserInterface|Proxy<UserInterface>|Factory<UserInterface> $user
+     * @param UserInterface $user
      *
      * @return static
      */
     public function actingAs(object $user, ?string $firewall = null): self
     {
-        if ($user instanceof Factory) {
-            $user = $user->create();
+        if ($user instanceof Factory) { // @phpstan-ignore-line
+            trigger_deprecation('zenstruck/browser', '1.9', 'Passing a Factory to actingAs() is deprecated, pass the created object instead.');
+            $user = $user->create(); // @phpstan-ignore-line
         }
 
-        if ($user instanceof Proxy) {
-            $user = $user->object();
+        if ($user instanceof Proxy) { // @phpstan-ignore-line
+            trigger_deprecation('zenstruck/browser', '1.9', 'Passing a Proxy to actingAs() is deprecated, pass the real object instead.');
+            $user = $user->object(); // @phpstan-ignore-line
         }
 
         if (!$user instanceof UserInterface) {
@@ -151,7 +153,7 @@ public function actingAs(object $user, ?string $firewall = null): self
     }
 
     /**
-     * @param string|UserInterface|Proxy<UserInterface>|Factory<UserInterface>|null $as
+     * @param string|UserInterface|null $as
      *
      * @return static
      */
@@ -171,12 +173,14 @@ public function assertAuthenticated($as = null): self
             return $this;
         }
 
-        if ($as instanceof Factory) {
-            $as = $as->create();
+        if ($as instanceof Factory) { // @phpstan-ignore-line
+            trigger_deprecation('zenstruck/browser', '1.9', 'Passing a Factory to assertAuthenticated() is deprecated, pass the created object instead.');
+            $as = $as->create(); // @phpstan-ignore-line
         }
 
-        if ($as instanceof Proxy) {
-            $as = $as->object();
+        if ($as instanceof Proxy) { // @phpstan-ignore-line
+            trigger_deprecation('zenstruck/browser', '1.9', 'Passing a Proxy to assertAuthenticated() is deprecated, pass the real object instead.');
+            $as = $as->object(); // @phpstan-ignore-line
         }
 
         if ($as instanceof UserInterface) {

tests/Fixture/Kernel.php

@@ -28,7 +28,6 @@
 use Symfony\Component\HttpKernel\Kernel as BaseKernel;
 use Symfony\Component\Routing\Loader\Configurator\RoutingConfigurator;
 use Symfony\Component\Security\Core\User\InMemoryUser;
-use Zenstruck\Foundry\ZenstruckFoundryBundle;
 
 /**
  * @author Kevin Bond <[email protected]>
@@ -173,7 +172,6 @@ public function registerBundles(): iterable
     {
         yield new FrameworkBundle();
         yield new SecurityBundle();
-        yield new ZenstruckFoundryBundle();
     }
 
     protected function configureContainer(ContainerBuilder $c, LoaderInterface $loader): void
@@ -209,9 +207,6 @@ protected function configureContainer(ContainerBuilder $c, LoaderInterface $load
         }
 
         $c->loadFromExtension('security', $security);
-        $c->loadFromExtension('zenstruck_foundry', [
-            'auto_refresh_proxies' => false,
-        ]);
         $c->register('logger', NullLogger::class); // disable logging
     }
 

tests/KernelBrowserAuthenticationTest.php

@@ -17,16 +17,13 @@
 use Symfony\Component\Security\Core\Authentication\Token\UsernamePasswordToken;
 use Symfony\Component\Security\Core\User\InMemoryUser;
 use Zenstruck\Browser\Test\HasBrowser;
-use Zenstruck\Foundry\Test\Factories;
-
-use function Zenstruck\Foundry\anonymous;
 
 /**
  * @author Kevin Bond <[email protected]>
  */
 final class KernelBrowserAuthenticationTest extends KernelTestCase
 {
-    use Factories, HasBrowser;
+    use HasBrowser;
 
     /**
      * @test
@@ -41,54 +38,20 @@ public function can_act_as_user(): void
         ;
     }
 
-    /**
-     * @test
-     */
-    public function can_act_as_user_with_foundry_factory(): void
-    {
-        $user = anonymous(InMemoryUser::class, ['username' => 'kevin', 'password' => 'pass']);
-
-        $this->browser()
-            ->throwExceptions()
-            ->actingAs($user)
-            ->visit('/user')
-            ->assertSee('user: kevin/pass')
-        ;
-    }
-
-    /**
-     * @test
-     */
-    public function can_act_as_user_with_foundry_proxy(): void
-    {
-        $user = anonymous(InMemoryUser::class)->create(['username' => 'kevin', 'password' => 'pass']);
-
-        $this->browser()
-            ->throwExceptions()
-            ->actingAs($user)
-            ->visit('/user')
-            ->assertSee('user: kevin/pass')
-        ;
-    }
-
     /**
      * @test
      */
     public function can_make_authentication_assertions(): void
     {
         $username = 'kevin';
         $user = new InMemoryUser('kevin', 'pass');
-        $factory = anonymous(InMemoryUser::class, ['username' => 'kevin', 'password' => 'pass']);
-        $proxy = anonymous(InMemoryUser::class)->create(['username' => 'kevin', 'password' => 'pass']);
 
         $this->browser()
             ->assertNotAuthenticated()
             ->actingAs($user)
             ->assertAuthenticated()
             ->assertAuthenticated($username)
             ->assertAuthenticated($user)
-            ->assertAuthenticated($factory)
-            ->assertAuthenticated($proxy)
             ->visit('/user')
             ->assertAuthenticated()
             ->assertAuthenticated($username)