Skip to content

Cleanup spec for ref_exists on GitalyClient

James Fargher requested to merge ref_service_spec_clean into master

What does this MR do and why?

#336749 (closed)

The specs in this file are only to check that the calls to the stubs are hooked up correctly. The actual functionality that we depend on is tested in Gitlab::Git::Repository. So remove the duplicate tests.

Additionally soon we will begin to use the praefect database in tests and when this happens we can no longer rely on being able to directly access repositories without praefect knowing.

So a small cleanup.

MR acceptance checklist

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

Merge request reports

Loading