Merge pull request #426 from range-of-motion/replace-usage-of-native-date-function-in-spaceinvite-model

Replace usage of native date function in SpaceInvite model

Author: range-of-motion

app/Models/SpaceInvite.php

@@ -47,11 +47,11 @@ protected function status(): Attribute
             }
 
             if ($this->accepted === true) {
-                return 'Accepted (' . date('d-m', strtotime($this->updated_at)) . ')';
+                return 'Accepted (' . $this->updated_at->format('d-m') . ')';
             }
 
             if ($this->accepted === false) {
-                return 'Denied (' . date('d-m', strtotime($this->updated_at)) . ')';
+                return 'Denied (' . $this->updated_at->format('d-m') . ')';
             }
 
             return 'Unknown';

tests/Unit/Models/SpaceInviteTest.php

@@ -3,6 +3,7 @@
 namespace Tests\Unit\Models;
 
 use App\Models\SpaceInvite;
+use Carbon\Carbon;
 use Tests\TestCase;
 
 class SpaceInviteTest extends TestCase
@@ -19,15 +20,17 @@ public function testStatusAttribute(): void
         $acceptedSpaceInvite = SpaceInvite::factory()
             ->make([
                 'accepted' => true,
+                'updated_at' => Carbon::parse('2021-02-14'),
             ]);
 
-        $this->assertStringStartsWith('Accepted', $acceptedSpaceInvite->status);
+        $this->assertEquals('Accepted (14-02)', $acceptedSpaceInvite->status);
 
         $deniedSpaceInvite = SpaceInvite::factory()
             ->make([
                 'accepted' => false,
+                'updated_at' => Carbon::parse('2021-02-14'),
             ]);
 
-        $this->assertStringStartsWith('Denied', $deniedSpaceInvite->status);
+        $this->assertEquals('Denied (14-02)', $deniedSpaceInvite->status);
     }
 }