Skip to content

Move GraphQL contact N+1 spec

Lee Tickett requested to merge 342914-tech-debt-move-spec into master

What does this MR do and why?

@splattael pointed out:

Question With run_with_clean_state it's not longer a request API spec as it's not running post_graphql at all and is skipping controllers auth etc. Should we move it to, say, spec/graphql/types/issue_type_spec.rb? 🤔

We could still leave a simple happy-path request API spec if we want to. Nothing fancy.

MR acceptance checklist

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

Related to #342914

Edited by Lee Tickett

Merge request reports