Skip to content
This repository has been archived by the owner on Feb 1, 2023. It is now read-only.

#7 - behat tests #45

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

#7 - behat tests #45

wants to merge 10 commits into from

Conversation

Anna-Sokolowska
Copy link
Contributor

I have written the first features:

  • CRUD for meetups
  • CRUD for organizations
  • CRUD for speakers
  • authentication
  • changing password
  • inviting users to admin panel

@Anna-Sokolowska Anna-Sokolowska changed the title #7 behat tests #7 - behat tests Apr 2, 2022
@krzysztofrewak krzysztofrewak mentioned this pull request Apr 4, 2022
Copy link
Member

@EwelinaSkrzypacz EwelinaSkrzypacz left a 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.

features/admin_panel/meetup/delete.feature Outdated Show resolved Hide resolved
features/admin_panel/meetup/show.feature Outdated Show resolved Hide resolved
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:
Copy link
Member

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?

@krzysztofrewak krzysztofrewak mentioned this pull request Apr 5, 2022
@krzysztofrewak
Copy link
Member

@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.

features/admin_panel/adding_user_to_admin_panel.feature Outdated Show resolved Hide resolved
@@ -0,0 +1,35 @@
Feature: Adding a user to admin panel
Copy link
Member

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).

features/authentication/register.feature Outdated Show resolved Hide resolved
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