Port EloquentHasOneIsTest and EloquentHasOneOfManyTest

- Update namespaces from Illuminate to Hypervel
- Add strict types to model properties and method signatures
- Adapt testItOnlyEagerLoadsRequiredModels for Hypervel's event dispatcher
  (wildcard listeners receive spread payload, not array)
- Comment out original test for restoration when illuminate/events is ported

Author: binaryfire

tests/Integration/Database/Laravel/EloquentHasOneIsTest.php

@@ -2,20 +2,20 @@
 
 declare(strict_types=1);
 
-namespace Illuminate\Tests\Integration\Database\EloquentHasOneIsTest;
+namespace Hypervel\Tests\Integration\Database\Laravel\EloquentHasOneIsTest;
 
-use Illuminate\Database\Eloquent\Model;
-use Illuminate\Database\Schema\Blueprint;
-use Illuminate\Support\Facades\Schema;
-use Illuminate\Tests\Integration\Database\DatabaseTestCase;
+use Hypervel\Database\Eloquent\Model;
+use Hypervel\Database\Schema\Blueprint;
+use Hypervel\Support\Facades\Schema;
+use Hypervel\Tests\Integration\Database\DatabaseTestCase;
 
 /**
  * @internal
  * @coversNothing
  */
 class EloquentHasOneIsTest extends DatabaseTestCase
 {
-    protected function afterRefreshingDatabase()
+    protected function afterRefreshingDatabase(): void
     {
         Schema::create('posts', function (Blueprint $table) {
             $table->increments('id');
@@ -92,7 +92,7 @@ public function testChildIsNotModelWithAnotherConnection()
 
 class Attachment extends Model
 {
-    public $timestamps = false;
+    public bool $timestamps = false;
 }
 
 class Post extends Model

tests/Integration/Database/Laravel/EloquentHasOneOfManyTest.php

@@ -2,21 +2,21 @@
 
 declare(strict_types=1);
 
-namespace Illuminate\Tests\Integration\Database\EloquentHasOneOfManyTest;
+namespace Hypervel\Tests\Integration\Database\Laravel\EloquentHasOneOfManyTest;
 
-use Illuminate\Database\Eloquent\Model;
-use Illuminate\Support\Facades\Schema;
-use Illuminate\Tests\Integration\Database\DatabaseTestCase;
+use Hypervel\Database\Eloquent\Model;
+use Hypervel\Support\Facades\Schema;
+use Hypervel\Tests\Integration\Database\DatabaseTestCase;
 
 /**
  * @internal
  * @coversNothing
  */
 class EloquentHasOneOfManyTest extends DatabaseTestCase
 {
-    public $retrievedLogins;
+    public int $retrievedLogins;
 
-    protected function afterRefreshingDatabase()
+    protected function afterRefreshingDatabase(): void
     {
         Schema::create('users', function ($table) {
             $table->id();
@@ -36,14 +36,16 @@ protected function afterRefreshingDatabase()
         });
     }
 
+    /**
+     * @TODO Replace with testItOnlyEagerLoadsRequiredModelsOriginal once illuminate/events package is ported.
+     *       Hypervel's event dispatcher spreads wildcard listener payload instead of passing array.
+     */
     public function testItOnlyEagerLoadsRequiredModels()
     {
         $this->retrievedLogins = 0;
-        User::getEventDispatcher()->listen('eloquent.retrieved:*', function ($event, $models) {
-            foreach ($models as $model) {
-                if (get_class($model) == Login::class) {
-                    ++$this->retrievedLogins;
-                }
+        User::getEventDispatcher()->listen('eloquent.retrieved:*', function ($event, $model) {
+            if ($model instanceof Login) {
+                ++$this->retrievedLogins;
             }
         });
 
@@ -59,6 +61,30 @@ public function testItOnlyEagerLoadsRequiredModels()
         $this->assertSame(2, $this->retrievedLogins);
     }
 
+    // @TODO Restore this test once illuminate/events package is ported (wildcard listeners receive array payload)
+    // public function testItOnlyEagerLoadsRequiredModelsOriginal()
+    // {
+    //     $this->retrievedLogins = 0;
+    //     User::getEventDispatcher()->listen('eloquent.retrieved:*', function ($event, $models) {
+    //         foreach ($models as $model) {
+    //             if (get_class($model) == Login::class) {
+    //                 ++$this->retrievedLogins;
+    //             }
+    //         }
+    //     });
+    //
+    //     $user = User::create();
+    //     $user->latest_login()->create();
+    //     $user->latest_login()->create();
+    //     $user = User::create();
+    //     $user->latest_login()->create();
+    //     $user->latest_login()->create();
+    //
+    //     User::with('latest_login')->get();
+    //
+    //     $this->assertSame(2, $this->retrievedLogins);
+    // }
+
     public function testItGetsCorrectResultUsingAtLeastTwoAggregatesDistinctFromId()
     {
         $user = User::create();
@@ -89,9 +115,9 @@ public function testItGetsCorrectResultUsingAtLeastTwoAggregatesDistinctFromId()
 
 class User extends Model
 {
-    protected $guarded = [];
+    protected array $guarded = [];
 
-    public $timestamps = false;
+    public bool $timestamps = false;
 
     public function latest_login()
     {
@@ -132,12 +158,12 @@ public function oldest_updated_oldest_created_state()
 
 class Login extends Model
 {
-    protected $guarded = [];
+    protected array $guarded = [];
 
-    public $timestamps = false;
+    public bool $timestamps = false;
 }
 
 class State extends Model
 {
-    protected $guarded = [];
+    protected array $guarded = [];
 }