test: split for readability and use mkdtemp & os.tmpdir

Author: fraxken

test/functions/download.spec.ts

@@ -0,0 +1,49 @@
+// Import Node.js Dependencies
+import path from "node:path";
+import fs from "node:fs";
+import assert from "node:assert";
+import { describe, test } from "node:test";
+
+// Import Third-party Dependencies
+import is from "@slimio/is";
+
+// Import Internal Dependency
+import * as gitlab from "../../src/index.js";
+
+describe("download", () => {
+  test("should be an asyncFunction", () => {
+    assert.ok(is.func(gitlab.download));
+    assert.ok(is.asyncFunction(gitlab.download));
+  });
+
+  test("must throw: repository must be a string!", async() => {
+    await assert.rejects(
+      async() => await gitlab.download(10 as any),
+      {
+        name: "TypeError",
+        message: "repository must be a string!"
+      }
+    );
+  });
+
+  test("extract tar.gz at in the current working dir", async() => {
+    try {
+      const { location, ...resultRest } = await gitlab.download("polychromatic.plombier-chauffagiste");
+
+      fs.accessSync(location);
+
+      assert.strictEqual(path.extname(location), ".gz");
+      assert.deepEqual(resultRest, {
+        branch: "master",
+        organization: "polychromatic",
+        repository: "plombier-chauffagiste"
+      });
+    }
+    finally {
+      const tarGzLocation = path.join(process.cwd(), "plombier-chauffagiste-master.tar.gz");
+      if (fs.existsSync(tarGzLocation)) {
+        fs.unlinkSync(tarGzLocation);
+      }
+    }
+  });
+});

test/functions/downloadAndExtract.spec.ts

@@ -0,0 +1,60 @@
+// Import Node.js Dependencies
+import os from "node:os";
+import path from "node:path";
+import fs from "node:fs";
+import assert from "node:assert";
+import { describe, test, it, beforeEach, afterEach } from "node:test";
+
+// Import Third-party Dependencies
+import is from "@slimio/is";
+
+// Import Internal Dependency
+import * as gitlab from "../../src/index.js";
+
+describe("downloadAndExtract", () => {
+  let tempDownloadDir: string;
+
+  beforeEach(() => {
+    tempDownloadDir = fs.mkdtempSync(
+      path.join(os.tmpdir(), "nsecure-gitlab-")
+    );
+  });
+
+  afterEach(() => {
+    fs.rmSync(tempDownloadDir, { recursive: true, force: true });
+  });
+
+  test("gitlab.downloadAndExtract should be an asyncFunction", () => {
+    assert.ok(is.func(gitlab.downloadAndExtract));
+    assert.ok(is.asyncFunction(gitlab.downloadAndExtract));
+  });
+
+  it("should download and extract the given gitlab repository (and not delete the tar.gz archive)", async() => {
+    const { location } = await gitlab.downloadAndExtract("polychromatic.plombier-chauffagiste", {
+      dest: tempDownloadDir,
+      removeArchive: false
+    });
+
+    assert.ok(
+      fs.existsSync(path.join(tempDownloadDir, "plombier-chauffagiste-master.tar.gz"))
+    );
+    assert.doesNotThrow(
+      () => fs.accessSync(location)
+    );
+  });
+
+  it("should download and extract the given gitlab repository (and delete the tar.gz archive)", async() => {
+    const { location } = await gitlab.downloadAndExtract("polychromatic.plombier-chauffagiste", {
+      dest: tempDownloadDir,
+      removeArchive: true
+    });
+
+    assert.strictEqual(
+      fs.existsSync(path.join(tempDownloadDir, "plombier-chauffagiste-master.tar.gz")),
+      false
+    );
+    assert.doesNotThrow(
+      () => fs.accessSync(location)
+    );
+  });
+});

test/test.spec.ts

@@ -1,14 +1,16 @@
 // Import Node.js Dependencies
-import { test } from "node:test";
+import { describe, test } from "node:test";
 import assert from "node:assert";
 
 // Import Internal Dependency
 import { getRepositoryPath } from "../src/utils.js";
 
-test("getRepositoryPath must return id", () => {
-  assert.equal(getRepositoryPath("10"), "10");
-});
+describe("getRepositoryPath", () => {
+  test("must return id", () => {
+    assert.strictEqual(getRepositoryPath("10"), "10");
+  });
 
-test("getRepositoryPath must return gitlab path", () => {
-  assert.equal(getRepositoryPath("nodesecure.boo"), "nodesecure%2Fboo");
+  test("must return GitLab path (encoded for URL)", () => {
+    assert.strictEqual(getRepositoryPath("nodesecure.boo"), "nodesecure%2Fboo");
+  });
 });