[WIP] Behat integration #57
Conversation
There was a problem hiding this comment.
Where's Then? Same questions to several scenarios below :)
There was a problem hiding this comment.
Oh right, its because in previous version of this feature "Services configuration" scenario was a "Background" :P Thanks for pointing this out! Now I'm not sure if it was a good move to change it into scenario. I must rethink this decision again. I would be really glad to hear any suggestions about this part of features.
There was a problem hiding this comment.
What's the outcome of configuring all this things? If there's none, there's no need of having them here :)
There was a problem hiding this comment.
Actually there is one, the main menu structure is result of this configuration. I think its worth to tell how this menu was builded.
Maybe...
Feature: Admin panel main page
In order to generate admin panel for Symfony2 project
As a developer
I need to install FSiAdminBundle and configure few admin elements
Scenario: Admin panelu menu configuration
Given the following services were registered
| Id | Class | Tag | Tag alias |
| fixtures_bundle.admin.news | FSi\FixturesBundle\Admin\News | admin.element | |
| fixtures_bundle.admin.home_page | FSi\FixturesBundle\Admin\HomePage | admin.element | structure |
| fixtures_bundle.admin.about_us_page | FSi\FixturesBundle\Admin\AboutUsPage | admin.element | structure |
And translations are enabled in application
When I open "Admin panel" page
Then I should see main menu with following elements
| Element name | Element group |
| News | |
| Home page | Site structure |
| About us page | Site structure |
What do you think about it?
There was a problem hiding this comment.
If something is not relevant to the current scenario, but it is required for the app to work, I'd hide it.
New scenario looks much better.
There was a problem hiding this comment.
Thanks a lot! 🍻
|
@chives review this please. I think its ready to merge |
2 participants
The goal of this pull request is to cover current functionalities with behat scenarios, this will give us more guaranty in the future that new things didnt break anything.
This PR must be finished before adding dashboards feature.