fix: missing trim() for name when options is an object (#197)

Author: jianyi-gronk

lib/Hook.js

@@ -66,11 +66,14 @@ class Hook {
 	_tap(type, options, fn) {
 		if (typeof options === "string") {
 			options = {
-				name: options.trim()
+				name: options
 			};
 		} else if (typeof options !== "object" || options === null) {
 			throw new Error("Invalid tap options");
 		}
+		if (typeof options.name === "string") {
+			options.name = options.name.trim();
+		}
 		if (typeof options.name !== "string" || options.name === "") {
 			throw new Error("Missing name for tap");
 		}

lib/__tests__/Hook.js

@@ -100,5 +100,11 @@ describe("Hook", () => {
 		expect(() => hook.tap(" ", () => {})).toThrow(
 			new Error("Missing name for tap")
 		);
+		expect(() => hook.tap({ name: "" }, () => {})).toThrow(
+			new Error("Missing name for tap")
+		);
+		expect(() => hook.tap({ name: " " }, () => {})).toThrow(
+			new Error("Missing name for tap")
+		);
 	});
 });