Resolves #10056 (comment) (cc @patrickhulce)

Unit tests pass locally, but Travis seems unreliable for PRs at the moment. I've opened an experimental improvement in #10075.

@patrickhulce Any chance you could reset the Travis build cache for this PR? Doesn't look like I have permissions to, and I think that's all that's left to get this passing (I was quietly hoping that merging master might invalidate the cache, but no dice so far)

Just kicked a new one off without the cache, we'll see if it does the trick!

@connorjclark any more thoughts?

@patrickhulce @connorjclark One thing I was wondering about - the audit explanation field isn't present for the incomplete results here, which is reflected in the updated a11y test expectations (ref).

I've opened an upstream ticket in axe-core about it (linked in this PR's history) to see if that can be addressed, but was wondering if it's likely to cause any issues/problems in the meantime?

if it's likely to cause any issues/problems in the meantime?

It will be a slight regression in the available data, but I don't expect it to be too large of a problem as long as we get this in to 6.0. There's also not much we can do about it other than not upgrade or wait for axe to fix it, so it shouldn't be a blocker IMO.

Small update: failureSummary should be available for incomplete results from axe-core>=3.5 (ref).

@paulirish does this require further discussion or does the current state of a review requested from @connorjclark work?

@connorjclark if you'd prefer deferring to my review that's fine too, just wanted to give you the opportunity since you reviewed earlier.

Thanks a ton @patrickhulce @connorjclark !