Merge branch 'main' into pre-commit-ci-update-config

Author: tobiasraabe

.github/pull_request_template.md

@@ -6,4 +6,4 @@ Provide a description and/or bullet points to describe the changes in this PR.
 
 - [ ] Reference issues which can be closed due to this PR with "Closes #x".
 - [ ] Review whether the documentation needs to be updated.
-- [ ] Document PR in docs/changes.rst.
+- [ ] Document PR in docs/source/changes.md.

docs/source/changes.md

@@ -10,6 +10,9 @@ releases are available on [PyPI](https://pypi.org/project/pytask) and
 - {pull}`617` fixes an interaction with provisional nodes and `@mark.persist`.
 - {pull}`618` ensures that `root_dir` of `DirectoryNode` is created before the task is
   executed.
+- {pull}`619` makes coiled an optional import for tests. Thanks to {user}`erooke`.
+- {pull}`620` makes tests more flexible about their location. Thanks to {user}`erooke`.
+- {pull}`621` fixes the pull requests template.
 
 ## 0.5.0 - 2024-05-26
 

tests/test_console.py

@@ -153,7 +153,7 @@ def test_format_task_id(
             PathNode.from_path(_ROOT.joinpath("src/module.py")),
             [_ROOT.joinpath("alternative_src")],
             does_not_raise(),
-            Text("pytask/src/module.py"),
+            Text(f"{_ROOT.name}/src/module.py"),
             id="Common path found for PathNode not in 'paths' and 'paths'",
         ),
         pytest.param(
@@ -174,7 +174,7 @@ def test_format_task_id(
             PythonNode(name=_ROOT.as_posix() + "/task_a.py::task_a::a", value=None),
             [_ROOT],
             does_not_raise(),
-            Text("pytask/task_a.py::task_a::a"),
+            Text(f"{_ROOT.name}/task_a.py::task_a::a"),
             id="PythonNode with automatically assigned name",
         ),
     ],

tests/test_shared.py

@@ -4,7 +4,6 @@
 import textwrap
 from contextlib import ExitStack as does_not_raise  # noqa: N813
 
-import coiled
 import pytest
 from _pytask.shared import convert_to_enum
 from _pytask.shared import find_duplicates
@@ -76,5 +75,13 @@ def wrapper():
     decorated = decorator(task)
     assert unwrap_task_function(decorated) is task
 
+
+@pytest.mark.unit()
+def test_no_unwrap_coiled():
+    coiled = pytest.importorskip("coiled")
+
+    def task():
+        pass
+
     coiled_decorated = coiled.function()(task)
     assert unwrap_task_function(coiled_decorated) is coiled_decorated