Fix bug to allow ref tasks to pass arguments again

poethepoet/task/ref.py

@@ -49,11 +49,12 @@ def _validate_task_def(
         return a message describing the first encountered issue if any.
         """
         task_ref = task_def["ref"]
+        task_name_ref = shlex.split(task_ref)[0]
 
-        if shlex.split(task_ref)[0] not in config.tasks:
-            return f"Task {task_name!r} contains reference to unkown task {task_ref!r}"
+        if task_name_ref not in config.tasks:
+            return f"Task {task_name!r} contains reference to unkown task {task_name_ref!r}"
 
-        referenced_task = config.tasks[task_ref]
+        referenced_task = config.tasks[task_name_ref]
         if isinstance(referenced_task, dict) and referenced_task.get("use_exec"):
             return (
                 f"Invalid task: {task_name!r}. contains illegal reference to task with "

pyproject.toml

@@ -91,7 +91,6 @@ poethepoet = "poethepoet.plugin:PoetryPlugin"
   help   = "Execute poe from this repo (useful for testing)"
   script = "poethepoet:main"
 
-
 [tool.coverage.report]
 omit = ["**/site-packages/**", "poethepoet/completion/*", "poethepoet/plugin.py"]
 

tests/fixtures/sequences_project/pyproject.toml

@@ -24,8 +24,14 @@ travel = [
   { script = "my_package:print_var('DEST')" }
 ]
 
+
+[tool.poe.tasks.say]
+cmd  = "poe_test_echo ${thing}"
+args = ["thing"]
+
 [tool.poe.tasks.multiple-value-arg]
-sequence = [{ cmd = "poe_test_echo first: ${first}" }, { cmd = " poe_test_echo second: ${second}" }]
+sequence = [{ cmd = "poe_test_echo first: ${first}" }, { cmd = " poe_test_echo second: ${second}" }, { ref = "say --thing ${thing}" }]
+env      = {thing = "Done."}
 
   [[tool.poe.tasks.multiple-value-arg.args]]
   name       = "first"
@@ -36,3 +42,5 @@ sequence = [{ cmd = "poe_test_echo first: ${first}" }, { cmd = " poe_test_echo s
   positional = true
   multiple   = true
   type       = "integer"
+
+

tests/test_sequence_tasks.py

@@ -52,7 +52,7 @@ def test_sequence_task_with_multiple_value_arg(run_poe_subproc):
     )
     assert (
         result.capture
-        == "Poe => poe_test_echo first: hey\nPoe => poe_test_echo second: 1 2 3\n"
+        == "Poe => poe_test_echo first: hey\nPoe => poe_test_echo second: 1 2 3\nPoe => poe_test_echo Done.\n"
     )
-    assert result.stdout == "first: hey\nsecond: 1 2 3\n"
+    assert result.stdout == "first: hey\nsecond: 1 2 3\nDone.\n"
     assert result.stderr == ""