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

Added tests for widgets #49

Merged
merged 1 commit into from
Oct 28, 2023

Conversation

WindowGenerator
Copy link
Contributor

@WindowGenerator WindowGenerator commented Oct 24, 2023

Added a way to mock tui::widgets::Widget to test if expected text is rendered

WindowGenerator added a commit to WindowGenerator/donkeytype that referenced this pull request Oct 26, 2023
@WindowGenerator WindowGenerator force-pushed the add-way-to-mock-widgets branch from 10717e8 to c7caf6a Compare October 26, 2023 20:07
@WindowGenerator WindowGenerator changed the title Added tests for widgets (Work In Progress) Added tests for widgets Oct 26, 2023
WindowGenerator added a commit to WindowGenerator/donkeytype that referenced this pull request Oct 27, 2023
@WindowGenerator WindowGenerator force-pushed the add-way-to-mock-widgets branch from c7caf6a to 4f5cb79 Compare October 27, 2023 20:27
@WindowGenerator WindowGenerator force-pushed the add-way-to-mock-widgets branch from 4f5cb79 to dda677f Compare October 27, 2023 20:28
@WindowGenerator
Copy link
Contributor Author

@radlinskii
Would you mind reviewing this PR when you have a moment? Thank you!

Copy link
Owner

@radlinskii radlinskii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job, thanks!

@radlinskii radlinskii merged commit 1d987dc into radlinskii:main Oct 28, 2023
1 check passed
@WindowGenerator WindowGenerator deleted the add-way-to-mock-widgets branch October 29, 2023 10:31
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.

2 participants