Skip to content

Conversation

@thc202
Copy link
Member

@thc202 thc202 commented Jun 24, 2025

Use Firefox and Edge along with Chrome.
Replace Playwright with Selenium, which allows to use Firefox with extensions.
Extract base driver class since most code is common to all browsers.
Extract "ZAP" server to make it easier to use.
Update all test cases to use Selenium.
Update test timeout to allow all browsers enough time to load.

@psiinon
Copy link
Member

psiinon commented Jun 24, 2025

Logo
Checkmarx One – Scan Summary & Details8d93233e-7bc7-47ff-a689-4473df7867b3

Great job, no security vulnerabilities found in this Pull Request

@thc202 thc202 force-pushed the itests-browsers branch 3 times, most recently from b6c8c6d to 485b28c Compare June 24, 2025 12:38
@thc202 thc202 marked this pull request as ready for review June 24, 2025 12:52
@thc202 thc202 marked this pull request as draft June 24, 2025 17:00
Use Firefox and Edge along with Chrome.
Replace Playwright with Selenium, which allows to use Firefox with
extensions.
Extract base driver class since most code is common to all browsers.
Extract "ZAP" server to make it easier to use.
Update all test cases to use Selenium.
Update test timeout to allow all browsers enough time to load.

Signed-off-by: thc202 <thc202@gmail.com>
@thc202 thc202 force-pushed the itests-browsers branch from 485b28c to f73c618 Compare June 25, 2025 08:19
@thc202 thc202 marked this pull request as ready for review June 25, 2025 08:22
@kingthorin kingthorin merged commit 8f606b5 into zaproxy:main Jun 25, 2025
3 checks passed
@thc202 thc202 deleted the itests-browsers branch June 25, 2025 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants