test(alias): add unit tests for compileAliasOptions bucketing and onl…#628
test(alias): add unit tests for compileAliasOptions bucketing and onl…#628hey-amanthakur wants to merge 1 commit into
Conversation
…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
|
Codecov Report✅ All modified and coverable lines are covered by tests. 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
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Merging this PR will degrade performance by 75.54%
|
| 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)
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