fix: combining 'end' and 'strict'

[skirtles-code]

This Playground illustrates the problem:

• Playground

The bottom row is the one that isn't working correctly. This combination of options:

{
  path: '/user/',
  strict: true,
  end: false,
  component: ...
}

should match the location path /user/1, but it doesn't.

The reason why is the RegExp is checking for the / twice:

^/user/(?:/|$)

---

While I've added a lot of extra testing for end: false, only two of those assertions would have failed with the previous code. Specifically, this one of line 666:

matchParams('/home/', '/home/other', {}, options)

and this one on line 673:

matchParams('/home/:p/', '/home/a/b', { p: 'a' }, options)

In both cases they would previously have failed to resolve the route.

[netlify]

✅ Deploy Preview for vue-router canceled.

Name	Link
🔨 Latest commit	efc026e
🔍 Latest deploy log	https://app.netlify.com/sites/vue-router/deploys/65e99c58b9ed150008af83fe

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.90%. Comparing base (13303bd) to head (4fce5c0).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2154   +/-   ##
=======================================
  Coverage   90.90%   90.90%           
=======================================
  Files          24       24           
  Lines        1121     1121           
  Branches      347      347           
=======================================
  Hits         1019     1019           
  Misses         63       63           
  Partials       39       39           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[posva]

I'm still trying to figure out this one. I have never been a fan of the end option. I may not have included enough tests on how this should work. I will have to check them again