In the test, after registering a new account you will be logged in, so going to /user/login will give 403. The attached patch will add a logout first.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Mark Theunissen’s picture

YK85’s picture

Status: Active » Needs review
nod_’s picture

works for me.

nod_’s picture

Status: Needs review » Reviewed & tested by the community
greggles’s picture

Status: Reviewed & tested by the community » Fixed

I enabled simpletest on this project so that we can see this patch making it green.

Committed - http://drupalcode.org/project/email_registration.git/commit/5cf156b

Thanks, guys.

greggles’s picture

http://drupal.org/project/email_registration/testing-status was red before the patch and green after.

Thanks!

greggles’s picture

Version: 6.x-1.3 » 7.x-1.x-dev
Status: Fixed » Needs review
FileSize
1.27 KB

Here's a patch for 7.x to get it working.

It needed both this change and to default to letting people create their own account.

greggles’s picture

Status: Needs review » Fixed

Given that this is a small change that stays inline with the previous test and is ONLY to the test (and not the module) I'm going to commit it faster than I might commit some other patch of my own.

If someone disagrees with this change please let me know and I'll work to improve it.

Now committed to 7.x-1.x too http://drupalcode.org/project/email_registration.git/commit/95e9832 and testing has been enabled for 7.x-1.x branches.

Status: Fixed » Closed (fixed)

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