Closed (fixed)
Project:
Drupal.org BDD
Version:
6.x-1.x-dev
Component:
Feature/Scenarios
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
29 Sep 2012 at 14:46 UTC
Updated:
3 Dec 2012 at 14:17 UTC
The feature language doesn't always capture the business value of a feature from a stakeholder perspective. I was hoping you could review the feature block, as well as doing a quick check to see that the summaries of the scenarios are accurate and clear.
features/drupalorg/forum_ad.feature
features/drupalorg/forum_topic_post.feature
features/drupalorg/forum_file_uploads.feature
features/drupalorg/forum_breadcrumb.feature
features/drupalorg/forum_restrictions.feature
Comments
Comment #1
tvn commentedI edited feature and scenario descriptions a bit.
We also need tests to check that:
Page /forum exist and "New forum topics" block in sidebar shows up
Categories "Forums' and 'Drupal version' work when creating new topic
Newly created topics show up in "New forum topics" block
These tests could be added to forum_topic_post.feature and this file renamed to more general "forum.feature". It feels like we have too many files with 1-2 scenario per file for this section.
forum_ad.feature file:
This also needs to test if ads show up in "Paid Drupal services" forum
And I'm not sure if it's possible - test if ads show up on individual forum topics of services subforums.
Note also that those ads are not really popular among sponsors now, so sometimes slots might just be empty, even though functionality itself works.
forum_breadcrumb.feature file:
Scenario: User follows link in the Support forum without logging in
probably needs
"And I should not see the breadcrumb "Post installation" step
Some scenarios in this file are for not logged in users - should we tag them as @anon?
Comment #2
tvn commentedComment #3
sachin2dhoni commentedTaking this up
Comment #4
sachin2dhoni commentedUploading to dev branch with changes.
Renamed the file forum_topic_post.feature to forum.feature.
Please check the commit log:61173e8
Comment #5
eliza411 commentedThis step seems inconsistent with the way we're checking for other content in specific regions, which is more like
I should see something in the "right sidebar" region. Why is that? There might be a good reason that I'm just not seeing, so it's not necessarily in need of work :)+
+ /**
+ * @Then /^I should see latest forum topic in the rightside block$/
+ */
+ public function iShouldSeeLatestForumTopicInTheRightsideBlock() {
+ sleep(2);
+ $forumTitle = HackyDataRegistry::get('random:Forum subject');
+ if(empty($forumTitle)) {
+ throw new Exception('No Forum title exists in this page');
+ }
+ $this->iShouldSeeInArea('link', $forumTitle, "right sidebar");
+ }
}
Comment #6
sachin2dhoni commentedAs the blocks does not fall under structured region/area , so the the above step definition is written to find latest forum topic link in the rightside of the forum page.
Comment #7
eliza411 commentedGreat, thanks.
Comment #9
pradeeprkara commentedAdded background step.
tagged 2nd and 3rd scenarios as @known_git7failure.
Ported to 7.x
http://drupalcode.org/project/doobie.git/commit/e1096a6
6.x:
http://drupalcode.org/project/doobie.git/commit/9e1a0a7
Comment #9.0
pradeeprkara commentedadding list of forum files