From 34bc4f821bfc7f985efd3696f88d73a8b4fd836a Mon Sep 17 00:00:00 2001 From: Justin Horvitz Date: Wed, 29 Nov 2017 09:11:49 -0500 Subject: [PATCH] fix(check): fix bug with async chekcs The correct field to set on CheckResult is 'result' not 'value' --- lib/core/utils/check-helper.js | 2 +- test/core/base/check.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/core/utils/check-helper.js b/lib/core/utils/check-helper.js index cac9b5d3fe..4a8af0f3a6 100644 --- a/lib/core/utils/check-helper.js +++ b/lib/core/utils/check-helper.js @@ -13,7 +13,7 @@ axe.utils.checkHelper = function checkHelper(checkResult, options, resolve, reje this.isAsync = true; return function (result) { if (result instanceof Error === false) { - checkResult.value = result; + checkResult.result = result; resolve(checkResult); } else { reject(result); diff --git a/test/core/base/check.js b/test/core/base/check.js index 2026a371a2..850d620050 100644 --- a/test/core/base/check.js +++ b/test/core/base/check.js @@ -193,7 +193,7 @@ describe('Check', function () { } }).run(fixture, {}, function (d) { assert.instanceOf(d, CheckResult); - assert.deepEqual(d.value, data); + assert.deepEqual(d.result, data); done(); });