Problem/Motivation

Proposed resolution

  • Move this test method into the child class CommentInterfaceTest.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xjm’s picture

Title: testCommentNewCommentsIndicator() is inside CommentHelperCase and therefore does not get run » testCommentNewCommentsIndicator() is inside CommentHelperCase and therefore gets run repeatedly

Yeah, more coffee.

xjm’s picture

Issue tags: +Needs backport to D7

And yeah.

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

Yeah the CommentHelperCase shouldn't contain tests

sun’s picture

Funky. Good spot!

catch’s picture

Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Craziness.

Committed/pushed to 8.x, moving back to 7.x for backport.

xjm’s picture

Status: Patch (to be ported) » Reviewed & tested by the community
FileSize
5.22 KB
xjm’s picture

Without an extra line of whitespace that wasn't in the original patch. :)

webchick’s picture

Status: Reviewed & tested by the community » Fixed

LOL. Nice find.

Committed and pushed to 7.x. Thanks!

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

Anonymous’s picture

Issue summary: View changes

Updated issue summary.