From d8c3603224f20831ac7d67c0a9e945d8857eadbc Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Thu, 19 Mar 2026 08:31:16 +0000 Subject: [PATCH] =?UTF-8?q?=F0=9F=A7=B9=20Remove=20unused=20't'=20paramete?= =?UTF-8?q?r=20in=20test=20callbacks?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 🎯 **What:** Removed the unused `t` (test context) parameter from test callbacks in `tests/test.js`. 💡 **Why:** The test runner `node:test` passes a context object `t`, but since the tests use the native `node:assert` module directly, the parameter was unnecessary and cluttered the code. ✅ **Verification:** Verified syntax with `node --check tests/test.js` and ran `npm test` to ensure no regressions were introduced. ✨ **Result:** Improved code readability and maintainability by removing dead code. Co-authored-by: shenald-dev <245350826+shenald-dev@users.noreply.github.com> --- tests/test.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/test.js b/tests/test.js index 374a6cc..0ace078 100644 --- a/tests/test.js +++ b/tests/test.js @@ -1,13 +1,13 @@ const { test } = require('node:test'); const assert = require('node:assert'); -test('module loads', async (t) => { +test('module loads', async () => { const mod = require('../src/index.js'); assert.ok(mod, 'Module should be defined'); }); -test('main function exists', (t) => { +test('main function exists', () => { const mod = require('../src/index.js'); assert.ok(typeof mod.main === 'function' || typeof mod.runBenchmark === 'function', 'Should have main/benchmark function');