Follow-up from "Add filter to hide dismissed alerts"
The following discussion from !49975 (merged) should be addressed:
-
@ohoral started a discussion: it is advised against testing vm properties. Let's check the usage of it
- the usage of this is in the graphql query, which is not being properly tested
- migrate
ee/spec/frontend/threat_monitoring/components/alerts/alerts_list_spec.js
to mock the apollo client and test that the graphql query gets called with the proper variables - similar to this file