Merge pull request #42 from peterjc/patch-1

Resolve relative paths when adding sources, so that these are recognized.

src/slipcover/importer.py

@@ -64,9 +64,7 @@ def __init__(self):
     def addSource(self, source : Path):
         if isinstance(source, str):
             source = Path(source)
-        if not source.is_absolute():
-            source = self.cwd / source
-        self.sources.append(source)
+        self.sources.append(source.resolve())
 
     def addOmit(self, omit):
         if not omit.startswith('*'):

tests/importer_test.py

@@ -62,6 +62,7 @@ def test_filematcher_defaults_from_root(return_to_dir):
     site_packages = next(Path(p) for p in sys.path if p != '' and (Path(p) / "pip").exists())
     assert not fm.matches(site_packages / 'foo.py')
 
+
 def test_filematcher_source():
     from pathlib import Path
     cwd = str(Path.cwd())
@@ -94,6 +95,17 @@ def test_filematcher_source():
     assert not fm.matches(site_packages / 'foo.py')
 
 
+def test_filematcher_source_resolved(monkeypatch):
+    from pathlib import Path
+    monkeypatch.chdir('tests')
+
+    fm = im.FileMatcher()
+    fm.addSource('../src/')
+
+    p = (Path.cwd() / '..' / 'src' / 'foo.py').resolve()
+    assert fm.matches(p)
+
+
 def test_filematcher_omit_pattern():
     from pathlib import Path
     cwd = str(Path.cwd())