#971320: Administrator configures registration paths, each with customised role assignments has been fixed, but we need some tests around this to test the registration paths functionality.

CommentFileSizeAuthor
#13 administrator-configures-registration-paths-simpletests-2313197-14.patch9 KBmike.davis
#12 administrator-configures-registration-paths-simpletests-2313197-12.patch9.06 KBmike.davis
#10 administrator-configures-registration-paths-simpletests-2313197-10.patch9 KBmike.davis
#8 administrator-configures-registration-paths-simpletests-2313197-8.patch9.17 KBmike.davis
#6 administrator-configures-registration-paths-simpletests-2313197-6.patch9.04 KBmike.davis
#4 administrator-configures-registration-paths-simpletests-2313197-4.patch8.85 KBmike.davis
#1 administrator-configures-registration-paths-simpletests-2313197-1.patch8.75 KBmike.davis
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mike.davis’s picture

mike.davis CreditAttribution: mike.davis commented
Status: Active » Needs review
FileSize
administrator-configures-registration-paths-simpletests-2313197-1.patch8.75 KB

After some time of playing with SimpleTest, I finally have a patch for adding tests for the auto assign path feature.

Status: Needs review » Needs work

The last submitted patch, 1: administrator-configures-registration-paths-simpletests-2313197-1.patch, failed testing.

mike.davis’s picture

mike.davis CreditAttribution: mike.davis commented

I'm not sure what is happening, as running this on the simpletest.me site all passes fine?

mike.davis’s picture

mike.davis CreditAttribution: mike.davis commented
Status: Needs work » Needs review
FileSize
administrator-configures-registration-paths-simpletests-2313197-4.patch8.85 KB

OK, so here is the same patch with some debug to see what roles exist for a new user.

Status: Needs review » Needs work

The last submitted patch, 4: administrator-configures-registration-paths-simpletests-2313197-4.patch, failed testing.

mike.davis’s picture

mike.davis CreditAttribution: mike.davis commented
Status: Needs work » Needs review
FileSize
administrator-configures-registration-paths-simpletests-2313197-6.patch9.04 KB

Added further debug to get the list of checkboxes on the user page.

Status: Needs review » Needs work

The last submitted patch, 6: administrator-configures-registration-paths-simpletests-2313197-6.patch, failed testing.

mike.davis’s picture

mike.davis CreditAttribution: mike.davis commented
Status: Needs work » Needs review
FileSize
administrator-configures-registration-paths-simpletests-2313197-8.patch9.17 KB

More debug to see what roles are on the user form

Status: Needs review » Needs work

The last submitted patch, 8: administrator-configures-registration-paths-simpletests-2313197-8.patch, failed testing.

mike.davis’s picture

mike.davis CreditAttribution: mike.davis commented
Status: Needs work » Needs review
FileSize
administrator-configures-registration-paths-simpletests-2313197-10.patch9 KB

OK, I changed tact on how I was checking for if the user had the correct role assigned..... lets try this.

Status: Needs review » Needs work

The last submitted patch, 10: administrator-configures-registration-paths-simpletests-2313197-10.patch, failed testing.

mike.davis’s picture

mike.davis CreditAttribution: mike.davis commented
Status: Needs work » Needs review
FileSize
administrator-configures-registration-paths-simpletests-2313197-12.patch9.06 KB

try again.....

mike.davis’s picture

mike.davis CreditAttribution: mike.davis commented
FileSize
administrator-configures-registration-paths-simpletests-2313197-14.patch9 KB

finally got there ... just updated some text for the test...

  • mike.davis committed f96ee1c on 7.x-1.x 
    Issue #2313197 by mike.davis: Administrator configures registration...
mike.davis’s picture

mike.davis CreditAttribution: mike.davis commented
Status: Needs review » Fixed

This has been committed now

Status: Fixed » Closed (fixed)

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