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

bevy_ecs system stepping tests are failing and are not tested in CI #17652

Open
alice-i-cecile opened this issue Feb 3, 2025 · 1 comment
Open
Labels
A-Build-System Related to build systems or continuous integration A-ECS Entities, components, systems, and events C-Bug An unexpected or incorrect behavior S-Needs-Investigation This issue requires detective work to figure out what's going wrong

Comments

@alice-i-cecile
Copy link
Member

Bevy version

#17651

What you did

cargo test --package bevy_ecs --lib --all-features

What went wrong

failures:
    schedule::stepping::tests::clear_breakpoint
    schedule::stepping::tests::clear_schedule
    schedule::stepping::tests::clear_schedule_then_set_behavior
    schedule::stepping::tests::clear_system
    schedule::stepping::tests::continue_always_run
    schedule::stepping::tests::continue_breakpoint
    schedule::stepping::tests::continue_never_run
    schedule::stepping::tests::continue_step_continue_with_breakpoint
    schedule::stepping::tests::disabled_always_run
    schedule::stepping::tests::disabled_breakpoint
    schedule::stepping::tests::disabled_never_run
    schedule::stepping::tests::multiple_calls_per_frame_continue
    schedule::stepping::tests::multiple_calls_per_frame_step
    schedule::stepping::tests::remove_schedule
    schedule::stepping::tests::set_behavior_then_clear_schedule
    schedule::stepping::tests::step_always_run
    schedule::stepping::tests::step_breakpoint
    schedule::stepping::tests::step_never_run

test result: FAILED. 643 passed; 18 failed; 1 ignored; 0 measured; 0 filtered out; finished in 0.04s

Additional information

These don't seem to be tested in CI somehow? Bisecting the start of the failure would be nice.

We should fix these tests, and ensure that our build system is exercising these tests. @mockersf or @BD103 may have ideas on the latter.

@alice-i-cecile alice-i-cecile added A-Build-System Related to build systems or continuous integration A-ECS Entities, components, systems, and events C-Bug An unexpected or incorrect behavior S-Needs-Investigation This issue requires detective work to figure out what's going wrong labels Feb 3, 2025
@Nilirad
Copy link
Contributor

Nilirad commented Feb 4, 2025

I searched for the commit introducing the test failure using git bisect: it is #17051.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Build-System Related to build systems or continuous integration A-ECS Entities, components, systems, and events C-Bug An unexpected or incorrect behavior S-Needs-Investigation This issue requires detective work to figure out what's going wrong
Projects
None yet
Development

No branches or pull requests

2 participants