Adjust the sync logic

Author: cyrillkalita

src/Actions/SyncDefinedRole.php

@@ -3,28 +3,27 @@
 namespace BinaryCats\LaravelRbac\Actions;
 
 use BackedEnum;
+use Illuminate\Support\Facades\Artisan;
 use Lorisleiva\Actions\Action;
-use Spatie\Permission\Contracts\Role;
+use Spatie\Permission\Commands\CreateRole;
 
 class SyncDefinedRole extends Action
 {
-    public function __construct(
-        protected readonly Role $role
-    ) {
-    }
-
     /**
      * Handle syncing a defined role.
      */
     public function handle(string $name, string $guard, array $permissions): void
     {
         $permissions = collect($permissions)
-            ->map(fn ($permission) => match (true) {
+            ->map(fn ($permission): string => match (true) {
                 $permission instanceof BackedEnum => $permission->value,
-                default                           => (string) $permission
-            });
-
-        $this->role::findOrCreate($name, $guard)
-            ->syncPermissions($permissions);
+                default => (string) $permission
+            })->implode('|');
+            
+        Artisan::call(CreateRole::class, [
+            'name' => $name,
+            'guard' => $guard,
+            'permissions' => $permissions
+        ]);
     }
 }

src/Commands/AbilityMakeCommand.php

@@ -40,7 +40,12 @@ protected function getStub()
      */
     protected function getDefaultNamespace($rootNamespace)
     {
-        return $rootNamespace.'\Abilities';
+        $path = config('rbac.path', 'Abilities');
+
+        return str($path)
+            ->after(app()->path())
+            ->trim('/')
+            ->prepend($rootNamespace, '\\');
     }
 
     /**

src/Commands/DefinedRoleMakeCommand.php

@@ -25,7 +25,7 @@ class DefinedRoleMakeCommand extends GeneratorCommand
      */
     protected function getStub()
     {
-        return file_exists($customPath = base_path('/stubs/defined-role.stub'))
+        return file_exists($customPath = $this->laravel->basePath('/stubs/defined-role.stub'))
             ? $customPath
             : __DIR__.'/../../stubs/defined-role.stub';
     }

src/Jobs/ResetPermissions.php

@@ -18,18 +18,15 @@ class ResetPermissions
     use Queueable;
     use SerializesModels;
 
-    protected string $guard;
+    protected readonly string $guard;
 
-    /**
-     * @param string|null $guard
-     */
     public function __construct(?string $guard = null)
     {
         $this->guard = $guard ?? config('auth.defaults.guard');
     }
 
     /**
-     * @return void
+     * Handle resetting permissions
      */
     public function handle(): void
     {

tests/Actions/SyncDefinedRoleTest.php

@@ -7,7 +7,6 @@
 use BinaryCats\LaravelRbac\Tests\Fixtures\Abilities\FooAbility;
 use BinaryCats\LaravelRbac\Tests\TestCase;
 use PHPUnit\Framework\Attributes\Test;
-use Spatie\Permission\Exceptions\PermissionDoesNotExist;
 
 class SyncDefinedRoleTest extends TestCase
 {
@@ -37,6 +36,7 @@ public function it_will_defer_syncing_defined_role_to_artisan_with_custom_guard(
         SyncDefinedRole::run('foo role', 'admin', [
             'bar',
             FooAbility::One,
+            'this-permission-is-new-and-will-be-created',
         ]);
 
         $this->assertDatabaseHas(config('permission.table_names.roles'), [
@@ -51,16 +51,6 @@ public function it_will_defer_syncing_defined_role_to_artisan_with_custom_guard(
 
         $this->assertTrue($role->hasPermissionTo('bar', 'admin'));
         $this->assertTrue($role->hasPermissionTo(FooAbility::One, 'admin'));
-    }
-
-    #[Test]
-    public function it_will_throw_an_exception_on_missing_permission(): void
-    {
-        $this->expectException(PermissionDoesNotExist::class);
-        $this->expectExceptionMessage('There is no permission named `bar` for guard `web`');
-
-        SyncDefinedRole::run('foo role', 'web', [
-            'bar',
-        ]);
+        $this->assertTrue($role->hasPermissionTo('this-permission-is-new-and-will-be-created', 'admin'));
     }
 }