♻️ refactor: Rename benchmark cases

[alexandear]

Description

Currently, the Benchmark Markdown table is broken for the Benchmark_Path_matchParams and Benchmark_RoutePatternMatch benchmarks (e.g., see https://github.com/gofiber/fiber/actions/runs/26518416331):

It's because of the | used in the test names that is interpreted as the table border in the Markdown. This PR replaces | with _ to fix that.

Changes introduced

List the new features or adjustments introduced in this pull request. Provide details on benchmarks, documentation updates, changelog entries, and if applicable, the migration guide.

• Benchmarks: Describe any performance benchmarks and improvements related to the changes.
• Documentation Update: Detail the updates made to the documentation and links to the changed files.
• Changelog/What's New: Include a summary of the additions for the upcoming release notes.
• Migration Guide: If necessary, provide a guide or steps for users to migrate their existing code to accommodate these changes.
• API Alignment with Express: Explain how the changes align with the Express API.
• API Longevity: Discuss the steps taken to ensure that the new or updated APIs are consistent and not prone to breaking changes.
• Examples: Provide examples demonstrating the new features or changes in action.

Type of change

Please delete options that are not relevant.

• New feature (non-breaking change which adds functionality)
• Enhancement (improvement to existing features and functionality)
• Documentation update (changes to documentation)
• Performance improvement (non-breaking change which improves efficiency)
• Code consistency (non-breaking change which improves code reliability and robustness)

Checklist

• Followed the inspiration of the Express.js framework for new functionalities, making them similar in usage.
• Conducted a self-review of the code and provided comments for complex or critical parts.
• Updated the documentation in the /docs/ directory for Fiber's documentation.
• Added or updated unit tests to validate the effectiveness of the changes or new features.
• Ensured that new and existing unit tests pass locally with the changes.
• Verified that any new dependencies are essential and have been agreed upon by the maintainers/community.
• Aimed for optimal performance with minimal allocations in the new code.
• Provided benchmarks for the new code to analyze and improve upon.

[coderabbitai]

Walkthrough

Two benchmark subtest names in path_test.go are updated to use underscore separators instead of pipe separators. The changes affect Benchmark_Path_matchParams and Benchmark_RoutePatternMatch naming conventions only.

Changes

Benchmark subtest naming

Layer / File(s)	Summary
Benchmark subtest name formatting path_test.go	Subtest names in Benchmark_Path_matchParams (line 358) and Benchmark_RoutePatternMatch (line 391) updated from pipe-delimited (`"

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

🧹 Updates, v3, codex

Suggested reviewers

• sixcolors
• efectn
• ReneWerner87

Poem

🐰 A dash becomes a line so clean,
Underscore flow, a benchmark's sheen,
Two tests rename with gentle ease,
Path-matching names now sure to please! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The PR title 'refactor: Rename benchmark cases' is partially related but misleading—the actual purpose is fixing broken Markdown table rendering by replacing pipe characters with underscores, not general refactoring.	Consider a more accurate title like 'test: rename benchmark cases to fix Markdown table' or 'test: replace pipe with underscore in benchmark names to fix Markdown rendering'.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description provides clear context about the problem (broken Markdown table due to pipe characters), explains the solution (replacing pipe with underscore), and completes the required template sections appropriately.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[gemini-code-assist]

Code Review

This pull request updates the benchmark subtest names in path_test.go by replacing spaces and pipe characters with underscores, which improves compatibility with test runners and command-line tools. There are no review comments, and I have no additional feedback to provide.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.34%. Comparing base (f0752ce) to head (26b8bd0).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4383      +/-   ##
==========================================
- Coverage   91.38%   91.34%   -0.05%     
==========================================
  Files         132      132              
  Lines       13108    13108              
==========================================
- Hits        11979    11973       -6     
- Misses        711      716       +5     
- Partials      418      419       +1     

Flag	Coverage Δ
unittests	91.34% <ø> (-0.05%)	⬇️

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

☔ View full report in Codecov by Sentry.
📢 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.

[ReneWerner87]

thx