fix bug and added test

Author: Nyholm

src/DependencyInjection/SentryExtension.php

@@ -78,7 +78,7 @@ protected function loadInternal(array $mergedConfig, ContainerBuilder $container
         $this->registerCacheTracingConfiguration($container, $mergedConfig['tracing']);
         $this->registerHttpClientTracingConfiguration($container, $mergedConfig['tracing']);
 
-        if (!class_exists(TokenStorageInterface::class)) {
+        if (!interface_exists(TokenStorageInterface::class)) {
             $container->removeDefinition(LoginListener::class);
         }
     }

tests/DependencyInjection/Compiler/AddLoginListenerTagPassTest.php

@@ -28,4 +28,20 @@ public function testProcess(): void
 
         $this->assertSame([['event' => AuthenticationSuccessEvent::class, 'method' => 'handleAuthenticationSuccessEvent']], $listenerDefinition->getTag('kernel.event_listener'));
     }
+
+    public function testProcessLoginSuccess(): void
+    {
+        if (!class_exists(LoginSuccessEvent::class)) {
+            $this->markTestSkipped('Skipping this test because LoginSuccessEvent does not exist.');
+        }
+
+        $container = new ContainerBuilder();
+        $container->register(LoginListener::class)->setPublic(true);
+        $container->addCompilerPass(new AddLoginListenerTagPass());
+        $container->compile();
+
+        $listenerDefinition = $container->getDefinition(LoginListener::class);
+
+        $this->assertSame([['event' => LoginSuccessEvent::class, 'method' => 'handleLoginSuccessEvent']], $listenerDefinition->getTag('kernel.event_listener'));
+    }
 }