search

DDMAL / MEI2Volpiano

An MEI to Volpiano converter for CWMN or Neume MEI files.
MIT License
0 stars 0 forks source link

Feedback from code coverage #48

Open napulen opened 3 years ago

napulen commented 3 years ago

Not sure if everyone is able to see the reports in codecov. If not, we need to find a way to make those publicly available for convenience.

From a quick look I gave to them:

The function sylb_note_map is not currently covered by the tests:

image

From the driver, almost no code is being covered by the tests. Here is the overall picture:

image

Having tests that exercise all the code is desirable. This is what these tools (code coverage) reveal and are useful for.

Yet, good thing code coverage is running now!

A low-priority issue derived from this is to extend the tests to exercise more of the codebase.

raviraina commented 3 years ago

Worth looking into if we have downtime @kemalkongar

kemalkongar commented 3 years ago

I'm looking through Calvo right now but I'll take a look tomorrow.

napulen commented 3 years ago

Sure, this can stay on hold for the time being. No rush.