From a4925a68db6fb1e90f75407d1017402dba6efd62 Mon Sep 17 00:00:00 2001 From: ionutboangiu Date: Fri, 22 Nov 2024 16:20:18 +0200 Subject: [PATCH] Remove duplicate sep param from ers load method --- config/config.go | 2 +- config/erscfg.go | 2 +- config/erscfg_test.go | 22 +++++++++++----------- 3 files changed, 13 insertions(+), 13 deletions(-) diff --git a/config/config.go b/config/config.go index 95a35bd6a..64a958b39 100644 --- a/config/config.go +++ b/config/config.go @@ -802,7 +802,7 @@ func (cfg *CGRConfig) loadErsCfg(jsnCfg *CgrJsonCfg) (err error) { if jsnERsCfg, err = jsnCfg.ERsJsonCfg(); err != nil { return } - return cfg.ersCfg.loadFromJSONCfg(jsnERsCfg, cfg.templates, cfg.generalCfg.RSRSep, cfg.dfltEvRdr, cfg.generalCfg.RSRSep) + return cfg.ersCfg.loadFromJSONCfg(jsnERsCfg, cfg.templates, cfg.generalCfg.RSRSep, cfg.dfltEvRdr) } // loadEesCfg loads the Ees section of the configuration diff --git a/config/erscfg.go b/config/erscfg.go index e8c1b3b1c..4fdecbf33 100644 --- a/config/erscfg.go +++ b/config/erscfg.go @@ -47,7 +47,7 @@ func (erS *ERsCfg) ReaderCfg(ID string) *EventReaderCfg { return nil } -func (erS *ERsCfg) loadFromJSONCfg(jsnCfg *ERsJsonCfg, msgTemplates map[string][]*FCTemplate, sep string, dfltRdrCfg *EventReaderCfg, separator string) (err error) { +func (erS *ERsCfg) loadFromJSONCfg(jsnCfg *ERsJsonCfg, msgTemplates map[string][]*FCTemplate, sep string, dfltRdrCfg *EventReaderCfg) (err error) { if jsnCfg == nil { return } diff --git a/config/erscfg_test.go b/config/erscfg_test.go index b807ef0c2..2da4ac51e 100644 --- a/config/erscfg_test.go +++ b/config/erscfg_test.go @@ -206,7 +206,7 @@ func TestEventReaderloadFromJsonCase1(t *testing.T) { } expected := "time: unknown unit \"ss\" in duration \"1ss\"" jsoncfg := NewDefaultCGRConfig() - if err := jsoncfg.ersCfg.loadFromJSONCfg(cfgJSON, jsoncfg.templates, jsoncfg.generalCfg.RSRSep, jsoncfg.dfltEvRdr, jsoncfg.generalCfg.RSRSep); err == nil || err.Error() != expected { + if err := jsoncfg.ersCfg.loadFromJSONCfg(cfgJSON, jsoncfg.templates, jsoncfg.generalCfg.RSRSep, jsoncfg.dfltEvRdr); err == nil || err.Error() != expected { t.Errorf("Expected %+v, received %+v", expected, err) } cfgJson := &ERsJsonCfg{ @@ -214,7 +214,7 @@ func TestEventReaderloadFromJsonCase1(t *testing.T) { Partial_cache_ttl: utils.StringPointer("test"), } - if err := jsoncfg.ersCfg.loadFromJSONCfg(cfgJson, jsoncfg.templates, jsoncfg.generalCfg.RSRSep, jsoncfg.dfltEvRdr, jsoncfg.generalCfg.RSRSep); err == nil { + if err := jsoncfg.ersCfg.loadFromJSONCfg(cfgJson, jsoncfg.templates, jsoncfg.generalCfg.RSRSep, jsoncfg.dfltEvRdr); err == nil { t.Error(err) } } @@ -229,7 +229,7 @@ func TestEventReaderloadFromJsonCase3(t *testing.T) { } expected := "invalid converter terminator in rule: " jsoncfg := NewDefaultCGRConfig() - if err := jsoncfg.ersCfg.loadFromJSONCfg(cfgJSON, jsoncfg.templates, jsoncfg.generalCfg.RSRSep, jsoncfg.dfltEvRdr, jsoncfg.generalCfg.RSRSep); err == nil || err.Error() != expected { + if err := jsoncfg.ersCfg.loadFromJSONCfg(cfgJSON, jsoncfg.templates, jsoncfg.generalCfg.RSRSep, jsoncfg.dfltEvRdr); err == nil || err.Error() != expected { t.Errorf("Expected %+v, received %+v", expected, err) } } @@ -397,7 +397,7 @@ func TestERSloadFromJsonCfg(t *testing.T) { Readers: nil, } jsonCfg := NewDefaultCGRConfig() - if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, jsonCfg.templates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr, jsonCfg.generalCfg.RSRSep); err != nil { + if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, jsonCfg.templates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr); err != nil { t.Error(err) } } @@ -416,7 +416,7 @@ func TestEventReaderFieldsloadFromJsonCfg(t *testing.T) { } expected := "invalid converter terminator in rule: " jsonCfg := NewDefaultCGRConfig() - if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, jsonCfg.templates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr, jsonCfg.generalCfg.RSRSep); err == nil || err.Error() != expected { + if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, jsonCfg.templates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr); err == nil || err.Error() != expected { t.Errorf("Expected %+v, received %+v", expected, err) } } @@ -436,7 +436,7 @@ func TestERSloadFromJsonCase1(t *testing.T) { } expected := "no template with id: <>" jsonCfg := NewDefaultCGRConfig() - if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, jsonCfg.templates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr, jsonCfg.generalCfg.RSRSep); err == nil || err.Error() != expected { + if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, jsonCfg.templates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr); err == nil || err.Error() != expected { t.Errorf("Expected %+v, received %+v", expected, err) } } @@ -455,7 +455,7 @@ func TestERSloadFromJsonCase2(t *testing.T) { } expected := "no template with id: <>" jsonCfg := NewDefaultCGRConfig() - if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, jsonCfg.templates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr, jsonCfg.generalCfg.RSRSep); err == nil || err.Error() != expected { + if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, jsonCfg.templates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr); err == nil || err.Error() != expected { t.Errorf("Expected %+v, received %+v", expected, err) } } @@ -615,7 +615,7 @@ func TestERSloadFromJsonCase3(t *testing.T) { v.ComputePath() } jsonCfg := NewDefaultCGRConfig() - if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, msgTemplates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr, jsonCfg.generalCfg.RSRSep); err != nil { + if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, msgTemplates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr); err != nil { t.Error(err) } else if !reflect.DeepEqual(expectedERsCfg, jsonCfg.ersCfg) { t.Errorf("Expected %+v \n, received %+v", utils.ToJSON(expectedERsCfg), utils.ToJSON(jsonCfg.ersCfg)) @@ -779,7 +779,7 @@ func TestERSloadFromJsonCase4(t *testing.T) { v.ComputePath() } jsonCfg := NewDefaultCGRConfig() - if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, msgTemplates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr, jsonCfg.generalCfg.RSRSep); err != nil { + if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, msgTemplates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr); err != nil { t.Error(err) } else if !reflect.DeepEqual(expectedERsCfg, jsonCfg.ersCfg) { t.Errorf("Expected %+v \n, received %+v", utils.ToJSON(expectedERsCfg), utils.ToJSON(jsonCfg.ersCfg)) @@ -800,7 +800,7 @@ func TestEventReaderCacheDumpFieldsloadFromJsonCfg(t *testing.T) { } expected := "invalid converter terminator in rule: " jsonCfg := NewDefaultCGRConfig() - if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, jsonCfg.templates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr, jsonCfg.generalCfg.RSRSep); err == nil || err.Error() != expected { + if err := jsonCfg.ersCfg.loadFromJSONCfg(cfgJSON, jsonCfg.templates, jsonCfg.generalCfg.RSRSep, jsonCfg.dfltEvRdr); err == nil || err.Error() != expected { t.Errorf("Expected %+v, received %+v", expected, err) } } @@ -1404,7 +1404,7 @@ func TestERsloadFromJsonCfg(t *testing.T) { } } cfgCgr := NewDefaultCGRConfig() - if err := cfgCgr.ersCfg.loadFromJSONCfg(cfgJSON, cfgCgr.templates, cfgCgr.generalCfg.RSRSep, cfgCgr.dfltEvRdr, cfgCgr.generalCfg.RSRSep); err != nil { + if err := cfgCgr.ersCfg.loadFromJSONCfg(cfgJSON, cfgCgr.templates, cfgCgr.generalCfg.RSRSep, cfgCgr.dfltEvRdr); err != nil { t.Error(err) } else if !reflect.DeepEqual(cfgCgr.ersCfg, expectedERsCfg) { t.Errorf("Expected %+v \n, received %+v", utils.ToJSON(expectedERsCfg), utils.ToJSON(cfgCgr.ersCfg))