Use VirtualMachine instead of Value to update monitor values

Author: adazem009

include/scratchcpp/imonitorhandler.h

@@ -17,7 +17,7 @@ class LIBSCRATCHCPP_EXPORT IMonitorHandler
 
         virtual void init(Monitor *) = 0;
 
-        virtual void onValueChanged(const Value &value) = 0;
+        virtual void onValueChanged(const VirtualMachine *vm) = 0;
         virtual void onVisibleChanged(bool visible) = 0;
 };
 

include/scratchcpp/monitor.h

@@ -49,7 +49,7 @@ class LIBSCRATCHCPP_EXPORT Monitor : public Entity
 
         const std::string &opcode() const;
 
-        void updateValue(const Value &value);
+        void updateValue(const VirtualMachine *vm);
 
         unsigned int width() const;
         void setWidth(unsigned int width);

src/scratch/monitor.cpp

@@ -80,11 +80,14 @@ const std::string &Monitor::opcode() const
     return impl->block->opcode();
 }
 
-/*! Updates the monitor's value and notifies its interface about it (if there's any). */
-void Monitor::updateValue(const Value &value)
+/*!
+ * Notifies the monitor's interface about value change.
+ * The interaface is supposed to read it from the VirtualMachine.
+ */
+void Monitor::updateValue(const VirtualMachine *vm)
 {
     if (impl->iface)
-        impl->iface->onValueChanged(value);
+        impl->iface->onValueChanged(vm);
 }
 
 /*! Returns the monitor's width. */

test/mocks/monitorhandlermock.h

@@ -10,6 +10,6 @@ class MonitorHandlerMock : public IMonitorHandler
     public:
         MOCK_METHOD(void, init, (Monitor *), (override));
 
-        MOCK_METHOD(void, onValueChanged, (const Value &), (override));
+        MOCK_METHOD(void, onValueChanged, (const VirtualMachine *), (override));
         MOCK_METHOD(void, onVisibleChanged, (bool), (override));
 };

test/scratch_classes/monitor_test.cpp

@@ -2,6 +2,7 @@
 #include <scratchcpp/block.h>
 #include <scratchcpp/sprite.h>
 #include <scratchcpp/rect.h>
+#include <scratchcpp/virtualmachine.h>
 #include <scratch/monitor_p.h>
 #include <monitorhandlermock.h>
 #include <randomgeneratormock.h>
@@ -91,17 +92,22 @@ TEST(MonitorTest, Sprite)
 TEST(MonitorTest, UpdateValue)
 {
     Monitor monitor("", "");
-    monitor.updateValue(5);
+    VirtualMachine vm1, vm2;
+    monitor.updateValue(nullptr);
+    monitor.updateValue(&vm1);
 
     MonitorHandlerMock handler;
     EXPECT_CALL(handler, init);
     monitor.setInterface(&handler);
 
-    EXPECT_CALL(handler, onValueChanged(Value(6.5)));
-    monitor.updateValue(6.5);
+    EXPECT_CALL(handler, onValueChanged(&vm1));
+    monitor.updateValue(&vm1);
 
-    EXPECT_CALL(handler, onValueChanged(Value("test")));
-    monitor.updateValue("test");
+    EXPECT_CALL(handler, onValueChanged(nullptr));
+    monitor.updateValue(nullptr);
+
+    EXPECT_CALL(handler, onValueChanged(&vm2));
+    monitor.updateValue(&vm2);
 }
 
 TEST(MonitorTest, Width)