<div dir="ltr">Source: node-on-finished<br>Version: 2.4.1+~1.1.1-2<br>Severity: minor<br>Tags: patch<br><br>Dear Maintainer,<br><br>Please consider removing node-should <!nocheck> from Build-Depends.<br><br>The main tests and the embedded ee-first tests use the Node.js built-in assert module. They do not import should.<br><br>I tested the original source and a version with only this Build-Depends<br>entry removed in two independent clean local Debian sid Docker containers<br>(amd64), using the same freshly updated signed APT indexes and the same<br>128-package base. Build dependencies were installed from each variant's<br>control file; no tests were disabled and no dependencies were added by hand.<br><br>Both builds succeeded, with 8+44 tests passing in each build. The patched<br>build did not have node-should installed. Installed package counts were<br>267 and 266; the omitted packages were node-should. No new packages<br>appeared and the versions/architectures of common packages were unchanged.<br>The generated node-on-finished_2.4.1+~1.1.1-2_all.deb files were byte-for-byte identical:<br>SHA256 84b53765e7d961ec7584def80ab90af05839ce0f79cff80f2ff3c5963da6aa19<br><br>The patch changes only Build-Depends; binary Depends are unchanged. This<br>validation covers the default profiles on amd64, one build per variant;<br>I did not run a separate autopkgtest or test other architectures.<br><br>I checked the current and archived source-package BTS reports and open<br>Salsa merge requests on 2026-11-12 and did not find a duplicate.<br><br>Attached are the patch, both complete build logs and a verification summary.<br><br>Regards,<br>Zixi Liu</div>