CommentFileSizeAuthor
#2 drupal8.pifr-verbose.2.patch1.03 KBsun
drupal8.pifr-verbose.0.patch947 bytessun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented

Yup, PIFR test result shows no verbose test results and no "debug messages" in the review log.

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
FileSize
drupal8.pifr-verbose.2.patch1.03 KB

But I can only guess that the logic in attached patch is a bit more clear and simpler to understand :-)

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented

Yeah, this is still weird, but restores the behavior we hat before the $conf rename and we can deal with it properly in #1774002: Correct mistake introduced in simpletest cmi conversion by introducing --verbose-browser flag.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Needs review » Reviewed & tested by the community
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Yeah, this looks like a decent workaround for now. All of those verbose messages are making the results a bit painful.

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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