Bump ty from 0.0.14 to 0.0.17 (#794)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Tobias Raabe <raabe@posteo.de>

Author: dependabot[bot]

src/_pytask/coiled_utils.py

@@ -3,31 +3,51 @@
 from dataclasses import dataclass
 from typing import TYPE_CHECKING
 from typing import Any
+from typing import cast
 
 if TYPE_CHECKING:
     from collections.abc import Callable
+    from typing import Protocol
 
 try:
     from coiled.function import Function
 except ImportError:
 
     @dataclass
     class Function:
-        cluster_kwargs: dict[str, Any]
-        environ: dict[str, Any]
+        _cluster_kwargs: dict[str, Any]
+        _environ: dict[str, Any] | None
+        _local: bool
+        _name: str
         function: Callable[..., Any] | None
-        keepalive: str | None
+        keepalive: Any | None
 
 
 __all__ = ["Function"]
 
 
+if TYPE_CHECKING:
+
+    class _CoiledFunctionPrivateAttrs(Protocol):
+        _cluster_kwargs: dict[str, Any]
+        _environ: dict[str, Any] | None
+        _local: bool
+        _name: str
+        keepalive: Any | None
+
+
+def _as_coiled_private_attrs(func: Function) -> _CoiledFunctionPrivateAttrs:
+    """Cast to the private-attribute layout used by coiled's Function class."""
+    return cast("_CoiledFunctionPrivateAttrs", func)
+
+
 def extract_coiled_function_kwargs(func: Function) -> dict[str, Any]:
     """Extract the kwargs for a coiled function."""
+    coiled_function = _as_coiled_private_attrs(func)
     return {
-        "cluster_kwargs": func._cluster_kwargs,  # ty: ignore[possibly-missing-attribute]
-        "keepalive": func.keepalive,
-        "environ": func._environ,  # ty: ignore[possibly-missing-attribute]
-        "local": func._local,  # ty: ignore[possibly-missing-attribute]
-        "name": func._name,  # ty: ignore[possibly-missing-attribute]
+        "cluster_kwargs": coiled_function._cluster_kwargs,
+        "keepalive": coiled_function.keepalive,
+        "environ": coiled_function._environ,
+        "local": coiled_function._local,
+        "name": coiled_function._name,
     }

src/_pytask/path.py

@@ -331,10 +331,10 @@ def _import_module_using_spec(
             break
     else:
         spec = importlib.util.spec_from_file_location(module_name, str(module_path))
-    if spec is not None:
+    if spec is not None and spec.loader is not None:
         mod = importlib.util.module_from_spec(spec)
         sys.modules[module_name] = mod
-        spec.loader.exec_module(mod)  # ty: ignore[possibly-missing-attribute]
+        spec.loader.exec_module(mod)
         _insert_missing_modules(sys.modules, module_name)
         return mod
 

src/_pytask/task_utils.py

@@ -186,7 +186,7 @@ def wrapper(func: T) -> TaskDecorated[T]:
 
     # When decorator is used without parentheses, call wrapper directly.
     if is_task_function(name) and kwargs is None:
-        return wrapper(cast("T", name))
+        return wrapper(cast("T", name))  # ty: ignore[invalid-argument-type]
     return wrapper
 
 

tests/test_collect.py

@@ -10,6 +10,7 @@
 
 from _pytask.collect import _find_shortest_uniquely_identifiable_name_for_tasks
 from _pytask.collect import pytask_collect_node
+from _pytask.node_protocols import PPathNode
 from pytask import CollectionOutcome
 from pytask import ExitCode
 from pytask import NodeInfo
@@ -237,6 +238,7 @@ def test_pytask_collect_node_does_not_raise_error_if_path_is_not_normalized(
         )
         assert not record
 
+    assert isinstance(result, PPathNode)
     assert str(result.path) == str(real_node)
 
 

tests/test_dag_command.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import importlib
 import os
 import subprocess
 import sys
@@ -12,7 +13,7 @@
 from pytask import cli
 
 try:
-    import pygraphviz  # type: ignore[unresolved-import]  # noqa: F401
+    importlib.import_module("pygraphviz")
 except ImportError:  # pragma: no cover
     _IS_PYGRAPHVIZ_INSTALLED = False
 else: