Skip to content

E2E Test: fix fail logging in audit streaming spec

Jay McCure requested to merge jmc-smocker-logging into master

What does this MR do and why?

Fixes logging smocker logs when qa/qa/specs/features/ee/api/10_govern/instance_audit_event_streaming_spec.rb fails. This secondary error can be seen here: https://gitlab.com/gitlab-org/gitlab/-/jobs/5591476140#L3004

Class Service::DockerRun::Smocker no longer has a logs method since !130134 (diffs) was merged.

Screenshots or screen recordings

Note: a dummy test failure commit was added and removed to test this logging. (hence the failing second pipeline)

Example logging (forced failure):

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Jay McCure

Merge request reports