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

Snapshot tests fail withunity feature #5297

Open
lucasmerlin opened this issue Oct 22, 2024 · 0 comments
Open

Snapshot tests fail withunity feature #5297

lucasmerlin opened this issue Oct 22, 2024 · 0 comments
Labels
bug Something is broken CI Continues Integration test tests Unit tests, regression tests, manual tests, …

Comments

@lucasmerlin
Copy link
Collaborator

lucasmerlin commented Oct 22, 2024

Describe the bug
Running the tests with cargo test --workspace --all-features currently fails because the unity feature isn't compatible with egui_wgpu. This means we currently aren't running tests with all possible features.

Expected behavior
Should be possible to run tests with --all-features

This was broken in #5166 (intentionally, in order to not make that PR even bigger)

@lucasmerlin lucasmerlin added bug Something is broken CI Continues Integration test tests Unit tests, regression tests, manual tests, … labels Oct 22, 2024
lucasmerlin added a commit that referenced this issue Oct 24, 2024
The check.sh script was broken in #5166, this fixes it
* Related to #5297 
* [x] I have followed the instructions in the PR template

---------

Co-authored-by: Emil Ernerfeldt <[email protected]>
lucasmerlin added a commit that referenced this issue Oct 27, 2024
The check.sh script was broken in #5166, this fixes it
* Related to #5297 
* [x] I have followed the instructions in the PR template

---------

Co-authored-by: Emil Ernerfeldt <[email protected]>
lucasmerlin added a commit that referenced this issue Oct 29, 2024
The check.sh script was broken in #5166, this fixes it
* Related to #5297 
* [x] I have followed the instructions in the PR template

---------

Co-authored-by: Emil Ernerfeldt <[email protected]>
hacknus pushed a commit to hacknus/egui that referenced this issue Oct 30, 2024
The check.sh script was broken in emilk#5166, this fixes it
* Related to emilk#5297 
* [x] I have followed the instructions in the PR template

---------

Co-authored-by: Emil Ernerfeldt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken CI Continues Integration test tests Unit tests, regression tests, manual tests, …
Projects
None yet
Development

No branches or pull requests

1 participant