LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-11 05:25:32 +03:00
Code Issues Packages Projects Releases Wiki Activity

LibCore: Make the FileWatcher test resilient against outside events

Browse Source 
The test currently watches /tmp, which the OS can create/modify files
under at any time outside of our control. So just ignore events that we
aren't interested in.

Also test removing an item from the FileWatcher.
This commit is contained in:
Timothy Flynn 2023-01-18 14:01:37 -05:00 committed by Linus Groh
parent 0e7a48bd7d
commit 8438c509e9
Notes: sideshowbarker 2024-07-17 06:28:38 +09:00 
Author: https://github.com/trflynn89
Commit: https://github.com/SerenityOS/serenity/commit/8438c509e9
Pull-request: https://github.com/SerenityOS/serenity/pull/17086
Reviewed-by: https://github.com/linusg ✅
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
Tests/LibCore/TestLibCoreFileWatcher.cpp
View File

@ -26,12 +26,16 @@ TEST_CASE(file_watcher_child_events)
int event_count = 0;
file_watcher->on_change = [&](Core::FileWatcherEvent const& event) {
// Ignore path events under /tmp that can occur for anything else the OS is
// doing to create/delete files there.
if (event.event_path != "/tmp/testfile"sv)
return;
if (event_count == 0) {
EXPECT_EQ(event.event_path, "/tmp/testfile");
EXPECT(has_flag(event.type, Core::FileWatcherEvent::Type::ChildCreated));
} else if (event_count == 1) {
EXPECT_EQ(event.event_path, "/tmp/testfile");
EXPECT(has_flag(event.type, Core::FileWatcherEvent::Type::ChildDeleted));
EXPECT(MUST(file_watcher->remove_watch("/tmp/"sv)));
event_loop.quit(0);
}