Take into account directory updates

When watching the whole test directory we'll also get some events for
the directory, and those events can come with different order depending
on the OS, so we can't rely on them. We check that if a directory event
comes, it is a modification.

Signed-off-by: Leandro Lucarella <[email protected]>

Author: llucax

tests/test_file_watcher_integration.py

@@ -10,7 +10,7 @@
 import pytest
 
 from frequenz.channels import ReceiverStoppedError, select, selected_from
-from frequenz.channels.file_watcher import Event, EventType, FileWatcher
+from frequenz.channels.file_watcher import EventType, FileWatcher
 from frequenz.channels.timer import SkipMissedAndDrift, Timer
 
 
@@ -34,11 +34,18 @@ async def test_file_watcher(tmp_path: pathlib.Path) -> None:
             filename.write_text(f"{selected.message}")
         elif selected_from(selected, file_watcher):
             event_type = EventType.CREATE if number_of_writes == 0 else EventType.MODIFY
-            assert selected.message == Event(type=event_type, path=filename)
-            number_of_writes += 1
-            # After receiving a write 3 times, unsubscribe from the writes channel
-            if number_of_writes == expected_number_of_writes:
-                break
+            event = selected.message
+            # If we receive updates for the directory itself, they should be only
+            # modifications, we only check that because we can have ordering issues if
+            # we try check also the order compared to events in the file.
+            if event.path == tmp_path:
+                assert event.type == EventType.MODIFY
+            elif event.path == filename:
+                assert event.type == event_type
+                number_of_writes += 1
+                # After receiving a write 3 times, unsubscribe from the writes channel
+                if number_of_writes == expected_number_of_writes:
+                    break
 
     assert number_of_writes == expected_number_of_writes