Move fully or partially away from snapshots in security dashboard integration test
The following discussion from !199755 (merged) should be addressed:
-
@dpisek started a discussion: (+1 comment) thought: Maybe instead of using snapshot-tests across the whole file, we could limit them and use something like
expect(root.querySelector('[data-testid="group-security-dashboard-new"]')).toBeDefined();(and related query selectors for the new dashboard) in other places?Or perhaps even move away from snapshots?
I found them to be not a 100% useful while iterating over the new dashboard, as they need to get updated whenever we change the dashboard component or add panels, etc.
But happy to at least have a couple of snapshot tests if you feel that they provide another layer of safety
👍 Having said that, this is not blocking, and we can tackle it in a follow-up!