WIP: setup required config in WithManager helper

model/instance/instance_test.go

@@ -59,11 +59,6 @@ func TestInstance(t *testing.T) {
 		was := cfg.Contexts
 		defer func() { cfg.Contexts = was }()
 
-		cfg.Clouderies = map[string]config.ClouderyConfig{
-			"context": {
-				API: config.ClouderyAPI{URL: "http://manager.example.org", Token: ""},
-			},
-		}
 		cfg.Contexts = map[string]interface{}{
 			"context": map[string]interface{}{
 				"logos": map[string]interface{}{

model/instance/manager.go

@@ -28,20 +28,24 @@ const (
 func (i *Instance) ManagerURL(k ManagerURLKind) (string, error) {
 	c := clouderyConfig(i)
 	if c == nil {
+		fmt.Printf("NO CLOUDERY CONFIG")
 		return "", nil
 	}
 
 	if i.UUID == "" {
+		fmt.Printf("NO INSTANCE UUID")
 		return "", nil
 	}
 
 	base := c.API.URL
 	if base == "" {
+		fmt.Printf("EMPTY API URL")
 		return "", nil
 	}
 
 	baseURL, err := url.Parse(base)
 	if err != nil {
+		fmt.Printf("INVALID API URL")
 		return "", err
 	}
 

model/oauth/client_test.go

@@ -32,12 +32,6 @@ func TestClient(t *testing.T) {
 	}
 
 	config.UseTestFile(t)
-	conf := config.GetConfig()
-	conf.Clouderies = map[string]config.ClouderyConfig{
-		config.DefaultInstanceContext: {
-			API: config.ClouderyAPI{URL: "http://manager.example.org", Token: ""},
-		},
-	}
 	setup := testutils.NewSetup(t, t.Name())
 	testInstance := setup.GetTestInstance()
 
@@ -201,7 +195,7 @@ func TestClient(t *testing.T) {
 		premiumLink := assertClientsLimitAlertMailWasSent(t, testInstance, "notificationWithoutPremium", 1)
 		assert.Empty(t, premiumLink)
 
-		testutils.WithManager(t, testInstance)
+		testutils.WithManager(t, testInstance, testutils.ManagerConfig{URL: "http://manager.example.org", WithPremiumLinks: true})
 
 		notificationWithPremium = &oauth.Client{
 			ClientName:   "notificationWithPremium",

tests/testutils/test_utils.go

@@ -3,6 +3,7 @@ package testutils
 import (
 	"bytes"
 	"context"
+	"encoding/json"
 	"errors"
 	"flag"
 	"fmt"
@@ -469,34 +470,57 @@ func compress(content string) []byte {
 	return buf.Bytes()
 }
 
-func WithManager(t *testing.T, inst *instance.Instance) (shouldRemoveUUID bool) {
+type ManagerConfig struct {
+	URL              string
+	WithPremiumLinks bool
+}
+
+func WithManager(t *testing.T, inst *instance.Instance, managerConfig ManagerConfig) (shouldRemoveUUID bool) {
+	require.NotEmpty(t, managerConfig.URL, "Could not enable test instance manager: cloudery API URL is required")
+
 	if inst.UUID == "" {
 		uuid, err := uuid.NewV7()
 		require.NoError(t, err, "Could not enable test instance manager")
 		inst.UUID = uuid.String()
 		shouldRemoveUUID = true
 	}
 
-	config, ok := inst.SettingsContext()
-	require.True(t, ok, "Could not enable test instance manager: could not fetch test instance settings context")
+	contextName := inst.ContextName
+	if contextName == "" {
+		contextName = config.DefaultInstanceContext
+	}
 
-	managerURL, err := inst.ManagerURL(instance.ManagerBaseURL)
-	require.NoError(t, err, "Could not enable test instance manager: cloudery config is required")
-	require.NotEmpty(t, managerURL, "Could not enable test instance manager: cloudery API URL is required")
+	cfg := config.GetConfig()
+	originalCfg, _ := json.Marshal(cfg)
 
-	was := config["enable_premium_links"]
-	config["enable_premium_links"] = true
+	if cfg.Clouderies == nil {
+		cfg.Clouderies = map[string]config.ClouderyConfig{}
+	}
+	if _, ok := cfg.Clouderies[contextName]; !ok {
+		cfg.Clouderies[contextName] = config.ClouderyConfig{}
+	}
+	cloudery := cfg.Clouderies[contextName]
+	cloudery.API = config.ClouderyAPI{URL: managerConfig.URL, Token: ""}
+
+	if cfg.Contexts == nil {
+		cfg.Contexts = map[string]interface{}{}
+	}
+	if _, ok := cfg.Contexts[contextName]; !ok {
+		cfg.Contexts[contextName] = map[string]interface{}{}
+	}
+	context := cfg.Contexts[contextName].(map[string]interface{})
+	context["enable_premium_links"] = managerConfig.WithPremiumLinks
 
 	t.Cleanup(func() {
-		config["enable_premium_links"] = was
+		json.Unmarshal(originalCfg, cfg)
 
 		if shouldRemoveUUID {
 			inst.UUID = ""
 			require.NoError(t, instance.Update(inst))
 		}
 	})
 
-	err = instance.Update(inst)
+	err := instance.Update(inst)
 	require.NoError(t, err, "Could not enable test instance manager")
 
 	return shouldRemoveUUID

web/apps/apps_test.go

@@ -66,11 +66,6 @@ func TestApps(t *testing.T) {
 		Host:   "localhost",
 		Path:   tempdir,
 	}
-	cfg.Clouderies = map[string]config.ClouderyConfig{
-		config.DefaultInstanceContext: {
-			API: config.ClouderyAPI{URL: "http://manager.example.org", Token: ""},
-		},
-	}
 	was := cfg.Subdomains
 	cfg.Subdomains = config.NestedSubdomains
 	defer func() { cfg.Subdomains = was }()
@@ -209,7 +204,7 @@ func TestApps(t *testing.T) {
 
 		// TOS not signed warning
 
-		testutils.WithManager(t, testInstance)
+		testutils.WithManager(t, testInstance, testutils.ManagerConfig{URL: "http://manager.example.org"})
 
 		tosSigned := testInstance.TOSSigned
 		tosLatest := testInstance.TOSLatest

web/settings/settings_test.go

@@ -73,11 +73,6 @@ func TestSettings(t *testing.T) {
 	config.UseTestFile(t)
 	conf := config.GetConfig()
 	conf.Assets = "../../assets"
-	conf.Clouderies = map[string]config.ClouderyConfig{
-		config.DefaultInstanceContext: {
-			API: config.ClouderyAPI{URL: "http://manager.example.org", Token: ""},
-		},
-	}
 	conf.Contexts[config.DefaultInstanceContext] = map[string]interface{}{
 		"logos": map[string]interface{}{
 			"home": map[string]interface{}{
@@ -1025,7 +1020,7 @@ func TestSettings(t *testing.T) {
 			Contains("/#/connectedDevices").
 			NotContains("http://manager.example.org")
 
-		testutils.WithManager(t, testInstance)
+		testutils.WithManager(t, testInstance, testutils.ManagerConfig{URL: "http://manager.example.org", WithPremiumLinks: true})
 
 		e.GET("/settings/clients/limit-exceeded").
 			WithCookie(sessCookie, "connected").