[PR #998] [MERGED] Fix selenium tests broken after bootstrap upgrade #1201

New issue

Closed

opened 2026-02-25 21:37:32 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 21:37:32 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/998
Author: @Shadow243
Created: 5/2/2024
Status: ✅ Merged
Merged: 7/22/2024
Merged by: @kroky

Base: master ← Head: fix-selenium

📝 Commits (9)

d1eb5b4 Fix selenium tests broken after bootstrap upgrade
f2bbba4 add try to navigate_msg_test in case we don't have email_folders
a3d41ae Add postfix to servers.py
a25e12f Jos test
93698fa Correct good_logout test
faf1408 Fixed good logout
3ef02bd Fix pages.py tests
ae9b6ff Fix pages.py tests
55a38fa Merge branch 'fix-selenium' of github.com:Shadow243/cypht into fix-selenium

📊 Changes

22 files changed (+240 additions, -146 deletions)

View changed files

📝 .github/tests/.env (+5 -4)
📝 .github/tests/selenium/creds.py (+12 -1)
📝 .github/tests/setup.sh (+4 -4)
📝 .github/tests/test.sh (+2 -1)
📝 .github/workflows/Test-Build.yml (+1 -0)
📝 docker-compose.dev.yaml (+2 -2)
📝 modules/calendar/modules.php (+1 -1)
📝 modules/core/output_modules.php (+3 -4)
📝 modules/core/site.js (+4 -1)
📝 modules/profiles/modules.php (+1 -1)
📝 tests/phpunit/modules/core/output_modules.php (+1 -1)
📝 tests/selenium/base.py (+20 -9)
📝 tests/selenium/folder_list.py (+7 -6)
📝 tests/selenium/inline_msg.py (+5 -5)
📝 tests/selenium/login.py (+2 -2)
📝 tests/selenium/pages.py (+28 -24)
📝 tests/selenium/profiles.py (+10 -6)
📝 tests/selenium/runall.sh (+3 -1)
📝 tests/selenium/search.py (+6 -3)
📝 tests/selenium/send.py (+41 -14)

...and 2 more files

📄 Description

Update Selenium Tests to `selenium: 4.14.0`

This pull request updates the deprecated Selenium tests to use selenium: 4.14.0. As mentioned in the related issue #989, some tests have been commented out temporarily as they are not currently functional, and work is ongoing to address them.

Additionally, the servers.py file will require refactoring to accommodate recent changes, the tests will be updated accordingly to reflect these changes.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cypht-org/cypht/pull/998 **Author:** [@Shadow243](https://github.com/Shadow243) **Created:** 5/2/2024 **Status:** ✅ Merged **Merged:** 7/22/2024 **Merged by:** [@kroky](https://github.com/kroky) **Base:** `master` ← **Head:** `fix-selenium` --- ### 📝 Commits (9) - [`d1eb5b4`](https://github.com/cypht-org/cypht/commit/d1eb5b478197cb2fbf5eab3abab14fea20f3cd10) Fix selenium tests broken after bootstrap upgrade - [`f2bbba4`](https://github.com/cypht-org/cypht/commit/f2bbba46d16e5734833af6b8e8e21afcd7bc0a7b) add try to navigate_msg_test in case we don't have email_folders - [`a3d41ae`](https://github.com/cypht-org/cypht/commit/a3d41ae0d759784b6255ca117ff45b32343b6c9b) Add postfix to servers.py - [`a25e12f`](https://github.com/cypht-org/cypht/commit/a25e12fdca0c6c1b0a352eb1b034a42cd3f3cce4) Jos test - [`93698fa`](https://github.com/cypht-org/cypht/commit/93698fa3a8dac054d403705251b72877079e809c) Correct good_logout test - [`faf1408`](https://github.com/cypht-org/cypht/commit/faf140868e280d97bed5f00b610e15f13c424ecb) Fixed good logout - [`3ef02bd`](https://github.com/cypht-org/cypht/commit/3ef02bdf213f6c0fd9a7d5e4f595732b9028fd8b) Fix pages.py tests - [`ae9b6ff`](https://github.com/cypht-org/cypht/commit/ae9b6ff206c7f7be01a4aa33cbeb78ab1bcc9bea) Fix pages.py tests - [`55a38fa`](https://github.com/cypht-org/cypht/commit/55a38fa092ab5e536b5f4b0915b174122929a6ab) Merge branch 'fix-selenium' of github.com:Shadow243/cypht into fix-selenium ### 📊 Changes **22 files changed** (+240 additions, -146 deletions) <details> <summary>View changed files</summary> 📝 `.github/tests/.env` (+5 -4) 📝 `.github/tests/selenium/creds.py` (+12 -1) 📝 `.github/tests/setup.sh` (+4 -4) 📝 `.github/tests/test.sh` (+2 -1) 📝 `.github/workflows/Test-Build.yml` (+1 -0) 📝 `docker-compose.dev.yaml` (+2 -2) 📝 `modules/calendar/modules.php` (+1 -1) 📝 `modules/core/output_modules.php` (+3 -4) 📝 `modules/core/site.js` (+4 -1) 📝 `modules/profiles/modules.php` (+1 -1) 📝 `tests/phpunit/modules/core/output_modules.php` (+1 -1) 📝 `tests/selenium/base.py` (+20 -9) 📝 `tests/selenium/folder_list.py` (+7 -6) 📝 `tests/selenium/inline_msg.py` (+5 -5) 📝 `tests/selenium/login.py` (+2 -2) 📝 `tests/selenium/pages.py` (+28 -24) 📝 `tests/selenium/profiles.py` (+10 -6) 📝 `tests/selenium/runall.sh` (+3 -1) 📝 `tests/selenium/search.py` (+6 -3) 📝 `tests/selenium/send.py` (+41 -14) _...and 2 more files_ </details> ### 📄 Description ## Update Selenium Tests to `selenium: 4.14.0` This pull request updates the deprecated Selenium tests to use `selenium: 4.14.0`. As mentioned in the related issue [#989](https://github.com/cypht-org/cypht/issues/989), some tests have been commented out temporarily as they are not currently functional, and work is ongoing to address them. Additionally, the `servers.py` file will require refactoring to accommodate recent changes, the tests will be updated accordingly to reflect these changes. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>