From a7166794bf84c158beb32ba58a177c1ef1264056 Mon Sep 17 00:00:00 2001 From: Jason Weathered Date: Fri, 10 Jun 2011 19:17:54 +1000 Subject: [PATCH] RecordNotFound errors now trigger 404s Exceptions that raise during view rendering do not trigger ActionDispatch's exception handler. --- app/controllers/posts_controller.rb | 4 ++++ spec/acceptance/posts_spec.rb | 7 +++++++ 2 files changed, 11 insertions(+) diff --git a/app/controllers/posts_controller.rb b/app/controllers/posts_controller.rb index 99afc2c..5d9d26d 100644 --- a/app/controllers/posts_controller.rb +++ b/app/controllers/posts_controller.rb @@ -1,4 +1,8 @@ class PostsController < ApplicationController + def show + resource + end + def feed max_age = 4.hours response.headers['Cache-Control'] = 'public, max-age=' + max_age.to_i.to_s diff --git a/spec/acceptance/posts_spec.rb b/spec/acceptance/posts_spec.rb index 9455f4e..4d554ea 100644 --- a/spec/acceptance/posts_spec.rb +++ b/spec/acceptance/posts_spec.rb @@ -136,4 +136,11 @@ end end + context 'Posts#show with invalid slug' do + it 'should raise an not found exception' do + lambda do + visit post_path('2011/05/01/invalid') + end.should raise_error(ActiveRecord::RecordNotFound) + end + end end