diff options
Diffstat (limited to 'spec/services/personal_access_tokens/revoke_service_spec.rb')
-rw-r--r-- | spec/services/personal_access_tokens/revoke_service_spec.rb | 20 |
1 files changed, 17 insertions, 3 deletions
diff --git a/spec/services/personal_access_tokens/revoke_service_spec.rb b/spec/services/personal_access_tokens/revoke_service_spec.rb index 5afa43cef76..a25484e218e 100644 --- a/spec/services/personal_access_tokens/revoke_service_spec.rb +++ b/spec/services/personal_access_tokens/revoke_service_spec.rb @@ -6,6 +6,11 @@ RSpec.describe PersonalAccessTokens::RevokeService do shared_examples_for 'a successfully revoked token' do it { expect(subject.success?).to be true } it { expect(service.token.revoked?).to be true } + it 'logs the event' do + expect(Gitlab::AppLogger).to receive(:info).with(/PAT REVOCATION: revoked_by: '#{current_user.username}', revoked_for: '#{token.user.username}', token_id: '\d+'/) + + subject + end end shared_examples_for 'an unsuccessfully revoked token' do @@ -19,10 +24,19 @@ RSpec.describe PersonalAccessTokens::RevokeService do let(:service) { described_class.new(current_user, token: token) } context 'when current_user is an administrator' do - let_it_be(:current_user) { create(:admin) } - let_it_be(:token) { create(:personal_access_token) } + context 'when admin mode is enabled', :enable_admin_mode do + let_it_be(:current_user) { create(:admin) } + let_it_be(:token) { create(:personal_access_token) } + + it_behaves_like 'a successfully revoked token' + end + + context 'when admin mode is disabled' do + let_it_be(:current_user) { create(:admin) } + let_it_be(:token) { create(:personal_access_token) } - it_behaves_like 'a successfully revoked token' + it_behaves_like 'an unsuccessfully revoked token' + end end context 'when current_user is not an administrator' do |