Skip to content

Commit

Permalink
Remove unused test resource in a bevy_ecs schedule unit test (#7551)
Browse files Browse the repository at this point in the history
Small commit to remove an unused resource scoped within a single bevy_ecs unit test. Also rearranged the initialization to follow initialization conventions of surrounding tests. World/Schedule initialization followed by resource initialization.

This change was tested locally with `cargo test`, and `cargo fmt` was run.

Risk should be tiny as change is scoped to a single unit test and very tiny, and I can't see any way that this resource is being used in the test.

Thank you so much!
  • Loading branch information
AndrewJakubowicz committed Feb 7, 2023
1 parent f0fc9e9 commit 978f7cd
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions crates/bevy_ecs/src/schedule/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -175,13 +175,11 @@ mod tests {

#[test]
fn add_systems_correct_order() {
#[derive(Resource)]
struct X(Vec<TestSet>);

let mut world = World::new();
let mut schedule = Schedule::new();

world.init_resource::<SystemOrder>();

let mut schedule = Schedule::new();
schedule.add_systems(
(
make_function_system(0),
Expand Down

0 comments on commit 978f7cd

Please sign in to comment.