SIGN-7469 Add unit test

Author: Taha-cmd

actions/submit-signing-request/helper-artifact-download.ts

@@ -67,7 +67,7 @@ export class HelperArtifactDownload {
         core.info(`The signed artifact has been successfully downloaded from SignPath and extracted to ${targetDirectory}`);
     }
 
-    private resolveOrCreateDirectory(directoryPath:string): string {
+    public resolveOrCreateDirectory(directoryPath:string): string {
         const workingDirectory =  process.env.GITHUB_WORKSPACE as string;
         const absolutePath = path.isAbsolute(directoryPath) ? directoryPath :
             path.join(workingDirectory as string, directoryPath);

actions/submit-signing-request/tests/helper-artifact-download.test.ts

@@ -0,0 +1,51 @@
+import { assert, expect } from "chai";
+import { HelperArtifactDownload } from "../helper-artifact-download"
+import * as path from 'path';
+import * as os from 'os';
+import * as uuid from 'uuid';
+import * as fs from 'fs'
+import { HelperInputOutput } from "../helper-input-output";
+
+describe("Artifact path resolving tests", () => {
+    before(() => { process.env.GITHUB_WORKSPACE = __dirname; });
+    after(() => { process.env.GITHUB_WORKSPACE = undefined; });
+
+    [".", "..", "fake", "fake/path"].forEach(relativePathBase => {
+        it("Should resolve relative paths to absolute paths", () => {
+            const sut = new HelperArtifactDownload(new HelperInputOutput());
+            const dirName = uuid.v4();
+            const relativePath = path.join(relativePathBase, dirName);
+            const expectedAbsolutePath = path.join(__dirname, relativePath);
+
+            let actualResolvedPath: fs.PathLike = "";
+            try {
+                // ACT
+                actualResolvedPath = sut.resolveOrCreateDirectory(relativePath);
+
+                // ASSERT
+                assert.equal(actualResolvedPath, expectedAbsolutePath);
+            }
+            finally {
+                fs.rmSync(actualResolvedPath, { force: true, recursive: true  })
+            }
+        })
+    })
+
+    it("Should detect and recognize absolute paths", () => {
+        const sut = new HelperArtifactDownload(new HelperInputOutput());
+        const dirName = uuid.v4();
+        const expectedAbsolutePath = path.join(__dirname, dirName);
+
+        let actualResolvedPath: fs.PathLike = "";
+        try {
+            // ACT
+            actualResolvedPath = sut.resolveOrCreateDirectory(expectedAbsolutePath);
+
+            // ASSERT
+            assert.equal(actualResolvedPath, expectedAbsolutePath);
+        }
+        finally {
+            fs.rmSync(actualResolvedPath, { force: true, recursive: true  })
+        }
+    })
+})

package-lock.json

@@ -17,6 +17,7 @@
     "@types/node": "^18.7.23",
     "@types/q": "^1.5.5",
     "@types/sinon": "^10.0.13",
+    "@types/uuid": "^10.0.0",
     "@vercel/ncc": "^0.36.1",
     "chai": "^4.3.7",
     "minimist": "^1.2.6",