We should be able to use the following steps:

# authenticated only; has not agreed to terms of service
I am logged in as a site user

# has agreed to terms of services but is not vetted
I am logged in as git user

# has agreed to terms of service and been granted vetted status
I am logged in as vetted git user

CommentFileSizeAuthor
#2 doobie-login-step-defs-1701494-1.patch3.05 KBkssundar
#1 doobie-login-step-defs-1701494-1.patch2.42 KBkssundar

Comments

kssundar’s picture

kssundar commented
Assigned: kssundar » Unassigned
Status: Active » Needs review
StatusFileSize
new doobie-login-step-defs-1701494-1.patch2.42 KB

Attached is the patch for review.

kssundar’s picture

kssundar commented
StatusFileSize
new doobie-login-step-defs-1701494-1.patch3.05 KB

Attached is the updated patch containing cleanup and missed variables

eliza411’s picture

eliza411 commented
Status: Needs review » Fixed

lotyrin did some work on the approach in the patch and this has been committed. Please review and re-open if needed.

Changes are in http://drupalcode.org/project/doobie.git/commit/023d3f3 and
http://drupalcode.org/project/doobie.git/commit/dc6ad67 I believe.

eliza411’s picture

eliza411 commented

Worth mentioning ... we're really logging in as users not roles, and that was the main reason for re-working it. It's confusing when the role and username are the same. Apologies I wasn't clear about that.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

removed last line