fix(#739): Remove else statement on route mismatch

[marcelom97]

What type of PR is this? (check all applicable)

• [ ] Refactor
• [ ] Feature
• [X] Bug Fix
• [ ] Optimization
• [ ] Documentation Update
• [ ] Go Version Update
• [ ] Dependency Update

Description

When a path with specific query params does not match we should return a ErrMethodMismatch instead of ErrNotFound

Related Tickets & Documents

• Related Issue #739
• Closes #739

Added/updated tests?

• [X] Yes
• [ ] No, and this is why: please replace this line with details on why tests have not been included
• [ ] I need help with writing tests

Run verifications and test

• [X] make verify is passing
• [X] make test is passing

[jaitaiwan]

I've reviewed and I'm happy with the code and tests to match. What needs to be considered now is if this is technically a breaking api change or not as folks utilising the library may expect the former error. @AlexVulaj would love your input on this.

[jaitaiwan]

Actually I got confused - this PR (thanks to #748) is now obsolete. Thank you heaps for the contribution @marcelom97. If you feel that this was in error, please open up another issue to discuss.