Scene: Device get value should deep clone device feature to avoid pollution on subsequent calls

server/lib/scene/scene.actions.js

@@ -195,7 +195,7 @@ const actionsFunc = {
       scope,
       `${columnIndex}`,
       {
-        [rowIndex]: deviceFeature,
+        [rowIndex]: cloneDeep(deviceFeature),
       },
       { merge: true },
     );

server/test/lib/scene/actions/scene.action.getValue.test.js

@@ -1,4 +1,4 @@
-const { fake } = require('sinon');
+const { fake, stub } = require('sinon');
 const { expect } = require('chai');
 const EventEmitter = require('events');
 
@@ -107,4 +107,49 @@ describe('device.getValue', () => {
       },
     });
   });
+  it('should execute multiple get value and get a merged scope with different value', async () => {
+    const stateManagerStub = stub();
+    const stateManager = {
+      get: stateManagerStub,
+    };
+    const objectToReturn = { last_value: 10 };
+    stateManagerStub.onCall(0).returns(objectToReturn);
+    stateManagerStub.onCall(1).returns({ last_value: 15 });
+    const device = {
+      setValue: fake.resolves(null),
+    };
+    const scope = {};
+    await executeActions(
+      { stateManager, event, device },
+      [
+        [
+          {
+            type: ACTIONS.DEVICE.GET_VALUE,
+            device_feature: 'my-device-feature',
+          },
+        ],
+        [
+          {
+            type: ACTIONS.DEVICE.GET_VALUE,
+            device_feature: 'my-device-feature',
+          },
+        ],
+      ],
+      scope,
+    );
+    // should not affect result
+    objectToReturn.last_value = 10000;
+    expect(scope).to.deep.equal({
+      0: {
+        0: {
+          last_value: 10,
+        },
+      },
+      1: {
+        0: {
+          last_value: 15,
+        },
+      },
+    });
+  });
 });