We're running Simpletest with a custom install profile (which is an edge case, I know) and the static in install_get_rid() is breaking DrupalWebTestCase::setUp(). When we run setUp() for the second time and our install profile runs, it tries to add a role using install_get_role(), but it fails because install_get_rid() is reporting that the role already exists.

CommentFileSizeAuthor
#2 1885738-2.static.patch518 bytesksenzee
#1 1885738-1.static.patch517 bytesksenzee

Comments

ksenzee’s picture

ksenzee
she/her
commented
Status: Active » Needs review
StatusFileSize
new 1885738-1.static.patch517 bytes

One approach would be to just get rid of the static as probably unnecessary optimization, but here's a patch that lets it be reset instead.

ksenzee’s picture

ksenzee
she/her
commented
StatusFileSize
new 1885738-2.static.patch518 bytes

Er, disregard that last patch. Clearly need more sleep. $reset parameter should default to FALSE.