It'd be great to have some automated tests that try to validate the behavior works as expected.

Comments

sun’s picture

I 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.

senpai’s picture

Version: » 7.x-1.x-dev
Assigned: Unassigned » bdragon

Yes, agreed, tests are good. And thank you @sun for adding this concurrent user sessions testing stuff.

Assigning to @bdragon for teh tests.

senpai’s picture

Assigned: bdragon » Unassigned
Issue tags: -project, -drupal.org D7

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 :)

drumm’s picture

drumm’s picture

Status: Active » Fixed

Marking as fixed, since the module does have tests now. More can be added with each issue.

Status: Fixed » Closed (fixed)

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