From 87fab588fb275ce8dfc3a7c2f27dc5ff0fe77336 Mon Sep 17 00:00:00 2001 From: fengyuchuanshen Date: Thu, 11 Sep 2025 14:41:54 +0800 Subject: [PATCH] refactor: use maps.Copy for cleaner map handling Signed-off-by: fengyuchuanshen --- provider/hana/hana_test.go | 9 +++------ provider/postgis/postgis_internal_test.go | 9 +++------ 2 files changed, 6 insertions(+), 12 deletions(-) diff --git a/provider/hana/hana_test.go b/provider/hana/hana_test.go index b53739653..a1596eb8f 100644 --- a/provider/hana/hana_test.go +++ b/provider/hana/hana_test.go @@ -4,6 +4,7 @@ import ( "context" "database/sql" "fmt" + "maps" "os" "strings" "testing" @@ -40,14 +41,10 @@ func (cfg TCConfig) Config() dict.Dict { mConfig = cfg.BaseConfig } config = make(map[string]interface{}, len(mConfig)) - for k, v := range mConfig { - config[k] = v - } + maps.Copy(config, mConfig) // set the config overrides - for k, v := range cfg.ConfigOverride { - config[k] = v - } + maps.Copy(config, cfg.ConfigOverride) if len(cfg.LayerConfig) > 0 { layerConfig, _ := config[hana.ConfigKeyLayers].([]map[string]interface{}) diff --git a/provider/postgis/postgis_internal_test.go b/provider/postgis/postgis_internal_test.go index f9b15f086..5fe1d7065 100644 --- a/provider/postgis/postgis_internal_test.go +++ b/provider/postgis/postgis_internal_test.go @@ -3,6 +3,7 @@ package postgis import ( "bytes" "context" + "maps" "reflect" "strings" "testing" @@ -57,14 +58,10 @@ func (cfg TCConfig) Config(mConfig map[string]any) dict.Dict { mConfig = cfg.BaseConfig } config = make(map[string]any, len(mConfig)) - for k, v := range mConfig { - config[k] = v - } + maps.Copy(config, mConfig) // set the config overrides - for k, v := range cfg.ConfigOverride { - config[k] = v - } + maps.Copy(config, cfg.ConfigOverride) if len(cfg.LayerConfig) > 0 { layerConfig, _ := config[ConfigKeyLayers].([]map[string]any)