Follow-up from "Remove extra GraphQL query when loading alert list"
The following discussion from !86274 (merged) should be addressed:
-
@mrincon started a discussion: Suggestion (non-blocking): @syasonik thanks for this!
This optimization is something we can actually test in jest by using mock handlers in
createMockApollo
in our tests. I did something like this to ensure we didn't refetch data unless needed:Do you want to take a stab a this? I could be a bit involved because it requires rewriting how
alert_management_table_spec.js
mounts the component, but then we would be covered against this regression.Happy to add it as a follow up as well.