bevy_ecs/observer/

mod.rs

//! Observers are a push-based tool for responding to [`Event`]s. The [`Observer`] component holds a [`System`] that runs whenever a matching [`Event`]
//! is triggered.
//!
//! See [`Event`] and [`Observer`] for in-depth documentation and usage examples.

mod centralized_storage;
mod condition;
mod distributed_storage;
mod entity_cloning;
mod runner;
mod system_param;

pub use centralized_storage::*;
pub use condition::*;
pub use distributed_storage::*;
pub use runner::*;
pub use system_param::*;

use crate::{
    change_detection::MaybeLocation,
    event::Event,
    prelude::*,
    world::{DeferredWorld, *},
};

impl World {
    /// Spawns a "global" [`Observer`] which will watch for the given event.
    /// Returns its [`Entity`] as a [`EntityWorldMut`].
    ///
    /// `system` can be any system whose first parameter is [`On`].
    ///
    /// # Example
    ///
    /// ```
    /// # use bevy_ecs::prelude::*;
    /// #[derive(Component)]
    /// struct A;
    ///
    /// # let mut world = World::new();
    /// world.add_observer(|_: On<Add, A>| {
    ///     // ...
    /// });
    /// world.add_observer(|_: On<Remove, A>| {
    ///     // ...
    /// });
    /// ```
    ///
    /// **Calling [`observe`](EntityWorldMut::observe) on the returned
    /// [`EntityWorldMut`] will observe the observer itself, which you very
    /// likely do not want.**
    ///
    /// # Panics
    ///
    /// Panics if the given system is an exclusive system.
    pub fn add_observer<M>(&mut self, observer: impl IntoObserver<M>) -> EntityWorldMut<'_> {
        self.spawn(observer.into_observer())
    }

    /// Triggers the given [`Event`], which will run any [`Observer`]s watching for it.
    ///
    /// For a variant that borrows the `event` rather than consuming it, use [`World::trigger_ref`] instead.
    #[track_caller]
    pub fn trigger<'a, E: Event<Trigger<'a>: Default>>(&mut self, mut event: E) {
        self.trigger_ref_with_caller(
            &mut event,
            &mut <E::Trigger<'a> as Default>::default(),
            MaybeLocation::caller(),
        );
    }

    /// Triggers the given [`Event`] using the given [`Trigger`](crate::event::Trigger), which will run any [`Observer`]s watching for it.
    ///
    /// For a variant that borrows the `event` rather than consuming it, use [`World::trigger_ref`] instead.
    #[track_caller]
    pub fn trigger_with<'a, E: Event>(&mut self, mut event: E, mut trigger: E::Trigger<'a>) {
        self.trigger_ref_with_caller(&mut event, &mut trigger, MaybeLocation::caller());
    }

    /// Triggers the given mutable [`Event`] reference, which will run any [`Observer`]s watching for it.
    ///
    /// Compared to [`World::trigger`], this method is most useful when it's necessary to check
    /// or use the event after it has been modified by observers.
    #[track_caller]
    pub fn trigger_ref<'a, E: Event<Trigger<'a>: Default>>(&mut self, event: &mut E) {
        self.trigger_ref_with_caller(
            event,
            &mut <E::Trigger<'a> as Default>::default(),
            MaybeLocation::caller(),
        );
    }

    /// Triggers the given mutable [`Event`] reference using the given mutable [`Trigger`](crate::event::Trigger) reference, which
    /// will run any [`Observer`]s watching for it.
    ///
    /// Compared to [`World::trigger`], this method is most useful when it's necessary to check
    /// or use the event after it has been modified by observers.
    pub fn trigger_ref_with<'a, E: Event>(&mut self, event: &mut E, trigger: &mut E::Trigger<'a>) {
        self.trigger_ref_with_caller(event, trigger, MaybeLocation::caller());
    }

    pub(crate) fn trigger_ref_with_caller<'a, E: Event>(
        &mut self,
        event: &mut E,
        trigger: &mut E::Trigger<'a>,
        caller: MaybeLocation,
    ) {
        let event_key = self.register_event_key::<E>();
        // SAFETY: event_key was just registered and matches `event`
        unsafe {
            DeferredWorld::from(self).trigger_raw(event_key, event, trigger, caller);
        }
    }

    /// Splits `&mut self` into a [`DeferredWorld`] and the [`CachedObservers`]
    /// registered for `event_key`, or returns `None` if no observers exist.
    ///
    /// # Safety
    ///
    /// Caller must not use the returned [`DeferredWorld`] to access observer
    /// storage, as it aliases with the returned [`CachedObservers`] reference.
    unsafe fn split_for_event(
        &mut self,
        event_key: crate::event::EventKey,
    ) -> Option<(DeferredWorld<'_>, &CachedObservers)> {
        let world_cell = self.as_unsafe_world_cell();
        let observers = world_cell.observers();
        let observers = observers.try_get_observers(event_key)?;
        // SAFETY: The caller guarantees the returned `DeferredWorld` will not
        // be used to access observer storage (which `observers` borrows).
        Some((unsafe { world_cell.into_deferred() }, observers))
    }

    /// Triggers global [`Observer`]s for `event_key` with untyped event and
    /// trigger data.
    ///
    /// Dynamic equivalent of [`World::trigger`]. Only fires global observers,
    /// not entity- or component-scoped ones.
    ///
    /// Use [`World::trigger_dynamic_targets`] to also fire entity-scoped
    /// observers.
    ///
    /// # Safety
    ///
    /// - `event_data` must point to a valid, aligned value whose layout matches
    ///   what observers registered for this `event_key` expect.
    /// - `trigger_data` must point to a valid, aligned value whose layout
    ///   matches what observers registered for this `event_key` expect.
    #[track_caller]
    pub unsafe fn trigger_dynamic(
        &mut self,
        event_key: crate::event::EventKey,
        mut event_data: bevy_ptr::PtrMut,
        mut trigger_data: bevy_ptr::PtrMut,
    ) {
        // SAFETY: We have exclusive access via `&mut self` and will not
        // access observer storage through the returned `DeferredWorld`.
        let Some((mut world, observers)) = (unsafe { self.split_for_event(event_key) }) else {
            return;
        };

        let context = TriggerContext {
            event_key,
            caller: MaybeLocation::caller(),
        };

        // SAFETY: no outstanding world references besides `observers`
        unsafe {
            world.as_unsafe_world_cell().increment_trigger_id();
        }

        for (observer, runner) in observers.global_observers() {
            // SAFETY:
            // - `observers` come from `world` and correspond to `event_key`
            // - caller guarantees `event_data` and `trigger_data` are valid
            unsafe {
                (runner)(
                    world.reborrow(),
                    *observer,
                    &context,
                    event_data.reborrow(),
                    trigger_data.reborrow(),
                );
            }
        }
    }

    /// Triggers [`Observer`]s for `event_key` targeting `entity`, with untyped
    /// event and trigger data.
    ///
    /// Fires global and entity-scoped observers. Dynamic equivalent of
    /// [`EntityWorldMut::trigger`].
    ///
    /// # Safety
    ///
    /// - `event_data` must point to a valid, aligned value whose layout matches
    ///   what observers registered for this `event_key` expect.
    /// - `trigger_data` must point to a valid, aligned value whose layout
    ///   matches what observers registered for this `event_key` expect.
    #[track_caller]
    pub unsafe fn trigger_dynamic_targets(
        &mut self,
        event_key: crate::event::EventKey,
        entity: Entity,
        event_data: bevy_ptr::PtrMut,
        trigger_data: bevy_ptr::PtrMut,
    ) {
        // SAFETY: We have exclusive access via `&mut self` and will not
        // access observer storage through the returned `DeferredWorld`.
        let Some((world, observers)) = (unsafe { self.split_for_event(event_key) }) else {
            return;
        };

        let context = TriggerContext {
            event_key,
            caller: MaybeLocation::caller(),
        };

        // SAFETY:
        // - `observers` come from `world` and correspond to `event_key`
        // - caller guarantees `event_data` and `trigger_data` are valid
        // - `trigger_entity_internal` increments the trigger id
        unsafe {
            crate::event::trigger_entity_internal(
                world,
                observers,
                event_data,
                trigger_data,
                entity,
                &context,
            );
        }
    }

    /// Triggers [`Observer`]s for `event_key` targeting `entity` and
    /// `components`, with untyped event and trigger data.
    ///
    /// Fires global, entity-scoped, and component-scoped observers.
    /// Dynamic equivalent of [`EntityComponentsTrigger`].
    ///
    /// [`EntityComponentsTrigger`]: crate::event::EntityComponentsTrigger
    ///
    /// # Safety
    ///
    /// - `event_data` must point to a valid, aligned value whose layout matches
    ///   what observers registered for this `event_key` expect.
    /// - `trigger_data` must point to a valid, aligned value whose layout
    ///   matches what observers registered for this `event_key` expect.
    #[track_caller]
    pub unsafe fn trigger_dynamic_targets_components(
        &mut self,
        event_key: crate::event::EventKey,
        entity: Entity,
        components: &[crate::component::ComponentId],
        mut event_data: bevy_ptr::PtrMut,
        mut trigger_data: bevy_ptr::PtrMut,
    ) {
        // SAFETY: We have exclusive access via `&mut self` and will not
        // access observer storage through the returned `DeferredWorld`.
        let Some((mut world, observers)) = (unsafe { self.split_for_event(event_key) }) else {
            return;
        };

        let context = TriggerContext {
            event_key,
            caller: MaybeLocation::caller(),
        };

        // SAFETY:
        // - `observers` come from `world` and correspond to `event_key`
        // - caller guarantees `event_data` and `trigger_data` are valid
        // - `trigger_entity_internal` increments the trigger id
        unsafe {
            crate::event::trigger_entity_internal(
                world.reborrow(),
                observers,
                event_data.reborrow(),
                trigger_data.reborrow(),
                entity,
                &context,
            );
        }

        // Trigger observers watching for specific components.
        for id in components {
            if let Some(component_observers) = observers.component_observers().get(id) {
                for (observer, runner) in component_observers.global_observers() {
                    // SAFETY: same as above, caller guarantees data validity
                    unsafe {
                        (runner)(
                            world.reborrow(),
                            *observer,
                            &context,
                            event_data.reborrow(),
                            trigger_data.reborrow(),
                        );
                    }
                }

                if let Some(map) = component_observers
                    .entity_component_observers()
                    .get(&entity)
                {
                    for (observer, runner) in map {
                        // SAFETY: same as above, caller guarantees data validity
                        unsafe {
                            (runner)(
                                world.reborrow(),
                                *observer,
                                &context,
                                event_data.reborrow(),
                                trigger_data.reborrow(),
                            );
                        }
                    }
                }
            }
        }
    }

    /// Register an observer to the cache, called when an observer is created
    pub(crate) fn register_observer(&mut self, observer_entity: Entity) {
        // SAFETY: References do not alias.
        let (observer_state, archetypes, observers) = unsafe {
            let observer_state: *const Observer = self.get::<Observer>(observer_entity).unwrap();
            // Populate ObservedBy for each observed entity.
            for watched_entity in (*observer_state).descriptor.entities.iter().copied() {
                let mut entity_mut = self.entity_mut(watched_entity);
                let mut observed_by = entity_mut.entry::<ObservedBy>().or_default().into_mut();
                observed_by.0.push(observer_entity);
            }
            (&*observer_state, &mut self.archetypes, &mut self.observers)
        };
        let descriptor = &observer_state.descriptor;

        for &event_key in &descriptor.event_keys {
            let cache = observers.get_observers_mut(event_key);

            if descriptor.components.is_empty() && descriptor.entities.is_empty() {
                cache
                    .global_observers
                    .insert(observer_entity, observer_state.runner);
            } else if descriptor.components.is_empty() {
                // Observer is not targeting any components so register it as an entity observer
                for &watched_entity in &observer_state.descriptor.entities {
                    let map = cache.entity_observers.entry(watched_entity).or_default();
                    map.insert(observer_entity, observer_state.runner);
                }
            } else {
                // Register observer for each watched component
                for &component in &descriptor.components {
                    let observers =
                        cache
                            .component_observers
                            .entry(component)
                            .or_insert_with(|| {
                                if let Some(flag) = Observers::is_archetype_cached(event_key) {
                                    archetypes.update_flags(component, flag, true);
                                }
                                CachedComponentObservers::default()
                            });
                    if descriptor.entities.is_empty() {
                        // Register for all triggers targeting the component
                        observers
                            .global_observers
                            .insert(observer_entity, observer_state.runner);
                    } else {
                        // Register for each watched entity
                        for &watched_entity in &descriptor.entities {
                            let map = observers
                                .entity_component_observers
                                .entry(watched_entity)
                                .or_default();
                            map.insert(observer_entity, observer_state.runner);
                        }
                    }
                }
            }
        }
    }

    /// Remove the observer from the cache, called when an observer gets despawned
    pub(crate) fn unregister_observer(&mut self, entity: Entity, descriptor: ObserverDescriptor) {
        // Remove this observer from all the corresponding ObservedBy components.
        for &observing in descriptor.entities.iter() {
            let Ok(mut observing) = self.get_entity_mut(observing) else {
                // This can happen when ObservedBy is despawning and is despawning the related
                // observers.
                continue;
            };
            let Some(mut observed_by) = observing.get_mut::<ObservedBy>() else {
                // In "normal" usage, this should be impossible, but there's nothing stopping a user
                // from just removing the ObservedBy component themselves. While that's odd usage,
                // there's no reason to panic if a user does so.
                continue;
            };

            observed_by.0.retain(|e| *e != entity);
            if observed_by.0.is_empty() {
                observing.remove::<ObservedBy>();
            }
        }

        let archetypes = &mut self.archetypes;
        let observers = &mut self.observers;

        for &event_key in &descriptor.event_keys {
            let cache = observers.get_observers_mut(event_key);
            if descriptor.components.is_empty() && descriptor.entities.is_empty() {
                cache.global_observers.remove(&entity);
            } else if descriptor.components.is_empty() {
                for watched_entity in &descriptor.entities {
                    // This check should be unnecessary since this observer hasn't been unregistered yet
                    let Some(observers) = cache.entity_observers.get_mut(watched_entity) else {
                        continue;
                    };
                    observers.remove(&entity);
                    if observers.is_empty() {
                        cache.entity_observers.remove(watched_entity);
                    }
                }
            } else {
                for component in &descriptor.components {
                    let Some(observers) = cache.component_observers.get_mut(component) else {
                        continue;
                    };
                    if descriptor.entities.is_empty() {
                        observers.global_observers.remove(&entity);
                    } else {
                        for watched_entity in &descriptor.entities {
                            let Some(map) =
                                observers.entity_component_observers.get_mut(watched_entity)
                            else {
                                continue;
                            };
                            map.remove(&entity);
                            if map.is_empty() {
                                observers.entity_component_observers.remove(watched_entity);
                            }
                        }
                    }

                    if observers.global_observers.is_empty()
                        && observers.entity_component_observers.is_empty()
                    {
                        cache.component_observers.remove(component);
                        if let Some(flag) = Observers::is_archetype_cached(event_key)
                            && let Some(by_component) = archetypes.by_component.get(component)
                        {
                            for archetype in by_component.keys() {
                                let archetype = &mut archetypes.archetypes[archetype.index()];
                                if archetype.contains(*component) {
                                    let no_longer_observed = archetype
                                        .iter_components()
                                        .all(|id| !cache.component_observers.contains_key(&id));

                                    if no_longer_observed {
                                        archetype.flags.set(flag, false);
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    }
}

#[cfg(test)]
mod tests {
    use alloc::{vec, vec::Vec};
    use core::any::type_name;

    use bevy_ptr::OwningPtr;

    use crate::{
        archetype::{Archetype, ArchetypeId},
        change_detection::MaybeLocation,
        error::Result,
        event::{EntityComponentsTrigger, Event, GlobalTrigger},
        hierarchy::ChildOf,
        observer::{Discard, ObservedBy, Observer},
        prelude::*,
        world::DeferredWorld,
    };

    #[derive(Component)]
    struct A;

    #[derive(Component)]
    struct B;

    #[derive(Component)]
    #[component(storage = "SparseSet")]
    struct S;

    #[derive(Event)]
    struct EventA;

    #[derive(EntityEvent)]
    struct EntityEventA(Entity);

    #[derive(EntityEvent)]
    #[entity_event(trigger = EntityComponentsTrigger<'a>)]
    struct EntityComponentsEvent(Entity);

    #[derive(Event)]
    struct EventWithData {
        counter: usize,
    }

    #[derive(Resource, Default)]
    struct Order(Vec<&'static str>);

    impl Order {
        #[track_caller]
        fn observed(&mut self, name: &'static str) {
            self.0.push(name);
        }
    }

    #[derive(Component, EntityEvent)]
    #[entity_event(propagate, auto_propagate)]
    struct EventPropagating(Entity);

    #[test]
    fn observer_order_spawn_despawn() {
        let mut world = World::new();
        world.init_resource::<Order>();

        world.add_observer(|_: On<Add, A>, mut res: ResMut<Order>| res.observed("add"));
        world.add_observer(|_: On<Insert, A>, mut res: ResMut<Order>| res.observed("insert"));
        world.add_observer(|_: On<Discard, A>, mut res: ResMut<Order>| {
            res.observed("discard");
        });
        world.add_observer(|_: On<Remove, A>, mut res: ResMut<Order>| res.observed("remove"));

        let entity = world.spawn(A).id();
        world.despawn(entity);
        assert_eq!(
            vec!["add", "insert", "discard", "remove"],
            world.resource::<Order>().0
        );
    }

    #[test]
    fn observer_order_insert_remove() {
        let mut world = World::new();
        world.init_resource::<Order>();

        world.add_observer(|_: On<Add, A>, mut res: ResMut<Order>| res.observed("add"));
        world.add_observer(|_: On<Insert, A>, mut res: ResMut<Order>| res.observed("insert"));
        world.add_observer(|_: On<Discard, A>, mut res: ResMut<Order>| {
            res.observed("discard");
        });
        world.add_observer(|_: On<Remove, A>, mut res: ResMut<Order>| res.observed("remove"));

        let mut entity = world.spawn_empty();
        entity.insert(A);
        entity.remove::<A>();
        entity.flush();
        assert_eq!(
            vec!["add", "insert", "discard", "remove"],
            world.resource::<Order>().0
        );
    }

    #[test]
    fn observer_order_insert_remove_sparse() {
        let mut world = World::new();
        world.init_resource::<Order>();

        world.add_observer(|_: On<Add, S>, mut res: ResMut<Order>| res.observed("add"));
        world.add_observer(|_: On<Insert, S>, mut res: ResMut<Order>| res.observed("insert"));
        world.add_observer(|_: On<Discard, S>, mut res: ResMut<Order>| {
            res.observed("discard");
        });
        world.add_observer(|_: On<Remove, S>, mut res: ResMut<Order>| res.observed("remove"));

        let mut entity = world.spawn_empty();
        entity.insert(S);
        entity.remove::<S>();
        entity.flush();
        assert_eq!(
            vec!["add", "insert", "discard", "remove"],
            world.resource::<Order>().0
        );
    }

    #[test]
    fn observer_order_replace() {
        let mut world = World::new();
        world.init_resource::<Order>();

        let entity = world.spawn(A).id();

        world.add_observer(|_: On<Add, A>, mut res: ResMut<Order>| res.observed("add"));
        world.add_observer(|_: On<Insert, A>, mut res: ResMut<Order>| res.observed("insert"));
        world.add_observer(|_: On<Discard, A>, mut res: ResMut<Order>| {
            res.observed("discard");
        });
        world.add_observer(|_: On<Remove, A>, mut res: ResMut<Order>| res.observed("remove"));

        let mut entity = world.entity_mut(entity);
        entity.insert(A);
        entity.flush();
        assert_eq!(vec!["discard", "insert"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_order_recursive() {
        let mut world = World::new();
        world.init_resource::<Order>();
        world.add_observer(
            |add: On<Add, A>, mut res: ResMut<Order>, mut commands: Commands| {
                res.observed("add_a");
                commands.entity(add.entity).insert(B);
            },
        );
        world.add_observer(
            |remove: On<Remove, A>, mut res: ResMut<Order>, mut commands: Commands| {
                res.observed("remove_a");
                commands.entity(remove.entity).remove::<B>();
            },
        );

        world.add_observer(
            |add: On<Add, B>, mut res: ResMut<Order>, mut commands: Commands| {
                res.observed("add_b");
                commands.entity(add.entity).remove::<A>();
            },
        );
        world.add_observer(|_: On<Remove, B>, mut res: ResMut<Order>| {
            res.observed("remove_b");
        });

        let entity = world.spawn(A).flush();
        let entity = world.get_entity(entity).unwrap();
        assert!(!entity.contains::<A>());
        assert!(!entity.contains::<B>());
        assert_eq!(
            vec!["add_a", "add_b", "remove_a", "remove_b"],
            world.resource::<Order>().0
        );
    }

    #[test]
    fn observer_trigger_ref() {
        let mut world = World::new();

        world.add_observer(|mut event: On<EventWithData>| event.counter += 1);
        world.add_observer(|mut event: On<EventWithData>| event.counter += 2);
        world.add_observer(|mut event: On<EventWithData>| event.counter += 4);

        let mut event = EventWithData { counter: 0 };
        world.trigger_ref(&mut event);
        assert_eq!(7, event.counter);
    }

    #[test]
    fn observer_multiple_listeners() {
        let mut world = World::new();
        world.init_resource::<Order>();

        world.add_observer(|_: On<Add, A>, mut res: ResMut<Order>| res.observed("add_1"));
        world.add_observer(|_: On<Add, A>, mut res: ResMut<Order>| res.observed("add_2"));

        world.spawn(A).flush();
        assert_eq!(vec!["add_2", "add_1"], world.resource::<Order>().0);
        // we have one A entity and two observers
        assert_eq!(world.query::<&A>().query(&world).count(), 1);
        assert_eq!(world.query::<&Observer>().query(&world).count(), 2);
    }

    #[test]
    fn observer_multiple_events() {
        let mut world = World::new();
        world.init_resource::<Order>();
        let on_remove = world.register_event_key::<Remove>();
        world.spawn(
            // SAFETY: Add and Remove are both unit types, so this is safe
            unsafe {
                Observer::new(|_: On<Add, A>, mut res: ResMut<Order>| {
                    res.observed("add/remove");
                })
                .with_event_key(on_remove)
            },
        );

        let entity = world.spawn(A).id();
        world.despawn(entity);
        assert_eq!(
            vec!["add/remove", "add/remove"],
            world.resource::<Order>().0
        );
    }

    #[test]
    fn observer_multiple_components() {
        let mut world = World::new();
        world.init_resource::<Order>();
        world.register_component::<A>();
        world.register_component::<B>();

        world.add_observer(|_: On<Add, (A, B)>, mut res: ResMut<Order>| {
            res.observed("add_ab");
        });

        let entity = world.spawn(A).id();
        world.entity_mut(entity).insert(B);
        assert_eq!(vec!["add_ab", "add_ab"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_despawn() {
        let mut world = World::new();

        let system: fn(On<Add, A>) = |_| {
            panic!("Observer triggered after being despawned.");
        };
        let observer = world.add_observer(system).id();
        world.despawn(observer);
        world.spawn(A).flush();
    }

    // Regression test for https://github.com/bevyengine/bevy/issues/14961
    #[test]
    fn observer_despawn_archetype_flags() {
        let mut world = World::new();
        world.init_resource::<Order>();

        let entity = world.spawn((A, B)).flush();

        world.add_observer(|_: On<Remove, A>, mut res: ResMut<Order>| {
            res.observed("remove_a");
        });

        let system: fn(On<Remove, B>) = |_: On<Remove, B>| {
            panic!("Observer triggered after being despawned.");
        };

        let observer = world.add_observer(system).flush();
        world.despawn(observer);

        world.despawn(entity);

        assert_eq!(vec!["remove_a"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_multiple_matches() {
        let mut world = World::new();
        world.init_resource::<Order>();

        world.add_observer(|_: On<Add, (A, B)>, mut res: ResMut<Order>| {
            res.observed("add_ab");
        });

        world.spawn((A, B)).flush();
        assert_eq!(vec!["add_ab"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_entity_routing() {
        let mut world = World::new();
        world.init_resource::<Order>();

        let system: fn(On<EntityEventA>) = |_| {
            panic!("Trigger routed to non-targeted entity.");
        };

        world.spawn_empty().observe(system);
        let entity = world
            .spawn_empty()
            .observe(|_: On<EntityEventA>, mut res: ResMut<Order>| res.observed("a_1"))
            .id();
        world.add_observer(move |event: On<EntityEventA>, mut res: ResMut<Order>| {
            assert_eq!(event.event_target(), entity);
            res.observed("a_2");
        });

        world.trigger(EntityEventA(entity));
        assert_eq!(vec!["a_2", "a_1"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_multiple_targets() {
        #[derive(Resource, Default)]
        struct R(i32);

        let mut world = World::new();
        let component_a = world.register_component::<A>();
        let component_b = world.register_component::<B>();
        world.init_resource::<R>();

        // targets (entity_1, A)
        let entity_1 = world
            .spawn_empty()
            .observe(|_: On<EntityComponentsEvent, A>, mut res: ResMut<R>| res.0 += 1)
            .id();
        // targets (entity_2, B)
        let entity_2 = world
            .spawn_empty()
            .observe(|_: On<EntityComponentsEvent, B>, mut res: ResMut<R>| res.0 += 10)
            .id();
        // targets any entity or component
        world.add_observer(|_: On<EntityComponentsEvent>, mut res: ResMut<R>| res.0 += 100);
        // targets any entity, and components A or B
        world
            .add_observer(|_: On<EntityComponentsEvent, (A, B)>, mut res: ResMut<R>| res.0 += 1000);
        // test all tuples
        world.add_observer(
            |_: On<EntityComponentsEvent, (A, B, (A, B))>, mut res: ResMut<R>| res.0 += 10000,
        );
        world.add_observer(
            |_: On<EntityComponentsEvent, (A, B, (A, B), ((A, B), (A, B)))>, mut res: ResMut<R>| {
                res.0 += 100000;
            },
        );
        world.add_observer(
            |_: On<EntityComponentsEvent, (A, B, (A, B), (B, A), (A, B, ((A, B), (B, A))))>,
             mut res: ResMut<R>| res.0 += 1000000,
        );

        // trigger for an entity and a component
        world.trigger_with(
            EntityComponentsEvent(entity_1),
            EntityComponentsTrigger {
                components: &[component_a],
                old_archetype: None,
                new_archetype: None,
            },
        );
        // only observer that doesn't trigger is the one only watching entity_2
        assert_eq!(1111101, world.resource::<R>().0);
        world.resource_mut::<R>().0 = 0;

        // trigger for both entities, but no components: trigger once per entity target
        world.trigger_with(
            EntityComponentsEvent(entity_1),
            EntityComponentsTrigger {
                components: &[],
                old_archetype: None,
                new_archetype: None,
            },
        );
        world.trigger_with(
            EntityComponentsEvent(entity_2),
            EntityComponentsTrigger {
                components: &[],
                old_archetype: None,
                new_archetype: None,
            },
        );

        // only the observer that doesn't require components triggers - once per entity
        assert_eq!(200, world.resource::<R>().0);
        world.resource_mut::<R>().0 = 0;

        // trigger for both entities and both components: trigger once per entity target
        // we only get 2222211 because a given observer can trigger only once per entity target
        world.trigger_with(
            EntityComponentsEvent(entity_1),
            EntityComponentsTrigger {
                components: &[component_a, component_b],
                old_archetype: None,
                new_archetype: None,
            },
        );
        world.trigger_with(
            EntityComponentsEvent(entity_2),
            EntityComponentsTrigger {
                components: &[component_a, component_b],
                old_archetype: None,
                new_archetype: None,
            },
        );
        assert_eq!(2222211, world.resource::<R>().0);
        world.resource_mut::<R>().0 = 0;
    }

    #[test]
    fn observer_dynamic_component() {
        let mut world = World::new();
        world.init_resource::<Order>();

        let component_id = world.register_component::<A>();
        world.spawn(
            Observer::new(|_: On<Add>, mut res: ResMut<Order>| res.observed("event_a"))
                .with_component(component_id),
        );

        let mut entity = world.spawn_empty();
        OwningPtr::make(A, |ptr| {
            // SAFETY: we registered `component_id` above.
            unsafe { entity.insert_by_id(component_id, ptr) };
        });

        assert_eq!(vec!["event_a"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_dynamic_trigger() {
        let mut world = World::new();
        world.init_resource::<Order>();
        let event_a = world.register_event_key::<EventA>();

        // SAFETY: we registered `event_a` above and it matches the type of EventA
        let observe = unsafe {
            Observer::with_dynamic_runner(
                |mut world, _observer, _trigger_context, _event, _trigger| {
                    world.resource_mut::<Order>().observed("event_a");
                },
            )
            .with_event_key(event_a)
        };
        world.spawn(observe);

        world.commands().queue(move |world: &mut World| {
            // SAFETY: we registered `event_a` above and it matches the type of EventA
            unsafe {
                DeferredWorld::from(world).trigger_raw(
                    event_a,
                    &mut EventA,
                    &mut GlobalTrigger,
                    MaybeLocation::caller(),
                );
            }
        });
        world.flush();
        assert_eq!(vec!["event_a"], world.resource::<Order>().0);
    }

    /// Collects `u32` values read by dynamic observers through `PtrMut`.
    #[derive(Resource, Default)]
    struct DynamicValues(Vec<u32>);

    #[test]
    fn observer_fully_dynamic_trigger() {
        use core::alloc::Layout;

        let mut world = World::new();
        world.init_resource::<Order>();
        world.init_resource::<DynamicValues>();

        // Register a dynamic event whose data is a u32.
        let event_id = world.register_component_with_descriptor(
            // SAFETY: u32 layout with no drop
            unsafe {
                crate::component::ComponentDescriptor::new_with_layout(
                    "DynamicEvent",
                    crate::component::StorageType::Table,
                    Layout::new::<u32>(),
                    None,
                    false,
                    crate::component::ComponentCloneBehavior::Ignore,
                    None,
                )
            },
        );
        // SAFETY: event_id was just registered for use as an event
        let event_key = unsafe { crate::event::EventKey::new(event_id) };

        // SAFETY: event_key was just created, observer reads event_data as u32
        let observe = unsafe {
            Observer::with_dynamic_runner(
                |mut world, _observer, _trigger_context, event, _trigger| {
                    // SAFETY: caller passes a valid u32 pointer as event data
                    let value = *event.as_ref().deref::<u32>();
                    world.resource_mut::<Order>().observed("dynamic_event");
                    world.resource_mut::<DynamicValues>().0.push(value);
                },
            )
            .with_event_key(event_key)
        };
        world.spawn(observe);

        let mut event_data: u32 = 42;
        let mut trigger_data: u32 = 0;
        // SAFETY: pointers are valid u32s matching the registered layout
        unsafe {
            world.trigger_dynamic(
                event_key,
                bevy_ptr::PtrMut::from(&mut event_data),
                bevy_ptr::PtrMut::from(&mut trigger_data),
            );
        }

        assert_eq!(vec!["dynamic_event"], world.resource::<Order>().0);
        assert_eq!(vec![42], world.resource::<DynamicValues>().0);
    }

    #[test]
    fn observer_fully_dynamic_trigger_targets() {
        use core::alloc::Layout;

        let mut world = World::new();
        world.init_resource::<Order>();
        world.init_resource::<DynamicValues>();

        let event_id = world.register_component_with_descriptor(
            // SAFETY: u32 layout with no drop
            unsafe {
                crate::component::ComponentDescriptor::new_with_layout(
                    "DynamicEntityEvent",
                    crate::component::StorageType::Table,
                    Layout::new::<u32>(),
                    None,
                    false,
                    crate::component::ComponentCloneBehavior::Ignore,
                    None,
                )
            },
        );
        // SAFETY: event_id was just registered for use as an event
        let event_key = unsafe { crate::event::EventKey::new(event_id) };

        let target = world.spawn_empty().id();
        let other = world.spawn_empty().id();

        // SAFETY: event_key was just created, observer reads event_data as u32
        let global = unsafe {
            Observer::with_dynamic_runner(
                |mut world, _observer, _trigger_context, event, _trigger| {
                    let value = *event.as_ref().deref::<u32>();
                    world.resource_mut::<Order>().observed("global");
                    world.resource_mut::<DynamicValues>().0.push(value);
                },
            )
            .with_event_key(event_key)
        };
        world.spawn(global);

        // SAFETY: event_key was just created, observer reads event_data as u32
        let entity_scoped = unsafe {
            Observer::with_dynamic_runner(
                |mut world, _observer, _trigger_context, event, _trigger| {
                    let value = *event.as_ref().deref::<u32>();
                    world.resource_mut::<Order>().observed("entity_scoped");
                    world.resource_mut::<DynamicValues>().0.push(value);
                },
            )
            .with_event_key(event_key)
            .with_entity(target)
        };
        world.spawn(entity_scoped);

        // Trigger targeting `target`: both global and entity-scoped should fire.
        let mut event_data: u32 = 7;
        let mut trigger_data: u32 = 0;
        // SAFETY: pointers are valid u32s matching the registered layout
        unsafe {
            world.trigger_dynamic_targets(
                event_key,
                target,
                bevy_ptr::PtrMut::from(&mut event_data),
                bevy_ptr::PtrMut::from(&mut trigger_data),
            );
        }

        assert_eq!(vec!["global", "entity_scoped"], world.resource::<Order>().0);
        assert_eq!(vec![7, 7], world.resource::<DynamicValues>().0);

        // Trigger targeting `other`: only global should fire.
        world.resource_mut::<Order>().0.clear();
        world.resource_mut::<DynamicValues>().0.clear();
        let mut event_data: u32 = 99;
        let mut trigger_data: u32 = 0;
        // SAFETY: pointers are valid u32s matching the registered layout
        unsafe {
            world.trigger_dynamic_targets(
                event_key,
                other,
                bevy_ptr::PtrMut::from(&mut event_data),
                bevy_ptr::PtrMut::from(&mut trigger_data),
            );
        }

        assert_eq!(vec!["global"], world.resource::<Order>().0);
        assert_eq!(vec![99], world.resource::<DynamicValues>().0);
    }

    #[test]
    fn observer_fully_dynamic_trigger_targets_components() {
        use core::alloc::Layout;

        let mut world = World::new();
        world.init_resource::<Order>();
        world.init_resource::<DynamicValues>();

        let event_id = world.register_component_with_descriptor(
            // SAFETY: u32 layout with no drop
            unsafe {
                crate::component::ComponentDescriptor::new_with_layout(
                    "DynamicComponentEvent",
                    crate::component::StorageType::Table,
                    Layout::new::<u32>(),
                    None,
                    false,
                    crate::component::ComponentCloneBehavior::Ignore,
                    None,
                )
            },
        );
        // SAFETY: event_id was just registered for use as an event
        let event_key = unsafe { crate::event::EventKey::new(event_id) };

        // Register a dynamic component to scope an observer to.
        let comp_id = world.register_component_with_descriptor(
            // SAFETY: ZST layout with no drop
            unsafe {
                crate::component::ComponentDescriptor::new_with_layout(
                    "DynamicComp",
                    crate::component::StorageType::Table,
                    Layout::new::<()>(),
                    None,
                    false,
                    crate::component::ComponentCloneBehavior::Ignore,
                    None,
                )
            },
        );

        let target = world.spawn_empty().id();

        // SAFETY: event_key was just created, observer reads event_data as u32
        let global = unsafe {
            Observer::with_dynamic_runner(
                |mut world, _observer, _trigger_context, event, _trigger| {
                    let value = *event.as_ref().deref::<u32>();
                    world.resource_mut::<Order>().observed("global");
                    world.resource_mut::<DynamicValues>().0.push(value);
                },
            )
            .with_event_key(event_key)
        };
        world.spawn(global);

        // SAFETY: event_key was just created, observer reads event_data as u32
        let comp_scoped = unsafe {
            Observer::with_dynamic_runner(
                |mut world, _observer, _trigger_context, event, _trigger| {
                    let value = *event.as_ref().deref::<u32>();
                    world.resource_mut::<Order>().observed("comp_scoped");
                    world.resource_mut::<DynamicValues>().0.push(value);
                },
            )
            .with_event_key(event_key)
            .with_component(comp_id)
        };
        world.spawn(comp_scoped);

        // Trigger with `comp_id` in the components list: both should fire.
        let mut event_data: u32 = 5;
        let mut trigger_data: u32 = 0;
        // SAFETY: pointers are valid u32s matching the registered layout
        unsafe {
            world.trigger_dynamic_targets_components(
                event_key,
                target,
                &[comp_id],
                bevy_ptr::PtrMut::from(&mut event_data),
                bevy_ptr::PtrMut::from(&mut trigger_data),
            );
        }

        assert_eq!(vec!["global", "comp_scoped"], world.resource::<Order>().0);
        assert_eq!(vec![5, 5], world.resource::<DynamicValues>().0);

        // Trigger without components: only global should fire.
        world.resource_mut::<Order>().0.clear();
        world.resource_mut::<DynamicValues>().0.clear();
        let mut event_data: u32 = 10;
        let mut trigger_data: u32 = 0;
        // SAFETY: pointers are valid u32s matching the registered layout
        unsafe {
            world.trigger_dynamic_targets_components(
                event_key,
                target,
                &[],
                bevy_ptr::PtrMut::from(&mut event_data),
                bevy_ptr::PtrMut::from(&mut trigger_data),
            );
        }

        assert_eq!(vec!["global"], world.resource::<Order>().0);
        assert_eq!(vec![10], world.resource::<DynamicValues>().0);
    }

    #[test]
    fn observer_propagating() {
        let mut world = World::new();
        world.init_resource::<Order>();

        let parent = world.spawn_empty().id();
        let child = world.spawn(ChildOf(parent)).id();

        world.entity_mut(parent).observe(
            move |event: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("parent");

                assert_eq!(event.event_target(), parent);
                assert_eq!(event.original_event_target(), child);
            },
        );

        world.entity_mut(child).observe(
            move |event: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("child");
                assert_eq!(event.event_target(), child);
                assert_eq!(event.original_event_target(), child);
            },
        );

        world.trigger(EventPropagating(child));

        assert_eq!(vec!["child", "parent"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_propagating_redundant_dispatch_same_entity() {
        let mut world = World::new();
        world.init_resource::<Order>();

        let parent = world
            .spawn_empty()
            .observe(|_: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("parent");
            })
            .id();

        let child = world
            .spawn(ChildOf(parent))
            .observe(|_: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("child");
            })
            .id();

        world.trigger(EventPropagating(child));
        world.trigger(EventPropagating(child));

        assert_eq!(
            vec!["child", "parent", "child", "parent"],
            world.resource::<Order>().0
        );
    }

    #[test]
    fn observer_propagating_redundant_dispatch_parent_child() {
        let mut world = World::new();
        world.init_resource::<Order>();

        let parent = world
            .spawn_empty()
            .observe(|_: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("parent");
            })
            .id();

        let child = world
            .spawn(ChildOf(parent))
            .observe(|_: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("child");
            })
            .id();

        world.trigger(EventPropagating(child));
        world.trigger(EventPropagating(parent));

        assert_eq!(
            vec!["child", "parent", "parent"],
            world.resource::<Order>().0
        );
    }

    #[test]
    fn observer_propagating_halt() {
        let mut world = World::new();
        world.init_resource::<Order>();

        let parent = world
            .spawn_empty()
            .observe(|_: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("parent");
            })
            .id();

        let child = world
            .spawn(ChildOf(parent))
            .observe(|mut event: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("child");
                event.propagate(false);
            })
            .id();

        world.trigger(EventPropagating(child));

        assert_eq!(vec!["child"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_propagating_join() {
        let mut world = World::new();
        world.init_resource::<Order>();

        let parent = world
            .spawn_empty()
            .observe(|_: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("parent");
            })
            .id();

        let child_a = world
            .spawn(ChildOf(parent))
            .observe(|_: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("child_a");
            })
            .id();

        let child_b = world
            .spawn(ChildOf(parent))
            .observe(|_: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("child_b");
            })
            .id();

        world.trigger(EventPropagating(child_a));
        world.trigger(EventPropagating(child_b));

        assert_eq!(
            vec!["child_a", "parent", "child_b", "parent"],
            world.resource::<Order>().0
        );
    }

    #[test]
    fn observer_propagating_no_next() {
        let mut world = World::new();
        world.init_resource::<Order>();

        let entity = world
            .spawn_empty()
            .observe(|_: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("event");
            })
            .id();

        world.trigger(EventPropagating(entity));
        assert_eq!(vec!["event"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_propagating_parallel_propagation() {
        let mut world = World::new();
        world.init_resource::<Order>();

        let parent_a = world
            .spawn_empty()
            .observe(|_: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("parent_a");
            })
            .id();

        let child_a = world
            .spawn(ChildOf(parent_a))
            .observe(|mut event: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("child_a");
                event.propagate(false);
            })
            .id();

        let parent_b = world
            .spawn_empty()
            .observe(|_: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("parent_b");
            })
            .id();

        let child_b = world
            .spawn(ChildOf(parent_b))
            .observe(|_: On<EventPropagating>, mut res: ResMut<Order>| {
                res.observed("child_b");
            })
            .id();

        world.trigger(EventPropagating(child_a));
        world.trigger(EventPropagating(child_b));

        assert_eq!(
            vec!["child_a", "child_b", "parent_b"],
            world.resource::<Order>().0
        );
    }

    #[test]
    fn observer_propagating_world() {
        let mut world = World::new();
        world.init_resource::<Order>();

        world.add_observer(|_: On<EventPropagating>, mut res: ResMut<Order>| {
            res.observed("event");
        });

        let grandparent = world.spawn_empty().id();
        let parent = world.spawn(ChildOf(grandparent)).id();
        let child = world.spawn(ChildOf(parent)).id();

        world.trigger(EventPropagating(child));

        assert_eq!(vec!["event", "event", "event"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_propagating_world_skipping() {
        let mut world = World::new();
        world.init_resource::<Order>();

        world.add_observer(
            |event: On<EventPropagating>, query: Query<&A>, mut res: ResMut<Order>| {
                if query.get(event.event_target()).is_ok() {
                    res.observed("event");
                }
            },
        );

        let grandparent = world.spawn(A).id();
        let parent = world.spawn(ChildOf(grandparent)).id();
        let child = world.spawn((A, ChildOf(parent))).id();

        world.trigger(EventPropagating(child));

        assert_eq!(vec!["event", "event"], world.resource::<Order>().0);
    }

    // Originally for https://github.com/bevyengine/bevy/issues/18452
    #[test]
    fn observer_modifies_relationship() {
        fn on_add(add: On<Add, A>, mut commands: Commands) {
            commands
                .entity(add.entity)
                .with_related_entities::<crate::hierarchy::ChildOf>(|rsc| {
                    rsc.spawn_empty();
                });
        }

        let mut world = World::new();
        world.add_observer(on_add);
        world.spawn(A);
    }

    // Regression test for https://github.com/bevyengine/bevy/issues/14467
    // Fails prior to https://github.com/bevyengine/bevy/pull/15398
    #[test]
    fn observer_on_remove_during_despawn_spawn_empty() {
        let mut world = World::new();

        // Observe the removal of A - this will run during despawn
        world.add_observer(|_: On<Remove, A>, mut cmd: Commands| {
            // Spawn a new entity - this reserves a new ID and requires a flush
            // afterward before Entities::free can be called.
            cmd.spawn_empty();
        });

        let ent = world.spawn(A).id();

        // Despawn our entity, which runs the Remove observer and allocates a
        // new Entity.
        // Should not panic - if it does, then Entities was not flushed properly
        // after the observer's spawn_empty.
        world.despawn(ent);
    }

    #[test]
    #[should_panic]
    fn observer_invalid_params() {
        #[derive(Resource)]
        struct ResA;

        #[derive(Resource)]
        struct ResB;

        let mut world = World::new();
        // This fails because `ResA` is not present in the world
        world.add_observer(|_: On<EventA>, _: Res<ResA>, mut commands: Commands| {
            commands.insert_resource(ResB);
        });
        world.trigger(EventA);
    }

    #[test]
    fn observer_apply_deferred_from_param_set() {
        #[derive(Resource)]
        struct ResA;

        let mut world = World::new();
        world.add_observer(
            |_: On<EventA>, mut params: ParamSet<(Query<Entity>, Commands)>| {
                params.p1().insert_resource(ResA);
            },
        );

        world.trigger(EventA);
        world.flush();

        assert!(world.get_resource::<ResA>().is_some());
    }

    #[test]
    #[track_caller]
    fn observer_caller_location_event() {
        #[derive(Event)]
        struct EventA;

        let caller = MaybeLocation::caller();
        let mut world = World::new();
        world.add_observer(move |event: On<EventA>| {
            assert_eq!(event.caller(), caller);
        });
        world.trigger(EventA);
    }

    #[test]
    #[track_caller]
    fn observer_caller_location_command_archetype_move() {
        #[derive(Component)]
        struct Component;

        let caller = MaybeLocation::caller();
        let mut world = World::new();
        world.add_observer(move |event: On<Add, Component>| {
            assert_eq!(event.caller(), caller);
        });
        world.add_observer(move |event: On<Remove, Component>| {
            assert_eq!(event.caller(), caller);
        });
        world.commands().spawn(Component).clear();
    }

    #[test]
    fn observer_watch_entities() {
        let mut world = World::new();
        world.init_resource::<Order>();
        let entities = world
            .spawn_batch(core::iter::repeat_n((), 4))
            .collect::<Vec<_>>();
        let observer = Observer::new(|_: On<EntityEventA>, mut order: ResMut<Order>| {
            order.observed("a");
        });
        world.spawn(observer.with_entities(entities.iter().copied().take(2)));

        world.trigger(EntityEventA(entities[0]));
        world.trigger(EntityEventA(entities[1]));
        assert_eq!(vec!["a", "a"], world.resource::<Order>().0);
        world.trigger(EntityEventA(entities[2]));
        world.trigger(EntityEventA(entities[3]));
        assert_eq!(vec!["a", "a"], world.resource::<Order>().0);
    }

    #[test]
    fn unregister_global_observer() {
        let mut world = World::new();
        let mut observer = world.add_observer(|_: On<EventA>| {});
        observer.remove::<Observer>();
        let id = observer.id();
        let event_key = world.event_key::<EventA>().unwrap();
        assert!(!world
            .observers
            .get_observers_mut(event_key)
            .global_observers
            .contains_key(&id));
    }

    #[test]
    fn unregister_entity_observer() {
        let mut world = World::new();
        let entity = world.spawn_empty().id();
        let observer = Observer::new(|_: On<EventA>| {}).with_entity(entity);
        let mut observer = world.spawn(observer);
        observer.remove::<Observer>();
        let event_key = world.event_key::<EventA>().unwrap();
        assert!(!world
            .observers
            .get_observers_mut(event_key)
            .entity_observers
            .contains_key(&entity));
    }

    #[test]
    fn unregister_component_observer() {
        let mut world = World::new();
        let a = world.register_component::<A>();
        let observer = Observer::new(|_: On<EventA>| {}).with_component(a);
        let mut observer = world.spawn(observer);
        observer.remove::<Observer>();
        let event_key = world.event_key::<EventA>().unwrap();
        assert!(!world
            .observers
            .get_observers_mut(event_key)
            .component_observers()
            .contains_key(&a));
    }

    #[derive(Resource)]
    struct RunConditionFlag(bool);

    #[test]
    fn observer_run_condition_true() {
        let mut world = World::new();
        world.insert_resource(RunConditionFlag(true));
        world.init_resource::<Order>();

        world.add_observer(
            (|_: On<EventA>, mut order: ResMut<Order>| {
                order.observed("event");
            })
            .run_if(|flag: Res<RunConditionFlag>| flag.0),
        );

        world.trigger(EventA);
        assert_eq!(vec!["event"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_run_condition_false() {
        let mut world = World::new();
        world.insert_resource(RunConditionFlag(false));
        world.init_resource::<Order>();

        world.add_observer(
            (|_: On<EventA>, mut order: ResMut<Order>| {
                order.observed("event");
            })
            .run_if(|flag: Res<RunConditionFlag>| flag.0),
        );

        world.trigger(EventA);
        assert!(world.resource::<Order>().0.is_empty());
    }

    #[test]
    fn observer_run_condition_chained() {
        let mut world = World::new();
        world.insert_resource(RunConditionFlag(true));
        world.init_resource::<Order>();

        #[derive(Resource)]
        struct SecondFlag(bool);
        world.insert_resource(SecondFlag(true));

        world.add_observer(
            (|_: On<EventA>, mut order: ResMut<Order>| {
                order.observed("event");
            })
            .run_if(|flag: Res<RunConditionFlag>| flag.0)
            .run_if(|flag: Res<SecondFlag>| flag.0),
        );

        world.trigger(EventA);
        assert_eq!(vec!["event"], world.resource::<Order>().0);

        world.resource_mut::<Order>().0.clear();
        world.resource_mut::<SecondFlag>().0 = false;
        world.trigger(EventA);
        assert!(world.resource::<Order>().0.is_empty());
    }

    #[test]
    fn observer_run_condition_re_evaluated() {
        let mut world = World::new();
        world.insert_resource(RunConditionFlag(false));
        world.init_resource::<Order>();

        world.add_observer(
            (|_: On<EventA>, mut order: ResMut<Order>| {
                order.observed("event");
            })
            .run_if(|flag: Res<RunConditionFlag>| flag.0),
        );

        world.trigger(EventA);
        assert!(world.resource::<Order>().0.is_empty());

        world.resource_mut::<RunConditionFlag>().0 = true;
        world.trigger(EventA);
        assert_eq!(vec!["event"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_run_condition_result_bool() {
        let mut world = World::new();
        world.init_resource::<Order>();

        world.add_observer(
            (|_: On<EventA>, mut order: ResMut<Order>| {
                order.observed("err");
            })
            .run_if(|| -> Result<bool> { Err(core::fmt::Error.into()) }),
        );
        world.add_observer(
            (|_: On<EventA>, mut order: ResMut<Order>| {
                order.observed("false");
            })
            .run_if(|| -> Result<bool> { Ok(false) }),
        );
        world.add_observer(
            (|_: On<EventA>, mut order: ResMut<Order>| {
                order.observed("true");
            })
            .run_if(|| -> Result<bool> { Ok(true) }),
        );

        world.trigger(EventA);
        assert_eq!(vec!["true"], world.resource::<Order>().0);
    }

    #[test]
    fn observer_conditions_and_change_detection() {
        #[derive(Resource, Default)]
        struct Bool2(pub bool);

        let mut world = World::new();
        world.init_resource::<Order>();
        world.insert_resource(RunConditionFlag(false));
        world.insert_resource(Bool2(false));

        world.add_observer(
            (|_: On<EventA>, mut order: ResMut<Order>| {
                order.observed("event");
            })
            .run_if(|res1: Res<RunConditionFlag>| res1.is_changed())
            .run_if(|res2: Res<Bool2>| res2.is_changed()),
        );

        // both resources were just added.
        world.trigger(EventA);
        assert_eq!(vec!["event"], world.resource::<Order>().0);

        // nothing has changed
        world.resource_mut::<Order>().0.clear();
        world.trigger(EventA);
        assert!(world.resource::<Order>().0.is_empty());

        // RunConditionFlag has changed, but observer did not run
        world.resource_mut::<RunConditionFlag>().0 = true;
        world.trigger(EventA);
        assert!(world.resource::<Order>().0.is_empty());

        // internal state for the Bool2 condition was updated in the
        // previous run, so observer still does not run
        world.resource_mut::<Bool2>().0 = true;
        world.trigger(EventA);
        assert!(world.resource::<Order>().0.is_empty());

        // internal state for Bool2 was updated, so observer still does not run
        world.resource_mut::<RunConditionFlag>().0 = false;
        world.trigger(EventA);
        assert!(world.resource::<Order>().0.is_empty());

        // now check that it works correctly changing Bool2 first and then RunConditionFlag
        world.resource_mut::<Bool2>().0 = false;
        world.resource_mut::<RunConditionFlag>().0 = true;
        world.trigger(EventA);
        assert_eq!(vec!["event"], world.resource::<Order>().0);
    }

    #[test]
    fn entity_observer_with_run_condition() {
        let mut world = World::new();
        world.insert_resource(RunConditionFlag(true));
        world.init_resource::<Order>();

        let entity = world
            .spawn_empty()
            .observe(
                (|_: On<EntityEventA>, mut order: ResMut<Order>| {
                    order.observed("entity_event");
                })
                .run_if(|flag: Res<RunConditionFlag>| flag.0),
            )
            .id();

        world.trigger(EntityEventA(entity));
        assert_eq!(vec!["entity_event"], world.resource::<Order>().0);

        world.resource_mut::<Order>().0.clear();
        world.resource_mut::<RunConditionFlag>().0 = false;
        world.trigger(EntityEventA(entity));
        assert!(world.resource::<Order>().0.is_empty());
    }

    #[test]
    fn observer_builder_run_if() {
        let mut world = World::new();
        world.insert_resource(RunConditionFlag(true));
        world.init_resource::<Order>();

        let observer = Observer::new(|_: On<EventA>, mut order: ResMut<Order>| {
            order.observed("event");
        })
        .run_if(|flag: Res<RunConditionFlag>| flag.0);

        world.spawn(observer);

        world.trigger(EventA);
        assert_eq!(vec!["event"], world.resource::<Order>().0);

        world.resource_mut::<Order>().0.clear();
        world.resource_mut::<RunConditionFlag>().0 = false;
        world.trigger(EventA);
        assert!(world.resource::<Order>().0.is_empty());
    }

    #[test]
    fn observer_new_old_archetypes() {
        #[derive(Resource, Default)]
        struct Changes(Vec<(&'static str, Option<ArchetypeId>, Option<ArchetypeId>)>);

        let mut world = World::new();
        world.init_resource::<Changes>();

        fn observer<E: for<'a> Event<Trigger<'a> = EntityComponentsTrigger<'a>>>(
            e: On<E, A>,
            mut c: ResMut<Changes>,
        ) {
            c.0.push((
                type_name::<E>(),
                e.trigger().old_archetype.map(Archetype::id),
                e.trigger().new_archetype.map(Archetype::id),
            ));
        }

        let empty = world.spawn(()).archetype().id();
        let a = world.spawn(A).archetype().id();
        let ab = world.spawn((A, B)).archetype().id();

        world.add_observer(observer::<Add>);
        world.add_observer(observer::<Insert>);
        world.add_observer(observer::<Discard>);
        world.add_observer(observer::<Remove>);
        world.add_observer(observer::<Despawn>);

        let mut entity = world.spawn((A, B));
        entity.remove::<(A, B)>();
        entity.insert(A);
        entity.insert(A);
        entity.despawn();

        assert_eq!(
            &world.resource_mut::<Changes>().0,
            &[
                ("bevy_ecs::lifecycle::Add", None, Some(ab)),
                ("bevy_ecs::lifecycle::Insert", None, Some(ab)),
                ("bevy_ecs::lifecycle::Discard", Some(ab), Some(empty)),
                ("bevy_ecs::lifecycle::Remove", Some(ab), Some(empty)),
                ("bevy_ecs::lifecycle::Add", Some(empty), Some(a)),
                ("bevy_ecs::lifecycle::Insert", Some(empty), Some(a)),
                ("bevy_ecs::lifecycle::Discard", Some(a), Some(a)),
                ("bevy_ecs::lifecycle::Insert", Some(a), Some(a)),
                ("bevy_ecs::lifecycle::Despawn", Some(a), None),
                ("bevy_ecs::lifecycle::Discard", Some(a), None),
                ("bevy_ecs::lifecycle::Remove", Some(a), None),
            ],
        );
    }

    #[test]
    fn despawning_observer_removes_observed_by() {
        let mut world = World::new();

        #[derive(EntityEvent)]
        struct Hi {
            entity: Entity,
        }

        let target = world.spawn_empty().id();
        let observer = world
            .spawn(Observer::new(|_: On<Hi>| {}).with_entity(target))
            .id();

        assert_eq!(
            world.entity(target).get::<ObservedBy>().unwrap().get(),
            &[observer]
        );

        world.entity_mut(observer).despawn();

        assert!(!world.entity(target).contains::<ObservedBy>());
    }

    #[test]
    fn despawning_observer_removes_observed_by_repeated() {
        let mut world = World::new();

        #[derive(EntityEvent)]
        struct Hi {
            entity: Entity,
        }

        let target = world.spawn_empty().id();
        let observer = world
            // Observe the same entity multiple times.
            .spawn(Observer::new(|_: On<Hi>| {}).with_entities([target, target]))
            .id();

        // Having an observer observe the same entity multiple times is likely not desired, but
        // preventing this is not worth it, so make sure it behaves correctly.
        assert_eq!(
            world.entity(target).get::<ObservedBy>().unwrap().get(),
            &[observer, observer]
        );

        world.entity_mut(observer).despawn();

        assert!(!world.entity(target).contains::<ObservedBy>());
    }
}