diff options
author | James Lopez <james@gitlab.com> | 2019-07-12 09:21:44 +0300 |
---|---|---|
committer | James Lopez <james@gitlab.com> | 2019-07-12 09:21:44 +0300 |
commit | 4c6bdd23b32940bc1687ef7879821cd8772340b2 (patch) | |
tree | 081fc42f3a02ee64a91aa2613e1bafcac89f04e9 /spec | |
parent | 0b71325dc00ac459636625e743133c0bc0b4cb33 (diff) | |
parent | e19499caf53c7da022333b1f354be058955991e8 (diff) |
Merge branch 'sh-audit-event-json-log-format-from-and-to' into 'master'
Format `from` and `to` fields in JSON audit log
See merge request gitlab-org/gitlab-ce!30333
Diffstat (limited to 'spec')
-rw-r--r-- | spec/services/audit_event_service_spec.rb | 32 |
1 files changed, 28 insertions, 4 deletions
diff --git a/spec/services/audit_event_service_spec.rb b/spec/services/audit_event_service_spec.rb index 32fd98e6ef9..e42bff607b2 100644 --- a/spec/services/audit_event_service_spec.rb +++ b/spec/services/audit_event_service_spec.rb @@ -10,11 +10,8 @@ describe AuditEventService do let(:logger) { instance_double(Gitlab::AuditJsonLogger) } describe '#security_event' do - before do - expect(service).to receive(:file_logger).and_return(logger) - end - it 'creates an event and logs to a file' do + expect(service).to receive(:file_logger).and_return(logger) expect(logger).to receive(:info).with(author_id: user.id, entity_id: project.id, entity_type: "Project", @@ -22,5 +19,32 @@ describe AuditEventService do expect { service.security_event }.to change(SecurityEvent, :count).by(1) end + + it 'formats from and to fields' do + service = described_class.new( + user, project, + { + from: true, + to: false, + action: :create, + target_id: 1 + }) + expect(service).to receive(:file_logger).and_return(logger) + expect(logger).to receive(:info).with(author_id: user.id, + entity_type: 'Project', + entity_id: project.id, + from: 'true', + to: 'false', + action: :create, + target_id: 1) + + expect { service.security_event }.to change(SecurityEvent, :count).by(1) + + details = SecurityEvent.last.details + expect(details[:from]).to be true + expect(details[:to]).to be false + expect(details[:action]).to eq(:create) + expect(details[:target_id]).to eq(1) + end end end |