Fix: do not fail when arguments are empty (fixes: #4)

Author: JPeer264

index.js

@@ -1,6 +1,6 @@
 import isGit from "is-git-repository";
 import process from "node:process";
-import { execSync } from "child_process";
+import { execSync } from "node:child_process";
 
 const cwd = process.cwd();
 const defaultOptions = {
@@ -9,9 +9,13 @@ const defaultOptions = {
 };
 
 const sanitize = (input) => {
-  if (!Array.isArray(input)) return input.replace(/[^a-zA-Z0-9-_]/g, "");
+  if (Array.isArray(input)) return input.map(sanitize).join(" ").trim();
 
-  return input.map(sanitize).join(" ");
+  if (typeof input !== "string") {
+    return "";
+  }
+
+  return input?.replace(/[^a-zA-Z0-9-_]/g, "");
 };
 
 const branchName = (options = defaultOptions) => {

index.spec.js

@@ -4,13 +4,24 @@ import path from "node:path";
 import process from "node:process";
 
 import branchName from "./index";
-import { afterAll, beforeAll, describe, expect, it } from "vitest";
+import {
+  afterAll,
+  beforeAll,
+  beforeEach,
+  describe,
+  expect,
+  it,
+  vi,
+} from "vitest";
+import { execSync } from "node:child_process";
 
 const cwd = process.cwd();
 const fixtures = path.join(cwd, "test", "fixtures");
 
 const folders = ["feat_test", "master"];
 
+vi.mock("node:child_process", { spy: true });
+
 describe("branchName", () => {
   beforeAll(() => {
     folders.map((folder) =>
@@ -21,6 +32,10 @@ describe("branchName", () => {
     );
   });
 
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
   afterAll(() => {
     folders.map((folder) =>
       fs.renameSync(
@@ -30,6 +45,25 @@ describe("branchName", () => {
     );
   });
 
+  it("check the default", () => {
+    branchName();
+
+    expect(execSync).toHaveBeenCalledWith("git branch", { cwd });
+  });
+
+  it("check if values are properly ignored", () => {
+    branchName({ cwd: path.join(fixtures, "master"), branchOptions: [] });
+    branchName({ branchOptions: [null, 0, "--no-color"] });
+
+    expect(execSync).toHaveBeenCalledTimes(2);
+    expect(execSync).toHaveBeenNthCalledWith(1, "git branch", {
+      cwd: path.join(fixtures, "master"),
+    });
+    expect(execSync).toHaveBeenNthCalledWith(2, "git branch --no-color", {
+      cwd,
+    });
+  });
+
   it("check if the given directory is the branch master", () => {
     expect(
       branchName({
@@ -53,4 +87,8 @@ describe("branchName", () => {
       false,
     );
   });
+
+  it("check any non existing dir", () => {
+    expect(branchName({ cwd: "ʕっ•ᴥ•ʔっ" })).toBe(false);
+  });
 });