test: fix mismatched test filenames across sibling packages#12991
Draft
Planeshifter wants to merge 1 commit into
Draft
test: fix mismatched test filenames across sibling packages#12991Planeshifter wants to merge 1 commit into
Planeshifter wants to merge 1 commit into
Conversation
Propagates fix from 8ca8ea2 ("test: fix filename") to sibling packages where the test filename does not match the lib module the test requires. In each site, the test's `require( './../lib/<X>.js' )` already points at the correct lib file; only the local test filename was misspelled or left over from a copy-pasted sibling. Each fix is a pure rename. Affected packages: - `math/base/special/polygamma`: `test.atinifnityplus.js` -> `test.atinfinityplus.js` - `blas/ext/base/csumkbn`: `test.zsumkbn.js` -> `test.csumkbn.js`, `test.zsumkbn.native.js` -> `test.csumkbn.native.js` - `blas/ext/base/dlast-index-of`: `test.dindex_of.native.js` -> `test.dlast_index_of.native.js` - `blas/ext/base/slast-index-of`: `test.sindex_of.native.js` -> `test.slast_index_of.native.js`
Contributor
Coverage ReportNo coverage information available. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Propagating fixes merged to
developbetween 2026-06-20 10:57 UTC and 2026-06-20 13:03 UTC to sibling packages.test:filename mismatches8ca8ea2b4renamedstats/base/dists/invgamma/logpdf/test/test.nativs.jstotest.native.js. The same defect — a test filename that does not match thelib/module the test requires, left behind by a typo or a copy-paste from a sibling package — survives in four other packages. Each fix is a pure rename; the offending tests already require the correct lib path internally.Source commit:
8ca8ea2b4(test: fix filename).Target packages:
math/base/special/polygamma:test.atinifnityplus.js→test.atinfinityplus.jsblas/ext/base/csumkbn:test.zsumkbn.js→test.csumkbn.jsandtest.zsumkbn.native.js→test.csumkbn.native.js(copy-paste leftover from the siblingzsumkbnpackage)blas/ext/base/dlast-index-of:test.dindex_of.native.js→test.dlast_index_of.native.jsblas/ext/base/slast-index-of:test.sindex_of.native.js→test.slast_index_of.native.jsRelated Issues
None.
Questions
No.
Other
Validation
What was checked:
require( './../lib/<X>.js' )already resolves to the correctly named lib module that exists on disk, so the rename merely aligns the filename to what the test was always testing.opusvalidation passes confirmed the defect at every site, that the test body genuinely exercises the lib function under the corrected name (last-index-of semantic, Complex64 vs Complex128 typing, polygammaatinfinityplusmath), and that nopackage.jsonscript, Makefile, README, fixture loader, or other build artifact references the misspelled filename anywhere in the repo.opusadaptation pass confirmed every site is a puregit mv— nopackage.jsonscript edits, no Makefile updates, no cross-package changes needed.sonnetstyle-consistency pass confirmed the proposed new filenames match the universaltest.<X>.js↔lib/<X>.jsconvention used acrossmath/base/specialandblas/ext/base(snake-case lib names retain snake-case in the matching test filename).What was deliberately excluded:
math/base/special/powf/test/test.x_is_zero.jstestinglib/x_is_zerof.js: the lib file'sfsuffix is a naming-convention choice rather than a typo, and a maintainer should pick between renaming the test vs. renaming the lib file. Logged asneeds-human.array/to-fancyand_tools/**where the test filename intentionally diverges from the lib (logical grouping with.separators,asyncvssynccomplement tests, etc.).docs: update related packages sections,docs: update namespace table of contents) — these are produced bystdlib-botfrom package metadata, so any propagation belongs in the generator.The full per-site audit log, including the dropped false positives, is recorded locally for future review.
Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This pull request was generated by a scheduled Claude Code routine that audits the prior 24 hours of merges to
developfor fix patterns and propagates the equivalent fix to sibling packages exhibiting the same defect. Two independentopuscorrectness passes, oneopusadaptation pass, and onesonnetstyle-consistency pass ran in parallel against the candidate sites; only unanimously approved propagations were committed. A human will audit before promoting the PR out of draft.@stdlib-js/reviewers
Generated by Claude Code