fix: jenkins' test_env options was overridden

internal/pkg/plugin/jenkins/create.go

@@ -11,7 +11,7 @@ func Create(options map[string]interface{}) (map[string]interface{}, error) {
 	// Initialize Operator with Operations
 	operator := &plugininstaller.Operator{
 		PreExecuteOperations: plugininstaller.PreExecuteOperations{
-			helm.SetDefaultConfig(&defaultHelmConfig),
+			setDefaultValue(defaultHelmConfig),
 			helm.Validate,
 			replaceStroageClass,
 		},

internal/pkg/plugin/jenkins/delete.go

@@ -9,7 +9,7 @@ func Delete(options map[string]interface{}) (bool, error) {
 	// Initialize Operator with Operations
 	operator := &plugininstaller.Operator{
 		PreExecuteOperations: plugininstaller.PreExecuteOperations{
-			helm.SetDefaultConfig(&defaultHelmConfig),
+			setDefaultValue(defaultHelmConfig),
 			helm.Validate,
 			replaceStroageClass,
 		},

internal/pkg/plugin/jenkins/jenkins.go

@@ -8,7 +8,7 @@ import (
 	"github.com/devstream-io/devstream/pkg/util/types"
 )
 
-var defaultHelmConfig = helm.Options{
+var defaultHelmConfig = &helm.Options{
 	Chart: helmCommon.Chart{
 		ChartName:   "jenkins/jenkins",
 		Timeout:     "5m",

internal/pkg/plugin/jenkins/options.go

@@ -3,8 +3,8 @@ package jenkins
 import (
 	"github.com/mitchellh/mapstructure"
 
-	"github.com/devstream-io/devstream/internal/pkg/plugin/common/helm"
 	"github.com/devstream-io/devstream/internal/pkg/plugininstaller"
+	"github.com/devstream-io/devstream/internal/pkg/plugininstaller/helm"
 )
 
 type jenkinsOptions struct {
@@ -28,3 +28,14 @@ func (opts *jenkinsOptions) encode() (map[string]interface{}, error) {
 	}
 	return options, nil
 }
+
+func setDefaultValue(defaultOpts *helm.Options) plugininstaller.MutableOperation {
+	return func(options plugininstaller.RawOptions) (plugininstaller.RawOptions, error) {
+		opts, err := newOptions(options)
+		if err != nil {
+			return nil, err
+		}
+		opts.FillDefaultValue(defaultOpts)
+		return opts.encode()
+	}
+}

internal/pkg/plugin/jenkins/read.go

@@ -10,7 +10,7 @@ func Read(options map[string]interface{}) (map[string]interface{}, error) {
 	// Initialize Operator with Operations
 	operator := &plugininstaller.Operator{
 		PreExecuteOperations: plugininstaller.PreExecuteOperations{
-			helm.SetDefaultConfig(&defaultHelmConfig),
+			setDefaultValue(defaultHelmConfig),
 			helm.Validate,
 			replaceStroageClass,
 		},

internal/pkg/plugin/jenkins/update.go

@@ -11,7 +11,7 @@ func Update(options map[string]interface{}) (map[string]interface{}, error) {
 	// Initialize Operator with Operations
 	operator := &plugininstaller.Operator{
 		PreExecuteOperations: plugininstaller.PreExecuteOperations{
-			helm.SetDefaultConfig(&defaultHelmConfig),
+			setDefaultValue(defaultHelmConfig),
 			helm.Validate,
 			replaceStroageClass,
 		},

internal/pkg/plugininstaller/helm/option.go

@@ -44,7 +44,7 @@ func (opts *Options) Encode() (map[string]interface{}, error) {
 	return options, nil
 }
 
-func (opts *Options) fillDefaultValue(defaultOpts *Options) {
+func (opts *Options) FillDefaultValue(defaultOpts *Options) {
 	chart := &opts.Chart
 	chart.FillDefaultValue(&defaultOpts.Chart)
 	repo := &opts.Repo

internal/pkg/plugininstaller/helm/validate.go

@@ -31,7 +31,7 @@ func SetDefaultConfig(defaultConfig *Options) plugininstaller.MutableOperation {
 		if err != nil {
 			return nil, err
 		}
-		opts.fillDefaultValue(defaultConfig)
+		opts.FillDefaultValue(defaultConfig)
 		return opts.Encode()
 	}
 }