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

Change final test in genre_spec.rb #11

Merged
merged 1 commit into from
Nov 2, 2017
Merged

Change final test in genre_spec.rb #11

merged 1 commit into from
Nov 2, 2017

Conversation

thebrianemory
Copy link
Contributor

The final test in genre_spec.rb uses the same wording as the previous test. The test for the new form and edit form have the same wording.

it 'shows a new form that submits content and redirects and prints out params'

This is a causing confusion for students as they think something is wrong with their new genre form when the issue is actually their edit genre form.

Updated the wording for the edit form test to instead read:

it 'shows an edit form that submits content and redirects and prints out params'

The last test uses the same wording as the previous test

This causes you to think your new form is failing when it is fine

The issue is actually your edit form is wrong

This is causing confusion for the students

Update: it 'shows a new form that submits content and redirects and prints out params'

To now read: it 'shows an edit form that submits content and redirects and prints out params'
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.

None yet

3 participants