Actually 100% test coverage (#126)

adamchainz · Oct 28, 2024 · c1630a4 · c1630a4
1 parent ae727e7
commit c1630a4
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/src/django_watchfiles/__init__.py b/src/django_watchfiles/__init__.py
@@ -91,8 +91,8 @@ def tick(self) -> Generator[None]:
         )
         self.watcher.set_roots(roots)
 
-        for changes in self.watcher:
-            for _, path in changes:
+        for changes in self.watcher:  # pragma: no branch
+            for _, path in changes:  # pragma: no cover
                 self.notify_file_changed(Path(path))
             yield
 

diff --git a/tests/test_django_watchfiles.py b/tests/test_django_watchfiles.py
@@ -98,7 +98,6 @@ def setUp(self):
 
     def test_file_filter_watched_file(self):
         test_txt = self.temp_path / "test.txt"
-        test_txt.touch()
         self.reloader.watched_files_set = {test_txt}
 
         result = self.reloader.file_filter(Change.modified, str(test_txt))
@@ -107,7 +106,6 @@ def test_file_filter_watched_file(self):
 
     def test_file_filter_unwatched_file(self):
         test_txt = self.temp_path / "test.txt"
-        test_txt.touch()
 
         result = self.reloader.file_filter(Change.modified, str(test_txt))
 
@@ -156,10 +154,14 @@ def test_file_filter_glob_relative_path_impossible(self):
 
     def test_tick(self):
         test_txt = self.temp_path / "test.txt"
-        test_txt.touch()
-        self.reloader.watched_files_set = {test_txt}
+        self.reloader.extra_files = {test_txt}
+
+        iterator = self.reloader.tick()
+        result = next(iterator)
+        assert result is None
 
-        self.reloader.tick()
+        result = self.reloader.file_filter(Change.modified, str(test_txt))
+        assert result is True
 
 
 class ReplacedGetReloaderTests(SimpleTestCase):