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

Document what actions are made available in an anonymous controller #415

Merged

Conversation

alanshields
Copy link
Contributor

It's a somewhat long test, but it shows what calls you can make and
what calls you cannot make on an anonymous controller. I ran into this
issue myself when I thought that all actions were routed.

They weren't.

Written on time paid for by onetotheworld.net

It's a somewhat long test, but it shows what calls you can make and
what calls you cannot make on an anonymous controller. I ran into this
issue myself when I thought that all actions were routed.

They weren't.

Written on time paid for by onetotheworld.net
@jdelStrother
Copy link
Contributor

The fact it only sets up routes for the regular index/show/create/update/destroy caught me off guard too : is this really desired behaviour, or would it be better to set up the routes to match any controller action that happens to be specified?

@alanshields
Copy link
Contributor Author

If someone wouldn't mind taking a look at this. I find I keep referring back to this code when doing anonymous controller work. That would indicate to me, at least, that this is good documentation. Though since I'm the one who wrote it, maybe not.

@travis
Copy link

travis commented Nov 3, 2011

+1 on this - definitely confusing as is

dchelimsky added a commit that referenced this pull request Dec 20, 2011
…r-functions

Document what actions are made available in an anonymous controller
@dchelimsky dchelimsky merged commit b8b6f79 into rspec:master Dec 20, 2011
dchelimsky added a commit that referenced this pull request Dec 20, 2011
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

4 participants