Skip to content
This repository has been archived by the owner on May 30, 2022. It is now read-only.

Add tests to the cmd line and env variables usage #410

Closed
arbulu89 opened this issue Oct 29, 2021 · 0 comments · Fixed by #662
Closed

Add tests to the cmd line and env variables usage #410

arbulu89 opened this issue Oct 29, 2021 · 0 comments · Fixed by #662
Labels
DX Developer Experience

Comments

@arbulu89
Copy link
Contributor

After suffering a quite tricky issue: #409

The cobra/viper combination is super interesting to attach command lines flags and configuration files to variables, but it has the risk that it includes a high degree of auto-magic (black-box operations).

In order to deal with this, we would need to start adding tests to the command line and environment variables loading. Basically we need to know that all existing variables are used properly, if they are loaded of course.
This would help us a lot to avoid regressions.

Besides this, tests on the LoadConfig functions would be interesting too.

@arbulu89 arbulu89 added the DX Developer Experience label Oct 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DX Developer Experience
Development

Successfully merging a pull request may close this issue.

1 participant