Problem/Motivation

There is no test coverage for hooks exposed by Authcache Enum.

Proposed resolution

Fix it.

Remaining tasks

User interface changes

API changes

Data model changes

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

znerol created an issue. See original summary.

znerol’s picture

znerol’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: add_test_coverage_for-2546624-2.patch, failed testing.

znerol’s picture

Oh, fun:


$hex='a0';
$dec='10';
$a[$hex] = 'hex';
$a[$dec] = 'dec';
var_dump($a);

# => array(2) {
#      'a0' =>
#      string(3) "hex"
#      [10] =>
#      string(3) "dec"
#    }
znerol’s picture

I think the correct way to resolve this is to ensure that authcache_enum_keys() returns an array of strings.

znerol’s picture

Status: Needs work » Needs review

  • znerol committed 318fc92 on 7.x-2.x
    Issue #2546624 by znerol: Add test coverage for Authcache Enum API
    
znerol’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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