search

binbashar / leverage

Binbash Leverage CLI intended to orchestrate Leverage Reference Architecture for AWS (www.binbash.co/leverage)
https://pypi.org/project/leverage/
Apache License 2.0
17 stars 2 forks source link

[BB-248] Improving test coverage #249

Closed Franr closed 5 months ago

Franr commented 6 months ago

What?

Improving tests coverage, mostly around modules/credentials Separating GHA files into 2: we need the "push" event for the unit tests, for the coverage integration, but not on the integration ones.

Why?

To improve the tools quality/realiability!

References

https://coveralls.io/builds/66184725

borland667 commented 6 months ago

@Franr General question. Perhaps I'm not familiar with the code yet but I'm finding it hard to understand what are you trying to test with some of the methods that do not describe what you are trying to test. Can we add docstrings with descriptions to all tests so its clear what is the intent ?