refactor: remove unused exported functions from config

Author: nexus49

config/config.go

@@ -212,17 +212,7 @@ func BindConfigToFlags(v *viper.Viper, cmd *cobra.Command, config any) error {
 	return nil
 }
 
-// GenerateFlagSet is exported for testing.
-func GenerateFlagSet(config any) (*pflag.FlagSet, error) {
-	return generateFlagSet(config)
-}
-
-// UnmarshalIntoStruct is exported for testing.
-func UnmarshalIntoStruct(v *viper.Viper, cfg any) func() {
-	return unmarshalIntoStruct(v, cfg)
-}
-
-// unmarshalIntoStruct returns a function that unmarshals viper config into cfg and panics on error.
+// unmarshalIntoStruct returns a function that unmarshal viper config into cfg and panics on error.
 func unmarshalIntoStruct(v *viper.Viper, cfg any) func() {
 	return func() {
 		if err := v.Unmarshal(cfg); err != nil {

config/config_test.go

@@ -177,25 +177,3 @@ func TestGenerateFlagSetUnsupportedType(t *testing.T) {
 	err := config.BindConfigToFlags(viper.New(), &cobra.Command{}, &testStruct)
 	assert.Error(t, err)
 }
-
-func TestGenerateFlagSetInvalidStruct(t *testing.T) {
-	notStruct := 123
-	_, err := config.GenerateFlagSet(notStruct)
-	assert.Error(t, err)
-}
-
-func TestUnmarshalIntoStructPanic(t *testing.T) {
-	v := viper.New()
-	cfg := struct {
-		InvalidField int `mapstructure:"invalid-field"`
-	}{}
-	// Set a string value to cause unmarshal error for int field
-	v.Set("invalid-field", "not-an-int")
-	defer func() {
-		if r := recover(); r == nil {
-			t.Errorf("expected panic but did not panic")
-		}
-	}()
-	unmarshal := config.UnmarshalIntoStruct(v, &cfg)
-	unmarshal()
-}