- Issue created by @the_g_bomb
- Merge request !10977Issue #3501457: Fix path used in a A11y Test Admin → (Closed) created by the_g_bomb
The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- 🇬🇧United Kingdom the_g_bomb
the_g_bomb → changed the visibility of the branch 3501457-incorrect-path-used to hidden.
- Merge request !10991Issue #3501457: Update path for create article in a A11y Test Admin → (Open) created by the_g_bomb
- 🇬🇧United Kingdom the_g_bomb
@smustgrave, it is curious as the test failure is now somewhere else, that seems unrelated to the changes. I am wondering if the fact that the test is now testing the correct bit, the issue is that there are problems with the article.
Looks like responsive images, if I am reading the failures correctly. I wonder if the limited profile that is enabled for the a11y tests doesn't have enough tools to do the job of setting up the article node as required.
There will also need to be follow up tickets to fix all the failures that led to the need for the disabled tests and the limited profile, so we can get to a place where everything passes and the full standard installation is tested.
- 🇺🇸United States smustgrave
Made a small tweak to keep things uniform in the file. Seems good though.
- 🇺🇸United States smustgrave
Oops now I see why it was done that way. Reverted