Skip to content
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

Use coverage in the CI #772

Open
moi15moi opened this issue Jan 26, 2025 · 2 comments
Open

Use coverage in the CI #772

moi15moi opened this issue Jan 26, 2025 · 2 comments
Labels
help wanted Extra attention is needed tests enhance or fix tests

Comments

@moi15moi
Copy link
Contributor

I think it would be beneficial to use coverage to track which part of comtypes is covered by tests.

The report created by coverage could be published on codecov.io.
In our CI, we could create the report and use codecov/codecov-action to publish it.

Here is a example of a project on codecov.io: https://app.codecov.io/github/moi15moi/FontCollector
As you can see, you can clearly see which part is covered and which part isn't which helps a lot to create some tests.

Note that I am note sure if it will properly detect the code covered in the server, but I still think it would be a plus.

@junkmd junkmd added the tests enhance or fix tests label Jan 26, 2025
@junkmd
Copy link
Collaborator

junkmd commented Jan 26, 2025

That's a great idea!

First, I’d appreciate it if you could test and verify what the appropriate codecov.io configuration for this project would be within your forked repository.

If applying codecov.io requires repository owner permissions, please let me know. I’m considering reaching out to Enthought, the host of this repository, to either allocate the necessary permissions or make the required configuration changes.

@moi15moi
Copy link
Contributor Author

Just so you know, this issue is currently blocked by this bug: nedbat/coveragepy#1920

@junkmd junkmd added the help wanted Extra attention is needed label Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed tests enhance or fix tests
Projects
None yet
Development

No branches or pull requests

2 participants