fix: substitute variable in include:file if it's array of files (#1579)

* fix: substitute variable in include:file if it's array of files

* Update .gitlab-ci-local-variables.yml

---------

Co-authored-by: Mads Jon Nielsen <[email protected]>

Author: rufoos

src/parser-includes.ts

@@ -219,7 +219,11 @@ export class ParserIncludes {
 
         for (const entry of include) {
             for (const [key, value] of Object.entries(entry)) {
-                entry[key] = Utils.expandText(value, variables);
+                if (Array.isArray(value)) {
+                    entry[key] = value.map((v) => Utils.expandText(v, variables));
+                } else {
+                    entry[key] = Utils.expandText(value, variables);
+                }
             }
         }
 

tests/test-cases/include-project-file-with-variable/.gitlab-ci-local-variables.yml

@@ -0,0 +1,2 @@
+---
+FILE_NAME: "test-file"

tests/test-cases/include-project-file-with-variable/.gitlab-ci.yml

@@ -0,0 +1,5 @@
+---
+include:
+  - project: test-group/gitlab-ci-local-test
+    file:
+      - $FILE_NAME.yml

tests/test-cases/include-project-file-with-variable/integration.test.ts

@@ -0,0 +1,45 @@
+import {WriteStreamsMock} from "../../../src/write-streams.js";
+import {handler} from "../../../src/handler.js";
+import chalk from "chalk";
+import {initBashSpy, initSpawnSpy} from "../../mocks/utils.mock.js";
+import {WhenStatics} from "../../mocks/when-statics.js";
+import fs from "fs-extra";
+
+beforeAll(() => {
+    initSpawnSpy(WhenStatics.all);
+});
+
+test("include-project-file-with-variable test-job", async () => {
+    const cwd = "tests/test-cases/include-project-file-with-variable";
+
+    await fs.rm(`${cwd}/.gitlab-ci-local/`, {recursive: true, force: true});
+
+    const spyGitRemote = {
+        cmdArgs: ["git", "remote", "get-url", "origin"],
+        returnValue: {stdout: "[email protected]:test-group/gitlab-ci-local-test.git"},
+    };
+    initSpawnSpy([...WhenStatics.all, spyGitRemote]);
+
+    const target = ".gitlab-ci-local/includes/gitlab.com/test-group/gitlab-ci-local-test/HEAD/";
+    const spyGitArchive1 = {
+        cmd: `git archive --remote=ssh://[email protected]:22/test-group/gitlab-ci-local-test.git HEAD test-file.yml | tar -f - -xC ${target}`,
+        returnValue: {output: ""},
+    };
+    initBashSpy([spyGitArchive1]);
+
+    const mock = `${cwd}/mock-test-file.yml`;
+    const mockTarget = `${cwd}/.gitlab-ci-local/includes/gitlab.com/test-group/gitlab-ci-local-test/HEAD/test-file.yml`;
+    await fs.ensureFile(mockTarget);
+    await fs.copyFile(mock, mockTarget);
+
+    const writeStreams = new WriteStreamsMock();
+    await handler({
+        cwd,
+        job: ["test-job"],
+    }, writeStreams);
+
+    const expected = [
+        chalk`{blueBright test-job} {greenBright >} Hello from test file`,
+    ];
+    expect(writeStreams.stdoutLines).toEqual(expect.arrayContaining(expected));
+});

tests/test-cases/include-project-file-with-variable/mock-test-file.yml

@@ -0,0 +1,5 @@
+---
+test-job:
+  stage: test
+  script:
+    - echo "Hello from test file"