Skip to content

AlertDialog doesn't support data-testid #8369

@drebin8

Description

@drebin8

Provide a general summary of the issue here

Adding "data-testid" to an AlertDialog does not add it to the dom element. As an addition, it'd be nice to have a way to add testid to the buttons - even if it's just taking a testid that's passed in, adding some suffix to it, and adding it to the button. Or adding confirmButtonProps etc.

Image

🤔 Expected Behavior?

data-testid should be an accepted prop and should be added to the dialog dom node.

😯 Current Behavior

data-testid is ignored for AlertDialog

💁 Possible Solution

No response

🔦 Context

QE is trying to write ui tests. Because of this issue, had to rewrite the component as a Dialog.

🖥️ Steps to Reproduce

https://codesandbox.io/p/sandbox/xwq44t

Version

3.42.1

What browsers are you seeing the problem on?

Other, Chrome, Firefox, Safari, Microsoft Edge

If other, please specify.

all

What operating system are you using?

osx

🧢 Your Company/Team

Adobe/DC

🕷 Tracking Issue

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions