search

NCATSTranslator / workflow-runner

1 stars 1 forks source link

Improve logging #10

Closed patrickkwang closed 2 years ago

kennethmorton commented 2 years ago

The majority of logging.py is not covered by current tests. Can we remove code and or add an additional test?

patrickkwang commented 2 years ago

When I run it, I see 97% coverage in logging.py... What are you seeing?

I would like to set up code coverage reporting for this repo, but we need to hook it up to Codecov. We will need help from @MarkDWilliams because I don't have the necessary permissions on the NCATSTranslator org.

kennethmorton commented 2 years ago

Agreed. pytest-asyncio was missing in requirements-test.txt. After that all looks good.