Add num_entities() to World (bevyengine#19780)

There is a lot of `world.entities().len()`, especially in tests. In
tests, usually, the assumption is made that empty worlds do not contain
any entities. This is about to change (bevyengine#19711), and as such all of these
tests are failing for that PR.

`num_entities` is a convenience method that returns the number of
entities inside a world. It can later be adapted to exclude 'unexpected'
entities, associated with internal data structures such as Resources,
Queries, Systems. In general I argue for a separation of concepts where
`World` ignores internal entities in methods such as `iter_entities()`
and `clear_entities()`, that discussion is, however, separate from this
PR.

I replaced most occurrences of `world.entities().len()` with
`world.num_entities()` and the tests passed.

---------

Co-authored-by: Alice Cecile <[email protected]>

Author: Trashtalk217

crates/bevy_app/src/app.rs

@@ -1582,8 +1582,7 @@ mod tests {
         app.add_systems(EnterMainMenu, (foo, bar));
 
         app.world_mut().run_schedule(EnterMainMenu);
-        let num_resources = app.world_mut().query::<&IsResource>().iter(&app.world()).count() as u32;
-        assert_eq!(app.world().entities().len(), num_resources + 2);
+        assert_eq!(app.world().num_entities(), 2);
     }
 
     #[test]

crates/bevy_ecs/src/lib.rs

@@ -1640,7 +1640,7 @@ mod tests {
 
         assert_eq!(q1.iter(&world).len(), 1);
         assert_eq!(q2.iter(&world).len(), 1);
-        assert_eq!(world.entities().len() - num_resources, 2);
+        assert_eq!(world.num_entities(), 2);
 
         world.clear_entities();
 
@@ -1655,7 +1655,7 @@ mod tests {
             "world should not contain sparse set components"
         );
         assert_eq!(
-            world.entities().len(),
+            world.num_entities(),
             0,
             "world should not have any entities"
         );

crates/bevy_ecs/src/observer/mod.rs

@@ -1316,7 +1316,7 @@ mod tests {
         world.spawn(A).flush();
         assert_eq!(vec!["add_2", "add_1"], world.resource::<Order>().0);
         // Our A entity plus our two observers
-        assert_eq!(world.entities().len() - num_resources, 3);
+        assert_eq!(world.num_entities(), 3);
     }
 
     #[test]

crates/bevy_ecs/src/system/commands/mod.rs

@@ -2343,7 +2343,7 @@ mod tests {
             .spawn((W(1u32), W(2u64)))
             .id();
         command_queue.apply(&mut world);
-        assert_eq!(world.entities().len() - num_resources, 1);
+        assert_eq!(world.num_entities(), 1);
         let results = world
             .query::<(&W<u32>, &W<u64>)>()
             .iter(&world)

crates/bevy_ecs/src/world/command_queue.rs

@@ -424,12 +424,12 @@ mod test {
         let num_resources = world.components().num_resources() as u32;
         queue.apply(&mut world);
 
-        assert_eq!(world.entities().len() - num_resources, 2);
+        assert_eq!(world.num_entities(), 2);
 
         // The previous call to `apply` cleared the queue.
         // This call should do nothing.
         queue.apply(&mut world);
-        assert_eq!(world.entities().len() - num_resources, 2);
+        assert_eq!(world.num_entities(), 2);
     }
 
     #[expect(
@@ -464,7 +464,7 @@ mod test {
         queue.push(SpawnCommand);
         queue.push(SpawnCommand);
         queue.apply(&mut world);
-        assert_eq!(world.entities().len() - num_resources, 3);
+        assert_eq!(world.num_entities(), 3);
     }
 
     #[test]

crates/bevy_ecs/src/world/mod.rs

@@ -218,6 +218,14 @@ impl World {
         &mut self.entities
     }
 
+    /// Retrieves the number of [`Entities`] in the world.
+    ///
+    /// This is helpful as a diagnostic, but it can also be used effectively in tests.
+    #[inline]
+    pub fn num_entities(&self) -> u32 {
+        self.entities.len()
+    }
+
     /// Retrieves this world's [`Archetypes`] collection.
     #[inline]
     pub fn archetypes(&self) -> &Archetypes {