Thanks for you contributions! As you can see from the broken tests this function is used in a lot of places, and reverting to an old behavior is not a valid solution...
Which is the exact behavior that you didn't expect? If you have an example would be great 🙂

@fain182

Reverting the changes and leaving the test as is, we can see that this assertion fails:

        $pattern = new PatternString('SoThisIsAnExample');
        $this->assertFalse($pattern->matches('*This'));

However, it shouldn't fail because it shouldn't match since there is no wildcard at the end of the string.

@fain182
I removed a duplicate assertion and added a few more to make sure all scenarios are covered.

@fain182 I believe I fixed the issues, can you check the last commits, please?

Codecov Report

Merging #404 (7f6ae0d) into main (976c200) will decrease coverage by 0.02%.
Report is 4 commits behind head on main.
The diff coverage is 100.00%.

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

@@             Coverage Diff              @@
##               main     #404       /-   ##
============================================
- Coverage     94.33%   94.31%   -0.02%     
  Complexity      571      570       -1     
============================================
  Files            67       67              
  Lines          1500     1495       -5     
============================================
- Hits           1415     1410       -5     
  Misses           85       85