The @requires directive on ApcuBackendTest is wrong, it should be "@requries apcu".

I guess there is no way we can test this?

CommentFileSizeAuthor
#2 phpunit-apcu-2742541-2.patch501 bytesklausi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

klausi created an issue. See original summary.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
CreditAttribution: klausi commented
Status: Active » Needs review
FileSize
phpunit-apcu-2742541-2.patch501 bytes

Patch.

neclimdul’s picture

neclimdul
He/Him
CreditAttribution: neclimdul commented
Status: Needs review » Reviewed & tested by the community

Always made sense and I finally got around to confirming it works as expected so RTBC.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

The DrupalCI logs confirm this fix - nice.

Committed 63645aa and pushed to 8.1.x and 8.2.x. Thanks!

  • alexpott committed 42d2ce9 on 8.2.x 
    Issue #2742541 by klausi: ApcuBackendTest is skipped in phpunit because...

  • alexpott committed 63645aa on 8.1.x 
    Issue #2742541 by klausi: ApcuBackendTest is skipped in phpunit because...

Status: Fixed » Closed (fixed)

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