Added more tests for ip2hex

agents/agentreq_test.go

@@ -1663,6 +1663,34 @@ func TestAgReqSetFieldsInTmp(t *testing.T) {
 	}
 }
 
+func TestAgReqSetFieldsIp2Hex(t *testing.T) {
+	cfg, _ := config.NewDefaultCGRConfig()
+	data := engine.NewInternalDB(nil, nil, true, cfg.DataDbCfg().Items)
+	dm := engine.NewDataManager(data, config.CgrConfig().CacheCfg(), nil)
+	filterS := engine.NewFilterS(cfg, nil, dm)
+	agReq := NewAgentRequest(nil, nil, nil, nil, nil, nil, "cgrates.org", "", filterS, nil, nil)
+	agReq.CGRRequest.Set(&utils.FullPath{Path: "IP", PathItems: utils.PathItems{{Field: "IP"}}}, utils.NewNMData("62.87.114.244"))
+
+	tplFlds := []*config.FCTemplate{
+		{Tag: "IP",
+			Path: utils.MetaTmp + utils.NestingSep + "IP", Type: utils.MetaVariable,
+			Value: config.NewRSRParsersMustCompile("~*cgreq.IP{*ip2hex}", utils.INFIELD_SEP)},
+	}
+	for _, v := range tplFlds {
+		v.ComputePath()
+	}
+	eMp := utils.NavigableMap2{}
+	eMp.Set(utils.PathItems{{Field: "IP"}}, &utils.NMSlice{
+		&config.NMItem{Data: "0x3e5772f4", Path: []string{"IP"},
+			Config: tplFlds[0]}})
+
+	if err := agReq.SetFields(tplFlds); err != nil {
+		t.Error(err)
+	} else if !reflect.DeepEqual(agReq.tmp, eMp) {
+		t.Errorf("expecting: %+v,\n received: %+v", eMp, agReq.tmp)
+	}
+}
+
 func TestAgReqSetFieldsWithRemove(t *testing.T) {
 	cfg, _ := config.NewDefaultCGRConfig()
 	data := engine.NewInternalDB(nil, nil, true, cfg.DataDbCfg().Items)

engine/datamanager.go

@@ -733,7 +733,6 @@ func (dm *DataManager) SetFilter(fltr *Filter, withIndex bool) (err error) {
 		}
 	}
 	return
-
 }
 
 func (dm *DataManager) RemoveFilter(tenant, id, transactionID string, withIndex bool) (err error) {

utils/dataconverter_test.go

@@ -643,6 +643,14 @@ func TestHexConvertor(t *testing.T) {
 	} else if !reflect.DeepEqual(expected, rpl) {
 		t.Errorf("expecting: %+v, received: %+v", expected, rpl)
 	}
+
+	val = "62.87.114.244"
+	expected = "0x3e5772f4"
+	if rpl, err := hx.Convert(val); err != nil {
+		t.Error(err)
+	} else if !reflect.DeepEqual(expected, rpl) {
+		t.Errorf("expecting: %+v, received: %+v", expected, rpl)
+	}
 }
 
 type testMockConverter struct{}