Skip to content

test(alias): add unit tests for compileAliasOptions bucketing and onl…#628

Open
hey-amanthakur wants to merge 1 commit into
webpack:mainfrom
hey-amanthakur:test/alias-utils-coverage
Open

test(alias): add unit tests for compileAliasOptions bucketing and onl…#628
hey-amanthakur wants to merge 1 commit into
webpack:mainfrom
hey-amanthakur:test/alias-utils-coverage

Conversation

@hey-amanthakur

Copy link
Copy Markdown
  • compileAliasOptions: verify bucket construction by first char code, useBuckets logic (disabled for single-char or empty-prefix wildcard), wildcard prefix/suffix splitting, absolutePath, and empty input

  • onlyModule flag: verify relative paths and path-prefixed requests are skipped when onlyModule is true, and that both are aliased when false

…yModule flag

- compileAliasOptions: verify bucket construction by first char code,
  useBuckets logic (disabled for single-char or empty-prefix wildcard),
  wildcard prefix/suffix splitting, absolutePath, and empty input
- onlyModule flag: verify relative paths and path-prefixed requests are
  skipped when onlyModule is true, and that both are aliased when false
@changeset-bot

changeset-bot Bot commented Jul 10, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 3583cab

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@codecov

codecov Bot commented Jul 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.11%. Comparing base (82026bf) to head (3583cab).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #628   +/-   ##
=======================================
  Coverage   98.11%   98.11%           
=======================================
  Files          49       49           
  Lines        9867     9867           
=======================================
  Hits         9681     9681           
  Misses        186      186           
Flag Coverage Δ
integration 98.11% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codspeed-hq

codspeed-hq Bot commented Jul 20, 2026

Copy link
Copy Markdown

Merging this PR will degrade performance by 75.54%

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 3 regressed benchmarks
✅ 139 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Memory extensions-many: 6-extension list (warm) 1.9 KB 96.6 KB -98.05%
Simulation concurrent-batch: Promise.all of 15 resolves (warm) 1.4 ms 1.6 ms -14.01%
Simulation realistic-midsize: mixed batch (cold cache) 6.9 ms 7.8 ms -12.59%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.


Comparing jhonsnow456:test/alias-utils-coverage (3583cab) with main (82026bf)

Open in CodSpeed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant