Improve sync+async test reporting

Wrap all example executions in a promise to avoid awkward pauses in
result reporting with a mix of sync/async tests

Author: wied03

CHANGELOG.md

@@ -5,6 +5,9 @@
 *   Arity checking enabled by default
 *   Dropped support for PhantomJS < 2.0
 
+## 0.5.1
+*   Wrap all example executions in a promise to avoid awkward pauses in result reporting with a mix of sync/async tests
+
 ## 0.5.0 (2015-12-08)
 
 *   By default, any subject, it example block, before(:each), after(:each), and around that returns a promise will be executed asynchronously. Async is NOT yet supported for context level hooks. Async approach from < 0.4.3 will still work.

opal/opal/rspec/async/async_example.rb

@@ -1,12 +1,15 @@
 class ::RSpec::Core::Example
   def core_block_run
-    example_promise = Promise.value(@example_group_instance.instance_exec(self, &@example_block))
-    example_promise.then do |result|
-      result
-    end.rescue do |ex|
-      ex ||= Exception.new 'Async promise failed for unspecified reason'
-      ex = Exception.new ex unless ex.kind_of?(Exception)
-      ex
+    # Use EG's method to avoid awkward pauses in result reporting with mix of sync/async examples
+    @example_group_instance.delay_with_promise 0 do
+      example_promise = Promise.value(@example_group_instance.instance_exec(self, &@example_block))
+      example_promise.then do |result|
+        result
+      end.rescue do |ex|
+        ex ||= Exception.new 'Async promise failed for unspecified reason'
+        ex = Exception.new ex unless ex.kind_of?(Exception)
+        ex
+      end
     end
   end