I think new step definition can be added to the DrupalExtension to check whether a link/text is present in the particular region or not. This will be similar to Then /^I should see the heading "(?P<heading>[^"]*)" in the "(?P<region>[^"]*)"(?:| region)$/

Is it worth? If so, i can start on this

CommentFileSizeAuthor
#4 check_region_link_text-1822680-4.patch2.67 KBlangworthy
#2 check_region_link_text-1822680-2.patch1.67 KBlangworthy

Comments

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented

This is worth it. For the link/follow, I already have a few features using this language (but no step definition yet)

When I follow "link text" in the "region name"

Step definitions at this point would be greatly appreciated!

langworthy’s picture

langworthy commented
Status: Active » Needs work
StatusFileSize
new check_region_link_text-1822680-2.patch1.67 KB

Here are 2 steps that accomplish checking for links/text in regions.

We need to add a Scenario to test.feature. What are we running this feature against? I think we need to add a behat.yml file to the repo. (might be a separate issue)

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented

I'm running tests against a default local install of D7.

langworthy’s picture

langworthy commented
Status: Needs work » Needs review
StatusFileSize
new check_region_link_text-1822680-4.patch2.67 KB
kssundar’s picture

kssundar commented
Status: Needs review » Needs work

This should be able to "see" link in the specified region as well.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Needs work » Fixed

Committed #4 (c153ace) with an additional definition to 'see' a link in a region. Thanks!

Also note that there is an api.feature and a blackbox.feature file. The latter can be run using behat.yml.dist that comes with the project, while the former still needs an instance that the user has api access to, and is expected to run against a default D7 install.

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 additional text

  • Commit c153ace on master, 1.0 by jhedstrom: 
    Issue #1822680 by jhedstrom, langworthy | ksbalajisundar: Added step...
  • Commit a506292 on master, 1.0 by jhedstrom:
    Follow-up to issue #1822680 to allow for broader feature language with...

  • Commit c153ace on master, 1.0, test-symfony by jhedstrom: 
    Issue #1822680 by jhedstrom, langworthy | ksbalajisundar: Added step...
  • Commit a506292 on master, 1.0, test-symfony by jhedstrom:
    Follow-up to issue #1822680 to allow for broader feature language with...