When running the simpletest-7.x-2.x tests, I consistently get Undefined variable: user Notice drupal_web_test_case.php 1321 DrupalWebTestCase->setUpInstall() in all tests.

CommentFileSizeAuthor
#7 simpletest_undefined_variable_user_976838_7.patch505 bytesBR0kEN
#1 976838_1_notice_user.patch762 bytesscor
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

scor’s picture

scor CreditAttribution: scor commented
Status: Active » Needs review
FileSize
976838_1_notice_user.patch762 bytes
lyricnz’s picture

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

This issue still in 7.x-2.0. Patch works. However simpletest is in core in D7.

thelee’s picture

thelee CreditAttribution: thelee commented

When will this get rolled into a recommended release? At least for me, this is a killer issue for 7.x-2.0 as it causes any DrupalWebTestCase to "fail" due to this php notice.

thelee’s picture

thelee CreditAttribution: thelee commented

Hello?? What's necessary to get this patch moving along? Does literally no one use 7.x-2.x Simpletest?

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

No, most people use the version of SimpleTest included with Drupal core.

thelee’s picture

thelee CreditAttribution: thelee commented

Still - is there anything I can do to kick this process along? Having a "recommended release" with such a game-breaking bug seems no good...

BR0kEN’s picture

BR0kEN
Primary language Ukrainian
CreditAttribution: BR0kEN commented
Issue summary: View changes
FileSize
simpletest_undefined_variable_user_976838_7.patch505 bytes

I hope that the modifications will be merged to core.

coderintherye’s picture

coderintherye CreditAttribution: coderintherye commented

Any reason to not roll this in? It does fix the primary problem.

If not into mainline could we at least get it committed to dev?

John Franklin’s picture

John Franklin CreditAttribution: John Franklin commented

When SimpleTest doesn't pass its own tests, it becomes hard to convince clients to use it. Please commit this soon.

  • BR0kEN committed a106dde on 7.x-2.x 
    Issue #976838 by scor, BR0kEN: Undefined variable: user when running...
BR0kEN’s picture

BR0kEN
Primary language Ukrainian
CreditAttribution: BR0kEN as a volunteer and for Propeople (now part of FFW), FFW commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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