Merge branch '35453-pending-delete-projects-error-in-admin-dashboard-fix' into 'master'

Fixes 500 error caused by pending delete projects in admin dashboard

Closes #35453

See merge request !13067
This commit is contained in:
Sean McGivern 2017-07-26 08:29:38 +00:00
commit f2da36f196
3 changed files with 26 additions and 1 deletions

View File

@ -1,6 +1,6 @@
class Admin::DashboardController < Admin::ApplicationController
def index
@projects = Project.with_route.limit(10)
@projects = Project.without_deleted.with_route.limit(10)
@users = User.limit(10)
@groups = Group.with_route.limit(10)
end

View File

@ -0,0 +1,4 @@
---
title: Fixes 500 error caused by pending delete projects in admin dashboard
merge_request: 13067
author:

View File

@ -0,0 +1,21 @@
require 'spec_helper'
describe Admin::DashboardController do
describe '#index' do
context 'with pending_delete projects' do
render_views
it 'does not retrieve projects that are pending deletion' do
sign_in(create(:admin))
project = create(:project)
pending_delete_project = create(:project, pending_delete: true)
get :index
expect(response.body).to match(project.name)
expect(response.body).not_to match(pending_delete_project.name)
end
end
end
end