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

Remove controller and view specs #9

Closed
xaviershay opened this issue Nov 19, 2010 · 5 comments
Closed

Remove controller and view specs #9

xaviershay opened this issue Nov 19, 2010 · 5 comments
Labels

Comments

@xaviershay
Copy link
Owner

They are too brittle. Replace with features, though 100% coverage of views probably not required.

@lucianosousa
Copy link
Contributor

it's a good idea

@recursive
Copy link

I guess I can help with this too. I'll make them not as brittle. As far as working on features, I think I'll pass on that one.

@migol
Copy link

migol commented Jun 24, 2011

To be honest I would leave them. Features is high-level testing while specs are low-level ones. There's no need to make a feature for a bug fix, but a spec that checks it it's a good idea.

@xaviershay
Copy link
Owner Author

Please see the comment regarding brittleness. They break all the time without actually catching errors, so there is a cost to maintaining them.

@xaviershay
Copy link
Owner Author

meh.

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

No branches or pull requests

4 participants