Add pending spec for User.those_who_can (#24724)
parent
105f8687e4
commit
4cfe52635c
|
@ -857,7 +857,6 @@ RSpec/PendingWithoutReason:
|
||||||
Exclude:
|
Exclude:
|
||||||
- 'spec/controllers/statuses_controller_spec.rb'
|
- 'spec/controllers/statuses_controller_spec.rb'
|
||||||
- 'spec/models/account_spec.rb'
|
- 'spec/models/account_spec.rb'
|
||||||
- 'spec/models/user_spec.rb'
|
|
||||||
|
|
||||||
# This cop supports unsafe autocorrection (--autocorrect-all).
|
# This cop supports unsafe autocorrection (--autocorrect-all).
|
||||||
# Configuration parameters: Strict, EnforcedStyle, AllowedExplicitMatchers.
|
# Configuration parameters: Strict, EnforcedStyle, AllowedExplicitMatchers.
|
||||||
|
|
|
@ -527,6 +527,28 @@ RSpec.describe User, type: :model do
|
||||||
end
|
end
|
||||||
|
|
||||||
describe '.those_who_can' do
|
describe '.those_who_can' do
|
||||||
pending
|
let!(:moderator_user) { Fabricate(:user, role: UserRole.find_by(name: 'Moderator')) }
|
||||||
|
|
||||||
|
context 'when there are not any user roles' do
|
||||||
|
before { UserRole.destroy_all }
|
||||||
|
|
||||||
|
it 'returns an empty list' do
|
||||||
|
expect(User.those_who_can(:manage_blocks)).to eq([])
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
context 'when there are not users with the needed role' do
|
||||||
|
it 'returns an empty list' do
|
||||||
|
expect(User.those_who_can(:manage_blocks)).to eq([])
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
context 'when there are users with roles' do
|
||||||
|
let!(:admin_user) { Fabricate(:user, role: UserRole.find_by(name: 'Admin')) }
|
||||||
|
|
||||||
|
it 'returns the users with the role' do
|
||||||
|
expect(User.those_who_can(:manage_blocks)).to eq([admin_user])
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
Reference in New Issue