test: add empty lines or so

To make more readable.

Author: kenjis

tests/system/Database/Live/BadQueryTest.php

@@ -31,8 +31,10 @@ final class BadQueryTest extends CIUnitTestCase
     public function testBadQueryDebugTrue()
     {
         $this->enableDBDebug();
+
         // expect an exception, class and message varies by DBMS
         $this->expectException(Exception::class);
+
         $this->db->query('SELECT * FROM table_does_not_exist');
 
         // this code is never executed
@@ -42,8 +44,10 @@ public function testBadQueryDebugFalse()
     {
         // WARNING this value will persist! take care to roll it back.
         $this->disableDBDebug();
+
         // this throws an exception when DBDebug is true, but it'll return FALSE when DBDebug is false
         $query = $this->db->query('SELECT * FROM table_does_not_exist');
+
         $this->assertFalse($query);
 
         $this->enableDBDebug();

tests/system/Database/Live/DbDebugTest.php

@@ -28,21 +28,26 @@ final class DbDebugTest extends CIUnitTestCase
     public function testDBDebugTrue()
     {
         $this->enableDBDebug();
+
         $this->expectException('Exception');
+
         $this->db->simpleQuery('SELECT * FROM db_error');
     }
 
     public function testDBDebugFalse()
     {
         // WARNING this value will persist! take care to roll it back.
         $this->disableDBDebug();
+
         $result = $this->db->simpleQuery('SELECT * FROM db_error');
+
         $this->assertFalse($result);
     }
 
     protected function tearDown(): void
     {
         $this->enableDBDebug();
+
         parent::tearDown();
     }
 }

tests/system/Database/Live/DbUtilsTest.php

@@ -118,6 +118,7 @@ public function testUtilsOptimizeTableFalseOptimizeDatabaseDebugTrue()
 
         $this->expectException(DatabaseException::class);
         $this->expectExceptionMessage('Unsupported feature of the database platform you are using.');
+
         $util->optimizeDatabase();
 
         // this point in code execution will never be reached

tests/system/Models/DeleteModelTest.php

@@ -36,10 +36,13 @@ public function testDeleteFail(): void
     {
         // WARNING this value will persist! take care to roll it back.
         $this->disableDBDebug();
+
         $this->createModel(JobModel::class);
+
         $this->seeInDatabase('job', ['name' => 'Developer']);
 
         $result = $this->model->where('name123', 'Developer')->delete();
+
         $this->assertFalse($result);
         $this->seeInDatabase('job', ['name' => 'Developer']);
 
@@ -73,10 +76,13 @@ public function testDeleteWithSoftDeleteFail(): void
     {
         // WARNING this value will persist! take care to roll it back.
         $this->disableDBDebug();
+
         $this->createModel(UserModel::class);
+
         $this->seeInDatabase('user', ['name' => 'Derek Jones', 'deleted_at IS NULL' => null]);
 
         $result = $this->model->where('name123', 'Derek Jones')->delete();
+
         $this->assertFalse($result);
         $this->seeInDatabase('user', ['name' => 'Derek Jones', 'deleted_at IS NULL' => null]);
 

tests/system/Models/InsertModelTest.php

@@ -148,12 +148,14 @@ public function testInsertResultFail(): void
             'name123'     => 'Apprentice',
             'description' => 'That thing you do.',
         ];
-
         $this->createModel(JobModel::class);
+
         $result = $this->model->protect(false)->insert($data, false);
+
         $this->assertFalse($result);
 
         $lastInsertId = $this->model->getInsertID();
+
         $this->assertSame(0, $lastInsertId);
         $this->dontSeeInDatabase('job', ['id' => $lastInsertId]);
 

tests/system/Models/SaveModelTest.php

@@ -63,8 +63,8 @@ public function testSaveNewRecordArrayFail(): void
             'name123'     => 'Apprentice',
             'description' => 'That thing you do.',
         ];
-
         $result = $this->model->protect(false)->save($data);
+
         $this->assertFalse($result);
         $this->dontSeeInDatabase('job', ['name' => 'Apprentice']);
 
@@ -96,8 +96,8 @@ public function testSaveUpdateRecordArrayFail(): void
             'name123'     => 'Apprentice',
             'description' => 'That thing you do.',
         ];
-
         $result = $this->model->protect(false)->save($data);
+
         $this->assertFalse($result);
         $this->dontSeeInDatabase('job', ['name' => 'Apprentice']);
 

tests/system/Models/UpdateModelTest.php

@@ -98,19 +98,20 @@ public function testUpdateResultFail(): void
     {
         // WARNING this value will persist! take care to roll it back.
         $this->disableDBDebug();
+        $this->createModel(UserModel::class);
 
         $data = [
             'name'    => 'Foo',
             'email'   => 'foo@example.com',
             'country' => 'US',
             'deleted' => 0,
         ];
-
-        $this->createModel(UserModel::class);
         $this->model->insert($data);
 
         $this->setPrivateProperty($this->model, 'allowedFields', ['name123']);
+
         $result = $this->model->update(1, ['name123' => 'Foo Bar 1']);
+
         $this->assertFalse($result);
         $this->dontSeeInDatabase('user', ['id' => 1, 'name' => 'Foo Bar 1']);