Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: attempt at unified stateful test #4263

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

asvae
Copy link
Member

@asvae asvae commented May 7, 2024

I attempted to somewhat unify a feature test. I took stateful as a simplest case. We may want to merge it its incomplete state or attempt to apply to other components, making this PR more useful.

I would like this test to be sort of like statefulWorksFor(VaComponent, testDetails).

I only managed to make it work with 2 components, and even that wasn't perfect (click on root element didn't work).

For VaCounter the problematic part was that it didn't even support data-testid on root element. (I detailed in #4262)

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant