- Issue created by @duaelfr
- @duaelfr opened merge request.
- Status changed to Needs review
almost 2 years ago 4:28pm 2 February 2023 - π«π·France duaelfr Montpellier, France
MR open for thoughts and reviews
I added a very basic test on Olivero to ensure it works (I never added a dependency so I'm not sure). - π«π·France duaelfr Montpellier, France
This is related to π Automated A11y tests in Nightwatch Fixed
We might discuss if we want both to be integrated or not. - Status changed to Needs work
over 1 year ago 9:35pm 18 February 2023 - πΊπΈUnited States smustgrave
Not sure if this is a valid test but I reused MediaSourceImageTest::testMediaImageSource
Disabled required alt text
Commented out line $page->fillField("{$source_field_id}[0][alt]", ''); so the image had no alt textThen ran
$this->drupalGet($this->assertLinkToCreatedMedia()); $this->executeAxe();;
Expected it to fail but I didn't and just continued along.
Is there something I'm missing?
- π«π·France duaelfr Montpellier, France
@smustgrave Thanks for testing! An empty alt text is not a failure. What's forbidden is no
alt
attribute at all!
It would be a failure if the image was the only content of a link for example. - π¨π¦Canada mgifford Ottawa, Ontario
Now we have https://www.drupal.org/project/drupal/issues/3293469 π Automated A11y tests in Nightwatch Fixed
Do we need this issue? Is PHPUnit going to give us different or better results?
- π«π·France duaelfr Montpellier, France
Hi @mgifford!
On its current state, I believe that my patch is less interesting that Nightwatch's one because you can only ignore some problems based on their severity. I believe it could be improved by adding a way to alter the axe rules like it's done in JS. I also considered adding some ways to assert expected success or failures in a future version of my patch.
All that to say that I think the only advantage of this one instead of Nightwatch's one would be that some devs could be more comfortable using PHPUnit than Nigthwatch.
Do you think it could be interesting to have both available?
- π¨π¦Canada mgifford Ottawa, Ontario
That I don't know. You've got the PHP developers & the JavaScript developers. Do you think PHPUnit is better supported by modules/themes than Nightwatch?
I think there's some split between nightwatch/cypress in the JS testing community. I'll ask around and see if we can get some other thoughts on this.
I'd suggest that we might want to test different things with them.
- πΊπΈUnited States dmundra Eugene, OR
I think since axe-core package is now available in version 10.1.x (with the nightwatch update) I think a PHP example could be good to have but I am not sure if we need the same setup for an install profile and such.
Is the AccessibilityTest being picked up by the pipeline? I am not seeing it https://dispatcher.drupalci.org/job/drupal_patches/164094/console if it is.