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

tvn’s picture

Assigned: tvn » Unassigned
Status: Active » Needs work

I 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?

tvn’s picture

Issue tags: +forums
sachin2dhoni’s picture

Assigned: Unassigned » sachin2dhoni
Issue tags: +sprint 12

Taking this up

sachin2dhoni’s picture

Status: Needs work » Needs review

Uploading to dev branch with changes.

Renamed the file forum_topic_post.feature to forum.feature.

Please check the commit log:61173e8

eliza411’s picture

Status: Needs review » Needs work

This 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");
+ }
}

sachin2dhoni’s picture

Status: Needs work » Needs review

As 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.

eliza411’s picture

Status: Needs review » Fixed

Great, thanks.

Status: Fixed » Closed (fixed)

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

pradeeprkara’s picture

Issue tags: +sprint 18

Added 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

pradeeprkara’s picture

Issue summary: View changes

adding list of forum files