Skip to content

Fix ResultsStore#read to return a CheckResult

Patrick Bajao requested to merge pb-mergeability-framework-caching-fix into master

What does this MR do and why?

MergeRequests::Mergeability::RunChecksService#run_check expects that MergeRequests::Mergeability::ResultsStore#read to return a MergeRequests::Mergeability::CheckResult object.

This fixes the ResultsStore#read to return a CheckResult object once it received a result from cache (e.g. Redis).

This is behind the mergeability_caching feature flag.

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