refactor: Rename ProviderManager to TenantProviderManager to keep con…

…sistency (#82)

* refactor: Rename ProviderManager to TenantProviderManager to keep consistency

* chore: Add missing changes

src/Facades/Sprout.php

@@ -8,7 +8,7 @@
 use Sprout\Contracts\Tenancy;
 use Sprout\Contracts\Tenant;
 use Sprout\Managers\IdentityResolverManager;
-use Sprout\Managers\ProviderManager;
+use Sprout\Managers\TenantProviderManager;
 use Sprout\Support\ResolutionHook;
 
 /**
@@ -31,7 +31,7 @@
  * @method static bool isBootableOverride(string $class)
  * @method static \Sprout\Sprout markAsInContext()
  * @method static \Sprout\Sprout markAsOutsideContext()
- * @method static ProviderManager providers()
+ * @method static TenantProviderManager providers()
  * @method static \Sprout\Sprout registerOverride(string $class)
  * @method static IdentityResolverManager resolvers()
  * @method static string route(string $name, Tenant $tenant, string|null $resolver = null, string|null $tenancy = null, array $parameters = [], bool $absolute = true)

src/Managers/TenancyManager.php

@@ -18,17 +18,17 @@
 final class TenancyManager extends BaseFactory
 {
     /**
-     * @var \Sprout\Managers\ProviderManager
+     * @var \Sprout\Managers\TenantProviderManager
      */
-    private ProviderManager $providerManager;
+    private TenantProviderManager $providerManager;
 
     /**
      * Create a new instance
      *
      * @param \Illuminate\Contracts\Foundation\Application $app
-     * @param \Sprout\Managers\ProviderManager             $providerManager
+     * @param \Sprout\Managers\TenantProviderManager       $providerManager
      */
-    public function __construct(Application $app, ProviderManager $providerManager)
+    public function __construct(Application $app, TenantProviderManager $providerManager)
     {
         parent::__construct($app);
 

src/Managers/ProviderManager.php → src/Managers/TenantProviderManager.php

@@ -21,7 +21,7 @@
  *
  * @package Core
  */
-final class ProviderManager extends BaseFactory
+final class TenantProviderManager extends BaseFactory
 {
     /**
      * Get the name used by this factory

src/Providers/EloquentTenantProvider.php

@@ -22,7 +22,7 @@
  *
  * @package  Providers
  *
- * @internal New instances are created with {@see \Sprout\Managers\ProviderManager::createEloquentProvider()}, and shouldn't be created manually
+ * @internal New instances are created with {@see \Sprout\Managers\TenantProviderManager::createEloquentProvider()}, and shouldn't be created manually
  */
 final class EloquentTenantProvider extends BaseTenantProvider
 {

src/Sprout.php

@@ -8,7 +8,7 @@
 use Sprout\Contracts\Tenancy;
 use Sprout\Contracts\Tenant;
 use Sprout\Managers\IdentityResolverManager;
-use Sprout\Managers\ProviderManager;
+use Sprout\Managers\TenantProviderManager;
 use Sprout\Managers\TenancyManager;
 use Sprout\Support\ResolutionHook;
 use Sprout\Support\SettingsRepository;
@@ -161,13 +161,13 @@ public function resolvers(): IdentityResolverManager
     /**
      * Get the tenant providers manager
      *
-     * @return \Sprout\Managers\ProviderManager
+     * @return \Sprout\Managers\TenantProviderManager
      *
      * @throws \Illuminate\Contracts\Container\BindingResolutionException
      */
-    public function providers(): ProviderManager
+    public function providers(): TenantProviderManager
     {
-        return $this->app->make(ProviderManager::class);
+        return $this->app->make(TenantProviderManager::class);
     }
 
     /**

src/SproutServiceProvider.php

@@ -12,7 +12,7 @@
 use Sprout\Http\RouterMethods;
 use Sprout\Listeners\IdentifyTenantOnRouting;
 use Sprout\Managers\IdentityResolverManager;
-use Sprout\Managers\ProviderManager;
+use Sprout\Managers\TenantProviderManager;
 use Sprout\Managers\TenancyManager;
 use Sprout\Support\ResolutionHook;
 use Sprout\Support\SettingsRepository;
@@ -49,8 +49,8 @@ private function registerSprout(): void
     private function registerManagers(): void
     {
         // Register the tenant provider manager
-        $this->app->singleton(ProviderManager::class, function ($app) {
-            return new ProviderManager($app);
+        $this->app->singleton(TenantProviderManager::class, function ($app) {
+            return new TenantProviderManager($app);
         });
 
         // Register the identity resolver manager
@@ -60,11 +60,11 @@ private function registerManagers(): void
 
         // Register the tenancy manager
         $this->app->singleton(TenancyManager::class, function ($app) {
-            return new TenancyManager($app, $app->make(ProviderManager::class));
+            return new TenancyManager($app, $app->make(TenantProviderManager::class));
         });
 
         // Alias the managers with simple names
-        $this->app->alias(ProviderManager::class, 'sprout.providers');
+        $this->app->alias(TenantProviderManager::class, 'sprout.providers');
         $this->app->alias(IdentityResolverManager::class, 'sprout.resolvers');
         $this->app->alias(TenancyManager::class, 'sprout.tenancies');
     }

tests/Unit/SproutServiceProviderTest.php

@@ -13,7 +13,7 @@
 use Sprout\Http\Middleware\TenantRoutes;
 use Sprout\Listeners\IdentifyTenantOnRouting;
 use Sprout\Managers\IdentityResolverManager;
-use Sprout\Managers\ProviderManager;
+use Sprout\Managers\TenantProviderManager;
 use Sprout\Managers\TenancyManager;
 use Sprout\Sprout;
 use Sprout\SproutServiceProvider;
@@ -54,17 +54,17 @@ public function sproutIsRegistered(): void
     #[Test]
     public function providerManagerIsRegistered(): void
     {
-        $this->assertTrue(app()->has(ProviderManager::class));
+        $this->assertTrue(app()->has(TenantProviderManager::class));
         $this->assertTrue(app()->has('sprout.providers'));
-        $this->assertTrue(app()->isShared(ProviderManager::class));
+        $this->assertTrue(app()->isShared(TenantProviderManager::class));
         $this->assertFalse(app()->isShared('sprout.providers'));
 
-        $this->assertSame(app()->make(ProviderManager::class), app()->make(ProviderManager::class));
+        $this->assertSame(app()->make(TenantProviderManager::class), app()->make(TenantProviderManager::class));
         $this->assertSame(app()->make('sprout.providers'), app()->make('sprout.providers'));
-        $this->assertSame(app()->make(ProviderManager::class), app()->make('sprout.providers'));
-        $this->assertSame(app()->make('sprout.providers'), app()->make(ProviderManager::class));
+        $this->assertSame(app()->make(TenantProviderManager::class), app()->make('sprout.providers'));
+        $this->assertSame(app()->make('sprout.providers'), app()->make(TenantProviderManager::class));
         $this->assertSame(app()->make(Sprout::class)->providers(), app()->make('sprout.providers'));
-        $this->assertSame(app()->make(Sprout::class)->providers(), app()->make(ProviderManager::class));
+        $this->assertSame(app()->make(Sprout::class)->providers(), app()->make(TenantProviderManager::class));
         $this->assertSame(sprout()->providers(), sprout()->providers());
         $this->assertSame(app()->make(Sprout::class)->providers(), sprout()->providers());
     }

tests/_Original/Providers/DatabaseTenantProviderTest.php

@@ -10,7 +10,7 @@
 use Orchestra\Testbench\Concerns\WithWorkbench;
 use Orchestra\Testbench\TestCase;
 use PHPUnit\Framework\Attributes\Test;
-use Sprout\Managers\ProviderManager;
+use Sprout\Managers\TenantProviderManager;
 use Sprout\Providers\DatabaseTenantProvider;
 use Sprout\Sprout;
 use Sprout\Support\GenericTenant;
@@ -34,7 +34,7 @@ protected function defineEnvironment($app): void
     #[Test]
     public function isRegisteredCorrectly(): void
     {
-        $manager  = app(ProviderManager::class);
+        $manager  = app(TenantProviderManager::class);
         $provider = $manager->get('backup');
 
         $this->assertNotNull($provider);

tests/_Original/Providers/EloquentTenantProviderTest.php

@@ -8,7 +8,7 @@
 use Orchestra\Testbench\Concerns\WithWorkbench;
 use Orchestra\Testbench\TestCase;
 use PHPUnit\Framework\Attributes\Test;
-use Sprout\Managers\ProviderManager;
+use Sprout\Managers\TenantProviderManager;
 use Sprout\Providers\EloquentTenantProvider;
 use Sprout\Sprout;
 use Workbench\App\Models\TenantModel;
@@ -29,7 +29,7 @@ protected function defineEnvironment($app): void
     #[Test]
     public function isRegisteredCorrectly(): void
     {
-        $manager  = app(ProviderManager::class);
+        $manager  = app(TenantProviderManager::class);
         $provider = $manager->get('tenants');
 
         $this->assertNotNull($provider);

tests/_Original/ServiceProviderTest.php

@@ -17,7 +17,7 @@
 use Sprout\Listeners\SetCurrentTenantContext;
 use Sprout\Listeners\SetCurrentTenantForJob;
 use Sprout\Managers\IdentityResolverManager;
-use Sprout\Managers\ProviderManager;
+use Sprout\Managers\TenantProviderManager;
 use Sprout\Managers\TenancyManager;
 use Sprout\Sprout;
 use Sprout\SproutServiceProvider;
@@ -60,17 +60,17 @@ public function coreSproutConfigExists(): void
     #[Test]
     public function providerManagerIsRegistered(): void
     {
-        $this->assertTrue(app()->has(ProviderManager::class));
+        $this->assertTrue(app()->has(TenantProviderManager::class));
         $this->assertTrue(app()->has('sprout.providers'));
-        $this->assertTrue(app()->isShared(ProviderManager::class));
+        $this->assertTrue(app()->isShared(TenantProviderManager::class));
         $this->assertFalse(app()->isShared('sprout.providers'));
 
-        $this->assertSame(app()->make(ProviderManager::class), app()->make(ProviderManager::class));
+        $this->assertSame(app()->make(TenantProviderManager::class), app()->make(TenantProviderManager::class));
         $this->assertSame(app()->make('sprout.providers'), app()->make('sprout.providers'));
-        $this->assertSame(app()->make(ProviderManager::class), app()->make('sprout.providers'));
-        $this->assertSame(app()->make('sprout.providers'), app()->make(ProviderManager::class));
+        $this->assertSame(app()->make(TenantProviderManager::class), app()->make('sprout.providers'));
+        $this->assertSame(app()->make('sprout.providers'), app()->make(TenantProviderManager::class));
         $this->assertSame(app()->make(Sprout::class)->providers(), app()->make('sprout.providers'));
-        $this->assertSame(app()->make(Sprout::class)->providers(), app()->make(ProviderManager::class));
+        $this->assertSame(app()->make(Sprout::class)->providers(), app()->make(TenantProviderManager::class));
     }
 
     #[Test]