Following error pops up from privatemsg tests:

Strict warning: call_user_func() expects parameter 1 to be a valid callback, non-static method PrivatemsgFilterWidgetTestCase::getInfo() should not be called statically in simpletest_test_get_all() (line 327 of /var/www/spr/modules/simpletest/simpletest.module).

Files: 
CommentFileSizeAuthor
#8 getinfo_warning_2.x-1807410-8.patch1.44 KBmikran
FAILED: [[SimpleTest]]: [MySQL] 3,118 pass(es), 7 fail(s), and 0 exception(s). View
#1 getinfo_warning-1807410-2.patch1.44 KBmikran
FAILED: [[SimpleTest]]: [MySQL] 3,118 pass(es), 7 fail(s), and 0 exception(s). View

Comments

mikran’s picture

Status: Active » Needs review
FileSize
1.44 KB
FAILED: [[SimpleTest]]: [MySQL] 3,118 pass(es), 7 fail(s), and 0 exception(s). View

Attached patch also adds public keyword to few other implementatiots of getInfo().

Status: Needs review » Needs work

The last submitted patch, getinfo_warning-1807410-2.patch, failed testing.

amitaibu’s picture

Project: Message » Privatemsg

Wrong project...

mikran’s picture

Status: Needs work » Needs review

too many tabs open it seems :)

mikran’s picture

#1: getinfo_warning-1807410-2.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, getinfo_warning-1807410-2.patch, failed testing.

Berdir’s picture

You can ignore those test fails.

mikran’s picture

FileSize
1.44 KB
FAILED: [[SimpleTest]]: [MySQL] 3,118 pass(es), 7 fail(s), and 0 exception(s). View

rerolled patch for 2.x branch as well

mikran’s picture

Status: Needs work » Needs review

and changing status for manual review then

Status: Needs review » Needs work

The last submitted patch, getinfo_warning_2.x-1807410-8.patch, failed testing.

Berdir’s picture

Status: Needs work » Fixed

Thanks, commited to both branches.

Status: Fixed » Closed (fixed)

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