I renamed user_dashboard_main.feature to user_dashboard.feature and fixed small typo in user_dashboard_add_block.feature: http://drupalcode.org/project/doobie.git/commit/9e8c39b

1. "Scenario: Restore to default link"
and
"Scenario: Orientation of blocks in columns"
are doing basically the same thing as user_dashboard_defaults.feature file. I suggest we merge them with existing tests in that file.

2. Almost all the scenarios include additional step to follow "Dashboard", which isn't really needed. I suggest we include it once, to make sure the link works, and remove from the rest of scenarios to save us some time.

And I follow "Your Dashboard"
And I wait until the page is loaded
And I follow "Dashboard"
And I wait until the page is loaded

Comments

tvn’s picture

tvn commented
Issue summary: View changes

.

pradeeprkara’s picture

pradeeprkara commented
Assigned: Unassigned » pradeeprkara
Issue tags: +sprint 17

Assigning and tagging

pradeeprkara’s picture

pradeeprkara commented
Status: Active » Needs review

Merged user_dashboard_defaults.feature scenarios to user_dashboard.feature.

Removed And I follow "Dashboard" from scenarios except from the first one.

http://drupalcode.org/project/doobie.git/commit/9235006
http://drupalcode.org/project/doobie.git/commit/fd8bc0c
http://drupalcode.org/project/doobie.git/commit/abb7dfa

tvn’s picture

tvn commented

Looks good, thanks.

pradeeprkara’s picture

pradeeprkara commented
Issue tags: +sprint 18

This is ported to 7.x branch: http://drupal.org/node/1697680#comment-6812684

eliza411’s picture

eliza411 commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Needs review » Needs work

Do you think we could use this test to try out the Spin function defined here? http://docs.behat.org/cookbook/using_spin_functions.html We need the step "I wait for the page to load" to fail faster than it is right now, for sure (not sure what the timeout is set to, but it's waiting over a minute right now).

pradeeprkara’s picture

pradeeprkara commented
Status: Needs work » Needs review
Issue tags: +sprint 19

We can depend on the spin method to force timeout but with limited capabilities.(Refer to: #1860452: Evaluate the Spin method#comment-6863638) As it now supports Goutte as well, I have added it in between steps that follow a page load. As it is not that helpful with selenium, have removed @javascript tag from some of the scenarios wherein it is actually not required.

http://drupalcode.org/project/doobie.git/commit/8ce0ce7

eliza411’s picture

eliza411 commented
Status: Needs review » Needs work

Everything here looks good except 

                  # FeatureContext::assertPageNotContainsText()
    When I create a new issue                                                                     # FeatureContext::iCreateANewIssue()
      Select option with value|text "6.x-1.0" not found.
eliza411’s picture

eliza411 commented
Assigned: pradeeprkara » Unassigned

Unassigning

eliza411’s picture

eliza411 commented
Issue summary: View changes

.