fix: attribute loss when updating entity

Author: GuillaumeOd

lib/services/ngsi/entities-NGSI-v2.js

@@ -463,7 +463,7 @@ function sendUpdateValueNgsi2(entityName, originMeasures, originTypeInformation,
             }
             //remove measures that has been shadowed by an alias (some may be left and managed later)
             //Maybe we must filter object_id if there is name == object_id
-            measures = measures.filter((item) => item.name !== currentAttr.object_id && item.name !== currentAttr.name);
+            measures = measures.filter((item) => item.name !== currentAttr.object_id);
 
             if (
                 currentAttr.expression !== undefined &&

test/functional/testCases.js

@@ -542,6 +542,16 @@ const testCases = [
                                 object_id: '.1.0.0.1',
                                 name: 'psBatteryVoltage',
                                 type: 'Number'
+                            },
+                            {
+                                object_id: 'fdt',
+                                name: 'fireDetectionThreshold',
+                                type: 'Number'
+                            },
+                            {
+                                object_id: 'fireDetectionThreshold3',
+                                name: 'fireDetectionThreshold3',
+                                type: 'Number'
                             }
                         ],
                         static_attributes: []
@@ -556,7 +566,7 @@ const testCases = [
         should: [
             {
                 shouldName:
-                    'A - WHEN sending defined object_ids with special format names (measures) through http IT should send measures to Context Broker preserving value types, name mappings and metadatas',
+                    'B - WHEN sending defined object_ids with special format names in attributes through http IT should send measures to Context Broker preserving value types, name mappings and metadatas',
                 type: 'single',
                 measure: {
                     url: 'http://localhost:' + config.http.port + '/iot/json',
@@ -567,7 +577,10 @@ const testCases = [
                     },
                     json: {
                         a: false,
-                        '.1.0.0.1': 23.5
+                        '.1.0.0.1': 10,
+                        fdt: 10,
+                        fireDetectionThreshold2: 20,
+                        fireDetectionThreshold3: 30
                     }
                 },
                 expectation: {
@@ -585,7 +598,54 @@ const testCases = [
                     },
                     psBatteryVoltage: {
                         type: 'Number',
-                        value: 23.5
+                        value: 10
+                    },
+                    fireDetectionThreshold: {
+                        type: 'Number',
+                        value: 10
+                    },
+                    fireDetectionThreshold2: {
+                        type: 'Text',
+                        value: 20
+                    },
+                    fireDetectionThreshold3: {
+                        type: 'Number',
+                        value: 30
+                    }
+                }
+            },
+            {
+                shouldName:
+                    'A - WHEN sending defined measures by attribute names through http IT should send measures to Context Broker preserving value types, name mappings and metadatas',
+                type: 'single',
+                measure: {
+                    url: 'http://localhost:' + config.http.port + '/iot/json',
+                    method: 'POST',
+                    qs: {
+                        i: globalEnv.deviceId,
+                        k: globalEnv.apikey
+                    },
+                    json: {
+                        a: false,
+                        fireDetectionThreshold: 10
+                    }
+                },
+                expectation: {
+                    id: globalEnv.entity_name,
+                    type: globalEnv.entity_type,
+                    attr_a: {
+                        value: false,
+                        type: 'Boolean',
+                        metadata: {
+                            accuracy: {
+                                value: 0.8,
+                                type: 'Float'
+                            }
+                        }
+                    },
+                    fireDetectionThreshold: {
+                        type: 'Number',
+                        value: 10
                     }
                 }
             }