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

Comments

klausi created an issue. See original summary.

klausi’s picture

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

Patch.

neclimdul’s picture

neclimdul
He/Him
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 🇪🇺🌍
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.