DevOps Adoption - Remove data-testid constants
The following discussion from !68447 (merged) should be addressed:
-
@pslaughter started a discussion:
suggestion (non-blocking): I recommend against putting test ID's in constants.
- There's not really a reusability win since the automated tests referencing them should be enough to verify the component applied them correctly.
- Ideally we could strip out
data-testid
from production, but moving these values to constants adds to their presence.