Fixed unit tests 1/2

handlers/handlers.go

@@ -19,10 +19,11 @@ import (
 // Handler holds the funcs and attributes for the
 // http communication
 type Handler struct {
-	config    config.Handlers
+	config                 config.Handlers
-	store     store.Store
+	store                  store.Store
-	engine    *gin.Engine
+	engine                 *gin.Engine
-	oAuthConf *oauth2.Config
+	oAuthConf              *oauth2.Config
+	DoNotCheckConfigViaGet bool // DoNotCheckConfigViaGet is for the unit testing usage
 }
 
 // New initializes the http handlers
@@ -56,15 +57,17 @@ func (h *Handler) setTemplateFromFS(name string) error {
 }
 
 func (h *Handler) checkIfSecretExist() error {
-	conf := config.Get()
+	if h.DoNotCheckConfigViaGet {
-	if conf.Handlers.Secret == nil {
+		conf := config.Get()
-		b := make([]byte, 128)
+		if conf.Handlers.Secret == nil {
-		if _, err := rand.Read(b); err != nil {
+			b := make([]byte, 128)
-			return err
+			if _, err := rand.Read(b); err != nil {
-		}
+				return err
-		conf.Handlers.Secret = b
+			}
-		if err := config.Set(conf); err != nil {
+			conf.Handlers.Secret = b
-			return err
+			if err := config.Set(conf); err != nil {
+				return err
+			}
 		}
 	}
 	return nil

handlers/handlers_test.go

@@ -244,13 +244,13 @@ func getBackend() (func(), error) {
 	}
 	handler, err := New(config.Handlers{
 		ListenAddr: ":8080",
-		Secret:     []byte(""),
+		Secret:     []byte("our really great secret"),
 		BaseURL:    "http://127.0.0.1",
 	}, *store)
 	if err != nil {
 		return nil, errors.Wrap(err, "could not create handler")
 	}
-
+	handler.DoNotCheckConfigViaGet = true
 	server = httptest.NewServer(handler.engine)
 	return func() {
 		server.Close()

store/store_test.go

@@ -20,7 +20,7 @@ var validConfig = config.Store{
 func TestGenerateRandomString(t *testing.T) {
 	tt := []struct {
 		name   string
-		length int
+		length uint
 	}{
 		{"fourtytwo long", 42},
 		{"sixteen long", 16},