Lack of tests and coverage report

djc / bb8

Full-featured async (tokio-based) postgres connection pool (like r2d2)

MIT License

754 stars 110 forks source link

Lack of tests and coverage report #78

Closed ar3s3ru closed 3 years ago

ar3s3ru commented 4 years ago

Given the scope of the project, to ensure production-readiness an adequate testing suite should be added.

cargo tarpaulin might also give out a coverage report of the testing suite.

djc commented 4 years ago

I currently have limited time to dedicate to this project, so this isn't going to happen unless someone does the work, or if someone is willing to pay me to do the work.

ar3s3ru commented 4 years ago

Not implying you need to do the work on this yourself.

I can try to help out in this regard, with a little guidance :smile:

djc commented 4 years ago

What kind of guidance do you need? Do you have particular scenarios in mind that you think merit more coverage?

djc commented 4 years ago

@adrianbenavides added basic coverage in CI in #79.

ar3s3ru commented 3 years ago

Sorry I lost track of this issue, @adrianbenavides well done with the coverage job :clap: