Merge pull request #25 from worksome/feature/enum-cases

feat!: update `Status` enum casing

Author: owenvoke

composer.json

@@ -32,8 +32,7 @@
     },
     "autoload": {
         "psr-4": {
-            "Worksome\\MultiFactorAuth\\": "src",
-            "Worksome\\MultiFactorAuth\\Database\\Factories\\": "database/factories"
+            "Worksome\\MultiFactorAuth\\": "src"
         }
     },
     "autoload-dev": {

src/Drivers/Email/NullEmailDriver.php

@@ -16,7 +16,7 @@ class NullEmailDriver extends AbstractEmailDriver
 
     public function make(Identifier $to): CreationResponse
     {
-        return new CreationResponse($this->status ?? Status::PENDING);
+        return new CreationResponse($this->status ?? Status::Pending);
     }
 
     public function verify(Identifier $to, string $code): bool

src/Drivers/Email/TwilioVerifyEmailDriver.php

@@ -35,6 +35,6 @@ public function verify(Identifier $to, string $code): bool
 
         assert(isset($data['status']));
 
-        return Status::fromTwilioVerify($data['status']) === Status::APPROVED;
+        return Status::fromTwilioVerify($data['status']) === Status::Approved;
     }
 }

src/Drivers/Sms/NullSmsDriver.php

@@ -16,7 +16,7 @@ class NullSmsDriver extends AbstractSmsDriver
 
     public function make(Identifier $to): CreationResponse
     {
-        return new CreationResponse($this->status ?? Status::PENDING);
+        return new CreationResponse($this->status ?? Status::Pending);
     }
 
     public function verify(Identifier $to, string $code): bool

src/Drivers/Sms/TwilioVerifySmsDriver.php

@@ -35,6 +35,6 @@ public function verify(Identifier $to, string $code): bool
 
         assert(isset($data['status']));
 
-        return Status::fromTwilioVerify($data['status']) === Status::APPROVED;
+        return Status::fromTwilioVerify($data['status']) === Status::Approved;
     }
 }

src/Drivers/Totp/BasicTotpDriver.php

@@ -19,7 +19,7 @@ public function __construct(private readonly Google2FA $engine, private readonly
 
     public function make(Identifier $to): CreationResponse
     {
-        return new CreationResponse(Status::PENDING);
+        return new CreationResponse(Status::Pending);
     }
 
     public function verify(Identifier $to, string $code): bool

src/Drivers/Totp/NullTotpDriver.php

@@ -20,7 +20,7 @@ public function __construct()
 
     public function make(Identifier $to): CreationResponse
     {
-        return new CreationResponse($this->status ?? Status::PENDING, [
+        return new CreationResponse($this->status ?? Status::Pending, [
             'secret' => 'TEST',
         ]);
     }

src/Enums/Status.php

@@ -6,18 +6,18 @@
 
 enum Status: string
 {
-    case APPROVED = 'approved';
-    case PENDING = 'pending';
-    case CANCELLED = 'cancelled';
-    case FAILED = 'failed';
+    case Approved = 'approved';
+    case Pending = 'pending';
+    case Cancelled = 'cancelled';
+    case Failed = 'failed';
 
     public static function fromTwilioVerify(string $status): self
     {
         return match ($status) {
-            'approved' => self::APPROVED,
-            'pending', 'unverified' => self::PENDING,
-            'canceled' => self::CANCELLED,
-            default => self::FAILED,
+            'approved' => self::Approved,
+            'pending', 'unverified' => self::Pending,
+            'canceled' => self::Cancelled,
+            default => self::Failed,
         };
     }
 }

tests/Feature/Casts/ToFieldCasterTest.php

@@ -15,7 +15,7 @@
         'channel' => $channel,
         'to' => $to,
         'user_id' => 1,
-        'status' => Status::PENDING,
+        'status' => Status::Pending,
     ])->refresh();
 
     expect($factor->channel)->toEqual($channel)

tests/Feature/Drivers/Email/NullDriverTest.php

@@ -9,20 +9,20 @@
 
 it('can send Email from the Null driver', function () {
     $driver = (new NullEmailDriver())
-        ->fakeStatus(Status::PENDING)
+        ->fakeStatus(Status::Pending)
         ->fakeVerified();
 
     $status = $driver->make(
         new EmailAddress('test@example.org'),
     );
 
     expect($status)->toBeInstanceOf(CreationResponse::class)
-        ->status->toBe(Status::PENDING);
+        ->status->toBe(Status::Pending);
 });
 
 it('can verify Email from the Null driver', function () {
     $driver = (new NullEmailDriver())
-        ->fakeStatus(Status::PENDING)
+        ->fakeStatus(Status::Pending)
         ->fakeVerified(false);
 
     $emailAddress = new EmailAddress('test@example.org');
@@ -32,7 +32,7 @@
     )->toBeFalse();
 
     $driver
-        ->fakeStatus(Status::APPROVED)
+        ->fakeStatus(Status::Approved)
         ->fakeVerified();
 
     expect(