forked from absmach/supermq
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
NOISSUE - Add tests for env pkg (absmach#1796)
* initial test outline Signed-off-by: SammyOina <[email protected]> * add env tests Signed-off-by: SammyOina <[email protected]> * structure tests in a loop Signed-off-by: SammyOina <[email protected]> * remove t.Run Signed-off-by: SammyOina <[email protected]> * separate test cases Signed-off-by: SammyOina <[email protected]> * use switch Signed-off-by: SammyOina <[email protected]> --------- Signed-off-by: SammyOina <[email protected]>
- Loading branch information
Showing
1 changed file
with
314 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,314 @@ | ||
| package env | ||
|
|
||
| import ( | ||
| "fmt" | ||
| "testing" | ||
| "time" | ||
|
|
||
| "github.com/mainflux/mainflux/pkg/errors" | ||
|
|
||
| "github.com/mainflux/mainflux/internal/clients/grpc" | ||
| "github.com/mainflux/mainflux/internal/server" | ||
| "github.com/stretchr/testify/assert" | ||
| ) | ||
|
|
||
| var errNotDuration error = errors.New("unable to parse duration") | ||
|
|
||
| func TestParseServerConfig(t *testing.T) { | ||
| tests := []struct { | ||
| description string | ||
| config *server.Config | ||
| expectedConfig *server.Config | ||
| options []Options | ||
| err error | ||
| }{ | ||
| { | ||
| "Parsing with Server Config", | ||
| &server.Config{}, | ||
| &server.Config{ | ||
| Host: "localhost", | ||
| Port: "8080", | ||
| CertFile: "cert", | ||
| KeyFile: "key", | ||
| }, | ||
| []Options{ | ||
| { | ||
| Environment: map[string]string{ | ||
| "HOST": "localhost", | ||
| "PORT": "8080", | ||
| "SERVER_CERT": "cert", | ||
| "SERVER_KEY": "key", | ||
| }, | ||
| }, | ||
| }, | ||
| nil, | ||
| }, | ||
| { | ||
| "Parsing with Server Config with Prefix", | ||
| &server.Config{}, | ||
| &server.Config{ | ||
| Host: "localhost", | ||
| Port: "8080", | ||
| CertFile: "cert", | ||
| KeyFile: "key", | ||
| }, | ||
| []Options{ | ||
| { | ||
| Environment: map[string]string{ | ||
| "MF-HOST": "localhost", | ||
| "MF-PORT": "8080", | ||
| "MF-SERVER_CERT": "cert", | ||
| "MF-SERVER_KEY": "key", | ||
| }, | ||
| Prefix: "MF-", | ||
| }, | ||
| }, | ||
| nil, | ||
| }, | ||
| { | ||
| "Parsing with Server Config with Alt-Prefix", | ||
| &server.Config{}, | ||
| &server.Config{ | ||
| Host: "localhost", | ||
| Port: "8080", | ||
| CertFile: "cert", | ||
| KeyFile: "key", | ||
| }, | ||
| []Options{ | ||
| { | ||
| Environment: map[string]string{ | ||
| "MF-HOST": "localhost", | ||
| "MF-HTTP-PORT": "8080", | ||
| "MF-SERVER_CERT": "cert", | ||
| "MF-SERVER_KEY": "key", | ||
| }, | ||
| Prefix: "MF-", | ||
| AltPrefix: "MF-HTTP-", | ||
| }, | ||
| }, | ||
| nil, | ||
| }, | ||
| { | ||
| "Parsing with conflicting configs", | ||
| &server.Config{}, | ||
| &server.Config{ | ||
| Host: "localhost", | ||
| Port: "8080", | ||
| CertFile: "cert", | ||
| KeyFile: "key", | ||
| }, | ||
| []Options{ | ||
| { | ||
| Environment: map[string]string{ | ||
| "PORT": "", | ||
| "MF-PORT": "", | ||
| "MF-HOST": "localhost", | ||
| "MF-HTTP-PORT": "8080", | ||
| "MF-SERVER_CERT": "", | ||
| "MF-SERVER_KEY": "", | ||
| "MF-HTTP-SERVER_CERT": "cert", | ||
| "MF-HTTP-SERVER_KEY": "key", | ||
| }, | ||
| Prefix: "MF-", | ||
| AltPrefix: "MF-HTTP-", | ||
| }, | ||
| }, | ||
| nil, | ||
| }, | ||
| } | ||
| for _, test := range tests { | ||
| err := Parse(test.config, test.options...) | ||
| switch test.err { | ||
| case nil: | ||
| assert.NoError(t, err, fmt.Sprintf("expected no error but got %v", err)) | ||
| default: | ||
| assert.Error(t, err, "expected error but got nil") | ||
| } | ||
| assert.Equal(t, test.expectedConfig, test.config, fmt.Sprintf("expected %v got %v", test.expectedConfig, test.config)) | ||
| } | ||
| } | ||
|
|
||
| func TestParseGRPCConfig(t *testing.T) { | ||
| tests := []struct { | ||
| description string | ||
| config *grpc.Config | ||
| expectedConfig *grpc.Config | ||
| options []Options | ||
| err error | ||
| }{ | ||
| { | ||
| "Parsing a grpc.Config struct", | ||
| &grpc.Config{}, | ||
| &grpc.Config{ | ||
| URL: "val.com", | ||
| Timeout: time.Second, | ||
| }, | ||
| []Options{ | ||
| { | ||
| Environment: map[string]string{ | ||
| "URL": "val.com", | ||
| "TIMEOUT": time.Second.String(), | ||
| }, | ||
| }, | ||
| }, | ||
| nil, | ||
| }, | ||
| { | ||
| "Invalid type parsing", | ||
| &grpc.Config{}, | ||
| &grpc.Config{URL: "val.com"}, | ||
| []Options{ | ||
| { | ||
| Environment: map[string]string{ | ||
| "URL": "val.com", | ||
| "TIMEOUT": "invalid", | ||
| }, | ||
| }, | ||
| }, | ||
| errNotDuration, | ||
| }, | ||
| { | ||
| "Parsing a grpc.Config with Alt-Prefix", | ||
| &grpc.Config{}, | ||
| &grpc.Config{ | ||
| URL: "val.com", | ||
| Timeout: time.Second, | ||
| }, | ||
| []Options{ | ||
| { | ||
| Environment: map[string]string{ | ||
| "MF-URL": "val.com", | ||
| "MF-GRPC-TIMEOUT": time.Second.String(), | ||
| }, | ||
| Prefix: "MF-", | ||
| AltPrefix: "MF-GRPC-", | ||
| }, | ||
| }, | ||
| nil, | ||
| }, | ||
| { | ||
| "Parsing a grpc.Config with Alt-Prefix and errors", | ||
| &grpc.Config{}, | ||
| &grpc.Config{ | ||
| URL: "val.com", | ||
| Timeout: time.Second, | ||
| }, | ||
| []Options{ | ||
| { | ||
| Environment: map[string]string{ | ||
| "MF-URL": "val.com", | ||
| "MF-GRPC-TIMEOUT": "not-duration", | ||
| }, | ||
| Prefix: "MF-", | ||
| AltPrefix: "MF-GRPC-", | ||
| }, | ||
| }, | ||
| errNotDuration, | ||
| }, | ||
| { | ||
| "Parsing conflicting configs", | ||
| &grpc.Config{}, | ||
| &grpc.Config{ | ||
| URL: "val.com", | ||
| Timeout: time.Second, | ||
| ClientTLS: true, | ||
| CACerts: "cert", | ||
| }, | ||
| []Options{ | ||
| { | ||
| Environment: map[string]string{ | ||
| "MF-URL": "val.com", | ||
| "MF-GRPC-TIMEOUT": "1s", | ||
| "MF-GRPC-CLIENT_TLS": "true", | ||
| "MF-GRPC-CA_CERTS": "cert", | ||
| "MF-CLIENT_TLS": "", | ||
| "MF-CA_CERTS": "", | ||
| }, | ||
| Prefix: "MF-", | ||
| AltPrefix: "MF-GRPC-", | ||
| }, | ||
| }, | ||
| nil, | ||
| }, | ||
| } | ||
| for _, test := range tests { | ||
| err := Parse(test.config, test.options...) | ||
| switch test.err { | ||
| case nil: | ||
| assert.NoError(t, err, fmt.Sprintf("expected no error but got %v", err)) | ||
| default: | ||
| assert.Error(t, err, "expected error but got nil") | ||
| } | ||
| assert.Equal(t, test.expectedConfig, test.config, fmt.Sprintf("expected %v got %v", test.expectedConfig, test.config)) | ||
| } | ||
| } | ||
|
|
||
| func TestParseCustomConfig(t *testing.T) { | ||
| type CustomConfig struct { | ||
| Field1 string `env:"FIELD1" envDefault:"val1"` | ||
| Field2 int `env:"FIELD2"` | ||
| } | ||
|
|
||
| tests := []struct { | ||
| description string | ||
| config *CustomConfig | ||
| expectedConfig *CustomConfig | ||
| options []Options | ||
| err error | ||
| }{ | ||
| { | ||
| "parse with missing required field", | ||
| &CustomConfig{}, | ||
| &CustomConfig{Field1: "test val"}, | ||
| []Options{ | ||
| { | ||
| Environment: map[string]string{ | ||
| "FIELD1": "test val", | ||
| }, | ||
| RequiredIfNoDef: true, | ||
| }, | ||
| }, | ||
| errors.New(`required environment variable "FIELD2" not set`), | ||
| }, | ||
| { | ||
| "parse with wrong type", | ||
| &CustomConfig{}, | ||
| &CustomConfig{Field1: "test val"}, | ||
| []Options{ | ||
| { | ||
| Environment: map[string]string{ | ||
| "FIELD1": "test val", | ||
| "FIELD2": "not int", | ||
| }, | ||
| }, | ||
| }, | ||
| errors.New(`strconv.ParseInt`), | ||
| }, | ||
| { | ||
| "parse with prefix", | ||
| &CustomConfig{}, | ||
| &CustomConfig{Field1: "test val", Field2: 2}, | ||
| []Options{ | ||
| { | ||
| Environment: map[string]string{ | ||
| "MF-FIELD1": "test val", | ||
| "MF-FIELD2": "2", | ||
| }, | ||
| Prefix: "MF-", | ||
| }, | ||
| }, | ||
| nil, | ||
| }, | ||
| } | ||
|
|
||
| for _, test := range tests { | ||
| err := Parse(test.config, test.options...) | ||
| switch test.err { | ||
| case nil: | ||
| assert.NoError(t, err, fmt.Sprintf("expected no error but got %v", err)) | ||
| default: | ||
| assert.Error(t, err, "expected error but got nil") | ||
| } | ||
| assert.Equal(t, test.expectedConfig, test.config, fmt.Sprintf("expected %v got %v", test.expectedConfig, test.config)) | ||
| } | ||
| } |