[FIX] _find_manifest_path when no manifest found

This happens if the test module is less than 5 directories
deep from root.

Author: petrus-v

pytest_odoo.py

@@ -233,10 +233,12 @@ def _find_manifest_path(collection_path: Path) -> Path:
     """Try to locate an Odoo manifest file in the collection path."""
     # check if collection_path is an addon directory
     path = collection_path
-    level = 0
-    while level < 5  and not (path.parent / "__manifest__.py").is_file():
+    for _ in range(0, 5): 
+        if (path.parent / "__manifest__.py").is_file():
+            break
         path = path.parent
-        level += 1
+    else:
+        return None
     return path.parent / "__manifest__.py"
 
 

tests/__init__.py

@@ -0,0 +1,39 @@
+from unittest import TestCase
+import tempfile
+from contextlib import contextmanager
+from pytest_odoo import _find_manifest_path
+from pathlib import Path
+
+class TestPytestOdoo(TestCase):
+
+    @contextmanager
+    def fake_module(self):
+        directory = tempfile.TemporaryDirectory()
+        try:
+            module_path = Path(directory.name)
+            manifest_path = module_path / "__manifest__.py"
+            manifest_path.touch()
+            test_path = module_path / "tests" / "test_module.py"
+            test_path.parent.mkdir(parents=True, exist_ok=True)
+            test_path.touch()
+            yield (module_path, manifest_path, test_path,)
+        finally:
+            directory.cleanup()
+
+
+    def test_find_manifest_path_less_than_5_directories(self):
+        self.assertIsNone(_find_manifest_path(Path("/some/path")))
+
+    def test_find_manifest_path_from_test_module(self):
+        with self.fake_module() as (_, manifest_path, test_path):
+            self.assertEqual(_find_manifest_path(test_path), manifest_path)
+
+    def test_find_manifest_path_from_itself(self):
+        with self.fake_module() as (_, manifest_path, _):
+            self.assertEqual(_find_manifest_path(manifest_path), manifest_path)
+
+    def test_find_manifest_path_from_brother(self):
+        with self.fake_module() as (module_path, manifest_path, _):
+            test = module_path / "test_something.py"
+            test.touch()
+            self.assertEqual(_find_manifest_path(test), manifest_path)