Specify a richer scheme to run the migration with

2 files changed, 26 insertions, 11 deletions

diff --git a/go/internal/config/config.go b/go/internal/config/config.go
index 64822c7..4069851 100644
--- a/go/internal/config/config.go
+++ b/go/internal/config/config.go
@@ -13,11 +13,16 @@ const (
 	logFile    = "gitlab-shell.log"
 )
 
+type MigrationConfig struct {
+	Enabled  bool     `yaml:"enabled"`
+	Features []string `yaml:"features"`
+}
+
 type Config struct {
-	RootDir      string
-	LogFile      string `yaml:"log_file"`
-	LogFormat    string `yaml:"log_format"`
-	Experimental bool   `yaml:"experimental"`
+	RootDir   string
+	LogFile   string          `yaml:"log_file"`
+	LogFormat string          `yaml:"log_format"`
+	Migration MigrationConfig `yaml:"migration"`
 }
 
 func New() (*Config, error) {
diff --git a/go/internal/config/config_test.go b/go/internal/config/config_test.go
index 552a600..368ab29 100644
--- a/go/internal/config/config_test.go
+++ b/go/internal/config/config_test.go
@@ -2,22 +2,28 @@ package config
 
 import (
 	"fmt"
+	"strings"
 	"testing"
 )
 
 func TestConfigLogFile(t *testing.T) {
 	testRoot := "/foo/bar"
 	testCases := []struct {
-		yaml         string
-		path         string
-		format       string
-		experimental bool
+		yaml      string
+		path      string
+		format    string
+		migration MigrationConfig
 	}{
 		{path: "/foo/bar/gitlab-shell.log", format: "text"},
 		{yaml: "log_file: my-log.log", path: "/foo/bar/my-log.log", format: "text"},
 		{yaml: "log_file: /qux/my-log.log", path: "/qux/my-log.log", format: "text"},
 		{yaml: "log_format: json", path: "/foo/bar/gitlab-shell.log", format: "json"},
-		{yaml: "experimental: true", path: "/foo/bar/gitlab-shell.log", format: "text", experimental: true},
+		{
+			yaml:      "migration:\n  enabled: true\n  features:\n    - foo\n    - bar",
+			path:      "/foo/bar/gitlab-shell.log",
+			format:    "text",
+			migration: MigrationConfig{Enabled: true, Features: []string{"foo", "bar"}},
+		},
 	}
 
 	for _, tc := range testCases {
@@ -27,8 +33,12 @@ func TestConfigLogFile(t *testing.T) {
 				t.Fatal(err)
 			}
 
-			if cfg.Experimental != tc.experimental {
-				t.Fatalf("expected %v, got %v", tc.experimental, cfg.Experimental)
+			if cfg.Migration.Enabled != tc.migration.Enabled {
+				t.Fatalf("migration.enabled: expected %v, got %v", tc.migration.Enabled, cfg.Migration.Enabled)
+			}
+
+			if strings.Join(cfg.Migration.Features, ":") != strings.Join(tc.migration.Features, ":") {
+				t.Fatalf("migration.features: expected %#v, got %#v", tc.migration.Features, cfg.Migration.Features)
 			}
 
 			if cfg.LogFile != tc.path {