deepmodeling / abacus-develop

An electronic structure package based on either plane wave basis or numerical atomic orbitals.
http://abacus.ustc.edu.cn
GNU Lesser General Public License v3.0
174 stars 136 forks source link

Code: Increase target code coverage ratio up to 80% before v3.9.0 #5470

Open WHUweiqingzhou opened 1 week ago

WHUweiqingzhou commented 1 week ago

Describe the Code Quality Issue

According to the latest coverage test, the current coverage is 74.11%. We need at least 6000 lines to cover and make the coverage up 80%. Image It is worth mentioning that, this file with more than 5000 lines shows zero coverage: Image

Additional Context

No response

Task list for Issue attackers (only for developers)

jinzx10 commented 1 week ago

Hi @WHUweiqingzhou , PR #5513 adds an accuracy test to the Lebedev quadrature and covered ~5000 lines of code.

Image

But I think I'm not able to figure out another ~1000 lines of coverage that I can test with ease. Maybe it's better to resort to respective module maintainers to do the job.