fix(errormessage): check data is an array of IDs

dequelabs · Aug 5, 2018 · d64bc5f · d64bc5f
1 parent 2cece25
commit d64bc5f
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/lib/checks/aria/errormessage.js b/lib/checks/aria/errormessage.js
@@ -21,7 +21,7 @@ function validateAttrValue() {
 // limit results to elements that actually have this attribute
 if (options.indexOf(attr) === -1 && hasAttr) {
 	if (!validateAttrValue()) {
-		this.data(attr);
+		this.data(axe.utils.tokenList(attr));
 		return false;
 	}
 }

diff --git a/test/checks/aria/errormessage.js b/test/checks/aria/errormessage.js
@@ -56,6 +56,16 @@ describe('aria-errormessage', function() {
 		);
 	});
 
+	it('sets an array of IDs in data', function() {
+		var testHTML = '<div></div>';
+		testHTML += '<div id="plain"></div>';
+		fixture.innerHTML = testHTML;
+		var target = fixture.children[0];
+		target.setAttribute('aria-errormessage', ' foo  bar \tbaz  ');
+		checks['aria-errormessage'].evaluate.call(checkContext, target);
+		assert.deepEqual(checkContext._data, ['foo', 'bar', 'baz']);
+	});
+
 	(shadowSupported ? it : xit)(
 		'should return false if aria-errormessage value crosses shadow boundary',
 		function() {