From 407c5aef2e74e0e18a6069dd926e604cf11907b5 Mon Sep 17 00:00:00 2001
From: Daniel Wehner <daniel@tag1consulting.com>
Date: Fri, 11 Dec 2015 15:46:44 +0100
Subject: [PATCH] use one example test entity type instead of multiple

---
 tests/Kernel/EntityRevisionLogTraitTest.php   |  4 +-
 tests/Kernel/RevisionBasicUITest.php          | 10 ++--
 ...hRevisionRoutes.php => EnhancedEntity.php} | 23 +++++----
 .../src/Entity/EntityWithRevisionLog.php      | 49 -------------------
 4 files changed, 20 insertions(+), 66 deletions(-)
 rename tests/modules/entity_module_test/src/Entity/{EntityWithRevisionRoutes.php => EnhancedEntity.php} (62%)
 delete mode 100644 tests/modules/entity_module_test/src/Entity/EntityWithRevisionLog.php

diff --git a/tests/Kernel/EntityRevisionLogTraitTest.php b/tests/Kernel/EntityRevisionLogTraitTest.php
index 7578636..d2c7a87 100644
--- a/tests/Kernel/EntityRevisionLogTraitTest.php
+++ b/tests/Kernel/EntityRevisionLogTraitTest.php
@@ -7,7 +7,7 @@
 
 namespace Drupal\Tests\entity\Kernel;
 
-use Drupal\entity_module_test\Entity\EntityWithRevisionLog;
+use Drupal\entity_module_test\Entity\EnhancedEntity;
 use Drupal\KernelTests\KernelTestBase;
 use Drupal\user\Entity\User;
 
@@ -42,7 +42,7 @@ class EntityRevisionLogTraitTest extends KernelTestBase {
     $user2->save();
 
     /** @var \Drupal\entity\Revision\EntityRevisionLogInterface $entity */
-    $entity = EntityWithRevisionLog::create([
+    $entity = EnhancedEntity::create([
       'revision_user' => $user->id(),
       'revision_created' => 1447941735,
       'revision_log_message' => 'Test message',
diff --git a/tests/Kernel/RevisionBasicUITest.php b/tests/Kernel/RevisionBasicUITest.php
index 69d7e8c..e19fdef 100644
--- a/tests/Kernel/RevisionBasicUITest.php
+++ b/tests/Kernel/RevisionBasicUITest.php
@@ -7,7 +7,7 @@
 
 namespace Drupal\Tests\entity\Kernel;
 
-use Drupal\entity_module_test\Entity\EntityWithRevisionRoutes;
+use Drupal\entity_module_test\Entity\EnhancedEntity;
 use Drupal\KernelTests\KernelTestBase;
 use Drupal\user\Entity\Role;
 use Drupal\user\Entity\User;
@@ -30,14 +30,14 @@ class RevisionBasicUITest extends KernelTestBase {
     parent::setUp();
 
     $this->installEntitySchema('user');
-    $this->installEntitySchema('entity_test__rev_routes');
+    $this->installEntitySchema('entity_test_enhanced');
     $this->installSchema('system', 'router');
 
     \Drupal::service('router.builder')->rebuild();
   }
 
   public function testRevisionView() {
-    $entity = EntityWithRevisionRoutes::create([
+    $entity = EnhancedEntity::create([
       'name' => 'rev 1',
     ]);
     $entity->save();
@@ -55,8 +55,8 @@ class RevisionBasicUITest extends KernelTestBase {
     $this->assertEquals(403, $response->getStatusCode());
 
     $role = Role::create(['id' => 'test_role']);
-    $role->grantPermission('view all entity_test__rev_routes revisions');
-    $role->grantPermission('administer entity_test__revision_routes');
+    $role->grantPermission('view all entity_test_enhanced revisions');
+    $role->grantPermission('administer entity_test_enhanced');
     $role->save();
 
     $user = User::create([
diff --git a/tests/modules/entity_module_test/src/Entity/EntityWithRevisionRoutes.php b/tests/modules/entity_module_test/src/Entity/EnhancedEntity.php
similarity index 62%
rename from tests/modules/entity_module_test/src/Entity/EntityWithRevisionRoutes.php
rename to tests/modules/entity_module_test/src/Entity/EnhancedEntity.php
index 5f6566b..9cefdac 100644
--- a/tests/modules/entity_module_test/src/Entity/EntityWithRevisionRoutes.php
+++ b/tests/modules/entity_module_test/src/Entity/EnhancedEntity.php
@@ -2,7 +2,7 @@
 
 /**
  * @file
- * Contains \Drupal\entity_module_test\Entity\EntityWithRevisionRoutes.
+ * Contains \Drupal\entity_module_test\Entity\EnhancedEntity.
  */
 
 namespace Drupal\entity_module_test\Entity;
@@ -14,33 +14,35 @@ use Drupal\entity\EntityKeysFieldsTrait;
 use Drupal\entity\Revision\EntityRevisionLogTrait;
 
 /**
+ * Provides a test entity which uses all the capabilities of entity module.
+ *
  * @ContentEntityType(
- *   id = "entity_test__rev_routes",
- *   label = @Translation("Entity test with revision routes"),
+ *   id = "entity_test_enhanced",
+ *   label = @Translation("Entity test with enhancements"),
  *   handlers = {
  *     "storage" = "\Drupal\Core\Entity\Sql\SqlContentEntityStorage",
  *     "route_provider" = {
  *       "revision" = "\Drupal\entity\Routing\RevisionRouteProvider",
  *     },
  *   },
- *   base_table = "entity_test__revision_routes",
- *   data_table = "entity_test__revision_routes__field_data",
- *   revision_table = "entity_test__revision_routes__revision",
- *   revision_data_table = "entity_test__revision_routes__field_revision",
+ *   base_table = "entity_test_enhanced",
+ *   data_table = "entity_test_enhanced_field_data",
+ *   revision_table = "entity_test_enhanced_revision",
+ *   revision_data_table = "entity_test_enhanced_field_revision",
  *   translatable = TRUE,
  *   revisionable = TRUE,
- *   admin_permission = "administer entity_test__revision_routes",
+ *   admin_permission = "administer entity_test_enhanced",
  *   entity_keys = {
  *     "id" = "id",
  *     "revision" = "vid",
  *     "langcode" = "langcode",
  *   },
  *   links = {
- *     "revision" = "/entity_test__rev_routes/{entity_test__rev_routes}/revisions/{entity_test__rev_routes_revision}/view",
+ *     "revision" = "/entity_test_enhanced/{entity_test_enhanced}/revisions/{entity_test_enhanced_revision}/view",
  *   }
  * )
  */
-class EntityWithRevisionRoutes extends ContentEntityBase {
+class EnhancedEntity extends ContentEntityBase {
 
   use EntityRevisionLogTrait;
   use EntityKeysFieldsTrait;
@@ -52,6 +54,7 @@ class EntityWithRevisionRoutes extends ContentEntityBase {
     $fields = [];
 
     $fields += static::entityKeysBaseFieldDefinitions($entity_type);
+    $fields += static::entityRevisionLogBaseFieldDefinitions();
 
     $fields['name'] = BaseFieldDefinition::create('string')
       ->setLabel('Name')
diff --git a/tests/modules/entity_module_test/src/Entity/EntityWithRevisionLog.php b/tests/modules/entity_module_test/src/Entity/EntityWithRevisionLog.php
deleted file mode 100644
index d44f34b..0000000
--- a/tests/modules/entity_module_test/src/Entity/EntityWithRevisionLog.php
+++ /dev/null
@@ -1,49 +0,0 @@
-<?php
-
-/**
- * @file
- * Contains \Drupal\entity_module_test\Entity\EntityWithRevisionLog.
- */
-
-namespace Drupal\entity_module_test\Entity;
-
-use Drupal\Core\Entity\ContentEntityBase;
-use Drupal\Core\Entity\EntityTypeInterface;
-use Drupal\Core\Field\BaseFieldDefinition;
-use Drupal\entity\EntityKeysFieldsTrait;
-use Drupal\entity\Revision\EntityRevisionLogTrait;
-
-/**
- * @ContentEntityType(
- *   id = "entity_test__revision_log",
- *   label = @Translation("Entity test with revision log"),
- *   handlers = {
- *     "storage" = "\Drupal\Core\Entity\Sql\SqlContentEntityStorage",
- *   },
- *   translatable = TRUE,
- *   revisionable = TRUE,
- *   entity_keys = {
- *     "id" = "id",
- *     "revision" = "vid",
- *     "langcode" = "langcode",
- *   }
- * )
- */
-class EntityWithRevisionLog extends ContentEntityBase {
-
-  use EntityRevisionLogTrait;
-  use EntityKeysFieldsTrait;
-
-  /**
-   * {@inheritdoc}
-   */
-  public static function baseFieldDefinitions(EntityTypeInterface $entity_type) {
-    $fields = [];
-
-    $fields += static::entityKeysBaseFieldDefinitions($entity_type);
-    $fields += static::entityRevisionLogBaseFieldDefinitions();
-
-    return $fields;
-  }
-
-}
-- 
GitLab