While working on performance issues, I needed to expand the drupal_render() test coverage. Sadly, these tests are very slow, which slowed me down. So I converted RenderTest to DrupalUnitTestBase and moved the integration tests into a RenderWebTest.

CommentFileSizeAuthor
#1 rendertest_dutb-2116827-1.patch14.18 KBWim Leers
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers commented
Status: Active » Needs review
FileSize
rendertest_dutb-2116827-1.patch14.18 KB
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Needs review » Reviewed & tested by the community

Mostly just code copy-paste.

msonnabaum’s picture

msonnabaum CreditAttribution: msonnabaum commented

Yeah, this looks pretty reasonable to me.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Looks good to me too. Committed.

Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers commented
Issue tags: -sprint

Thanks!

Status: Fixed » Closed (fixed)

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