Merge pull request #19 from mrloop/assert-throws

fix: make sure assert.throws uses our assert proxy

Author: jembezmamy

src/retry.js

@@ -12,6 +12,7 @@ export default class Retry {
     testFn(...args, async (assert, ...callbackArgs) => {
       this.assertProxy = new Proxy(assert, this.assertResultHandler)
       this.callbackArgs = callbackArgs
+      assert.test.assert = this.assertProxy
       this.currentRun = 1
       await this.retry(this.currentRun)
     })

test/retry.js

@@ -246,6 +246,22 @@ QUnit.module('assert.expect', function () {
   })
 })
 
+QUnit.module('assert.throws', function () {
+  const calls = []
+
+  retry('count retries', function (assert, currentRun) {
+    calls.push(currentRun)
+
+    assert.throws(() => {
+      if (currentRun === 2) throw new Error('fail')
+    })
+  })
+
+  QUnit.test('verify calls', function (assert) {
+    assert.deepEqual(calls, [1, 2])
+  })
+})
+
 QUnit.module('retry.todo', function () {
   const calls = []