chore: test refactor + change changelog entry (#226)

Author: alexander-akait

.changeset/perf-tap-registration.md

@@ -2,32 +2,4 @@
 "tapable": patch
 ---
 
-Perf: reduce allocations and work on the tap registration and compile paths.
-
-- `Hook#_tap` builds the final tap descriptor in a single allocation for the
-  common `hook.tap("name", fn)` string-options case instead of creating an
-  intermediate `{ name }` object that was then merged via `Object.assign`.
-- `Hook#_insert` takes an O(1) fast path for the common append case (no
-  `before`, and stage consistent with the last tap) - the previous
-  implementation always ran the shift loop once.
-- `Hook#_runRegisterInterceptors` early-returns when there are no
-  interceptors and uses an indexed loop instead of `for…of`.
-- `HookMap#for` inlines the `_map.get` lookup instead of routing through
-  `this.get(key)`, saving a method dispatch on a path hit once per hook
-  access in consumers like webpack.
-- `HookCodeFactory#setup` builds `_x` with a preallocated array + explicit
-  loop instead of `Array.prototype.map`.
-- `HookCodeFactory#init` uses `Array.prototype.slice` instead of spread to
-  skip the iterator protocol.
-- `HookCodeFactory#args` memoizes the common no-before/no-after result so
-  arguments are joined once per compile rather than once per tap.
-- `HookCodeFactory#needContext`, `callTapsSeries`, `callTapsParallel` and
-  `MultiHook`'s iteration use indexed loops with cached length, and the
-  series/parallel code hoists the per-tap `done`/`doneBreak` closures
-  out of the compile-time loop. Replaces `Array.prototype.findIndex`
-  with a local loop to avoid callback allocation.
-
-Registering 10 taps on a `SyncHook` is roughly 2× faster,
-`SyncHook: tap 5 + first call (compile)` is ~15% faster, and
-`HookMap#for (existing key)` is ~6% faster in the micro-benchmarks.
-The `.call()` path is unchanged.
+Improved performance in many places.

eslint.config.mjs

@@ -12,15 +12,15 @@ export default defineConfig([
 		}
 	},
 	{
-		files: ["lib/__tests__/**/*.js"],
+		files: ["test/**/*"],
 		languageOptions: {
 			parserOptions: {
 				ecmaVersion: 2018
 			}
 		}
 	},
 	{
-		files: ["benchmark/**/*.mjs"],
+		files: ["benchmark/**/*"],
 		languageOptions: {
 			parserOptions: {
 				ecmaVersion: 2022

package.json

@@ -20,7 +20,6 @@
   "types": "./tapable.d.ts",
   "files": [
     "lib",
-    "!lib/__tests__",
     "tapable.d.ts"
   ],
   "scripts": {
@@ -38,7 +37,7 @@
   },
   "jest": {
     "transform": {
-      "__tests__[\\\\/].+\\.js$": "babel-jest"
+      "test[\\\\/].+\\.js$": "babel-jest"
     },
     "snapshotFormat": {
       "escapeString": true,

lib/__tests__/AsyncParallelHooks.js test/AsyncParallelHooks.test.jslib/__tests__/AsyncParallelHooks.js renamed to test/AsyncParallelHooks.test.js

@@ -4,9 +4,9 @@
 */
 "use strict";
 
-const AsyncParallelBailHook = require("../AsyncParallelBailHook");
-const AsyncParallelHook = require("../AsyncParallelHook");
-const HookTester = require("./HookTester");
+const AsyncParallelBailHook = require("../lib/AsyncParallelBailHook");
+const AsyncParallelHook = require("../lib/AsyncParallelHook");
+const HookTester = require("./HookTester.test");
 
 describe("AsyncParallelHook", () => {
 	it("should have to correct behavior", async () => {

lib/__tests__/AsyncSeriesHooks.js test/AsyncSeriesHooks.test.jslib/__tests__/AsyncSeriesHooks.js renamed to test/AsyncSeriesHooks.test.js

@@ -4,11 +4,11 @@
 */
 "use strict";
 
-const AsyncSeriesBailHook = require("../AsyncSeriesBailHook");
-const AsyncSeriesHook = require("../AsyncSeriesHook");
-const AsyncSeriesLoopHook = require("../AsyncSeriesLoopHook");
-const AsyncSeriesWaterfallHook = require("../AsyncSeriesWaterfallHook");
-const HookTester = require("./HookTester");
+const AsyncSeriesBailHook = require("../lib/AsyncSeriesBailHook");
+const AsyncSeriesHook = require("../lib/AsyncSeriesHook");
+const AsyncSeriesLoopHook = require("../lib/AsyncSeriesLoopHook");
+const AsyncSeriesWaterfallHook = require("../lib/AsyncSeriesWaterfallHook");
+const HookTester = require("./HookTester.test");
 
 describe("AsyncSeriesHook", () => {
 	it("should not have call method", () => {

lib/__tests__/DefaultArgs.js test/DefaultArgs.test.jslib/__tests__/DefaultArgs.js renamed to test/DefaultArgs.test.js

@@ -3,16 +3,16 @@
 */
 "use strict";
 
-const AsyncParallelBailHook = require("../AsyncParallelBailHook");
-const AsyncParallelHook = require("../AsyncParallelHook");
-const AsyncSeriesBailHook = require("../AsyncSeriesBailHook");
-const AsyncSeriesHook = require("../AsyncSeriesHook");
-const AsyncSeriesLoopHook = require("../AsyncSeriesLoopHook");
-const AsyncSeriesWaterfallHook = require("../AsyncSeriesWaterfallHook");
-const SyncBailHook = require("../SyncBailHook");
-const SyncHook = require("../SyncHook");
-const SyncLoopHook = require("../SyncLoopHook");
-const SyncWaterfallHook = require("../SyncWaterfallHook");
+const AsyncParallelBailHook = require("../lib/AsyncParallelBailHook");
+const AsyncParallelHook = require("../lib/AsyncParallelHook");
+const AsyncSeriesBailHook = require("../lib/AsyncSeriesBailHook");
+const AsyncSeriesHook = require("../lib/AsyncSeriesHook");
+const AsyncSeriesLoopHook = require("../lib/AsyncSeriesLoopHook");
+const AsyncSeriesWaterfallHook = require("../lib/AsyncSeriesWaterfallHook");
+const SyncBailHook = require("../lib/SyncBailHook");
+const SyncHook = require("../lib/SyncHook");
+const SyncLoopHook = require("../lib/SyncLoopHook");
+const SyncWaterfallHook = require("../lib/SyncWaterfallHook");
 
 describe("Hooks without explicit args", () => {
 	it("should construct SyncHook without args", () => {

lib/__tests__/Hook.js test/Hook.test.jslib/__tests__/Hook.js renamed to test/Hook.test.js

@@ -4,12 +4,12 @@
 */
 "use strict";
 
-const Hook = require("../Hook");
-const SyncHook = require("../SyncHook");
+const HookTest = require("../lib/Hook");
+const SyncHook = require("../lib/SyncHook");
 
 describe("Hook", () => {
 	it("should throw when compile is not overridden", () => {
-		const hook = new Hook(["arg"]);
+		const hook = new HookTest(["arg"]);
 		expect(() =>
 			hook.compile({ taps: [], interceptors: [], args: [], type: "sync" })
 		).toThrow(/Abstract: should be overridden/);

lib/__tests__/HookCodeFactory.js test/HookCodeFactory.test.jslib/__tests__/HookCodeFactory.js renamed to test/HookCodeFactory.test.js

@@ -4,7 +4,7 @@
 */
 "use strict";
 
-const HookCodeFactory = require("../HookCodeFactory");
+const HookCodeFactoryTest = require("../lib/HookCodeFactory");
 
 const expectNoSyntaxError = (code) => {
 	// eslint-disable-next-line no-new
@@ -80,7 +80,7 @@ describe("HookCodeFactory", () => {
 				let factory;
 
 				beforeEach(() => {
-					factory = new HookCodeFactory();
+					factory = new HookCodeFactoryTest();
 					factory.init(factoryConfigurations[configurationName]);
 				});
 
@@ -233,7 +233,7 @@ describe("HookCodeFactory", () => {
 				let factory;
 
 				beforeEach(() => {
-					factory = new HookCodeFactory();
+					factory = new HookCodeFactoryTest();
 					factory.init(factoryConfigurations[configurationName]);
 				});
 

lib/__tests__/HookMap.js test/HookMap.test.jslib/__tests__/HookMap.js renamed to test/HookMap.test.js

@@ -3,18 +3,18 @@
 */
 "use strict";
 
-const HookMap = require("../HookMap");
-const SyncHook = require("../SyncHook");
+const HookMapTest = require("../lib/HookMap");
+const SyncHook = require("../lib/SyncHook");
 
 describe("HookMap", () => {
 	it("should return undefined from get when the key is unknown", () => {
-		const map = new HookMap(() => new SyncHook());
+		const map = new HookMapTest(() => new SyncHook());
 		expect(map.get("missing")).toBeUndefined();
 	});
 
 	it("should lazily create hooks through for(...) and cache them", () => {
 		const factory = jest.fn(() => new SyncHook(["a"]));
-		const map = new HookMap(factory, "myMap");
+		const map = new HookMapTest(factory, "myMap");
 
 		expect(map.name).toBe("myMap");
 
@@ -32,7 +32,7 @@ describe("HookMap", () => {
 	});
 
 	it("should apply interceptor factories when creating hooks", () => {
-		const map = new HookMap(() => new SyncHook());
+		const map = new HookMapTest(() => new SyncHook());
 		const wrapped = new SyncHook();
 
 		map.intercept({
@@ -47,7 +47,7 @@ describe("HookMap", () => {
 	});
 
 	it("should default the interceptor factory to pass-through", () => {
-		const map = new HookMap(() => new SyncHook());
+		const map = new HookMapTest(() => new SyncHook());
 		map.intercept({});
 		const hook = map.for("bar");
 		expect(hook).toBeDefined();
@@ -56,15 +56,15 @@ describe("HookMap", () => {
 
 	it("should forward deprecated tap helpers to the underlying hook", () => {
 		const warn = jest.spyOn(console, "warn").mockImplementation(() => {});
-		const map = new HookMap(() => new SyncHook(["a"]));
+		const map = new HookMapTest(() => new SyncHook(["a"]));
 
 		const syncMock = jest.fn();
 		map.tap("k", "plugin-sync", syncMock);
 		map.for("k").call(1);
 		expect(syncMock).toHaveBeenCalledWith(1);
 
-		const asyncMap = new HookMap(
-			() => new (require("../AsyncSeriesHook"))(["a"])
+		const asyncMap = new HookMapTest(
+			() => new (require("../lib/AsyncSeriesHook"))(["a"])
 		);
 		const asyncMock = jest.fn((_a, cb) => cb());
 		asyncMap.tapAsync("k", "plugin-async", asyncMock);
@@ -73,8 +73,8 @@ describe("HookMap", () => {
 			asyncMap.for("k").callAsync(2, () => {
 				expect(asyncMock).toHaveBeenCalled();
 
-				const promiseMap = new HookMap(
-					() => new (require("../AsyncSeriesHook"))(["a"])
+				const promiseMap = new HookMapTest(
+					() => new (require("../lib/AsyncSeriesHook"))(["a"])
 				);
 				const promiseMock = jest.fn(() => Promise.resolve());
 				promiseMap.tapPromise("k", "plugin-promise", promiseMock);

lib/__tests__/HookStackOverflow.js test/HookStackOverflow.test.jslib/__tests__/HookStackOverflow.js renamed to test/HookStackOverflow.test.js

@@ -1,6 +1,6 @@
 "use strict";
 
-const AsyncSeriesHook = require("../AsyncSeriesHook");
+const AsyncSeriesHook = require("../lib/AsyncSeriesHook");
 
 describe("HookStackOverflow", () => {
 	it("should not crash when compiling a large hook", () => {