Skip to content

refactor(tests): move all tests into isolated tests dir#15

Merged
nberlette merged 3 commits intofixes-and-testsfrom
refactor/isolate-tests
Jan 24, 2026
Merged

refactor(tests): move all tests into isolated tests dir#15
nberlette merged 3 commits intofixes-and-testsfrom
refactor/isolate-tests

Conversation

@nberlette
Copy link
Copy Markdown
Owner

  • refactor(tests): move all .test.ts files into an isolated tests dir
  • fix(tests): update import paths for all test files
  • chore: fmt

@nberlette nberlette added the enhancement New feature or request label Jan 24, 2026
Copilot AI review requested due to automatic review settings January 24, 2026 02:00
@nberlette nberlette merged commit ccc8711 into fixes-and-tests Jan 24, 2026
4 checks passed
@nberlette nberlette deleted the refactor/isolate-tests branch January 24, 2026 02:01
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Refactors the test suite to run from an isolated tests/ directory by updating test imports to reference implementation modules under src/.

Changes:

  • Updated test imports from local ./*.ts modules to ../src/** and ../../src/** paths.
  • Applied minor formatting changes (notably multi-line imports) as part of the refactor.

Reviewed changes

Copilot reviewed 89 out of 89 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/types/safe_integer.test.ts Update type import to src/types/safe_integer.ts.
tests/types/precision.test.ts Update type imports to src/types/precision.ts.
tests/types/integer.test.ts Update type imports to src/types/integer.ts.
tests/types/index.test.ts Update type imports to src/types/index.ts.
tests/types/float.test.ts Update type imports to src/types/float.ts (formatted as multi-line).
tests/types/finite.test.ts Update type import to src/types/finite.ts.
tests/trunc.test.ts Update imports to src/trunc.ts and src/internal/_test_utils.ts.
tests/trigonometry.test.ts Update imports to src/index.ts and src/internal/_test_utils.ts.
tests/tanh.test.ts Update imports to src/tanh.ts and src/internal/_test_utils.ts.
tests/tan.test.ts Update imports to src/tan.ts and src/internal/_test_utils.ts.
tests/sqrt.test.ts Update imports to src/sqrt.ts and src/internal/_test_utils.ts.
tests/sinh.test.ts Update imports to src/sinh.ts and src/internal/_test_utils.ts.
tests/sin.test.ts Update imports to src/sin.ts and src/internal/_test_utils.ts.
tests/sign.test.ts Update imports to src/sign.ts and src/internal/_test_utils.ts.
tests/round.test.ts Update imports to src/round.ts and src/internal/_test_utils.ts.
tests/random.test.ts Update imports to src/random.ts.
tests/pow.test.ts Update imports to src/pow.ts and src/internal/_test_utils.ts.
tests/min.test.ts Update imports to src/min.ts and src/internal/_test_utils.ts.
tests/max.test.ts Update imports to src/max.ts and src/internal/_test_utils.ts.
tests/log2.test.ts Update imports to src/log2.ts and src/internal/_test_utils.ts.
tests/log1p.test.ts Update imports to src/log1p.ts and src/internal/_test_utils.ts.
tests/log10.test.ts Update imports to src/log10.ts and src/internal/_test_utils.ts.
tests/log.test.ts Update imports to src/log.ts and src/internal/_test_utils.ts.
tests/index.test.ts Update imports to src/index.ts and src/internal/_test_utils.ts.
tests/imul.test.ts Update imports to src/imul.ts and src/internal/_test_utils.ts.
tests/ieee754.test.ts Update imports to src/ieee754.ts.
tests/hypot.test.ts Update imports to src/hypot.ts and src/internal/_test_utils.ts.
tests/guards/safe_integer.test.ts Update import to src/guards/safe_integer.ts.
tests/guards/positive_zero.test.ts Update import to src/guards/positive_zero.ts.
tests/guards/positive_infinity.test.ts Update import to src/guards/positive_infinity.ts.
tests/guards/negative_zero.test.ts Update import to src/guards/negative_zero.ts.
tests/guards/negative_infinity.test.ts Update import to src/guards/negative_infinity.ts.
tests/guards/nan.test.ts Update import to src/guards/nan.ts.
tests/guards/integer.test.ts Update import to src/guards/integer.ts.
tests/guards/infinity.test.ts Update imports to src/guards/infinity.ts (formatted as multi-line).
tests/guards/index.test.ts Update import to src/guards/index.ts.
tests/guards/finite.test.ts Update import to src/guards/finite.ts.
tests/fround.test.ts Update imports to src/fround.ts and src/internal/_test_utils.ts.
tests/floor.test.ts Update imports to src/floor.ts and src/internal/_test_utils.ts.
tests/float32/round.test.ts Update import to src/float32/round.ts.
tests/float32/index.test.ts Update import to src/float32/index.ts.
tests/float32/guards.test.ts Update imports to src/float32/guards.ts and src/float32/round.ts.
tests/float32/encode.test.ts Update import to src/float32/encode.ts.
tests/float32/decode.test.ts Update import to src/float32/decode.ts.
tests/float32/constants.test.ts Update imports to src/float32/constants.ts.
tests/float16/round.test.ts Update import to src/float16/round.ts.
tests/float16/index.test.ts Update import to src/float16/index.ts.
tests/float16/guards.test.ts Update imports to src/float16/guards.ts and src/float16/round.ts.
tests/float16/encode.test.ts Update import to src/float16/encode.ts.
tests/float16/decode.test.ts Update import to src/float16/decode.ts.
tests/float16/constants.test.ts Update imports to src/float16/constants.ts.
tests/f16round.test.ts Update imports to src/f16round.ts and src/internal/_test_utils.ts.
tests/expm1.test.ts Update imports to src/expm1.ts and src/internal/_test_utils.ts.
tests/exp.test.ts Update imports to src/exp.ts and src/internal/_test_utils.ts.
tests/cosh.test.ts Update imports to src/cosh.ts and src/internal/_test_utils.ts.
tests/cos.test.ts Update imports to src/cos.ts and src/internal/_test_utils.ts.
tests/constants/sqrt2.test.ts Update import to src/constants/sqrt2.ts.
tests/constants/sqrt1_2.test.ts Update import to src/constants/sqrt1_2.ts.
tests/constants/positive_zero.test.ts Update import to src/constants/positive_zero.ts.
tests/constants/positive_infinity.test.ts Update import to src/constants/positive_infinity.ts.
tests/constants/pi.test.ts Update import to src/constants/pi.ts.
tests/constants/negative_zero.test.ts Update import to src/constants/negative_zero.ts.
tests/constants/negative_infinity.test.ts Update import to src/constants/negative_infinity.ts.
tests/constants/nan.test.ts Update import to src/constants/nan.ts.
tests/constants/min_value.test.ts Update import to src/constants/min_value.ts.
tests/constants/min_safe_integer.test.ts Update import to src/constants/min_safe_integer.ts.
tests/constants/max_value.test.ts Update import to src/constants/max_value.ts.
tests/constants/max_safe_integer.test.ts Update import to src/constants/max_safe_integer.ts.
tests/constants/log2e.test.ts Update import to src/constants/log2e.ts.
tests/constants/log10e.test.ts Update import to src/constants/log10e.ts.
tests/constants/ln2.test.ts Update import to src/constants/ln2.ts.
tests/constants/ln10.test.ts Update import to src/constants/ln10.ts.
tests/constants/infinity.test.ts Update imports to src/constants/infinity.ts (formatted as multi-line).
tests/constants/index.test.ts Update import to src/constants/index.ts.
tests/constants/epsilon.test.ts Update import to src/constants/epsilon.ts.
tests/constants/e.test.ts Update import to src/constants/e.ts.
tests/constants.test.ts Update imports to src/constants/index.ts and src/internal/_test_utils.ts.
tests/clz32.test.ts Update imports to src/clz32.ts and src/internal/_test_utils.ts.
tests/clamp.test.ts Update import to src/clamp.ts.
tests/ceil.test.ts Update imports to src/ceil.ts and src/internal/_test_utils.ts.
tests/cbrt.test.ts Update imports to src/cbrt.ts and src/internal/_test_utils.ts.
tests/atanh.test.ts Update imports to src/atanh.ts and src/internal/_test_utils.ts.
tests/atan2.test.ts Update imports to src/atan2.ts, src/constants/pi.ts, src/guards/negative_zero.ts, and src/internal/_test_utils.ts.
tests/atan.test.ts Update imports to src/atan.ts and src/internal/_test_utils.ts.
tests/asinh.test.ts Update imports to src/asinh.ts and src/internal/_test_utils.ts.
tests/asin.test.ts Update imports to src/asin.ts and src/internal/_test_utils.ts.
tests/acosh.test.ts Update imports to src/acosh.ts and src/internal/_test_utils.ts.
tests/acos.test.ts Update imports to src/acos.ts and src/internal/_test_utils.ts.
tests/abs.test.ts Update imports to src/abs.ts and src/internal/_test_utils.ts.
Comments suppressed due to low confidence (38)

tests/asin.test.ts:4

  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/constants.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/expm1.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/f16round.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/min.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/sin.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/log2.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/atan2.test.ts:6
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/cos.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/index.test.ts:5
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/round.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/sqrt.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/max.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/acos.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/asinh.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e. g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/clz32.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/hypot.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/sign.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/tan.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/trigonometry.test.ts:2
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/atan.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/cbrt.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/floor.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/atanh.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/exp.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/tanh.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/abs.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/acosh.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/cosh.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/imul.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/log10.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/log1p.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/trunc.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/ceil.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/fround.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/log.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/pow.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.
    tests/sinh.test.ts:4
  • These tests now import assertion helpers from ../src/internal/_test_utils.ts. That module appears to be test-only (no usages in src/), yet it lives under src/ which is included in the published package (see deno.json publish include src/**/*.ts). This couples tests to an internal path and ships a Node-specific helper (node:assert) to consumers. Consider moving _test_utils.ts back under tests/ (e.g. tests/internal/_test_utils.ts) or excluding it from publish, and update imports accordingly.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants