-
|
Just a thought - is the test bootstrapper app still needed, or can it be removed now that everything is done via Github actions? |
Beta Was this translation helpful? Give feedback.
Replies: 1 comment 1 reply
-
|
The only reason for keeping it right now was in case it would be needed to get better coverage output, but I actually think I just found what causes the code coverage diff.... The tests are using random data: SharpZipLib/test/ICSharpCode.SharpZipLib.Tests/BZip2/Bzip2Tests.cs Lines 25 to 27 in 232d690 ...which means that it's random whether all code paths are followed... 😅 |
Beta Was this translation helpful? Give feedback.
-
|
Should be fixed in #636 and I see no reason to keep the bootstrapper anymore. |
Beta Was this translation helpful? Give feedback.
The only reason for keeping it right now was in case it would be needed to get better coverage output, but I actually think I just found what causes the code coverage diff....
The tests are using random data:
SharpZipLib/test/ICSharpCode.SharpZipLib.Tests/BZip2/Bzip2Tests.cs
Lines 25 to 27 in 232d690
...which means that it's random whether all code paths are followed... 😅
https://en.wikipedia.org/wiki/Bzip2#Run-length_encoding_of_MTF_result