404 on board index when no board configured. #64

7e99e271 · Felix Schäfer · 9ed2d8ed · 7e99e271 · 7e99e271
Commit 7e99e271 authored Feb 24, 2011 by Felix Schäfer
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 0 deletions

boards_controller.rb app/controllers/boards_controller.rb +1 -0

boards_controller_test.rb test/functional/boards_controller_test.rb +7 -0

No files found.
--- a/app/controllers/boards_controller.rb
+++ b/app/controllers/boards_controller.rb
@@ -29,6 +29,7 @@ class BoardsController < ApplicationController
 
  def index
    @boards = @project.boards
+    render_404 if @boards.empty?
    # show the board if there is only one
    if @boards.size == 1
      @board = @boards.first

--- a/test/functional/boards_controller_test.rb
+++ b/test/functional/boards_controller_test.rb
@@ -96,4 +96,11 @@ class BoardsControllerTest < ActionController::TestCase
    assert_redirected_to '/projects/ecookbook/settings/boards'
    assert_nil Board.find_by_id(2)
  end
+  
+  def test_index_should_404_with_no_board
+    Project.find(1).boards.each(&:destroy)
+    
+    get :index, :project_id => 1
+    assert_response 404
+  end
 end