Blocks #606840: Enable internal page cache by default.

CommentFileSizeAuthor
#4 interdiff.txt1.52 KBWim Leers
#4 cron_run_test-2458289-4.patch1.54 KBWim Leers
#1 cron_run_test-2458289-1.patch847 bytesWim Leers
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers at Acquia commented
Status: Active » Needs review
Issue tags: +Quickfix
FileSize
cron_run_test-2458289-1.patch847 bytes
Fabianx’s picture

Fabianx CreditAttribution: Fabianx commented
Status: Needs review » Reviewed & tested by the community

RTBC, looks great!

alexpott’s picture

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

Can't we just move the following lines up to here - doing less logins in the test?

    $admin_user = $this->drupalCreateUser(array('administer site configuration'));
    $this->drupalLogin($admin_user);
Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers at Acquia commented
Status: Needs work » Reviewed & tested by the community
FileSize
cron_run_test-2458289-4.patch1.54 KB
interdiff.txt1.52 KB

Well-spotted, Sir!

Done.

alexpott’s picture

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

Test only change - permitted in beta. Committed 0880f31 and pushed to 8.0.x. Thanks!

  • alexpott committed 0880f31 on 8.0.x 
    Issue #2458289 by Wim Leers: CronRunTest::testAutomaticCron() should...

Status: Fixed » Closed (fixed)

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