-
Notifications
You must be signed in to change notification settings - Fork 2
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
Added tests for extra event functions #35
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.
What makes us sure that the inserted event is of the same name
as the one we get with get_elixir_conf_event /0
?
What if we insert two events with the same name 🤔
@okothkongo , we can add a unique constraint to the name , would this suffice? |
be1313c
to
be7f141
Compare
f864473
to
346c580
Compare
346c580
to
be0b057
Compare
In this PR , I added tests for the following functions
get_all_available_tickets
,get_elixir_conf_event
,get_elixir_conf_event_and_ticket_types