planetfederal / registry

pycsw + elasticsearch + opensearch + swagger - Based on HHyperMap
MIT License
11 stars 9 forks source link

Coverage increased to 100% in tests #15

Closed JorgeMartinezG closed 7 years ago

coveralls commented 7 years ago

Coverage Status

Coverage increased (+3.7%) to 100.0% when pulling 520c3d55bdb319df8f409796971a156a55395b13 on JorgeMartinezG:master into 69c31b8cc37f62cf32283ec0ef8bc03e264b6719 on boundlessgeo:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+3.7%) to 100.0% when pulling de3023a08b080ff6794bbc55b170504c5e144fb6 on JorgeMartinezG:master into 69c31b8cc37f62cf32283ec0ef8bc03e264b6719 on boundlessgeo:master.

ingenieroariel commented 7 years ago

Hi @JorgeMartinezG

I made a few comments, I think the date parsing functionality should be tested instead of removed. I am interested in keeping the behavior but testing it and documenting it properly.

ingenieroariel commented 7 years ago

I think what we can do is merge this PR and ask you to create a new one that restores the deleted code with tests and documentation. Can you create a new issue to track it?