Bugfix/general fixes (#284)

* Fixed error log printing when it shouldn't

* Fixed a bug where asyn tests that finish sync could result in problems

* insure object key order when converting to string

* increased sleep at the end of the run to better flush the io logs

framework/src/source/BaseTestSuite.bs

@@ -199,6 +199,8 @@ namespace rooibos
           m.testRunner.top.unobserveFieldScoped("rooibosGroupFinished")
           ' m.testGroupDone()
           m.onAsyncGroupComplete(group)
+        else if m.testRunner.top.rooibosGroupFinished
+          m.onAsyncGroupComplete(group)
         end if
 
       end if

framework/src/source/CommonUtils.bs

@@ -334,7 +334,9 @@ namespace rooibos.common
         'bs:disable-next-line
         isFirst = false
       end if
-      for each key in input
+      keys = input.keys()
+      keys.sort()
+      for each key in keys
         if rooibos.common.canSafelyIterateAAKey(input, key)
           text = text + key + ":" + rooibos.common.asString(input[key], includeType)
         end if

framework/src/source/Rooibos.bs

@@ -38,7 +38,7 @@ namespace rooibos
         ' leading to an instant exit of the application
 
         ' Give the io port time to finish sending all the logs
-        sleep(200)
+        sleep(400)
         end
       end if
     end if

framework/src/source/TestRunner.bs

@@ -163,6 +163,7 @@ namespace rooibos
           ? "Running suite asynchronously!"
           m.nodeContext.top.observeFieldScoped("rooibosSuiteFinished", "Rooibos_onTestSuiteComplete")
           testSuite.run()
+          return invalid
         else
           ? "Running suite synchronously!"
           testSuite.run()