Test that user can enable his contact form, disable it and use other people's contact forms.

Feature: User contact form
In order to allow or stop other people from contacting me
As an authenticated user
I should be able to enable or disable my contact form

Comments

pradeeprkara’s picture

pradeeprkara commented

personal contact form is not loading for both authenticated user and anonymous user even after enabling Personal contact form checkbox. Instead it shows Access denied. Are there any other dependencies or is there any other way to access the form?

I tried with the below url:

http://git6site.devdrupal.org/user/2384543/contact

Same with drupal.org.

git-vetted’s picture

git-vetted commented

I am reproduce this as follows:

As user411 with my contact form enabled:

I cannot access my own form (perhaps normal)
I cannot access as an anonymous user (not sure if this is expected)
I can access if I'm logged in as a different user (git-vetted) or an admin user (eliza411)

Form: http://drupal.org/user/1102856/contact

site user should be able to access git user's form and visa versa, if that helps.

pradeeprkara’s picture

pradeeprkara commented
Assigned: Unassigned » pradeeprkara
Issue tags: +sprint 17
pradeeprkara’s picture

pradeeprkara commented
Status: Active » Needs review

Added the feature.

Considered the following cases:

  • Site user looks for his own contact form when enabled
  • Git user accesses site user's contact form when enabled
  • Anonymous user accesses site user's contact form when enabled
  • Git user accesses site user's contact form when disabled

Please review: c70c190

tvn’s picture

tvn commented

"I cannot access my own form" - yes, this is normal I think
"I cannot access as an anonymous user" - yes, this is expected
"I can access if I'm logged in as a different user" - correct

The file looks good to me.
Only question - can't we use direct urls to user pages e.g. user/44444 instead of doing whole that search thing?

pradeeprkara’s picture

pradeeprkara commented

The user profile url has the user id in it and later when we run the tests against drupal.org, the user id may change and it will be a rework. If this is okay, we can make it directly visits profile url.

Please confirm.

tvn’s picture

tvn commented

Do all the user accounts we use for testing already exist on Drupal.org? Or are they being created for git6 and git7 sites only?

If they are registered on Drupal.org - user id will always be the same on live and dev and staging sites and we could use direct profile urls.

pradeeprkara’s picture

pradeeprkara commented

Nope, the test users are specific to git6 and git6 test environments.

kssundar’s picture

kssundar commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Needs review » Patch (to be ported)

Need 7.x porting

pradeeprkara’s picture

pradeeprkara commented
Status: Patch (to be ported) » Needs review
Issue tags: +sprint 18

There is no equivalent user search module at git7 as that at http://git6site.devdrupal.org/search/user_search .
Marked those scenarios as @known_git7failures.

This is ported to 7.x

http://drupalcode.org/project/doobie.git/commit/85e7f79

pradeeprkara’s picture

pradeeprkara commented

Tagging

eliza411’s picture

eliza411 commented
Status: Needs review » Reviewed & tested by the community

I decided to remove known_failure tags since so much of the site is in progress and failing. I think this is as complete as we can make it and the missing search/user_search will have to be addressed during a feature set review when D7 Drupal.org is ready for feedback.

eliza411’s picture

eliza411 commented
Status: Reviewed & tested by the community » Closed (fixed)