Backport EntityKernelTestBase to 8.0.x [#2683391]

Problem/Motivation

Patches that add new kernel tests to 8.0.x (e.g. #2644088: DefaultTableMapping::getFieldTableName does not report table for fields with dedicated storage) should be able to use the same EntityKernelTestBase base test class that was committed only to 8.1.x and 8.2.x.

Proposed resolution

Backport the test class to 8.0.x.

Remaining tasks

None.

User interface changes

Nope.

API changes

Api addition: a new base test class is available for entity unit tests based on PHPUnit.

Data model changes

Nope.

Comment	File	Size	Author
#2	2683391.patch	3.72 KB	amateescu
#2	8.0.x: PHP 5.5 & MySQL 5.5 14,604 pass

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

8 March 2016 at 14:02

amateescu created an issue. See original summary.

File	Size
2683391.patch	3.72 KB
8.0.x: PHP 5.5 & MySQL 5.5 14,604 pass

Comment #3

Berdir

German

Switzerland

CreditAttribution: Berdir at MD Systems GmbH commented 8 March 2016 at 14:06

When discussing with alexpott, @dawehner and I discussed that it shouldn't be needed to backport the other issue.

I think it's enough if contrib tests work with the current and upcoming minor version, so once 8.1.x is out, you could convert to the new one.

But I'm also OK if someone thinks this is important enough. Just the base class is certainly already better and less disruption.

Comment #4

amateescu CreditAttribution: amateescu for Pfizer, Inc. commented 8 March 2016 at 15:24

IMO the disruption is 0 and it lets patches like #2644088: DefaultTableMapping::getFieldTableName does not report table for fields with dedicated storage be in sync for all three branches.

Comment #5

kevin.dutra CreditAttribution: kevin.dutra at Workday, Inc. commented 8 March 2016 at 17:11

Status:

Needs review

» Needs work

A little minor code style cleanup:

+  /**
+   * {@inheritdoc}
+   */
+  public static function assertEquals($expected, $actual, $message = '', $delta = 0.0, $maxDepth = 10, $canonicalize = false, $ignoreCase = false) {
+    $expected = static::castSafeStrings($expected);
+    $actual = static::castSafeStrings($actual);
+    parent::assertEquals($expected, $actual, $message, $delta, $maxDepth, $canonicalize, $ignoreCase);
+  }
+
+

FALSE should be in all caps.
$maxDepth and $ignoreCase should be lowercase with underscores.
There's an extra newline at the end.

Comment #6

amateescu CreditAttribution: amateescu for Pfizer, Inc. commented 8 March 2016 at 19:49

Status:

Needs work

» Needs review

This is just a part of the patch that was committed in #2679096: Convert Kernel Tests in Drupal\system\Tests\Entity to phpunit. I think the arguments were not changed to fit our coding standards in order to be consistent with the parent method.

Comment #7

kevin.dutra CreditAttribution: kevin.dutra at Workday, Inc. commented 8 March 2016 at 20:25

Status:

Needs review

» Reviewed & tested by the community

Ah, okay -- I guess I'll defer to the original issue on that. Since this change is pretty minor, low impact, and helps to avoid reworking patches among the different 8.x branches, I'm going to go ahead and mark this RTBC.

Comment #8

8 March 2016 at 23:43

catch committed bce8913 on 8.0.x

Issue #2683391 by amateescu: Backport EntityKernelTestBase to 8.0.x

Comment #9

catch

he/him

English

CreditAttribution: catch at Third and Grove commented 8 March 2016 at 23:43

Status:

Reviewed & tested by the community

» Fixed

+++ b/core/modules/simpletest/src/AssertHelperTrait.php
@@ -23,7 +23,7 @@
+  protected static function castSafeStrings($value) {

I had to double check on 3v4l.org that callers of this won't get a message, but it's only the other way 'round that you get E_DEPRECATED if you call it the wrong way.

Committed/pushed to 8.0.x, thanks!