psycofdj / coverxygen

Generate doxygen's documentation coverage report
GNU General Public License v3.0
49 stars 11 forks source link

c# support #32

Open noname77 opened 1 year ago

noname77 commented 1 year ago

Hi,

This project looks great, thanks!

I tried it with doxygen output for a c# project and run into some issues where the coverage is not reflected correctly.

I was wondering if this is meant to be supported and if you could point me in the right direction?

Unfortunately I cannot share the project and didn't get around to creating a minimum reproducible example, but the two issues im seeing at first glance are:

EDIT: this is same as https://github.com/psycofdj/coverxygen/issues/15

Screenshot 2023-10-31 at 13 50 24

Screenshot 2023-10-31 at 13 51 54

I'd be happy to share more debug output, create a minimum example or test stuff or submit a PR if this project scope would include c# support.

Best, wiktor

noname77 commented 1 year ago

I decided to give it a go and the property issue turned out to be a simple fix, just added the property kind, see https://github.com/noname77/coverxygen/commit/3aa40653b548f25af19fb7bb4132493ced723c80

$ python3 -m coverxygen --xml-dir doxygen/doxyxml --src-dir ../ --output doc-coverage.info --kind enum,enumvalue,variable,function,class,define,namespace,property

I have a feeling there will be more missing kinds, so not submitting a PR just yet, changes will be tracked in https://github.com/psycofdj/coverxygen/compare/master...noname77:coverxygen:feat/csharp-support

noname77 commented 1 year ago

To leave a trace somewhere, I added the interface kind in https://github.com/noname77/coverxygen/commit/8f949dde20f9ab974f24574ff53e2b48de590fff and it works for a simple definition

Screenshot 2023-10-31 at 15 05 28

but for some reason it fails on a generic interface:

image

weirdly, a similar class is hadndled correctly

Screenshot 2023-10-31 at 15 09 59

xmarcelet commented 12 months ago

I'll be happy to merge your PR if can provide one :+1: