fix enums

Astrotomic · Feb 28, 2024 · c685020 · c685020
1 parent 8bb97db
commit c685020
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 22 deletions.
diff --git a/README.md b/README.md
@@ -148,8 +148,8 @@ use Astrotomic\Tmdb\Enums\WatchProviderType;
 
 Movie::findOrFail(335983)->watchProviders();
 Movie::findOrFail(335983)->watchProviders('DE');
-Movie::findOrFail(335983)->watchProviders(null, WatchProviderType::FLATRATE());
-Movie::findOrFail(335983)->watchProviders('DE', WatchProviderType::FLATRATE());
+Movie::findOrFail(335983)->watchProviders(null, WatchProviderType::FLATRATE);
+Movie::findOrFail(335983)->watchProviders('DE', WatchProviderType::FLATRATE);
 ```
 
 And there are some helper methods on the movie model to easier work with some attributes.

diff --git a/src/Enums/WatchProviderType.php b/src/Enums/WatchProviderType.php
@@ -2,21 +2,9 @@
 
 namespace Astrotomic\Tmdb\Enums;
 
-use Spatie\Enum\Laravel\Enum;
-
-/**
- * @method static self BUY()
- * @method static self RENT()
- * @method static self FLATRATE()
- */
-class WatchProviderType extends Enum
+enum WatchProviderType: string
 {
-    protected static function values(): array
-    {
-        return [
-            'BUY' => 'buy',
-            'RENT' => 'rent',
-            'FLATRATE' => 'flatrate',
-        ];
-    }
+    case BUY = 'buy';
+    case RENT = 'rent';
+    case FLATRATE = 'flatrate';
 }
diff --git a/tests/Feature/Models/MovieTest.php b/tests/Feature/Models/MovieTest.php
@@ -301,31 +301,31 @@
 });
 
 it('loads buy watch providers', function (): void {
-    $providers = Movie::query()->findOrFail(335983)->watchProviders(null, WatchProviderType::BUY());
+    $providers = Movie::query()->findOrFail(335983)->watchProviders(null, WatchProviderType::BUY);
 
     expect($providers)
         ->toHaveCount(31)
         ->each->toBeInstanceOf(WatchProvider::class);
 });
 
 it('loads rent watch providers', function (): void {
-    $providers = Movie::query()->findOrFail(335983)->watchProviders(null, WatchProviderType::RENT());
+    $providers = Movie::query()->findOrFail(335983)->watchProviders(null, WatchProviderType::RENT);
 
     expect($providers)
         ->toHaveCount(36)
         ->each->toBeInstanceOf(WatchProvider::class);
 });
 
 it('loads flatrate watch providers', function (): void {
-    $providers = Movie::query()->findOrFail(335983)->watchProviders(null, WatchProviderType::FLATRATE());
+    $providers = Movie::query()->findOrFail(335983)->watchProviders(null, WatchProviderType::FLATRATE);
 
     expect($providers)
         ->toHaveCount(24)
         ->each->toBeInstanceOf(WatchProvider::class);
 });
 
 it('loads german flatrate watch providers', function (): void {
-    $providers = Movie::query()->findOrFail(335983)->watchProviders('DE', WatchProviderType::FLATRATE());
+    $providers = Movie::query()->findOrFail(335983)->watchProviders('DE', WatchProviderType::FLATRATE);
 
     expect($providers)
         ->toHaveCount(1)