Rename is_valid_*_source() to validate_*_source()

Co-authored-by: Maria Knorps <[email protected]>

Author: jherland

fawltydeps/extract_declared_dependencies.py

@@ -368,7 +368,7 @@ def parse_sources(
         yield from parse_source(source, parser_choice=parser_choice)
 
 
-def is_valid_deps_source(
+def validate_deps_source(
     path: Path,
     parser_choice: Optional[ParserChoice] = None,
     filter_by_parser: bool = False,

fawltydeps/extract_imports.py

@@ -209,7 +209,7 @@ def parse_sources(
         yield from parse_source(source, stdin)
 
 
-def is_valid_code_source(
+def validate_code_source(
     path: PathOrSpecial, base_dir: Optional[Path] = None
 ) -> Optional[CodeSource]:
     """Check if the given file path is a valid source for parsing imports.

fawltydeps/traverse_project.py

@@ -3,8 +3,8 @@
 from pathlib import Path
 from typing import AbstractSet, Dict, Iterator, Optional, Set, Type
 
-from fawltydeps.extract_declared_dependencies import is_valid_deps_source
-from fawltydeps.extract_imports import is_valid_code_source
+from fawltydeps.extract_declared_dependencies import validate_deps_source
+from fawltydeps.extract_imports import validate_code_source
 from fawltydeps.settings import Settings
 from fawltydeps.types import CodeSource, DepsSource, Source, UnparseablePathException
 from fawltydeps.utils import walk_dir
@@ -26,7 +26,7 @@ def find_sources(
     dirs_to_traverse: Dict[Path, Set[Type[Source]]] = {}
 
     for path in settings.code if CodeSource in source_types else []:
-        validated: Optional[Source] = is_valid_code_source(path)  # propagate exceptions
+        validated: Optional[Source] = validate_code_source(path)  # propagate exceptions
         if validated is not None:  # parse-able file given directly
             assert isinstance(validated, CodeSource)  # sanity check
             yield validated
@@ -35,7 +35,7 @@ def find_sources(
             dirs_to_traverse.setdefault(path, set()).add(CodeSource)
 
     for path in settings.deps if DepsSource in source_types else []:
-        validated = is_valid_deps_source(
+        validated = validate_deps_source(
             path, settings.deps_parser_choice, filter_by_parser=False
         )  # propagate exceptions
         if validated is not None:  # parse-able file given directly
@@ -49,14 +49,14 @@ def find_sources(
         for file in walk_dir(dir_path):  # traverse directories only _once_
             if CodeSource in types:
                 try:  # catch all exceptions while traversing dirs
-                    validated = is_valid_code_source(file, dir_path)
+                    validated = validate_code_source(file, dir_path)
                     assert isinstance(validated, CodeSource)  # sanity check
                     yield validated
                 except UnparseablePathException:  # don't abort directory walk for this
                     pass
             if DepsSource in types:
                 try:  # catch all exceptions while traversing dirs
-                    validated = is_valid_deps_source(
+                    validated = validate_deps_source(
                         file, settings.deps_parser_choice, filter_by_parser=True
                     )
                     assert isinstance(validated, DepsSource)  # sanity check