Allow valid targets alongside empty directories

Author: nightcityblade

mypy/find_sources.py

@@ -38,6 +38,7 @@ def create_source_list(
     finder = SourceFinder(fscache, options)
 
     sources = []
+    empty_dirs = []
     for path in paths:
         path = os.path.normpath(path)
         if path.endswith(PY_EXTENSIONS):
@@ -47,11 +48,13 @@ def create_source_list(
         elif fscache.isdir(path):
             sub_sources = finder.find_sources_in_dir(path)
             if not sub_sources and not allow_empty_dir:
-                raise InvalidSourceList(f"There are no .py[i] files in directory '{path}'")
+                empty_dirs.append(path)
             sources.extend(sub_sources)
         else:
             mod = os.path.basename(path) if options.scripts_are_modules else None
             sources.append(BuildSource(path, mod, None))
+    if not sources and empty_dirs:
+        raise InvalidSourceList(f"There are no .py[i] files in directory '{empty_dirs[0]}'")
     return sources
 
 

mypy/test/test_find_sources.py

@@ -6,7 +6,6 @@
 import unittest
 
 import pytest
-
 from mypy.find_sources import InvalidSourceList, SourceFinder, create_source_list
 from mypy.fscache import FileSystemCache
 from mypy.modulefinder import BuildSource
@@ -261,6 +260,14 @@ def test_find_sources_in_dir_namespace_multi_dir(self) -> None:
         finder = SourceFinder(FakeFSCache({"/a/pkg/a.py", "/b/pkg/b.py"}), options)
         assert find_sources_in_dir(finder, "/") == [("pkg.a", "/a"), ("pkg.b", "/b")]
 
+    def test_find_sources_with_empty_directory(self) -> None:
+        options = Options()
+        fscache = FakeFSCache({"/empty/README", "/src/a.py"})
+
+        assert find_sources(["/empty", "/src"], options, fscache) == [("a", "/src")]
+        with pytest.raises(InvalidSourceList, match=r"directory '.*[\\/]empty'"):
+            find_sources(["/empty"], options, fscache)
+
     def test_find_sources_exclude(self) -> None:
         options = Options()
         options.namespace_packages = True