Skip to content

Commit

Permalink
tests/UdevWrapper: Drop test relying on UDev behaviour that has changed.
Browse files Browse the repository at this point in the history
None of our code *really* cares that we don't want to get events
before calling `monitor.enable()`, and UDev in Ubuntu 25.04 has
changed behaviour so that events *are* received before enabling
the monitor, so just drop the test.
  • Loading branch information
RAOF committed Jan 16, 2025
1 parent a0335c8 commit c8a01ab
Showing 1 changed file with 0 additions and 13 deletions.
13 changes: 0 additions & 13 deletions tests/unit-tests/test_udev_wrapper.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -317,19 +317,6 @@ TEST_F(UdevWrapperTest, MemberDereferenceWorks)
EXPECT_STREQ("drm", iter->subsystem());
}

TEST_F(UdevWrapperTest, UdevMonitorDoesNotTriggerBeforeEnabling)
{
mir::udev::Monitor monitor{mir::udev::Context()};
bool event_handler_called = false;

udev_environment.add_device("drm", "control64D", NULL, {}, {});

monitor.process_events([&event_handler_called](mir::udev::Monitor::EventType,
mir::udev::Device const&) {event_handler_called = true;});

EXPECT_FALSE(event_handler_called);
}

TEST_F(UdevWrapperTest, UdevMonitorTriggersAfterEnabling)
{
mir::udev::Monitor monitor{mir::udev::Context()};
Expand Down

0 comments on commit c8a01ab

Please sign in to comment.