Closed (fixed)
Project:
Drupal.org BDD
Version:
7.x-1.x-dev
Component:
Feature/Scenarios
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
20 Sep 2012 at 11:13 UTC
Updated:
22 Jul 2013 at 22:18 UTC
Things to be tested for Training section of the Marketplace:
- /training page exists +
- sidebar navigation works
- links on top of the page "Events section" and "Global Training Days" work +
- "Marketplace guidelines" link works +
- anonymous users can't see "Add your listing link" and authenticated users can +
- when user creates/edits organization page - if he clicks "Request listing in the Training section" and saves:
Comments
Comment #1
pradeeprkara commentedassigning and tagging
@important
"Add your listing" is linked to http://drupal.org/node/add/organization instead of http://git6site.devdrupal.org/node/add/organization.
Comment #2
sachin2dhoni commentedUploaded to dev branch.
Please check the commit log: be5f79f
Comment #3
eliza411 commentedThis looks good. I'm assigning to tvn to open an issue in the appropriate development queue for the known failure where an anonymous user sees the "Add your listing" link, which is not visible to anonymous on production and leads to an Access denied page:
Comment #4
eliza411 commentedThis has also been cherry-picked into the 7.x branch.
Comment #5
eliza411 commentedand ... actually assigning to tvn
Comment #6
tvn commented"Add your listing" link for anonymous was fixed on d.o last week #1791128: Hide "Add your listing" button from anonymous users.
Comment #7
eliza411 commentedGreat, thanks! Adding a tag to retest after the next build so we can remove the known_git6failure tag.
Comment #8
kssundar commentedRan this test on d.o and below is the report:
Ran this on git6 as well:
Comment #9
sachin2dhoni commentedAs per comment #8,we can see the failed cases because there are text changes after rebuilt/refresh of git6.
Updated to dev branch with the consistent data.
Please check the commit log:6f5f04d
Comment #10
tvn commentedLast week we did some changes to Marketplace and emptied "Locations" field for everyone, it will take some time for companies to update their pages. In the future I'm sure we'll have over 10 links there.
It was changed to DrupalCon yesterday.
The text above the map is changing all the time, it's more safe to test for the text right below the map (e.g. Drupal Global Training Days is an initiative by the Drupal Association to introduce new and beginning users to Drupal.). This one might change too someday, but shouldn't anytime soon.
Only authenticated users can now see "Add your listing" button and guidelines link on /training page.
Comment #11
tvn commentedcross post :)
Comment #12
sachin2dhoni commentedMade some minor modifications in the feature(marketplace_training.feature) file for reusability.
Also added @retest-after-next-build tag as the user login are not working in git 6 currently
Updated to dev branch.
Please check the commit log:8d946f1
Comment #13
eliza411 commentedRemoved retest tags from files. Retest after next build should be tagged on the issue.
I think this looks done. tvn might want to review more closely.
http://drupalcode.org/project/doobie.git/commit/57d268f
Comment #14
sachin2dhoni commentedUpdating the title with file name.
Comment #15
tvn commentedI moved tests related to page creation to marketplace_organization.feature and did some small changes to the file.
http://drupalcode.org/project/doobie.git/commit/5c878ec
This file needs:
1. Check that organizations actually shown on the page.
Similar to services scenario (http://drupalcode.org/project/doobie.git/blob/5c878ec:/features/drupalor...) - "Then I should see at least 10 records".
2. Scenario similar to this one (http://drupalcode.org/project/doobie.git/blob/5c878ec:/features/drupalor...) to open org page and check it has training provider message on top
3. If possible test somehow that sidebar navigation actually works, as in list changes when you choose specific country.
Comment #16
sachin2dhoni commentedUpdated to 6.x branch and implemented new scenarios as per comment 15.
Please check the commit log:
http://drupalcode.org/project/doobie.git/commit/90f43f9
Comment #17
tvn commented"Scenario: See a paged list of training providers"
"+ When I click on page "2"" should be changed to page "last", because we are adding more and more companies to the Training section, so page 2 will not be the last one for too long.
I am sorry, my point 2. in comment #15 was not clear. I was linking to scenario starting on line 87
So similar scenario only with last line "This organization is a Drupal training provider." is needed instead of
Last scenario in the file looks good.
Comment #18
tvn commentedComment #19
sachin2dhoni commentedUpdated to 6x branch with the changes.
Please check the commit log:
http://drupalcode.org/project/doobie.git/commit/2358bdf
Comment #20
tvn commentedLooks good. I only modified scenario description and changed number of required records for the last page, as it can be less than 5.
http://drupalcode.org/project/doobie.git/commit/7f94272
Comment #21
sachin2dhoni commentedPorted to 7.x with the changes.
http://drupalcode.org/project/doobie.git/commit/116ea62
Comment #22
eliza411 commentedReady as this can be until D7 Drupal.org is ready for feedback.
Comment #23
eliza411 commentedUpdated css selectors to D7
Comment #23.0
eliza411 commented.