Merge pull request #709 from daleglass-overte/script-test-infinite-require

Script test infinite require

tests/script-engine/src/ScriptEngineNetworkedTests.cpp

@@ -155,3 +155,31 @@ void ScriptEngineNetworkedTests::testRequire() {
 
 
 
+void ScriptEngineNetworkedTests::testRequireInfinite() {
+    auto sm = makeManager(
+        "print(\"Starting\");"
+        "Script.require('./tests/require_inf_a.js');"
+        "print(\"Done\");"
+        "Script.stop(true);", "testRequireInf.js");
+    QString errors;
+
+
+
+    QVERIFY(!sm->isRunning());
+    QVERIFY(!sm->isStopped());
+    QVERIFY(!sm->isFinished());
+
+    connect(sm.get(), &ScriptManager::errorMessage, [&errors](const QString& message, const QString& engineName){
+        errors.append(message);
+    });
+
+
+    qInfo() << "About to run script";
+    sm->run();
+
+    QVERIFY(!sm->isRunning());
+    QVERIFY(!sm->isStopped());
+    QVERIFY(sm->isFinished());
+
+    QVERIFY(errors.contains("Maximum call stack size exceeded"));
+}

tests/script-engine/src/ScriptEngineNetworkedTests.h

@@ -27,6 +27,7 @@ class ScriptEngineNetworkedTests : public QObject {
 private slots:
     void initTestCase();
     void testRequire();
+    void testRequireInfinite();
 
 
 private:

tests/script-engine/src/tests/require_inf_a.js

@@ -0,0 +1,2 @@
+module.exports = { value: 1 };
+var a = Script.require('./require_inf_b.js');

tests/script-engine/src/tests/require_inf_b.js

@@ -0,0 +1,2 @@
+module.exports = { value: 1 };
+var a = Script.require('./require_inf_c.js');

tests/script-engine/src/tests/require_inf_c.js

@@ -0,0 +1,2 @@
+module.exports = { value: 1 };
+var a = Script.require('./require_inf_a.js');