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

Rspec Added for Testing #27

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

sherrimitchell
Copy link
Contributor

I added Rspec to the project to use for testing. I created tests for the User model and Users Controller. The gemfile and gemfile.lock have both been updated to include the Rspec additions. A spec folder with Controllers and Models folders which include the spec files for both and a rails_helper spec_helper file are included.

The tests are a starting point, and we can add more and refactor as needed.

…s for Users controllers. Tests for show, new, and create methods are included. User_spec file includes test for User model. All tests are currently passing in pending status.The spec directory which includes the rails_helper and spec_helper files is created as part of the rspec install. More tests will be added and refined as we continue to add to the project.
@KyFaSt
Copy link
Member

KyFaSt commented Nov 18, 2015

I think this is a really good start! check my note on the last commit about test syntax but otherwise I think these are ready for you to write some tests 👍

I would just keep working on this PR and we'll merge it once we have some working tests.

…dding changes to user.rb and user_spec.rb files. Updated rails_helper.rb and spec_helper files for Factory Girl gems.
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.

2 participants