Skip to content
This repository was archived by the owner on Mar 17, 2020. It is now read-only.

Conversation

pritalpatel
Copy link
Contributor

@pritalpatel pritalpatel commented Jun 12, 2016

Related issue: #16

Sending users.feature scenario in separate PR from #18

@pritalpatel pritalpatel force-pushed the gherkin-feature-users branch 5 times, most recently from e586cc7 to e428944 Compare June 15, 2016 16:02
@pritalpatel pritalpatel force-pushed the gherkin-feature-users branch from e428944 to 69413af Compare June 15, 2016 16:03
$I->click(['xpath' => "//div[@id='toolbar-featured']//button"]);
break;
}
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functions like the previous one will be moved to step or helper files. But since we will be improving the current code through iterations will merge.

@javigomez javigomez merged commit feccb60 into joomla-projects:staging Jun 23, 2016
Scenario: perform a add new user
Given There is a add user link
When I create new user with fields Name "register", Login Name "register", Password "register" and Email "[email protected]"
Then I Save the user
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be "when" not "then"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Javi.
I am thinking to refactor it like this,

When I fill following fields Name "register", Login Name "register", Password "register" and Email "[email protected]"
And  I Save the  user
Then I see the "User successfully saved." message

How about this?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's perfect!

@pritalpatel pritalpatel deleted the gherkin-feature-users branch October 14, 2016 15:06
yvesh pushed a commit that referenced this pull request Jun 17, 2017
* codestyle

* code style

* codestyle

* codestyle

* codestyle

* thanks @wojsmol

* corrections - thanks @Quy

* corrections - thanks @Quy

* oops

* make @Quy happy

* Update article.xml

* Remove space

* Update config.xml (#14)

* Update filter.xml (#15)

* Update config.xml (#16)

* Update profile.xml (#17)

* Update application.xml (#18)

* Update article.xml (#19)

* Update filter_articles.xml (#20)

* Update config.xml (#24)

* Update config.xml (#23)

* Update filter_fields.xml (#22)

* Update filter_featured.xml (#21)

* Update override.xml (#25)

* Update config.xml

* Update config.xml (#26)

* Update itemadmin_alias.xml (#30)

* Update itemadmin.xml (#29)

* Update item.xml (#27)

* Update item_alias.xml (#28)

* Update itemadmin_url.xml (#31)

* Update module.xml (#32)

* Update plugin.xml (#33)

* Update config.xml (#34)

* Update link.xml (#35)

* Update config.xml (#36)

* Update style.xml (#38)

* Update config.xml (#37)

* Update note.xml (#42)

* Update group.xml (#41)

* Update filter_debuggroup.xml (#40)

* Update config.xml (#39)

* corrections for @andrepereiradasilva

* gotya
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants