-
Notifications
You must be signed in to change notification settings - Fork 2
#7 - behat tests #45
base: main
Are you sure you want to change the base?
#7 - behat tests #45
Conversation
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.
For now scenarios are okay. In the future we can write more.
Given the administrator is on the deleting meetup page with meetup id <id> | ||
When the administrator clicks on the delete meetup button | ||
Then the administrator should see the message "Meetup successfully deleted" | ||
Examples: |
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.
Do we really need table "examples", if we have only one example?
@Anna-Sokolowska, probably you need to update your codestyle. The best option would be probably to merge current master to your branch and then update all packages. |
@@ -0,0 +1,35 @@ | |||
Feature: Adding a user to admin panel |
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.
It could be good to add unhappy paths scenarios (what happen if email will be empty or invalid).
Co-authored-by: Marcin Tracz <[email protected]>
I have written the first features: