Fix avp

ngcp/avp.lua

@@ -23,8 +23,10 @@ NGCPAvp_MT = {
                 error("value is not a number or string")
             end
         end
-        setmetatable( t, NGCPAvp_MT )
-        return t
+        function t.all()
+            return sr.pv.get("$(avp(" .. id .. ")[*])")
+        end
+        return setmetatable( t, NGCPAvp_MT )
     end
 
     function NGCPAvp:clean()

tests/ngcp_avp.lua

@@ -22,37 +22,30 @@ TestNGCPAvp = {} --class
         sr.pv.sets("$avp(s:testid)", "value")
         assertEquals(self.avp(), "value")
         sr.pv.sets("$avp(s:testid)", "1")
-        assertItemsEquals(self.avp(),{"1","value"})
+        assertItemsEquals(self.avp(), "1")
+        assertItemsEquals(self.avp:all(),{"1","value"})
     end
 
     function TestNGCPAvp:test_avp_set()
         local vals = {1,2,3}
+        local okvals = {3,2,1}
+        local i
         for i=1,#vals do
             self.avp(vals[i])
+            assertEquals(self.avp(),vals[i])
         end
-        local l = self.avp()
-        assertTrue(type(l), 'table')
-        --print(table.tostring(l))
-        v = 1
-        for i=#vals,1,-1 do
-           assertEquals(l[i],vals[v])
-           v = v + 1 
-        end        
+        assertEquals(self.avp:all(), okvals)
     end
 
     function TestNGCPAvp:test_avp_set2()
         local vals = {1,2,"3"}
+        local okvals = {"3",2,1}
+        local i
         for i=1,#vals do
             self.avp(vals[i])
+            assertEquals(self.avp(),vals[i])
         end
-        local l = self.avp()
-        assertTrue(type(l), 'table')
-        --print(table.tostring(l))
-        v = 1
-        for i=#vals,1,-1 do
-           assertEquals(l[i],vals[v])
-           v = v + 1 
-        end        
+        assertEquals(self.avp:all(), okvals)
     end
 
     function TestNGCPAvp:test_clean()