diff --git a/.claude/settings.local.json b/.claude/settings.local.json index e5f9503..4dc6383 100644 --- a/.claude/settings.local.json +++ b/.claude/settings.local.json @@ -1,6 +1,10 @@ { "permissions": { "allow": [ +<<<<<<< HEAD +======= + "mcp__laravel-boost__database-schema", +>>>>>>> role-module "Bash(php artisan *)", "Bash(vendor/bin/pint --dirty --format agent)", "mcp__laravel-boost__search-docs" diff --git a/app/Http/Controllers/RoleController.php b/app/Http/Controllers/RoleController.php new file mode 100644 index 0000000..cb60165 --- /dev/null +++ b/app/Http/Controllers/RoleController.php @@ -0,0 +1,36 @@ +paginate(10); + + if ($request->ajax()) { + return view('roles._table', compact('roles')); + } + + return view('roles.index', compact('roles')); + } + + public function create(): View + { + return view('roles.create'); + } + + public function store(StoreRoleRequest $request): RedirectResponse + { + Role::create($request->validated()); + + return redirect()->route('roles.index') + ->with('status', 'role-created'); + } +} diff --git a/app/Http/Requests/StoreRoleRequest.php b/app/Http/Requests/StoreRoleRequest.php new file mode 100644 index 0000000..ef2c983 --- /dev/null +++ b/app/Http/Requests/StoreRoleRequest.php @@ -0,0 +1,28 @@ +|string> + */ + public function rules(): array + { + return [ + 'name' => ['required', 'string', 'max:255', 'unique:roles,name'], + 'description' => ['nullable', 'string', 'max:500'], + ]; + } +} diff --git a/app/Models/Role.php b/app/Models/Role.php new file mode 100644 index 0000000..a68d904 --- /dev/null +++ b/app/Models/Role.php @@ -0,0 +1,15 @@ + */ + use HasFactory; + + protected $fillable = ['name', 'description']; +} diff --git a/database/factories/RoleFactory.php b/database/factories/RoleFactory.php new file mode 100644 index 0000000..ffbe866 --- /dev/null +++ b/database/factories/RoleFactory.php @@ -0,0 +1,25 @@ + + */ +class RoleFactory extends Factory +{ + /** + * Define the model's default state. + * + * @return array + */ + public function definition(): array + { + return [ + 'name' => fake()->unique()->word(), + 'description' => fake()->sentence(), + ]; + } +} diff --git a/database/migrations/2026_05_11_042341_create_roles_table.php b/database/migrations/2026_05_11_042341_create_roles_table.php new file mode 100644 index 0000000..5ea3089 --- /dev/null +++ b/database/migrations/2026_05_11_042341_create_roles_table.php @@ -0,0 +1,29 @@ +id(); + $table->string('name')->unique(); + $table->string('description')->nullable(); + $table->timestamps(); + }); + } + + /** + * Reverse the migrations. + */ + public function down(): void + { + Schema::dropIfExists('roles'); + } +}; diff --git a/database/seeders/RoleSeeder.php b/database/seeders/RoleSeeder.php new file mode 100644 index 0000000..1933f21 --- /dev/null +++ b/database/seeders/RoleSeeder.php @@ -0,0 +1,22 @@ + 'Admin', 'description' => 'Full access to all resources'], + ['name' => 'Editor', 'description' => 'Can create and edit content'], + ['name' => 'Viewer', 'description' => 'Read-only access'], + ]; + + foreach ($roles as $role) { + Role::firstOrCreate(['name' => $role['name']], $role); + } + } +} diff --git a/resources/views/layouts/navigation.blade.php b/resources/views/layouts/navigation.blade.php index af4fd24..63fc66a 100644 --- a/resources/views/layouts/navigation.blade.php +++ b/resources/views/layouts/navigation.blade.php @@ -17,6 +17,8 @@ {{ __('Users') }} + + {{ __('Roles') }} @@ -75,6 +77,8 @@ {{ __('Users') }} + + {{ __('Roles') }} diff --git a/resources/views/roles/_table.blade.php b/resources/views/roles/_table.blade.php new file mode 100644 index 0000000..b6b38b2 --- /dev/null +++ b/resources/views/roles/_table.blade.php @@ -0,0 +1,34 @@ + + + + + + + + + + + @foreach ($roles as $role) + + + + + + + @endforeach + + @if ($roles->isEmpty()) + + + + @endif + +
#{{ __('Name') }}{{ __('Description') }}{{ __('Created') }}
{{ $roles->firstItem() + $loop->index }}{{ $role->name }}{{ $role->description ?? '—' }}{{ $role->created_at->format('d M Y') }}
+ {{ __('No roles found.') }} +
+ +@if ($roles->hasPages()) +
+ {{ $roles->links() }} +
+@endif diff --git a/resources/views/roles/create.blade.php b/resources/views/roles/create.blade.php new file mode 100644 index 0000000..3a24f01 --- /dev/null +++ b/resources/views/roles/create.blade.php @@ -0,0 +1,56 @@ + + +

+ {{ __('New Role') }} +

+
+ +
+
+
+
+
+
+

+ {{ __('Role Details') }} +

+ +

+ {{ __('Create a new role to assign to users in the system.') }} +

+
+ +
+ @csrf + +
+ + + +
+ +
+ + + +
+ +
+ {{ __('Create Role') }} + + + {{ __('Cancel') }} + +
+
+
+
+
+
+
+
diff --git a/resources/views/roles/index.blade.php b/resources/views/roles/index.blade.php new file mode 100644 index 0000000..dae2c3c --- /dev/null +++ b/resources/views/roles/index.blade.php @@ -0,0 +1,59 @@ + + +
+

+ {{ __('Roles') }} +

+ + {{ __('New Role') }} + +
+
+ +
+
+ @if (session('status') === 'role-created') +
+ {{ __('Role created successfully.') }} +
+ @endif + +
+
+
+ {{ __('Loading...') }} +
+ +
+ @include('roles._table') +
+
+
+
+
+
diff --git a/routes/web.php b/routes/web.php index ce44b28..fb24986 100644 --- a/routes/web.php +++ b/routes/web.php @@ -2,6 +2,7 @@ use App\Http\Controllers\ProfileController; use App\Http\Controllers\UserController; +use App\Http\Controllers\RoleController; use Illuminate\Support\Facades\Route; Route::get('/', function () { @@ -14,6 +15,9 @@ Route::get('/dashboard', function () { Route::middleware('auth')->group(function () { Route::get('/users', [UserController::class, 'index'])->name('users.index'); + Route::get('/roles', [RoleController::class, 'index'])->name('roles.index'); + Route::get('/roles/create', [RoleController::class, 'create'])->name('roles.create'); + Route::post('/roles', [RoleController::class, 'store'])->name('roles.store'); Route::get('/profile', [ProfileController::class, 'edit'])->name('profile.edit'); Route::patch('/profile', [ProfileController::class, 'update'])->name('profile.update'); Route::delete('/profile', [ProfileController::class, 'destroy'])->name('profile.destroy'); diff --git a/tests/Feature/RoleIndexTest.php b/tests/Feature/RoleIndexTest.php new file mode 100644 index 0000000..78a2f2f --- /dev/null +++ b/tests/Feature/RoleIndexTest.php @@ -0,0 +1,41 @@ +get('/roles')->assertRedirect('/login'); +}); + +test('authenticated users can view the roles page', function () { + $this->actingAs(User::factory()->create()) + ->get('/roles') + ->assertOk() + ->assertViewIs('roles.index'); +}); + +test('roles page passes paginated roles to view', function () { + $this->actingAs(User::factory()->create()) + ->get('/roles') + ->assertOk() + ->assertViewHas('roles'); +}); + +test('ajax request returns roles table partial', function () { + $this->actingAs(User::factory()->create()) + ->withHeader('X-Requested-With', 'XMLHttpRequest') + ->get('/roles') + ->assertOk() + ->assertViewIs('roles._table'); +}); + +test('ajax pagination returns correct page', function () { + Role::factory()->count(15)->create(); + + $this->actingAs(User::factory()->create()) + ->withHeader('X-Requested-With', 'XMLHttpRequest') + ->get('/roles?page=2') + ->assertOk() + ->assertViewIs('roles._table') + ->assertViewHas('roles', fn ($roles) => $roles->currentPage() === 2); +}); diff --git a/tests/Feature/StoreRoleTest.php b/tests/Feature/StoreRoleTest.php new file mode 100644 index 0000000..c6b0994 --- /dev/null +++ b/tests/Feature/StoreRoleTest.php @@ -0,0 +1,52 @@ +get('/roles/create')->assertRedirect('/login'); +}); + +test('authenticated users can access the create role page', function () { + $this->actingAs(User::factory()->create()) + ->get('/roles/create') + ->assertOk() + ->assertViewIs('roles.create'); +}); + +test('authenticated users can create a role', function () { + $this->actingAs(User::factory()->create()) + ->post('/roles', ['name' => 'Manager', 'description' => 'Manages things']) + ->assertRedirect(route('roles.index')) + ->assertSessionHas('status', 'role-created'); + + $this->assertDatabaseHas('roles', ['name' => 'Manager', 'description' => 'Manages things']); +}); + +test('role name is required', function () { + $this->actingAs(User::factory()->create()) + ->post('/roles', ['name' => '', 'description' => 'Some description']) + ->assertSessionHasErrors('name'); +}); + +test('role name must be unique', function () { + Role::factory()->create(['name' => 'Admin']); + + $this->actingAs(User::factory()->create()) + ->post('/roles', ['name' => 'Admin']) + ->assertSessionHasErrors('name'); +}); + +test('description is optional', function () { + $this->actingAs(User::factory()->create()) + ->post('/roles', ['name' => 'Viewer']) + ->assertRedirect(route('roles.index')); + + $this->assertDatabaseHas('roles', ['name' => 'Viewer', 'description' => null]); +}); + +test('guests cannot create a role', function () { + $this->post('/roles', ['name' => 'Admin'])->assertRedirect('/login'); + + $this->assertDatabaseMissing('roles', ['name' => 'Admin']); +});