minor #1774 [make:crud] Ensure the generated functional tests run in MakerBundle's CI (GromNaN)

This PR was merged into the 1.x branch.

Discussion
----------

[make:crud] Ensure the generated functional tests run in MakerBundle's CI

In order to ensure there is no regression in the generated test case (#1773), they must be run in MakerBundle's CI.

The `markTestIncomplete` call stops the execution by throwing and exception. Skipping all the test body. By moving this call at the end of the method, the tests are fully run.

I even wonder if marking tests as incomplete serves any purpose.

For reference, this was added in #307, this point was not discussed.

Commits
-------

69e911a [make:crud] Move markTestIncomplete at the end to ensure the test is run in MakerBundle CI

Author: GromNaN

templates/crud/test/Test.EntityManager.tpl.php

@@ -40,7 +40,6 @@ public function testIndex(): void
 
     public function testNew(): void
     {
-        $this->markTestIncomplete();
         $this->client->request('GET', sprintf('%snew', $this->path));
 
         self::assertResponseStatusCodeSame(200);
@@ -54,11 +53,12 @@ public function testNew(): void
         self::assertResponseRedirects('<?= $route_path; ?>');
 
         self::assertSame(1, $this-><?= lcfirst($entity_var_singular); ?>Repository->count([]));
+
+        $this->markTestIncomplete('This test was generated');
     }
 
     public function testShow(): void
     {
-        $this->markTestIncomplete();
         $fixture = new <?= $entity_class_name; ?>();
 <?php foreach ($form_fields as $form_field => $typeOptions): ?>
         $fixture->set<?= Str::asCamelCase($form_field); ?>('My Title');
@@ -73,11 +73,11 @@ public function testShow(): void
         self::assertPageTitleContains('<?= ucfirst($entity_var_singular); ?>');
 
         // Use assertions to check that the properties are properly displayed.
+        $this->markTestIncomplete('This test was generated');
     }
 
     public function testEdit(): void
     {
-        $this->markTestIncomplete();
         $fixture = new <?= $entity_class_name; ?>();
 <?php foreach ($form_fields as $form_field => $typeOptions): ?>
         $fixture->set<?= Str::asCamelCase($form_field); ?>('Value');
@@ -101,11 +101,12 @@ public function testEdit(): void
 <?php foreach ($form_fields as $form_field => $typeOptions): ?>
         self::assertSame('Something New', $fixture[0]->get<?= Str::asCamelCase($form_field); ?>());
 <?php endforeach; ?>
+
+        $this->markTestIncomplete('This test was generated');
     }
 
     public function testRemove(): void
     {
-        $this->markTestIncomplete();
         $fixture = new <?= $entity_class_name; ?>();
 <?php foreach ($form_fields as $form_field => $typeOptions): ?>
         $fixture->set<?= Str::asCamelCase($form_field); ?>('Value');
@@ -119,5 +120,7 @@ public function testRemove(): void
 
         self::assertResponseRedirects('<?= $route_path; ?>');
         self::assertSame(0, $this-><?= lcfirst($entity_var_singular); ?>Repository->count([]));
+
+        $this->markTestIncomplete('This test was generated');
     }
 }