Skip to content
This repository has been archived by the owner on Dec 5, 2022. It is now read-only.

Document known CC issues #160

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mayankbansal018
Copy link
Contributor

No description provided.

@@ -0,0 +1,17 @@
# Syntax
```vstest.console.exe --collect:"code coverage"``` or ```dotnet test --collec:"code coverage"```
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Collect*

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also the name of the file. codecoverage*


# Output

A coverage file, which contains detailed report on what all code was hit when unit tests were run for specific(s) project. The generated file can be viewed in Visual Studio Enterprise, for a detailed analysis.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

project(s)* instead.


* We have often observed failures/crashes when tests are run with Code Coverage enabled on Windows 8.1(&Lower), or Windows Server 2012(& Lower), failing with ```Access Violation in covrun32.dll```

For such issues we recommened users to use coverage configuration specified at https://github.com/Microsoft/vstest/blob/master/src/DataCollectors/TraceDataCollector/VanguardCollector/DefaultCodeCoverageConfig.xml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Recommend*

@@ -0,0 +1,17 @@
# Syntax
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about /enablecodecoverage or via settings?
Make it clear by adding one liner that this doc is related to the specified syntax


* Coverage file fails to open in VS E.g. https://github.com/Microsoft/vstest/issues/1814

One of the possible reason for this is when there are too many blocks in a single function. Currently we support ```65535(USHRT_MAX)``` blocks in a single method, beyond that we go into serialization issue while reading this data from coverage file.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the recommendation here?
Also, I see some folks reported the Shimdll issue, or related to verifiable, should we cover that as well ?

Copy link
Contributor

@pvlakshm pvlakshm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What prompted the creation of this doc? I think I may be missing some context here, hence I ask.
Is this already an exhaustive list of all known issues?
BTW, there is a spelling mistake in the name of the .md file.

@singhsarab
Copy link
Contributor

@mayankbansal018 Are planning to take this anytime soon ?

Copy link
Member

@PBoraMSFT PBoraMSFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feedback same as Sarab's.

@AbhitejJohn
Copy link
Contributor

Tagging @nohwnd , @jakubch1 on this one. Perhaps this would fit in better in a troubleshooting doc. What do you guys think?

@nohwnd
Copy link
Member

nohwnd commented Mar 25, 2020

@AbhitejJohn probably yes. Will let @jakubch1 decide what to do with this next.

@nohwnd nohwnd changed the base branch from master to main May 13, 2021 11:49
@Haplois Haplois changed the title Document know CC issues Document known CC issues Jun 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants