🚀 v3 Feature: Make app.Test accept a time.Duration timeout

[leonklingele]

Cases found by running semgrep with an empty .semgrepignore file

semgrep --lang=go -e '$O.Test($X, $Y)'

Description

Make app.Test accept a time.Duration timeout instead of an timeout specified in milliseconds.

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

Checklist:

• For new functionalities I follow the inspiration of the express js framework and built them similar in usage
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation - https://github.com/gofiber/docs for https://docs.gofiber.io/
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• If new dependencies exist, I have checked that they are really necessary and agreed with the maintainers/community (we want to have as few dependencies as possible)
• I tried to make my code as fast as possible with as few allocations as possible
• For new code I have written benchmarks so that they can be analyzed and improved

[ReneWerner87]

Thx

[leonklingele]

I see this not being part of v3 atm, even though this got merged. Why?!

[ReneWerner87]

I see this not being part of v3 atm, even though this got merged. Why?!

sorry, i will look why this is the case

[ReneWerner87]

unfortunately i have not found an explanation
normally beta-v3 is in main and main contains your commit, yet the code is not customized

I have updated the code
370cc8b