ContainerSolutions/externalsecret-operator

Increase our unit test coverage

Aperta

#103 aperta il 7 ott 2020

 (1 commento) (0 reazioni) (2 assegnatari)Go (27 fork)auto 404
enhancementgood first issuehacktoberfesthelp wantedtesting

Metriche repository

Star
 (192 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

I am opening this issue as a general issue to attract hacktoberfestians, but we can later open sub-issues to be more specific on what types of testes and where we want them.

Generally we want tests for our main controller (responsible for our reconciliation loop) or/and our backends (responsible for implementing our interface to external secrets solutions support).

Example:

This is the Google Secrets Manager Backend implementation: pkg/gsm/backend.go

This is where its tests are implemented: pkg/gsm/backend_test.go

Our controller logic: controllers/secrets/externalsecret_controller.go

Its tests: controllers/secrets/externalsecret_controller_test.go

I will try to add a checklist here later. But generally if you implement a meaningful test in these places they will be welcomed.

You can also checkout codecov to see where we have less coverage to bump it up a bit.

Guida contributor