jnioche opened a new pull request, #1906: URL: https://github.com/apache/stormcrawler/pull/1906
The issue was that ProtocolTest and PageActionsLiveTest both extended AbstractProtocolTest and shared the same static Jetty server instance. When running tests together, the server would be stopped by one test class before the other could use it, causing ERR_CONNECTION_REFUSED errors. This fix creates a new BasePlaywrightTest class specifically for the Playwright module that gives each test class its own isolated server instance. Changes: - Created BasePlaywrightTest with proper server management - Updated ProtocolTest to extend BasePlaywrightTest - Updated PageActionsLiveTest to extend BasePlaywrightTest - Removed redundant getHandlers() overrides Testing • Ran full test suite: mvn test -pl external/playwright - PASS • Ran individual tests: mvn test -pl external/playwright -Dtest=PageActionsLiveTest - PASS • Ran individual tests: mvn test -pl external/playwright -Dtest=ProtocolTest - PASS -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
