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

Modify tests for selectList and dropdown #245

Merged
merged 1 commit into from
Jan 25, 2023

Conversation

Deltaspace0
Copy link
Contributor

Hi @fjvallarino!
In #227 you said you weren't able to generate a failing unit test that your PR (#230) fixes. I added a test for dropdown and modified existing test for selectList. They both fail before your commit and pass after.

@fjvallarino
Copy link
Owner

@Deltaspace0 I just tested and they are indeed great for validating the fix. Thank you!

@fjvallarino fjvallarino merged commit 2c52bc8 into fjvallarino:main Jan 25, 2023
@Deltaspace0 Deltaspace0 deleted the test branch January 25, 2023 13:33
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.

None yet

2 participants