diff options
| author | Coen van der Kamp <coen@fourdigits.nl> | 2023-06-02 12:20:17 +0100 |
|---|---|---|
| committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2023-06-05 14:43:51 +0200 |
| commit | 10d9d0ccb24ebefbcc12f8bc1b90becec64340ee (patch) | |
| tree | 9a61c4488d8e2c6ce60d02074fc84b58569d9347 /tests/admin_views/test_related_object_lookups.py | |
| parent | 0faad0193815c5f07a08dba4eaa04193ca9eb268 (diff) | |
Fixed #34622 -- Improved accessibility of related widget wrapper in admin.
This improves accessibility for screen reader users by adding
"aria-disabled" and removing "alt".
Thanks Thibaud Colas for the report.
Diffstat (limited to 'tests/admin_views/test_related_object_lookups.py')
| -rw-r--r-- | tests/admin_views/test_related_object_lookups.py | 76 |
1 files changed, 76 insertions, 0 deletions
diff --git a/tests/admin_views/test_related_object_lookups.py b/tests/admin_views/test_related_object_lookups.py new file mode 100644 index 0000000000..d478214d90 --- /dev/null +++ b/tests/admin_views/test_related_object_lookups.py @@ -0,0 +1,76 @@ +from django.contrib.admin.tests import AdminSeleniumTestCase +from django.contrib.auth.models import User +from django.test import override_settings +from django.urls import reverse + + +@override_settings(ROOT_URLCONF="admin_views.urls") +class SeleniumTests(AdminSeleniumTestCase): + available_apps = ["admin_views"] + AdminSeleniumTestCase.available_apps + + def setUp(self): + self.superuser = User.objects.create_superuser( + username="super", password="secret", email="super@example.com" + ) + self.admin_login( + username="super", password="secret", login_url=reverse("admin:index") + ) + + def test_related_object_link_images_empty_alt(self): + from selenium.webdriver.common.by import By + + album_add_url = reverse("admin:admin_views_album_add") + self.selenium.get(self.live_server_url + album_add_url) + + tests = [ + "add_id_owner", + "change_id_owner", + "delete_id_owner", + "view_id_owner", + ] + for link_id in tests: + with self.subTest(link_id): + link_image = self.selenium.find_element( + By.XPATH, f'//*[@id="{link_id}"]/img' + ) + self.assertEqual(link_image.get_attribute("alt"), "") + + def test_related_object_lookup_link_initial_state(self): + from selenium.webdriver.common.by import By + + album_add_url = reverse("admin:admin_views_album_add") + self.selenium.get(self.live_server_url + album_add_url) + + tests = [ + "change_id_owner", + "delete_id_owner", + "view_id_owner", + ] + for link_id in tests: + with self.subTest(link_id): + link = self.selenium.find_element(By.XPATH, f'//*[@id="{link_id}"]') + self.assertEqual(link.get_attribute("aria-disabled"), "true") + + def test_related_object_lookup_link_enabled(self): + from selenium.webdriver.common.by import By + from selenium.webdriver.support.select import Select + + album_add_url = reverse("admin:admin_views_album_add") + self.selenium.get(self.live_server_url + album_add_url) + + select_element = self.selenium.find_element(By.XPATH, '//*[@id="id_owner"]') + option = Select(select_element).options[-1] + self.assertEqual(option.text, "super") + select_element.click() + option.click() + + tests = [ + "add_id_owner", + "change_id_owner", + "delete_id_owner", + "view_id_owner", + ] + for link_id in tests: + with self.subTest(link_id): + link = self.selenium.find_element(By.XPATH, f'//*[@id="{link_id}"]') + self.assertIsNone(link.get_attribute("aria-disabled")) |
