Remove common prefixes and suffixes from actual and expected single-line strings

[staabm]

closes #116

[staabm]

//cc @Schrank

[staabm]

should be good to go

[staabm]

hopefully this will fix this build :)

[sebastianbergmann]

AFAICS, the build failed due to GitHub Actions' outage.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.00%. Comparing base (5ef523a) to head (ee79ea0).
Report is 7 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #117      +/-   ##
============================================
+ Coverage     98.91%   99.00%   +0.08%     
- Complexity      127      140      +13     
============================================
  Files            15       15              
  Lines           370      400      +30     
============================================
+ Hits            366      396      +30     
  Misses            4        4              

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

[staabm]

I saw errors regarding test executed code it should not cover

[staabm]

Grreeeeen :-)

[sebastianbergmann]

Looking at the changes, I only see removeOverlongCommonPrefix() and findCommonPrefix(), but not similar methods for suffixes. Are common suffixes handled or not? Sorry, cannot look at it more closely just now.

[staabm]

you are right. I totally forgot about the suffixes. added them now.