Follow-up from "Add metrics image UI for Alerts"
The following discussion from !84129 (merged) should be addressed:
-
@mrincon started a discussion: (+2 comments) Suggestion: The logic in this file is getting gradually more complex and we don't have text coverage for this file.
Perhaps is time to split it for each page?
If not, we can also define util functions that we can test in isolation? E.g.
getAlertDetailsProvide(page)
,getAlertDetailsAppOptions(page)
, ....