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

Sending content.feature scenario in separate PR from #18

}
else
{
if ($row[0] == "title")
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you take a look at the comments i made at Puneets PR? There are several issues here, starting from code style to readability ;)

->arg('tests/acceptance/content.feature')
->run()
->stopOnFail();

Copy link
Contributor

Choose a reason for hiding this comment

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

interesting ^_^

@javigomez
Copy link
Contributor

closing for #27

@javigomez javigomez closed this Jun 23, 2016
$I->click(['xpath' => "//div[@id='toolbar-featured']//button"]);
break;
}
}
Copy link
Member

Choose a reason for hiding this comment

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

Better:
$buttons = [
'new' => ['xpath' => "//div[@id='toolbar-new']//button"],
'edit' => ['xpath' => "//div[@id='toolbar-edit']//button"],
...
];
if (isset($button[$input]))
{
$I->click($button[$input]);
}

@pritalpatel pritalpatel deleted the gherkin-feature-content 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.

4 participants