Task to convert MimeTypeMatcherTest to phpunit. See #1938068: Convert UnitTestBase to PHPUnit.

CommentFileSizeAuthor
#6 mimetypematcher-phpunit-2130551-06.patch9.76 KBjhedstrom
#6 interdiff.txt3.75 KBjhedstrom
#3 system-mimetypematcher-phpunit-2130551-03.patch6.28 KBjhedstrom
#3 interdiff.txt1.13 KBjhedstrom
#1 system-mimetypematcher-phpunit-2130551-01.patch5.14 KBjhedstrom
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom commented
Status: Active » Needs review
FileSize
system-mimetypematcher-phpunit-2130551-01.patch5.14 KB

Here's an initial attempt.

Status: Needs review » Needs work

The last submitted patch, 1: system-mimetypematcher-phpunit-2130551-01.patch, failed testing.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom commented
Status: Needs work » Needs review
FileSize
interdiff.txt1.13 KB
system-mimetypematcher-phpunit-2130551-03.patch6.28 KB

This fixes the fatals in simpletest caused by moving the RoutingFixtures class.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom commented
Issue tags: +PHPUnit

Tagging.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

Is there a reason why we do not let testFilterRoutes just use a dataProvider? The three code hunks look really similar.

Otherwise this looks perfect.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom commented
FileSize
interdiff.txt3.75 KB
mimetypematcher-phpunit-2130551-06.patch9.76 KB

This moves to using a dataProvider. I also added groups.

dawehner’s picture

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

Thank you!

webchick’s picture

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

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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