Skip to content

Add security training urls request spec

Malcolm Locke requested to merge 406623-training-providers-request-spec into master

What does this MR do and why?

This change replaces the resolver unit spec for Resolvers::SecurityTrainingUrlsResolve with a request spec to better exercise the full stack on this endpoint.

The main motivation is to provide coverage for a test gap we had with errors in the background job for reactive caching on this endpoint, see !129561 (merged) for details. Additionally our GraphQL style guide says to prefer request specs over unit tests

Related to #406623 (closed)

Edited by Malcolm Locke

Merge request reports