Also includes an example test case. I hope the phpDoc as well as inline comments are sufficiently clear for how to use the multi-session feature.
Oh, and it also retains + restores the internal browser content when switching between users. So writing tests for Conflict should be super-duper-easy now.
Since this concept requires a core patch for simpletest, and there are already some tests in Conflict that sorta work, this won't happen until after the D7 Upgrade launched (if ever simpletest itself is fixed :)
Comments
Comment #1
sunI added a base test case that enables the functionality from #1378126: Simplify tests needing multiple, concurrent user sessions / logins
Also includes an example test case. I hope the phpDoc as well as inline comments are sufficiently clear for how to use the multi-session feature.
Oh, and it also retains + restores the internal browser content when switching between users. So writing tests for Conflict should be super-duper-easy now.
Comment #2
senpai commentedYes, agreed, tests are good. And thank you @sun for adding this concurrent user sessions testing stuff.
Assigning to @bdragon for teh tests.
Comment #3
senpai commentedSince this concept requires a core patch for simpletest, and there are already some tests in Conflict that sorta work, this won't happen until after the D7 Upgrade launched (if ever simpletest itself is fixed :)
Comment #4
drumm#2210937: Conflict module over-reporting conflicts during crosspost, preventing issue updates incorrectly is something that specifically needs tests.
Comment #5
drummMarking as fixed, since the module does have tests now. More can be added with each issue.