feat: implement variable-substitution in trigger:include:local (#1494)

Author: ANGkeith

src/job.ts

@@ -1469,8 +1469,9 @@ export class Job {
 
         for (const include of this.jobData.trigger?.include ?? []) {
             if (include["local"]) {
-                validateIncludeLocal(include["local"]);
-                const files = await globby(include["local"].replace(/^\//, ""), {dot: true, cwd});
+                const expandedInclude = Utils.expandText(include["local"], this._variables);
+                validateIncludeLocal(expandedInclude);
+                const files = await globby(expandedInclude.replace(/^\//, ""), {dot: true, cwd});
                 if (files.length == 0) {
                     throw new AssertionError({message: `Local include file \`${include["local"]}\` specified in \`.${this.name}\` cannot be found!`});
                 }

tests/test-cases/parent-child/.gitlab-ci-8.yml

@@ -0,0 +1,7 @@
+---
+trigger:
+  variables:
+    PROJECT_DIR: "8"
+  trigger:
+    include:
+      - local: $PROJECT_DIR/.gitlab-ci.yml

tests/test-cases/parent-child/8/.gitlab-ci.yml

@@ -0,0 +1,5 @@
+---
+job:
+  image: busybox
+  script:
+    - echo yay variable substitution works

tests/test-cases/parent-child/integration.test.ts

@@ -115,3 +115,21 @@ test("2 dynamic pipeline with concurrency set to 1", async () => {
     ).join("\n");
     expect(filteredStdout).toEqual(expected);
 });
+
+test("trigger:include:local should support variable substitution", async () => {
+    const writeStreams = new WriteStreamsMock();
+    await handler({
+        cwd: cwd,
+        file: ".gitlab-ci-8.yml",
+        noColor: true,
+    }, writeStreams);
+
+    const expected = `	[trigger] -> job $ echo yay variable substitution works
+	[trigger] -> job > yay variable substitution works`;
+
+    const filteredStdout = writeStreams.stdoutLines.filter(f =>
+        f.startsWith("	[trigger] -> job >")
+            || f.startsWith("	[trigger] -> job $")
+    ).join("\n");
+    expect(filteredStdout).toEqual(expected);
+});