Fix false positive specs
This commit is contained in:
parent
e4b2c3b08b
commit
e285fb5049
|
@ -96,16 +96,20 @@ describe 'Private Group access' do
|
|||
|
||||
describe 'GET /groups/:path for shared projects' do
|
||||
let(:project) { create(:project, :public) }
|
||||
let(:linking_user) { create(:user) }
|
||||
before do
|
||||
group.add_guest(linking_user)
|
||||
|
||||
Projects::GroupLinks::CreateService.new(
|
||||
project,
|
||||
create(:user),
|
||||
linking_user,
|
||||
link_group_access: ProjectGroupLink::DEVELOPER
|
||||
).execute(group)
|
||||
end
|
||||
|
||||
subject { group_path(group) }
|
||||
|
||||
it { expect(group.shared_projects).not_to be_empty }
|
||||
it { is_expected.to be_allowed_for(:admin) }
|
||||
it { is_expected.to be_allowed_for(:owner).of(group) }
|
||||
it { is_expected.to be_allowed_for(:maintainer).of(group) }
|
||||
|
|
|
@ -80,6 +80,8 @@ describe GroupPolicy do
|
|||
let(:current_user) { nil }
|
||||
|
||||
before do
|
||||
group.add_developer(user)
|
||||
|
||||
Projects::GroupLinks::CreateService.new(
|
||||
project,
|
||||
user,
|
||||
|
@ -87,7 +89,10 @@ describe GroupPolicy do
|
|||
).execute(group)
|
||||
end
|
||||
|
||||
it { expect_disallowed(:read_group) }
|
||||
it do
|
||||
expect(group.shared_projects).not_to be_empty
|
||||
expect_disallowed(:read_group)
|
||||
end
|
||||
end
|
||||
|
||||
context 'with foreign user and public project' do
|
||||
|
@ -96,6 +101,8 @@ describe GroupPolicy do
|
|||
let(:current_user) { create(:user) }
|
||||
|
||||
before do
|
||||
group.add_developer(user)
|
||||
|
||||
Projects::GroupLinks::CreateService.new(
|
||||
project,
|
||||
user,
|
||||
|
@ -103,7 +110,10 @@ describe GroupPolicy do
|
|||
).execute(group)
|
||||
end
|
||||
|
||||
it { expect_disallowed(:read_group) }
|
||||
it do
|
||||
expect(group.shared_projects).not_to be_empty
|
||||
expect_disallowed(:read_group)
|
||||
end
|
||||
end
|
||||
|
||||
context 'has projects' do
|
||||
|
|
Loading…
Reference in New Issue