Merge addd5d9 into 2d7d298

michaelDCurran · web-flow · commit 2f4fdc30a8de · 2020-10-27T12:28:40.000+10:00
diff --git a/source/NVDAObjects/IAccessible/__init__.py b/source/NVDAObjects/IAccessible/__init__.py
@@ -85,10 +85,11 @@ def normalizeIA2TextFormatField(formatField):
 	except KeyError:
 		invalid=None
 	if invalid:
-		invalid=invalid.lower()
-		if invalid=="spelling":
+		# aria-invalid can contain multiple values separated by a comma.
+		invalidList = [x.lower().strip() for x in invalid.split(',')]
+		if "spelling" in invalidList:
 			formatField["invalid-spelling"]=True
-		elif invalid=="grammar":
+		if "grammar" in invalidList:
 			formatField["invalid-grammar"]=True
 	color=formatField.get('color')
 	if color:
diff --git a/tests/system/robot/chromeTests.py b/tests/system/robot/chromeTests.py
@@ -288,3 +288,30 @@ def test_ariaTreeGrid_browseMode():
 			"level 1  Treegrids are awesome Want to learn how to use them? aaron at thegoogle dot rocks  expanded",
 		])
 	)
+
+def ARIAInvalid_spellingAndGrammar():
+	"""
+	Tests ARIA invalid values of "spelling", "grammar" and "spelling, grammar".
+	"""
+	_chrome.prepareChrome(
+		r"""
+			<p>Big <span aria-invalid="spelling">caat</span> meos</p>
+			<p>Small <span aria-invalid="grammar">a dog</span> woofs</p>
+			<p>Fat <span aria-invalid="grammar, spelling">a ffrog</span> crokes</p>
+		"""
+	)
+	actualSpeech = _chrome.getSpeechAfterKey("downArrow")
+	_asserts.strings_match(
+		actualSpeech,
+		"Big  spelling error  caat  meos"
+	)
+	actualSpeech = _chrome.getSpeechAfterKey("downArrow")
+	_asserts.strings_match(
+		actualSpeech,
+		"Small  grammar error  a dog  woofs"
+	)
+	actualSpeech = _chrome.getSpeechAfterKey("downArrow")
+	_asserts.strings_match(
+		actualSpeech,
+		"Fat  spelling error  grammar error  a ffrog  crokes"
+	)
diff --git a/tests/system/robot/chromeTests.robot b/tests/system/robot/chromeTests.robot
@@ -39,3 +39,6 @@ ARIA treegrid
 	[Documentation]	Ensure that ARIA treegrids are accessible as a standard table in browse mode.
 	# Excluded due to regular failures.
 	test_ariaTreeGrid_browseMode
+ARIA invalid spelling and grammar
+	[Documentation]	Tests ARIA invalid values of "spelling", "grammar" and "spelling, grammar".
+	ARIAInvalid_spellingAndGrammar
