[go: up one dir, main page]

Skip to content

Address query limit override in specs for Mergerequests

In MR !155732 (merged), we added validation for presence of Organizations.

As a consequence, logic that is checking organization membership is issuing additional queries. This broke specs that are counting the number of queries. One of those specs is spec/requests/api/graphql/project/merge_requests_spec.rb

The additional queries are executed because we call https://gitlab.com/gitlab-org/gitlab/-/blob/53a78bd4393764a6b2fca42da73addf5b96cd5a6/app/models/group.rb#L732 this

The number of queries for the test should be lowered to under 200 The exception for the rate limit should be lowered as well (or even removed)

Edited by Rutger Wessels