Add DotCover Report

[bjorkstromm]

Once DotCover Cover command is implemented #722 , there will be use cases for Merge, Report and Delete. https://www.jetbrains.com/dotcover/help/dotCover__Console_Runner_Commands.html

[bjorkstromm]

I was thinking of starting implementing this (or at least Report command). Any objections @patriksvensson?

By adding DotCover Report, I can change my cake-script from:

Task("Run-Unit-Tests")
    .IsDependentOn("Build")
    .Does(() =>
{
    if(isRunningOnTeamCity)
    {
        DotCoverCover(tool => {
            tool.NUnit3("/path/to/*.Test.dll",
                new NUnit3Settings {
                    TeamCity = isRunningOnTeamCity
                });
            },
            coverageResult,
            new DotCoverCoverSettings()
                .WithFilter("+:Foo.*")
        );
    }
    else
    {
        DotCoverAnalyse(tool => {
            tool.NUnit3("/path/to/*.Test.dll",
                new NUnit3Settings {
                    TeamCity = isRunningOnTeamCity
                });
            },
            coverageResult,
            new DotCoverAnalyseSettings {
                ReportType = DotCoverReportType.HTML
            }.WithFilter("+:Foo.*")
        );
    }
})
.Finally(() =>
{
    if(isRunningOnTeamCity)
    {
        TeamCity.ImportDotCoverCoverage(coverageResult, MakeAbsolute(Directory("./tools/JetBrains.dotCover.CommandLineTools/tools")));
    }
});

To:

Task("Run-Unit-Tests")
    .IsDependentOn("Build")
    .Does(() =>
{
    DotCoverCover(tool => {
        tool.NUnit3("/path/to/*.Test.dll",
            new NUnit3Settings {
                TeamCity = isRunningOnTeamCity
            });
        },
        coverageResult,
        new DotCoverCoverSettings()
            .WithFilter("+:Foo.*")
    );
})
.Finally(() =>
{
    if(isRunningOnTeamCity)
    {
        TeamCity.ImportDotCoverCoverage(coverageResult, MakeAbsolute(Directory("./tools/JetBrains.dotCover.CommandLineTools/tools")));
    }
    else
    {
        DotCoverReport(coverageResult, "/path/to/output.html");
    }
});

And eliminate risk of running with different DotCoverSettings on TeamCity vs. Local

[gep13]

@mholo65 you mentioned in your last PR that this partially solves this issue. Is there still work that needs to be done in order to close out this issue?

[bjorkstromm]

@gep13 well, implement Delete and Merge... 😄

1. Currently, I don't see use case for Delete at least...
2. Merge might be useful in complex scenarios (if having multiple snapshot files that needs to be merged prior to running TeamCity import or DotCover Report).

[bjorkstromm]

@gep13 as I see it, we could rename this issue and create separate issue for Merge..? Thoughts?

[gep13]

@mholo65 sounds good to me.

I am just signing off for the night. Can I ask that you do that, and then close this issue when you are happy?

Thanks!

[bjorkstromm]

Implemented in #1153