-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Include Integration tests in code coverage #147
Changes from 20 commits
2eb2837
dadc552
654af0d
76e59f9
14d5c07
931e96f
1a83983
dd5a391
92c262d
bafbf68
31e4a46
d17e817
7be9d82
968bb47
699b7ea
193990f
5e4cab7
01c1efe
a8d06ed
4923f01
f981b0d
92349c1
3c607e0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
<?xml version="1.0" encoding="utf-8"?> | ||
<RunSettings> | ||
<DataCollectionRunSettings> | ||
<DataCollectors> | ||
<DataCollector friendlyName="Code Coverage"> | ||
<Configuration> | ||
<SplitCoverage>True</SplitCoverage> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. what does SplitCoverage mean? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. SplitCoverage is so that we can see the Code Coverage tab in ADO. |
||
<CodeCoverage> | ||
<ModulePaths> | ||
<Include> | ||
<ModulePath>.*\Microsoft.Azure.WebJobs.Extensions.Sql.dll$</ModulePath> | ||
</Include> | ||
</ModulePaths> | ||
</CodeCoverage> | ||
</Configuration> | ||
</DataCollector> | ||
</DataCollectors> | ||
</DataCollectionRunSettings> | ||
</RunSettings> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are we really using this for the build? Or just for running the tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be for tests - the Microsoft.NET.Test.Sdk v17 did not work with .NET Core SDK v3.1