Skip to content

Conversation

@kitizz
Copy link
Contributor

@kitizz kitizz commented Dec 13, 2025

Includes the following changes:

  • Add egui_kittest, log, env_logger, assertables as dev-dependencies to egui_plot crate
  • Add test-only init_test_logger function to enable capturing log::info! and friends in tests
  • Add tests for basic auto_bounds behavior to be an example for additional test cases

Includes the following changes:
- Add `egui_kittest`, `log`, `env_logger`, `assertables` as dev-dependencies to egui_plot crate
- Add test-only `init_test_logger` function to enable capturing `log::info!` and friends in tests
- Add tests for basic auto_bounds behavior to be an example for additional test cases
@github-actions
Copy link

View snapshot changes at kitdiff

@kitizz kitizz marked this pull request as ready for review December 13, 2025 00:52

/// Test that auto_bounds(true) fits the data exactly when the margin fraction is zero.
#[test]
fn auto_bounds_true() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Love it! This is exactly the step in the right direction.

@michalsustr michalsustr added the exclude from changelog This change will NOT be included in the changelog label Dec 13, 2025
@michalsustr michalsustr merged commit af87f3a into emilk:main Dec 13, 2025
10 checks passed
@kitizz
Copy link
Contributor Author

kitizz commented Dec 13, 2025

Thanks for the review! So speedy

@michalsustr
Copy link
Collaborator

I get to work on this library usually on the weekends, during weekdays not as much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exclude from changelog This change will NOT be included in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add basic infrastructure for non-screenshot tests with kittest

2 participants