docs: finish up source_dirs. bump to 7.8.0

Author: nedbat

CHANGES.rst

@@ -23,6 +23,11 @@ upgrading your version of coverage.py.
 Unreleased
 ----------
 
+- Added a new ``source_dirs`` setting for symmetry with the existing
+  ``source_pkgs`` setting. It's preferable to the existing ``source`` setting,
+  because you'll get a clear error when directories don't exist. Fixes `issue
+  1942`_.  Thanks, `Jeremy Fleischman <pull 1943_>`_.
+
 - Fix: the PYTHONSAFEPATH environment variable new in Python 3.11 is properly
   supported, closing `issue 1696`_.  Thanks, `Philipp A. <pull 1700_>`_.  This
   works properly except for a detail when using the ``coverage`` command on
@@ -31,14 +36,8 @@ Unreleased
 
 .. _issue 1696: https://github.com/nedbat/coveragepy/issues/1696
 .. _pull 1700: https://github.com/nedbat/coveragepy/pull/1700
-
-- Added a new ``source_dirs`` setting for symmetry with the existing
-  ``source_pkgs`` setting. It's preferable to the existing ``source`` setting,
-  because you'll get a clear error when directories don't exist. Fixes `issue
-  1942`_.
-
 .. _issue 1942: https://github.com/nedbat/coveragepy/issues/1942
-
+.. _pull 1943: https://github.com/nedbat/coveragepy/pull/1943
 
 .. start-releases
 

CONTRIBUTORS.txt

@@ -110,6 +110,7 @@ James Valleroy
 Jan Kühle
 Jan Rusak
 Janakarajan Natarajan
+Jeremy Fleischman
 Jerin Peter George
 Jessamyn Smith
 Joanna Ejzel

coverage/control.py

@@ -240,7 +240,7 @@ def __init__(                       # pylint: disable=too-many-arguments
         .. versionadded:: 7.7
             The `plugins` parameter.
 
-        .. versionadded:: ???
+        .. versionadded:: 7.8
             The `source_dirs` parameter.
         """
         # Start self.config as a usable default configuration. It will soon be

coverage/inorout.py

@@ -196,11 +196,9 @@ def __init__(
         # Canonicalize everything in `source_dirs`.
         # Also confirm that they actually are directories.
         for i, src in enumerate(self.source_dirs):
-            self.source_dirs[i] = canonical_filename(src)
-
             if not os.path.isdir(src):
-                raise ConfigError(f"Source dir doesn't exist, or is not a directory: {src}")
-
+                raise ConfigError(f"Source dir is not a directory: {src!r}")
+            self.source_dirs[i] = canonical_filename(src)
 
         self.source_pkgs_unmatched = self.source_pkgs[:]
 

coverage/version.py

@@ -8,7 +8,7 @@
 
 # version_info: same semantics as sys.version_info.
 # _dev: the .devN suffix if any.
-version_info = (7, 7, 2, "alpha", 0)
+version_info = (7, 8, 0, "alpha", 0)
 _dev = 1
 
 

doc/config.rst

@@ -485,7 +485,7 @@ ambiguities between packages and directories.
 Operates the same as ``source``, but only names directories, for resolving
 ambiguities between packages and directories.
 
-.. versionadded:: ???
+.. versionadded:: 7.8
 
 
 .. _config_run_timid:

tests/test_api.py

@@ -975,7 +975,7 @@ def test_source_dirs(self) -> None:
     def test_non_existent_source_dir(self) -> None:
         with pytest.raises(
             ConfigError,
-            match=re.escape("Source dir doesn't exist, or is not a directory: i-do-not-exist"),
+            match=re.escape("Source dir is not a directory: 'i-do-not-exist'"),
         ):
             self.coverage_usepkgs_counts(source_dirs=["i-do-not-exist"])