Comments

naveenvalecha created an issue. See original summary.

naveenvalecha’s picture

Status: Active » Needs review
FileSize
2.43 KB

Here we go

//Naveen

jibran’s picture

+++ b/core/modules/contact/tests/src/Functional/Views/ContactLinkTest.php
@@ -39,10 +39,12 @@ class ContactLinkTest extends ViewTestBase {
+    if ($import_test_views) {

Why do we need this?

naveenvalecha’s picture

FileSize
2.25 KB
759 bytes

@jibran,
we don't need this :) Removed this. Thanks for your review.

//Naveen

jibran’s picture

Status: Needs review » Reviewed & tested by the community

Thanks!

dawehner’s picture

Maybe a child class want's to defer the importing to do some questions, but yeah I guess in this case we don't need it.

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Needs work
+++ b/core/modules/contact/tests/src/Functional/Views/ContactFieldsTest.php
@@ -28,8 +28,8 @@ class ContactFieldsTest extends ViewTestBase {
+  protected function setUp($import_test_views = TRUE) {
+    parent::setUp($import_test_views);

+++ b/core/modules/contact/tests/src/Functional/Views/ContactLinkTest.php
@@ -39,8 +39,8 @@ class ContactLinkTest extends ViewTestBase {
+  protected function setUp($import_test_views = TRUE) {
+    parent::setUp($import_test_views);

Hm, we did not do this before, why do we need it now?

naveenvalecha’s picture

Status: Needs work » Needs review

To fix the warning b/c ViewTestBase::setup requires the argument $import_test_views

Back to N/R
//Naveen

Lendude’s picture

Status: Needs review » Reviewed & tested by the community

The child class setUp() needs to match the parent, so the change is correct.

  • Gábor Hojtsy committed b6f1820 on 8.3.x
    Issue #2889352 by naveenvalecha, jibran, dawehner, Lendude: Convert...

  • Gábor Hojtsy committed 09ee23b on 8.4.x
    Issue #2889352 by naveenvalecha, jibran, dawehner, Lendude: Convert...
Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Fixed

Makes sense. Thanks for setting me straight :) Committed :)

Gábor Hojtsy’s picture

Version: 8.4.x-dev » 8.3.x-dev

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.