formatting, more assertions

Author: limenet

tests/App/Elasticsearch/CustomerIndex.php

@@ -26,6 +26,6 @@ public function getMapping(): array
                     'type' => 'keyword',
                 ],
              ],
-                ];
+        ];
     }
 }

tests/IndexTest.php

@@ -20,6 +20,7 @@ public function setUp():void
     public function raw_index()
     {
         $index = $this->customerIndex->getElasticaIndex();
+
         $this->assertInstanceOf(Index::class, $index);
         $this->assertSame($this->customerIndex->getName(),$index->getName());
     }
@@ -29,8 +30,9 @@ public function settings()
     {
         $settings = $this->customerIndex->getCreateArguments();
         $mappings = $this->customerIndex->getMapping();
+
         $this->assertTrue($this->customerIndex->hasMapping());
-            $this->assertArrayHasKey('mappings', $settings);
-            $this->assertSame($settings['mappings'],$mappings);
+        $this->assertArrayHasKey('mappings', $settings);
+        $this->assertSame($settings['mappings'],$mappings);
     }
 }

tests/ModelTest.php

@@ -3,6 +3,8 @@
 namespace Limenet\LaravelElasticaBridge\Tests;
 
 use Elastica\Document;
+use Limenet\LaravelElasticaBridge\Index\IndexInterface;
+use Limenet\LaravelElasticaBridge\Model\ElasticsearchableInterface;
 use Limenet\LaravelElasticaBridge\Tests\App\Elasticsearch\CustomerIndex;
 use Limenet\LaravelElasticaBridge\Tests\App\Models\Customer;
 
@@ -19,12 +21,20 @@ public function setUp():void
     /** @test */
     public function convert_to_elastica_document()
     {
-        /** @var Customer $customer */
-        $customer = Customer::first();
-        $document = $customer->toElasticaDocument($this->customerIndex);
-        $this->assertInstanceOf(Document::class, $document);
+        Customer::all()
+            ->each(function(Customer $customer):void{
+                $document = $customer->toElasticaDocument($this->customerIndex);
+                $this->assertInstanceOf(Document::class, $document);
 
-        $this->assertSame($customer->name, $document->get('name'));
-        $this->assertSame($customer->email, $document->get('email'));
+                $this->assertSame($customer->name, $document->get('name'));
+                $this->assertSame($customer->email, $document->get('email'));
+                $this->assertSame($customer->type, $document->get('type'));
+
+                $this->assertStringContainsString('|'.$customer->id, $document->getId());
+                $this->assertStringContainsString($customer::class . '|', $document->getId());
+
+                $this->assertSame($customer->id, $document->get(IndexInterface::DOCUMENT_MODEL_ID));
+                $this->assertSame($customer::class, $document->get(IndexInterface::DOCUMENT_MODEL_CLASS));
+            });
     }
 }

tests/TestCase.php

@@ -33,6 +33,7 @@ public function getEnvironmentSetUp($app)
     {
         config()->set('database.default', 'testing');
     }
+
     protected function defineDatabaseMigrations():void
     {
         include_once __DIR__.'/database/migrations/SetupTables.php';

tests/database/factories/CustomerFactory.php

@@ -19,7 +19,7 @@ class CustomerFactory extends Factory
      *
      * @return array
      */
-    public function definition()
+    public function definition():array
     {
         return [
             'name' => $this->faker->name(),

tests/database/migrations/SetupTables.php

@@ -6,7 +6,7 @@
 
 class SetupTables extends Migration
 {
-    public function up()
+    public function up():void
     {
         Schema::create('customers', function (Blueprint $table) {
             $table->id();

tests/database/seeders/DatabaseSeeder.php

@@ -12,7 +12,7 @@ class DatabaseSeeder extends Seeder
      *
      * @return void
      */
-    public function run()
+    public function run():void
     {
         Customer::factory()
                 ->count(50)