-
Notifications
You must be signed in to change notification settings - Fork 89
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding Test for Deleting Category #103
Conversation
@test works fine. |
Thanks, I will refactor this by making use of Step Object. Need some time |
The build will fail since this PR depends on joomla-projects/joomla-browser#71 |
@test tested again, still working fine (after "composer install"). |
* | ||
* @return void | ||
*/ | ||
public function deleteCategory($categoryName) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
Just one comment. Could this method be renamed to "trashCategory" and keep the deleteCategory
for when we completely delete the category from the database.
I have updated the function name! thanks for the feedback! |
|
||
$I->amGoingTo('Try to unpublish a weblink category'); | ||
$I->click(['xpath' => "//button[@onclick=\"if (document.adminForm.boxchecked.value==0){alert('Please first make a selection from the list.');}else{ Joomla.submitbutton('categories.unpublish')}\"]"]); | ||
$I->clickToolbarButton('unpublish'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice 👍
related to joomla-projects/joomla-browser#71
Worked:
|
Thank you @puneet0191! Merged with d42948b |
Thanks @Kubik-Rubik 👌 |
No description provided.