Fix moderator rights inconsistencies (#26729)
This commit is contained in:
parent
93d051e47d
commit
b83e487502
6 changed files with 47 additions and 9 deletions
|
@ -7,7 +7,8 @@ describe Admin::StatusPolicy do
|
|||
let(:policy) { described_class }
|
||||
let(:admin) { Fabricate(:user, role: UserRole.find_by(name: 'Admin')).account }
|
||||
let(:john) { Fabricate(:account) }
|
||||
let(:status) { Fabricate(:status) }
|
||||
let(:status) { Fabricate(:status, visibility: status_visibility) }
|
||||
let(:status_visibility) { :public }
|
||||
|
||||
permissions :index?, :update?, :review?, :destroy? do
|
||||
context 'with an admin' do
|
||||
|
@ -26,7 +27,7 @@ describe Admin::StatusPolicy do
|
|||
permissions :show? do
|
||||
context 'with an admin' do
|
||||
context 'with a public visible status' do
|
||||
before { allow(status).to receive(:public_visibility?).and_return(true) }
|
||||
let(:status_visibility) { :public }
|
||||
|
||||
it 'permits' do
|
||||
expect(policy).to permit(admin, status)
|
||||
|
@ -34,11 +35,21 @@ describe Admin::StatusPolicy do
|
|||
end
|
||||
|
||||
context 'with a not public visible status' do
|
||||
before { allow(status).to receive(:public_visibility?).and_return(false) }
|
||||
let(:status_visibility) { :direct }
|
||||
|
||||
it 'denies' do
|
||||
expect(policy).to_not permit(admin, status)
|
||||
end
|
||||
|
||||
context 'when the status mentions the admin' do
|
||||
before do
|
||||
status.mentions.create!(account: admin)
|
||||
end
|
||||
|
||||
it 'permits' do
|
||||
expect(policy).to permit(admin, status)
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue