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

Tests #73

Closed
wants to merge 10 commits into from
Closed

Tests #73

wants to merge 10 commits into from

Conversation

rohitpaulk
Copy link
Member

✋ Work in progress

  • Model tests
    • User model
    • Comment model
    • Project model
  • Controller tests
    • Sessions controller
    • Comments controller
    • Dashboard controller
    • Users controller
  • Feature tests
  • Integration tests

@sarupbanskota
Copy link
Contributor

Thanks for these commits! When you're done adding all tests, you can just remove the todo/WIP marker and I'll merge it in.

I'm not going to review these - I'm not great at tests myself, so I'll just assume that some tests are better than no tests ;) Thanks again!

@rohitpaulk rohitpaulk mentioned this pull request Mar 7, 2014
@EmilyDirsh
Copy link
Contributor

You might want to revisit the project and maybe user tests, given PR #118.

@rohitpaulk
Copy link
Member Author

Kinda busy now, I'll get this done next week :)

@rohitpaulk
Copy link
Member Author

I'm closing this for now. We're pretty much rewriting the app for now - it makes more sense to write tests step by step.

@rohitpaulk rohitpaulk closed this May 25, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants