Merge pull request #30 from laravel-enso/feature/roles

Feature/roles

config/calendar.php

@@ -0,0 +1,7 @@
+<?php
+
+return [
+    'birthdays' => [
+        'roles' => ['*'],
+    ],
+];

src/AppServiceProvider.php

@@ -18,13 +18,16 @@ public function boot()
             ->publishProvider()
             ->publishFactories()
             ->publishMail()
+            ->publishConfig()
             ->commands(SendReminders::class);
     }
 
     private function load()
     {
         $this->loadRoutesFrom(__DIR__.'/../routes/api.php');
 
+        $this->mergeConfigFrom(__DIR__.'/../config/calendar.php', 'enso.calendar');
+
         $this->loadMigrationsFrom(__DIR__.'/../database/migrations');
 
         $this->loadViewsFrom(__DIR__.'/../resources/views', 'laravel-enso/calendar');
@@ -68,4 +71,13 @@ private function publishMail()
 
         return $this;
     }
+
+    private function publishConfig()
+    {
+        $this->publishes([
+            __DIR__.'/../config' => config_path('enso'),
+        ], ['enso-config', 'calendar-config']);
+
+        return $this;
+    }
 }

src/Calendars/BirthdayCalendar.php

@@ -4,6 +4,7 @@
 
 use Carbon\Carbon;
 use Illuminate\Support\Collection;
+use Illuminate\Support\Facades\Config;
 use LaravelEnso\Calendar\Contracts\CustomCalendar;
 use LaravelEnso\Calendar\Enums\Colors;
 use LaravelEnso\People\Models\Person;
@@ -42,8 +43,11 @@ public function events(Carbon $startDate, Carbon $endDate): Collection
     {
         $this->startDate = $startDate;
         $this->endDate = $endDate;
+        $roles = Config::get('enso.calendar.birthdays.roles');
 
         return Person::query()
+            ->unless($roles === ['*'], fn ($query) => $query
+                ->whereHas('user', fn ($query) => $query->whereIn('role_id', $roles)))
             ->when(! $this->withinSameYear(), $this->differentYearQuery())
             ->when(
                 $this->withinSameYear() && $this->withinSameMonth(),

tests/features/CalendarTest.php

@@ -0,0 +1,51 @@
+<?php
+
+use Carbon\Carbon;
+use Illuminate\Foundation\Testing\RefreshDatabase;
+use Illuminate\Support\Facades\Config;
+use LaravelEnso\Calendar\Calendars\BirthdayCalendar;
+use LaravelEnso\Calendar\Enums\Colors;
+use LaravelEnso\Calendar\Models\Calendar;
+use LaravelEnso\Core\Models\User;
+use LaravelEnso\People\Models\Person;
+use Tests\TestCase;
+
+class CalendarTest extends TestCase
+{
+    use RefreshDatabase;
+
+    protected Calendar $calendar;
+    protected Carbon $date;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->seed();
+
+        $this->actingAs(User::first());
+
+        $this->calendar = Calendar::factory()->create();
+    }
+
+    /** @test */
+    public function create_calendar()
+    {
+        $result = $this->post(route('core.calendar.store'), [
+            'name' => 'test',
+            'color' => Colors::Red,
+            'private' => false,
+        ]);
+
+        $this->assertTrue(Calendar::whereName('test')->exists());
+    }
+
+    /** @test */
+    public function can_limit_birthday_calendar_roles()
+    {
+        Config::set('enso.calendar.birthdays.roles', []);
+
+        $this->assertEmpty((new BirthdayCalendar())
+            ->events(Person::first()->birthday, Person::first()->birthday->addDay()));
+    }
+}

tests/features/CreateTest.php

@@ -1,7 +1,5 @@
 <?php
 
-namespace LaravelEnso\Calendar\tests\features;
-
 use Carbon\Carbon;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use LaravelEnso\Calendar\Enums\Frequencies;

tests/features/EventTest.php

@@ -1,7 +1,5 @@
 <?php
 
-namespace LaravelEnso\Calendar\tests\features;
-
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Illuminate\Support\Collection;
 use LaravelEnso\Calendar\Enums\Frequencies;

tests/features/SequenceTest.php

@@ -1,7 +1,5 @@
 <?php
 
-namespace LaravelEnso\Calendar\tests\features;
-
 use Carbon\Carbon;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use LaravelEnso\Calendar\Enums\Frequencies;