-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Improve code coverage for Microsoft.VisualBasic.dll (8.9%) #19615
Comments
The only reason there aren't exactly zero is because if there were zero, it wouldn't show up in the code coverage report at all, so I added one test to help highlight how bad we are here ;) |
@stephentoub Well, that's pretty bad. ;) I could take this one. Is there an existing test codebase somewhere I could use as a starting point or do I have to start from scratch (I mean, from the one test you added :))? |
Thanks, @dennisdietrich. That's a nice offer. @VSadov, do you know if there are existing tests for this internally somewhere? |
Maybe from whatever repo @JeremyKuhne found the serial port tests recently. |
Sure thing. I'll hold of with coding until we get a response regarding the (non)exists of test code for the assembly but please feel free to assign this issue to me. |
I will take a look if there is something salvageable. |
@VSadov any luck? |
Info on code coverage work:
|
#26744) * Added tests for Microsoft.VisualBasic.CompilerServices.Conversions #14344 * added tests for ToBoolean fro Object
…tnet#1… (dotnet#26744) * Added tests for Microsoft.VisualBasic.CompilerServices.Conversions #14344 * added tests for ToBoolean fro Object
I'm planing to contribute with this issue, but for some reason the there is no coverage (0 classes) for this assembly (actually Microsoft.VisualBasic.Core.dll). Any help in how to make code coverage show for this assembly? |
@danmosemsft yes, I tried that. I also tried test with coverage on the entire repository, the report showed for all the other assemblies, except the VisualBasic one. |
@ViktorHofer is VB supposed to work? I would expect the tools to not be aware of the language used. |
Seems to be a coverlet bug:
|
I'm OOF for a week I'll take a look asap @ViktorHofer please can you attach command line/project folder to speed up repro? |
Submitted coverlet-coverage/coverlet#495 to fix the crash. |
@ehasis that's the current code coverage status for Microsoft.VisualBasic.Core:
HTML report: |
Very nice, thanks @ViktorHofer ! |
Merged thanks Viktor |
Submitted dotnet/corefx#39728 to consume the new version. |
Closing. If we identify specific types in Microsoft.VisualBasic.Core that require additional test coverage, we can address those cases with specific issues. |
There are almost zero tests so there is essentially zero code coverage.
The text was updated successfully, but these errors were encountered: