diff --git a/ee/config/audit_events/types/saml_group_links_created.yml b/ee/config/audit_events/types/saml_group_links_created.yml new file mode 100644 index 0000000000000000000000000000000000000000..9a7aa6511239f1081607f1c3ba5a0882c27fd6c5 --- /dev/null +++ b/ee/config/audit_events/types/saml_group_links_created.yml @@ -0,0 +1,9 @@ +--- +name: saml_group_links_created +description: Event triggered when a SAML Group Link is created +introduced_by_issue: https://gitlab.com/gitlab-org/gitlab/-/issues/373954 +introduced_by_mr: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/110525 +feature_category: compliance_management +milestone: '15.9' +saved_to_database: true +streamed: true diff --git a/ee/config/audit_events/types/saml_group_links_removed.yml b/ee/config/audit_events/types/saml_group_links_removed.yml new file mode 100644 index 0000000000000000000000000000000000000000..8ca2971496b4bcadd3a132be36c0277c22c7293a --- /dev/null +++ b/ee/config/audit_events/types/saml_group_links_removed.yml @@ -0,0 +1,9 @@ +--- +name: saml_group_links_removed +description: Event triggered when a SAML Group Link is destroyed +introduced_by_issue: https://gitlab.com/gitlab-org/gitlab/-/issues/373954 +introduced_by_mr: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/110525 +feature_category: compliance_management +milestone: '15.9' +saved_to_database: true +streamed: true diff --git a/ee/spec/services/group_saml/saml_group_links/create_service_spec.rb b/ee/spec/services/group_saml/saml_group_links/create_service_spec.rb index 63306ebcb246a2724428bb8a96a8ccdd0cd4228a..c0dd59a52f64e4653b19c079676da97f879bf0fe 100644 --- a/ee/spec/services/group_saml/saml_group_links/create_service_spec.rb +++ b/ee/spec/services/group_saml/saml_group_links/create_service_spec.rb @@ -32,7 +32,14 @@ let_it_be(:saml_provider) { create(:saml_provider, group: group, enabled: true) } it "create a new saml_group_link entry against the group" do - expect(::Gitlab::Audit::Auditor).to receive(:audit).once.and_call_original + audit_context = { + name: 'saml_group_links_created', + author: current_user, + scope: group, + target: group, + message: audit_event_message + } + expect(::Gitlab::Audit::Auditor).to receive(:audit).with(audit_context).once.and_call_original response = service.execute diff --git a/ee/spec/services/group_saml/saml_group_links/destroy_service_spec.rb b/ee/spec/services/group_saml/saml_group_links/destroy_service_spec.rb index 9acb4c4d3be3d2594d5d1b278f6b5bae36e4d061..6ea88150c8c4433f08efeebfe173fd791f1d11f0 100644 --- a/ee/spec/services/group_saml/saml_group_links/destroy_service_spec.rb +++ b/ee/spec/services/group_saml/saml_group_links/destroy_service_spec.rb @@ -26,8 +26,16 @@ before do stub_licensed_features(group_saml: true, saml_group_sync: true) end + it "create a new saml_group_link entry against the group" do - expect(::Gitlab::Audit::Auditor).to receive(:audit).once.and_call_original + audit_context = { + name: 'saml_group_links_removed', + author: current_user, + scope: group, + target: group, + message: audit_event_message + } + expect(::Gitlab::Audit::Auditor).to receive(:audit).with(audit_context).once.and_call_original response = service.execute