Add sub-delim path tests

[kriswuollett]

Excluded * since it is known to not be supported yet in #23.

[ibraheemdev]

I'm not sure what the point of these tests are? matchit doesn't do any percent encoding at all, it just looks for literal matches. : and * are the only exception because they are used to mark parameters.

[kriswuollett]

I'm not sure what the point of these tests are? matchit doesn't do any percent encoding at all, it just looks for literal matches. : and * are the only exception because they are used to mark parameters.

It adds test coverage to prove that matchit covers sub-delims, i.e. doesn't panic nor return error, as defined in RFC 3986 except for the exceptions that you mention. Currently the only likely other error-prone in URLs character that was covered was ? in:

matchit/tests/tree.rs

Line 285 in ab4cee7

"/???" :: "/*bar" => { "bar" => "???" },

Since there were multiple /[a-z]+ examples in basic, I thought it would be reasonable to propose making tests to cover all of the defined sub-delims.

It was done in an effort to see if a workaround was possible for #23.

Thanks. It is good to know that matchit doesn't do percent encoding, and so that would be the responsibility of its users.

[ibraheemdev]

Alright, that seems reasonable. Thanks!