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

[marcelom97]

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

• Refactor
• Feature
• 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 [BUG] MethodNotAllowedHandler does not work for subrouter with different routes #739
• Closes [BUG] MethodNotAllowedHandler does not work for subrouter with different routes #739

Added/updated tests?

• 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

• make verify is passing
• 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.