From c4277d66779b7b6c9f77cbc71c74d60e7668dfb1 Mon Sep 17 00:00:00 2001
From: George Cook <george@tantawowa.com>
Date: Thu, 21 May 2020 17:39:41 -0500
Subject: [PATCH] fix(TestRunner): Adds more logoutput when a node test does
 not complete

---
 frameworkTests/source/tests/BasicTests.bs | 1 -
 src/TestRunner.bs                         | 7 +++++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/frameworkTests/source/tests/BasicTests.bs b/frameworkTests/source/tests/BasicTests.bs
index 784871c8..68eac50d 100644
--- a/frameworkTests/source/tests/BasicTests.bs
+++ b/frameworkTests/source/tests/BasicTests.bs
@@ -66,7 +66,6 @@ end function
 '@It tests async tests
 '+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 
-'@Only
 '@Test times out
 function async_timeout()
   item = { "id" : "item" }
diff --git a/src/TestRunner.bs b/src/TestRunner.bs
index 3b32d905..a3ad20b8 100644
--- a/src/TestRunner.bs
+++ b/src/TestRunner.bs
@@ -106,8 +106,11 @@ public sub run()
           "runtimeConfig": m.runtimeConfig
         }
         nodeStatResults = node.callFunc("Rooibos_RunNodeTests", args)
-        RBS_STATS.MergeTotalStatistic(totalStatObj, nodeStatResults)
-        
+        if nodeStatResults <> invalid
+          RBS_STATS.MergeTotalStatistic(totalStatObj, nodeStatResults)
+        else
+          ? " ERROR! The node "; nodeType; " did not return stats from the Rooibos_RunNodeTests method. This usually means you are not importing rooibosDist.brs, or rooibosFunctionMap.brs"
+        end if
         m.testScene.RemoveChild(node)
         
       else