The getInfo() method should be public static and the strings should not be translated: https://drupal.org/node/325974.

The setUp() method should take the same parameters as their parent.

CommentFileSizeAuthor
simplenews-test_setup_static.patch5.04 KBlund.mikkel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lund.mikkel’s picture

lund.mikkel CreditAttribution: lund.mikkel commented
Status: Active » Needs review
sdrycroft’s picture

sdrycroft CreditAttribution: sdrycroft commented

I've just run into this too. Cheers for the patch lund.

TR’s picture

TR CreditAttribution: TR commented
Version: 7.x-1.0-rc1 » 7.x-1.x-dev
Issue summary: View changes

setUp() should be protected ...