[rel/5.x.x] Implement Cobertura coverage format#2572
Conversation
* wip: Implement Cobertura coverage format * fix unit test * revert editor settings change * remove ?? operator * fix attribute ordering * make coverage report test work on all platforms * Fix windows paths * fix unit test for Windows paths * kick the build * re-implement Cobertura coverage report generation * fix compatibility issues * fix tests * removing Cobertura from v4 parameter options * fix compatibility with ReportGenerator * Update src/functions/Coverage.ps1 Co-authored-by: Frode Flaten <3436158+fflaten@users.noreply.github.com> * fix whitespace * fix output * fix windows paths * order packages,classes,methods by name * change Cobertura DTD to loose * Tune coverage report for performance * Remove outdated condition * Add Cobertura DTD file * Apply suggestions from code review Co-authored-by: Jakub Jareš <me@jakubjares.com> * Fix typo and update JaCoCo starttime * Fix tests * Use epoch time for Cobertura and JaCoCo * Update test --------- Co-authored-by: Frode Flaten <3436158+fflaten@users.noreply.github.com> Co-authored-by: Jakub Jareš <me@jakubjares.com>
|
regarding your comments on the source PR @fflaten why do you say this one will need to use a different implementation? the api seems to exist in .NET Framework 462: https://learn.microsoft.com/en-us/dotnet/api/system.datetimeoffset.tounixtimemilliseconds?view=netframework-4.6.2 which is the oldest one we support. the comments: |
* Ignore calls to base class constructor in code coverage * Update tests
|
Too tired to deal with the build errors right now. |
Pester v5 targets .NET 4.5.2. |
| function MyClass | ||
| { | ||
| 'I am the constructor.' | ||
| MyBaseClass # calls the base constructor |
There was a problem hiding this comment.
This was originally MyBaseClass; "I am the constructor" but that adds one more BP than needed, removed the string so we simulate the call to base constructor like the new class base code does it.
3 participants
Backport #2298 to 5.x.x
Backport #2553 as well