Support err as first argument for done callbacks in async tests

ospec/ospec.js

@@ -87,7 +87,13 @@ module.exports = new function init(name) {
 				var timeout = 0, delay = 200, s = new Date
 				var isDone = false
 
-				function done() {
+				function done(err) {
+					if (err) {
+						if (err.message) record(err.message, err)
+						else record(err)
+						subjects.pop()
+						next()
+					}
 					if (timeout !== undefined) {
 						timeout = clearTimeout(timeout)
 						if (delay !== Infinity) record(null)
@@ -114,9 +120,7 @@ module.exports = new function init(name) {
 						fn(done, function(t) {delay = t})
 					}
 					catch (e) {
-						record(e.message, e)
+						done(e)
-						subjects.pop()
-						next()
 					}
 					if (timeout === 0) {
 						startTimer()
@@ -126,11 +130,7 @@ module.exports = new function init(name) {
 					var p = fn()
 					if (p && p.then) {
 						startTimer()
-						p.then(done, e => {
+						p.then(function() { done() }, done)
-							record(e.message, e)
-							subjects.pop()
-							next()
-						})
 					} else {
 						nextTickish(next)
 					}

ospec/tests/test-ospec.js

@@ -153,10 +153,14 @@ o.spec("ospec", function() {
 		var a = 0, b = 0
 
 		function wrapPromise(fn) {
-			return new Promise(resolve => {
+			return new Promise((resolve, reject) => {
 				callAsync(() => {
-					fn()
+					try {
-					resolve()
+						fn()
+						resolve()
+					} catch(e) {
+						reject(e)
+					}
 				})
 			})
 		}