Fixes N+1 on ExperimentsController
requested to merge 444689-test-spec-requests-projects-ml-experiments_controller_spec-rb-projects-ml into master
What does this MR do and why?
Test was quarantined due to flakiness: an N+1 was hiding under skip_cached=False, setting to true makes the test consistently fail. The query mismatch was being caused within rendering the page, and avoiding the query fixes it.
Changelog: fixed
Reproducing
Changing !150271 (diffs) to link_to_experiment(experiment.project, experiment)
should make the test fail.
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Related to #444689 (closed)
Edited by Eduardo Bonet