Set change function of monitors

Author: adazem009

src/engine/internal/engine.cpp

@@ -229,10 +229,13 @@ void Engine::compile()
         auto container = blockSectionContainer(block->opcode());
 
         if (container) {
-            MonitorNameFunc f = container->resolveMonitorNameFunc(block->opcode());
+            MonitorNameFunc nameFunc = container->resolveMonitorNameFunc(block->opcode());
 
-            if (f)
-                monitor->impl->name = f(block.get());
+            if (nameFunc)
+                monitor->impl->name = nameFunc(block.get());
+
+            MonitorChangeFunc changeFunc = container->resolveMonitorChangeFunc(block->opcode());
+            monitor->setValueChangeFunction(changeFunc);
         }
 
         if (section) {
@@ -1466,10 +1469,13 @@ void Engine::addVarOrListMonitor(std::shared_ptr<Monitor> monitor, Target *targe
     auto container = blockSectionContainer(monitor->opcode());
 
     if (container) {
-        MonitorNameFunc f = container->resolveMonitorNameFunc(monitor->opcode());
+        MonitorNameFunc nameFunc = container->resolveMonitorNameFunc(monitor->opcode());
+
+        if (nameFunc)
+            monitor->impl->name = nameFunc(monitor->block().get());
 
-        if (f)
-            monitor->impl->name = f(monitor->block().get());
+        MonitorChangeFunc changeFunc = container->resolveMonitorChangeFunc(monitor->opcode());
+        monitor->setValueChangeFunction(changeFunc);
     }
 
     // Auto-position the monitor

test/engine/engine_test.cpp

@@ -130,6 +130,9 @@ TEST(EngineTest, CompileAndExecuteMonitors)
 
     engine.addMonitorNameFunction(section.get(), m2->opcode(), [](Block *block) -> const std::string & { return block->opcode(); });
 
+    engine.addMonitorChangeFunction(section.get(), m1->opcode(), [](Block *block, const Value &newValue) { std::cout << "change 1!" << std::endl; });
+    engine.addMonitorChangeFunction(section.get(), m2->opcode(), [](Block *block, const Value &newValue) { std::cout << "change 2!" << std::endl; });
+
     // Compile the monitor blocks
     engine.compile();
     auto script1 = m1->script();
@@ -170,6 +173,17 @@ TEST(EngineTest, CompileAndExecuteMonitors)
     })));
     EXPECT_CALL(iface3, onValueChanged).Times(0);
     engine.updateMonitors();
+
+    // Change the monitor values
+    testing::internal::CaptureStdout();
+    EXPECT_CALL(iface1, onValueChanged);
+    m1->changeValue(0);
+    ASSERT_EQ(testing::internal::GetCapturedStdout(), "change 1!\n");
+
+    testing::internal::CaptureStdout();
+    EXPECT_CALL(iface2, onValueChanged);
+    m2->changeValue(0);
+    ASSERT_EQ(testing::internal::GetCapturedStdout(), "change 2!\n");
 }
 
 TEST(EngineTest, IsRunning)