Skip to content

Remove CycleAnalyticsHelpers timecop mocking

James Fargher requested to merge remove_rugged_repo into master

What does this MR do and why?

&8396

Example failure https://gitlab.com/gitlab-org/gitlab/-/jobs/2796510718

This mock was originally used because gitaly is a separate process and it is not possible to mock time on commits that gitaly creates. This has since changed, times are generated in ruby before sending to gitaly via OperationService#user_commit_files_request_header

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 James Fargher

Merge request reports

Loading