Skip to content

Preload correct latest test report for requirements

Felipe Cardozo requested to merge issue_323481 into master

What does this MR do?

Preloads requirements test reports with correct order when using GraphQL lookahead.

Fixes bug where last test report state for a requirement was incorrect because we were not enforcing order on SQL when preloading the association.

Related to #323481 (closed)

Query executed when preloading test reports

SELECT "requirements_management_test_reports".* FROM "requirements_management_test_reports" 
WHERE "requirements_management_test_reports"."requirement_id" IN (12286, 3548, 3547, 6257) 
ORDER BY "requirements_management_test_reports"."created_at" DESC

https://explain.depesz.com/s/jj71

It looks good to me. We could consider adding an index on created_at in the future to support sorting.

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by Felipe Cardozo

Merge request reports

Loading