feat: allow to remap directories

felipecrs · Aug 29, 2023 · f59d6da · f59d6da
1 parent c33860a
commit f59d6da
Show file tree

Hide file tree

Showing 3 changed files with 77 additions and 3 deletions.
diff --git a/src/command.ts b/src/command.ts
@@ -102,17 +102,46 @@ export class MainCommand extends Command {
               filter: (file) => {
                 if (binary.files) {
                   return Boolean(
-                    binary.files.some((f) => f.source === file.path),
+                    binary.files.some((f) => {
+                      if (f.source === file.path) {
+                        return true;
+                      }
+                      // Compare by path. For example, if source is shellcheck/ and
+                      // file is shellcheck/LICENSE.txt, we want to return true.
+                      if (
+                        f.source.endsWith("/") &&
+                        file.path.startsWith(f.source)
+                      ) {
+                        return true;
+                      }
+                    }),
                   );
                 }
 
                 return true;
               },
               map: (file) => {
                 if (binary.files) {
-                  const f = binary.files.find((f) => f.source === file.path);
+                  let remapDirectory = false;
+                  const f = binary.files.find((f) => {
+                    if (f.source === file.path) {
+                      return true;
+                    }
+                    // Compare by path. For example, if source is shellcheck/ and
+                    // target is directory/, and file is shellcheck/LICENSE.txt, we
+                    // want to map it to directory/LICENSE.txt.
+                    if (
+                      f.source.endsWith("/") &&
+                      file.path.startsWith(f.source)
+                    ) {
+                      remapDirectory = true;
+                      return true;
+                    }
+                  });
                   if (f) {
-                    file.path = f.target;
+                    file.path = remapDirectory
+                      ? file.path.replace(f.source, f.target)
+                      : f.target;
                   }
                 }
 

diff --git a/test/index.test.ts b/test/index.test.ts
@@ -57,6 +57,19 @@ describe("bindl", () => {
     ).toBeTruthy();
   });
 
+  it("downloads shellcheck remapping directory", async () => {
+    const result = await execa.command(
+      `${bin} --config ${configDirectory}/remap-directory.bindl.config.cjs`,
+    );
+    expect(result.stdout).toContain(`downloading and extracting`);
+    expect(result.exitCode).toBe(0);
+
+    expect(
+      shell.test("-f", "./binaries/linux/x64/directory/shellcheck"),
+    ).toBeTruthy();
+    expect(shell.test("-f", "./binaries/linux/x64/shellcheck")).toBeTruthy();
+  });
+
   it("downloads only linux-arm64 binary when npm_config_arch is set", async () => {
     const result = await execa.command(`${bin} --config ${configPath}`, {
       // eslint-disable-next-line camelcase

diff --git a/test/res/remap-directory.bindl.config.cjs b/test/res/remap-directory.bindl.config.cjs
@@ -0,0 +1,32 @@
+/* eslint-env node */
+
+const version = "v0.8.0";
+const shellcheckReleaseUrl = `https://github.com/koalaman/shellcheck/releases/download/${version}/shellcheck-${version}`;
+
+module.exports = {
+  decompressPlugins: ["@felipecrs/decompress-tarxz"],
+  binaries: [
+    {
+      platform: "linux",
+      arch: "x64",
+      url: `${shellcheckReleaseUrl}.linux.x86_64.tar.xz`,
+      files: [
+        {
+          source: `shellcheck-${version}/`,
+          target: "directory/",
+        },
+      ],
+    },
+    {
+      platform: "linux",
+      arch: "x64",
+      url: `${shellcheckReleaseUrl}.linux.x86_64.tar.xz`,
+      files: [
+        {
+          source: `shellcheck-${version}/`,
+          target: "",
+        },
+      ],
+    },
+  ],
+};