Commit 250dbecd authored by Tiago Botelho's avatar Tiago Botelho

Pending delete projects should not show in deploy keys

parent ea6dfcad
...@@ -9,7 +9,7 @@ class DeployKeyEntity < Grape::Entity ...@@ -9,7 +9,7 @@ class DeployKeyEntity < Grape::Entity
expose :created_at expose :created_at
expose :updated_at expose :updated_at
expose :projects, using: ProjectEntity do |deploy_key| expose :projects, using: ProjectEntity do |deploy_key|
deploy_key.projects.select { |project| options[:user].can?(:read_project, project) } deploy_key.projects.without_deleted.select { |project| options[:user].can?(:read_project, project) }
end end
expose :can_edit expose :can_edit
......
---
title: Pending delete projects should not show in deploy keys.
merge_request: 13088
author:
...@@ -2,13 +2,15 @@ require 'spec_helper' ...@@ -2,13 +2,15 @@ require 'spec_helper'
describe DeployKeyEntity do describe DeployKeyEntity do
include RequestAwareEntity include RequestAwareEntity
let(:user) { create(:user) } let(:user) { create(:user) }
let(:project) { create(:empty_project, :internal)} let(:project) { create(:empty_project, :internal)}
let(:project_private) { create(:empty_project, :private)} let(:project_private) { create(:empty_project, :private)}
let!(:project_pending_delete) { create(:empty_project, :internal, pending_delete: true) }
let(:deploy_key) { create(:deploy_key) } let(:deploy_key) { create(:deploy_key) }
let!(:deploy_key_internal) { create(:deploy_keys_project, project: project, deploy_key: deploy_key) } let!(:deploy_key_internal) { create(:deploy_keys_project, project: project, deploy_key: deploy_key) }
let!(:deploy_key_private) { create(:deploy_keys_project, project: project_private, deploy_key: deploy_key) } let!(:deploy_key_private) { create(:deploy_keys_project, project: project_private, deploy_key: deploy_key) }
let!(:deploy_key_pending_delete) { create(:deploy_keys_project, project: project_pending_delete, deploy_key: deploy_key) }
let(:entity) { described_class.new(deploy_key, user: user) } let(:entity) { described_class.new(deploy_key, user: user) }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment