Skip to content
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

Implement create performance with a related cause #143

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

aonomike
Copy link
Collaborator

@aonomike aonomike commented Jan 8, 2020

What does this PR do?

  • Ensures performance can be created with artists
  • Implements creating performance with cause

How should this be manually tested?

  1. On terminal, run command ...

mix test

What are the relevant Github Issues ?

Fixes #142

Co-authored-by: Arslan <arslan_21@rocketmail.com>
@aonomike aonomike force-pushed the 142_create-performance-with-related-cause branch from b00ca93 to bca95a5 Compare January 8, 2020 17:48
@yakryder
Copy link
Collaborator

yakryder commented Oct 7, 2020

Thank you Mike! Build failing

 1) test create_performance/1 creates a performance with a related cause (SingForNeeds.PerformancesTest)
     test/sing_for_needs/performances/performances_test.exs:47
     Assertion with == failed
     code:  assert performance.title() == valid_attrs_with_cause_id.title()
     left:  nil
     right: "Awesome Performance A"
     stacktrace:
       test/sing_for_needs/performances/performances_test.exs:54: (test)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement create performance with related cause
2 participants