See #2735005: Convert all Simpletest web tests to BrowserTestBase (or UnitTestBase/KernelTestBase).

Scope:
- All tests are in scope

CommentFileSizeAuthor
#14 interdiff-2872059-12-14.txt1.87 KBnaveenvalecha
#14 2872059-14.patch19.47 KBnaveenvalecha
#12 interdiff-2872059-10-12.txt6.06 KBnaveenvalecha
#12 2872059-12.patch19.49 KBnaveenvalecha
#10 interdiff-2872059-8-10.txt1.07 KBnaveenvalecha
#10 2872059-10.patch19.61 KBnaveenvalecha
#8 interdiff-2872059-6-8.txt8.73 KBnaveenvalecha
#8 2872059-8.patch19.66 KBnaveenvalecha
#6 2872059-6.patch12.79 KBnaveenvalecha
#2 2872059-02.patch12.71 KBmpdonadio
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mpdonadio created an issue. See original summary.

mpdonadio’s picture

mpdonadio
he/him
Primary language English
Location Philadelphia/PA/USA (UTC-5)
CreditAttribution: mpdonadio at DiD Agency commented
Status: Active » Needs review
FileSize
2872059-02.patch12.71 KB

Just the results of the browsertest-convert.php script to see what the starting point is. Some of these pass as-is, but lots of fails to dig into.

Status: Needs review » Needs work

The last submitted patch, 2: 2872059-02.patch, failed testing.

michielnugter’s picture

michielnugter CreditAttribution: michielnugter as a volunteer and at Synetic commented
Issue tags: -PHPUnit +phpunit initiative
mpdonadio’s picture

mpdonadio
he/him
Primary language English
Location Philadelphia/PA/USA (UTC-5)
CreditAttribution: mpdonadio at DiD Agency commented

Dug into this a bit. I think nearly all of the fails are because the local .po files in the tests are not being picked up, and I am not sure why. Been debugging this for a bit and am stumped :/

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Status: Needs work » Needs review
FileSize
2872059-6.patch12.79 KB

Here we go.LocaleConfigTranslationImportTest:: testLocaleRemovalAndConfigOverridePreserve failed when running whole testsuite on local but passed while running explicitly.Let's see at testbot

//Naveen

Status: Needs review » Needs work

The last submitted patch, 6: 2872059-6.patch, failed testing. View results

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Status: Needs work » Needs review
FileSize
2872059-8.patch19.66 KB
interdiff-2872059-6-8.txt8.73 KB

Here we go with fixes.

//Naveen

Status: Needs review » Needs work

The last submitted patch, 8: 2872059-8.patch, failed testing. View results

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Status: Needs work » Needs review
FileSize
2872059-10.patch19.61 KB
interdiff-2872059-8-10.txt1.07 KB

Here we go with more fixes.

//Naveen

Lendude’s picture

Lendude
Primary language Dutch
Location Amsterdam
CreditAttribution: Lendude as a volunteer and at Dx Experts commented
Issue summary: View changes
Status: Needs review » Needs work 
+++ b/core/modules/locale/tests/src/Functional/LocaleConfigTranslationImportTest.php
@@ -206,7 +206,7 @@ public function testLocaleRemovalAndConfigOverridePreserve() {
+    $lid = (string) $textarea->getAttribute('name');

+++ b/core/modules/locale/tests/src/Functional/LocaleConfigTranslationTest.php
@@ -76,7 +76,7 @@ public function testConfigTranslation() {
+    $lid = (string) $textarea->getAttribute('name');

@@ -100,7 +100,7 @@ public function testConfigTranslation() {
+    $lid = (string) $textarea->getAttribute('name');

All the (string) casts can be removed, getAttribute returns a string.

The rest looks really nice and clean.

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Status: Needs work » Needs review
FileSize
2872059-12.patch19.49 KB
interdiff-2872059-10-12.txt6.06 KB

Addressed #11
Also removed the string casts from a couple of other places.

//Naveen

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer commented 
  1. +++ b/core/modules/locale/tests/src/Functional/LocaleJavascriptTranslationTest.php
@@ -138,8 +138,9 @@ public function testLocaleTranslationJsDependencies() {
+    $content = $this->getSession()->getPage()->getcontent();

    I think the actual method name is getContent, but PHP doesn't care 

  2. +++ b/core/modules/locale/tests/src/Functional/LocaleTranslationUiTest.php
@@ -317,7 +317,7 @@ public function testStringValidation() {
-      $this->assertNotIdentical(FALSE, strpos($form_class[0], 'error'), 'The string was rejected as unsafe.');
+      $this->assertNotIdentical(FALSE, strpos($form_class[0]->getText(), 'error'), 'The string was rejected as unsafe.');

    I am wondering whether we should use assertContains here instead.

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
FileSize
2872059-14.patch19.47 KB
interdiff-2872059-12-14.txt1.87 KB

#13.1
good catch
#13.2
Done

//Naveen

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Thank you!

  • Gábor Hojtsy committed 0f37866 on 8.4.x 
    Issue #2872059 by naveenvalecha, mpdonadio, dawehner, Lendude: Convert...

  • Gábor Hojtsy committed a779d66 on 8.3.x 
    Issue #2872059 by naveenvalecha, mpdonadio, dawehner, Lendude: Convert...
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Version: 8.4.x-dev » 8.3.x-dev
Status: Reviewed & tested by the community » Fixed

Looks great, thanks all!

Status: Fixed » Closed (fixed)

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